From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f199.google.com (mail-wr0-f199.google.com [209.85.128.199]) by kanga.kvack.org (Postfix) with ESMTP id A6F826B0313 for ; Wed, 14 Jun 2017 10:04:54 -0400 (EDT) Received: by mail-wr0-f199.google.com with SMTP id l34so407857wrc.12 for ; Wed, 14 Jun 2017 07:04:54 -0700 (PDT) Received: from mx1.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id z69si139691wrc.261.2017.06.14.07.04.52 for (version=TLS1 cipher=AES128-SHA bits=128/128); Wed, 14 Jun 2017 07:04:52 -0700 (PDT) Date: Wed, 14 Jun 2017 16:04:50 +0200 From: Michal Hocko Subject: Re: [RFC PATCH 1/4] mm, hugetlb: unclutter hugetlb allocation layers Message-ID: <20170614140450.GQ6045@dhcp22.suse.cz> References: <20170613090039.14393-1-mhocko@kernel.org> <20170613090039.14393-2-mhocko@kernel.org> <1babcd50-a90e-a3e4-c45c-85b1b8b93171@suse.cz> <20170614134258.GP6045@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170614134258.GP6045@dhcp22.suse.cz> Sender: owner-linux-mm@kvack.org List-ID: To: Vlastimil Babka Cc: linux-mm@kvack.org, Naoya Horiguchi , Mike Kravetz , Mel Gorman , Andrew Morton , LKML On Wed 14-06-17 15:42:58, Michal Hocko wrote: > On Wed 14-06-17 15:18:26, Vlastimil Babka wrote: > > On 06/13/2017 11:00 AM, Michal Hocko wrote: > [...] > > > @@ -1717,13 +1640,22 @@ struct page *alloc_huge_page_node(struct hstate *h, int nid) > > > page = dequeue_huge_page_node(h, nid); > > > spin_unlock(&hugetlb_lock); > > > > > > - if (!page) > > > - page = __alloc_buddy_huge_page_no_mpol(h, nid); > > > + if (!page) { > > > + nodemask_t nmask; > > > + > > > + if (nid != NUMA_NO_NODE) { > > > + nmask = NODE_MASK_NONE; > > > + node_set(nid, nmask); > > > > TBH I don't like this hack too much, and would rather see __GFP_THISNODE > > involved, which picks a different (short) zonelist. Also it's allocating > > nodemask on stack, which we generally avoid? Although the callers > > currently seem to be shallow. > > Fair enough. That would require pulling gfp mask handling up the call > chain. This on top of this patch + refreshes for other patches later in > the series as they will conflict now? I've rebase the attempts/hugetlb-zonelists branch for an easier review. -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org