From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [External] Re: [RFC PATCH v2 00/12] get rid of GFP_ZONE_TABLE/BAD Date: Wed, 30 May 2018 11:11:19 +0200 Message-ID: <20180530091119.GA30154@lst.de> References: <1526916033-4877-1-git-send-email-yehs2007@gmail.com> <20180522183728.GB20441@dhcp22.suse.cz> <20180524121853.GG20441@dhcp22.suse.cz> <20180528133733.GF27180@dhcp22.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Huaisheng HS1 Ye Cc: "kstewart-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org" , Ocean HY1 He , "gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "willy-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org" , Michal Hocko , "linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org" , "colyli-l3A5Bk7waGM@public.gmane.org" , "iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org" , "linux-btrfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , NingTing Cheng , "xen-devel-GuqFBffKawtpuQazS67q72D2FQJk+8+b@public.gmane.org" , "akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org" , "alexander.levin-H+0wwilmMs1BDgjK7y7TUQ@public.gmane.org" , "mgorman-3eNAlZScCAx27rWaFMvyedHuzzzSOjJt@public.gmane.org" , Christoph Hellwig , "vbabka-AlSwsSmVLrQ@public.gmane.org" List-Id: iommu@lists.linux-foundation.org On Wed, May 30, 2018 at 09:02:13AM +0000, Huaisheng HS1 Ye wrote: > > I don't quite understand that. I think those, mostly drivers, need to > get the correct zone they want. ZONE_DMA32 is an example, if drivers can be > satisfied with a low mem zone, why they mark the gfp flags as > 'GFP_KERNEL|__GFP_DMA32'? Drivers should never use GFP_DMA32 directly. The right abstraction is the DMA API, ZONE_DMA32 is just a helper.