Re: [PATCH 03/10] mm: tag reseve pages

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Andi Kleen
Date: Monday, August 6, 2007 - 1:18 pm

On Mon, Aug 06, 2007 at 12:10:53PM -0700, Andrew Morton wrote:

They can just use phys_to_nid() or equivalent instead. Putting the node
into the page flags is just a very minor optimization.  I doubt
actually you could benchmark the difference. While in theory the
hash lookup could be another cache miss in practice this should
be already hot since it's used elsewhere.


Really?  They forgot to document it then.

Anyways, if they don't have enough bits left they can always just
use the hash table and drop the node completely. Shouldn't make too much 
difference and IA64 has gobs of cache anyways.

I'm actually thinking about a PG_arch_2 on x86_64 too. arch_1 is already
used now but another one would be useful in c_p_a().

-Andi
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 03/10] mm: tag reseve pages, Peter Zijlstra, (Mon Aug 6, 3:29 am)
Re: [PATCH 03/10] mm: tag reseve pages, Christoph Lameter, (Mon Aug 6, 11:11 am)
Re: [PATCH 03/10] mm: tag reseve pages, Daniel Phillips, (Mon Aug 6, 11:13 am)
Re: [PATCH 03/10] mm: tag reseve pages, Peter Zijlstra, (Mon Aug 6, 11:28 am)
Re: [PATCH 03/10] mm: tag reseve pages, Christoph Lameter, (Mon Aug 6, 11:43 am)
Re: [PATCH 03/10] mm: tag reseve pages, Peter Zijlstra, (Mon Aug 6, 11:47 am)
Re: [PATCH 03/10] mm: tag reseve pages, Andi Kleen, (Mon Aug 6, 11:59 am)
Re: [PATCH 03/10] mm: tag reseve pages, Christoph Lameter, (Mon Aug 6, 12:09 pm)
Re: [PATCH 03/10] mm: tag reseve pages, Andrew Morton, (Mon Aug 6, 12:10 pm)
Re: [PATCH 03/10] mm: tag reseve pages, Christoph Lameter, (Mon Aug 6, 12:16 pm)
Re: [PATCH 03/10] mm: tag reseve pages, Andi Kleen, (Mon Aug 6, 12:34 pm)
Re: [PATCH 03/10] mm: tag reseve pages, Matt Mackall, (Mon Aug 6, 12:38 pm)
Re: [PATCH 03/10] mm: tag reseve pages, Andi Kleen, (Mon Aug 6, 1:18 pm)