From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754903AbbK0RF3 (ORCPT ); Fri, 27 Nov 2015 12:05:29 -0500 Received: from mail-wm0-f48.google.com ([74.125.82.48]:36397 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754566AbbK0RF0 (ORCPT ); Fri, 27 Nov 2015 12:05:26 -0500 Date: Fri, 27 Nov 2015 18:05:22 +0100 From: Michal Hocko To: Vlastimil Babka Cc: Jan Kara , linux-mm@kvack.org, Andrew Morton , Tetsuo Handa , Mel Gorman , Dave Chinner , Mark Fasheh , ocfs2-devel@oss.oracle.com, ceph-devel@vger.kernel.org, linux-fsdevel@vger.kernel.org, LKML Subject: Re: [PATCH] mm: Allow GFP_IOFS for page_cache_read page cache allocation Message-ID: <20151127170522.GL2493@dhcp22.suse.cz> References: <1447251233-14449-1-git-send-email-mhocko@kernel.org> <20151112095301.GA25265@quack.suse.cz> <20151126150820.GI7953@dhcp22.suse.cz> <56588789.1010300@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56588789.1010300@suse.cz> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 27-11-15 17:40:41, Vlastimil Babka wrote: > On 11/26/2015 04:08 PM, Michal Hocko wrote: > >On Thu 12-11-15 10:53:01, Jan Kara wrote: > >>On Wed 11-11-15 15:13:53, mhocko@kernel.org wrote: > >>> > >>>Hi, > >>>this has been posted previously as a part of larger GFP_NOFS related > >>>patch set (http://lkml.kernel.org/r/1438768284-30927-1-git-send-email-mhocko%40kernel.org) > >>>but I think it makes sense to discuss it even out of that scope. > >>> > >>>I would like to hear FS and other MM people about the proposed interface. > >>>Using mapping_gfp_mask blindly doesn't sound good to me and vm_fault > >>>looks like a proper channel to communicate between MM and FS layers. > >>> > >>>Comments? Are there any better ideas? > >> > >>Makes sense to me and the filesystems I know should be fine with this > >>(famous last words ;). Feel free to add: > >> > >>Acked-by: Jan Kara > > > >Thanks a lot! Are there any objections from other fs/mm people? > > Please replace "GFP_IOFS" in the subject, as the "flag" has been removed > recently. Otherwise Done. mm: Allow GFP_{FS,IO} for page_cache_read page cache allocation > Acked-by: Vlastimil Babka Thanks -- Michal Hocko SUSE Labs