From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753008AbbAUL2S (ORCPT ); Wed, 21 Jan 2015 06:28:18 -0500 Received: from cantor2.suse.de ([195.135.220.15]:50842 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750974AbbAUL2I (ORCPT ); Wed, 21 Jan 2015 06:28:08 -0500 Message-ID: <54BF8D45.7030205@suse.cz> Date: Wed, 21 Jan 2015 12:28:05 +0100 From: Vlastimil Babka User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: "Aneesh Kumar K.V" , Andrew Morton CC: "Kirill A. Shutemov" , David Rientjes , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH V3] mm/thp: Allocate transparent hugepages on local node References: <1421393196-20915-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> <20150116160204.544e2bcf9627f5a4043ebf8d@linux-foundation.org> <54BD308A.4080905@suse.cz> <87fvb6uhfp.fsf@linux.vnet.ibm.com> <54BE1B00.3090102@suse.cz> In-Reply-To: <54BE1B00.3090102@suse.cz> Content-Type: text/plain; charset=iso-8859-2 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/20/2015 10:08 AM, Vlastimil Babka wrote: > On 01/20/2015 06:52 AM, Aneesh Kumar K.V wrote: >> Vlastimil Babka writes: >> >> is that check correct ? ie, >> >> if ((gfp & GFP_TRANSHUGE) == GFP_TRANSHUGE) >> >> may not always indicate transparent hugepage if defrag = 0 . With defrag >> cleared, we remove __GFP_WAIT from GFP_TRANSHUGE. > > Yep, that looks wrong. Sigh. I guess we can't spare an extra GFP flag to > indicate TRANSHUGE? I wanted to fix this in __alloc_pages_slowpath(), but actually there's no issue (other than being quite subtle) - if defrag == 0 and thus we don't have __GFP_WAIT, we reach "if (!wait) goto nopage;" and bail out before reaching the checks for GFP_TRANSHUGE. >> static inline gfp_t alloc_hugepage_gfpmask(int defrag, gfp_t extra_gfp) >> { >> return (GFP_TRANSHUGE & ~(defrag ? 0 : __GFP_WAIT)) | extra_gfp; >> } >> >> -aneesh >> >> -- >> 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 >> > > -- > 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 >