Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1278831

Re: linux-next: Tree for Nov 27 (mm stuff)

Show all headers | View raw

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 from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

linux-next: Tree for Nov 27 Stephen Rothwell <sfr@canb.auug.org.au> - 2015-11-27 06:10 +0100
  Re: linux-next: Tree for Nov 27 (mm stuff) Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2015-11-27 10:20 +0100
    Re: linux-next: Tree for Nov 27 (mm stuff) Vladimir Davydov <vdavydov@virtuozzo.com> - 2015-11-27 11:20 +0100
      Re: linux-next: Tree for Nov 27 (mm stuff) "Kirill A. Shutemov" <kirill@shutemov.name> - 2015-11-27 15:50 +0100
  Re: linux-next: Tree for Nov 27 (mm stuff) Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> - 2015-11-27 10:20 +0100

csiph-web