From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758142AbaAJT7m (ORCPT ); Fri, 10 Jan 2014 14:59:42 -0500 Received: from mx1.redhat.com ([209.132.183.28]:62474 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751258AbaAJT7i (ORCPT ); Fri, 10 Jan 2014 14:59:38 -0500 Message-ID: <52D0488B.9080806@redhat.com> Date: Fri, 10 Jan 2014 14:22:51 -0500 From: Rik van Riel User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: Johannes Weiner , Andrew Morton CC: Andi Kleen , Andrea Arcangeli , Bob Liu , Christoph Hellwig , Dave Chinner , Greg Thelen , Hugh Dickins , Jan Kara , KOSAKI Motohiro , Luigi Semenzato , Mel Gorman , Metin Doslu , Michel Lespinasse , Minchan Kim , Ozgun Erdogan , Peter Zijlstra , Roman Gushchin , Ryan Mallon , Tejun Heo , Vlastimil Babka , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [patch 4/9] mm: filemap: move radix tree hole searching here References: <1389377443-11755-1-git-send-email-hannes@cmpxchg.org> <1389377443-11755-5-git-send-email-hannes@cmpxchg.org> In-Reply-To: <1389377443-11755-5-git-send-email-hannes@cmpxchg.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/10/2014 01:10 PM, Johannes Weiner wrote: > The radix tree hole searching code is only used for page cache, for > example the readahead code trying to get a a picture of the area > surrounding a fault. > > It sufficed to rely on the radix tree definition of holes, which is > "empty tree slot". But this is about to change, though, as shadow > page descriptors will be stored in the page cache after the actual > pages get evicted from memory. > > Move the functions over to mm/filemap.c and make them native page > cache operations, where they can later be adapted to handle the new > definition of "page cache hole". > > Signed-off-by: Johannes Weiner Reviewed-by: Rik van Riel -- All rights reversed