linux-kernel - Re: [2.6 patch] mm/page

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 5 May 2008 14:13:52 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	"KOSAKI Motohiro" <kosaki.motohiro@...fujitsu.com>
Cc:	bunk@...nel.org, linux-kernel@...r.kernel.org, peterz@...radead.org
Subject: Re: [2.6 patch] mm/page_alloc.c: cleanups

On Tue, 6 May 2008 05:59:31 +0900
"KOSAKI Motohiro" <kosaki.motohiro@...fujitsu.com> wrote:

> >  > >  -void free_cold_page(struct page *page)
> >  > >  -{
> >  > >  -       free_hot_cold_page(page, 1);
> >  > >  -}
> >  >
> >  > Wow, I didn't know nobody use cold page :)
> >
> >  that's a bug, surely?  I bet there are plenty of places which should be
> >  using this hint.
> 
> Hm, as far as I investigate
> 
> 1. nobody used free_cold_page() in kernel code tree.
> 2. free_cold_page() doesn't exported.
> 
> I investigated to 2.6.25-mm1.
> I don't understand yet why you think plenty....

Any place in the kernel which frees a page which is probably cache-cold
should be using this function.  (Including any place which caused the page
to be evicted from CPU cache by placing it under DMA).

But nobody thinks about providing the hint.

Probably it doesn't matter much at all, except in those few places which
free a lot of pages.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/