From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754782AbbK0OkO (ORCPT ); Fri, 27 Nov 2015 09:40:14 -0500 Received: from mail-wm0-f43.google.com ([74.125.82.43]:33090 "EHLO mail-wm0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754599AbbK0OkK (ORCPT ); Fri, 27 Nov 2015 09:40:10 -0500 Date: Fri, 27 Nov 2015 16:40:06 +0200 From: "Kirill A. Shutemov" To: Vladimir Davydov Cc: Andrew Morton , Sergey Senozhatsky , Randy Dunlap , Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Linux MM Subject: Re: linux-next: Tree for Nov 27 (mm stuff) Message-ID: <20151127144006.GA15674@node.shutemov.name> References: <20151127160514.7b2022f2@canb.auug.org.au> <56580097.8050405@infradead.org> <20151127091047.GA585@swordfish> <20151127091739.GB585@swordfish> <20151127101640.GO29014@esperanza> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151127101640.GO29014@esperanza> User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 27, 2015 at 01:16:40PM +0300, Vladimir Davydov wrote: > On Fri, Nov 27, 2015 at 06:17:39PM +0900, Sergey Senozhatsky wrote: > > Cc Vladimir, Kirill, Andrew > > > > On (11/27/15 18:10), Sergey Senozhatsky wrote: > > > On (11/26/15 23:04), Randy Dunlap wrote: > > > > > > > > on i386: > > > > > > > > mm/built-in.o: In function `page_referenced_one': > > > > rmap.c:(.text+0x362a2): undefined reference to `pmdp_clear_flush_young' > > > > mm/built-in.o: In function `page_idle_clear_pte_refs_one': > > > > page_idle.c:(.text+0x4b2b8): undefined reference to `pmdp_test_and_clear_young' > > > > > > > > > > Hello, > > > > > > https://lkml.org/lkml/2015/11/24/160 > > > > > > corresponding patch mm-add-page_check_address_transhuge-helper-fix.patch added > > > to -mm tree. > > > > > > > my bad, it's in -next already. > > Sigh, this fails for me too :-( Kirill was right that this hack might > not always work. > > So, we still need to check explicitly if CONFIG_TRANSPARENT_HUGEPAGE is > enabled whenever we use page_check_address_transhuge, as Kirill proposed > initially. The patch below does the trick. The previous "fix" is still > useful though, because it reduces the size of kernels compiled w/o > tranparent huge page feature. > > Andrew, could you please merge this patch too? > > Sorry for all the trouble. > > Thanks, > Vladimir > --- > diff --git a/mm/page_idle.c b/mm/page_idle.c > index 374931f..aa7ca61 100644 > --- a/mm/page_idle.c > +++ b/mm/page_idle.c > @@ -66,7 +66,7 @@ static int page_idle_clear_pte_refs_one(struct page *page, > if (pte) { > referenced = ptep_clear_young_notify(vma, addr, pte); > pte_unmap(pte); > - } else > + } else if (IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE)) > referenced = pmdp_clear_young_notify(vma, addr, pmd); I would like to have yet another 'else' with warning just in case, as I proposed initially: https://lkml.kernel.org/g/20151124090930.GB15712@node.shutemov.name -- Kirill A. Shutemov From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kirill A. Shutemov" Subject: Re: linux-next: Tree for Nov 27 (mm stuff) Date: Fri, 27 Nov 2015 16:40:06 +0200 Message-ID: <20151127144006.GA15674@node.shutemov.name> References: <20151127160514.7b2022f2@canb.auug.org.au> <56580097.8050405@infradead.org> <20151127091047.GA585@swordfish> <20151127091739.GB585@swordfish> <20151127101640.GO29014@esperanza> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20151127101640.GO29014@esperanza> Sender: owner-linux-mm@kvack.org To: Vladimir Davydov Cc: Andrew Morton , Sergey Senozhatsky , Randy Dunlap , Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Linux MM List-Id: linux-next.vger.kernel.org On Fri, Nov 27, 2015 at 01:16:40PM +0300, Vladimir Davydov wrote: > On Fri, Nov 27, 2015 at 06:17:39PM +0900, Sergey Senozhatsky wrote: > > Cc Vladimir, Kirill, Andrew > > > > On (11/27/15 18:10), Sergey Senozhatsky wrote: > > > On (11/26/15 23:04), Randy Dunlap wrote: > > > > > > > > on i386: > > > > > > > > mm/built-in.o: In function `page_referenced_one': > > > > rmap.c:(.text+0x362a2): undefined reference to `pmdp_clear_flush_young' > > > > mm/built-in.o: In function `page_idle_clear_pte_refs_one': > > > > page_idle.c:(.text+0x4b2b8): undefined reference to `pmdp_test_and_clear_young' > > > > > > > > > > Hello, > > > > > > https://lkml.org/lkml/2015/11/24/160 > > > > > > corresponding patch mm-add-page_check_address_transhuge-helper-fix.patch added > > > to -mm tree. > > > > > > > my bad, it's in -next already. > > Sigh, this fails for me too :-( Kirill was right that this hack might > not always work. > > So, we still need to check explicitly if CONFIG_TRANSPARENT_HUGEPAGE is > enabled whenever we use page_check_address_transhuge, as Kirill proposed > initially. The patch below does the trick. The previous "fix" is still > useful though, because it reduces the size of kernels compiled w/o > tranparent huge page feature. > > Andrew, could you please merge this patch too? > > Sorry for all the trouble. > > Thanks, > Vladimir > --- > diff --git a/mm/page_idle.c b/mm/page_idle.c > index 374931f..aa7ca61 100644 > --- a/mm/page_idle.c > +++ b/mm/page_idle.c > @@ -66,7 +66,7 @@ static int page_idle_clear_pte_refs_one(struct page *page, > if (pte) { > referenced = ptep_clear_young_notify(vma, addr, pte); > pte_unmap(pte); > - } else > + } else if (IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE)) > referenced = pmdp_clear_young_notify(vma, addr, pmd); I would like to have yet another 'else' with warning just in case, as I proposed initially: https://lkml.kernel.org/g/20151124090930.GB15712@node.shutemov.name -- Kirill A. Shutemov -- 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