Re: [PATCH] fix CONFIG_DISCONTIGMEM and CONFIG_VIRTUAL_MEMMAP

From: Ian Wienand <ianw_at_gelato.unsw.edu.au>
Date: 2004-09-06 12:11:30
On Sun, Sep 05, 2004 at 08:33:46AM -0700, Jesse Barnes wrote:
> Looks like the patch that fixed Ian's problem introduced an ugly warning.  
> Maybe we could just switch to requiring CONFIG_DISCONTIGMEM and 
> CONFIG_VIRTUAL_MEMMAP for ia64 in general?  That would let us kill a lot of 
> code and only have one code path, etc.  Benchmarks have shown that the 
> overhead of CONFIG_DISCONTIGMEM is very difficult to measure (if you look at 
> the code, the only added overhead in any fast path is an additional pointer 
> dereference in alloc_pages).  CONFIG_VIRTUAL_MEMMAP is a little more 
> expensive iirc, but still difficult to measure.

Guilty as charged, I only checked it on my machine.

If performance isn't an issue [*], for mine, combining the the two
seems like the most logical thing to do to me, as it is only causing
confusion with the two being apart.  Setting myself the simple goal of
running with discontig and virtual mem map without NUMA, the diffstat
of the patch looks like

 Makefile                     |    2
 arch/ia64/Kconfig            |    4
 arch/ia64/mm/discontig.c     |  180 ++-----------------------------------------
 arch/ia64/mm/numa.c          |  171 ++++++++++++++++++++++++++++++++++++++++
 include/asm-ia64/meminit.h   |    2
 include/asm-ia64/processor.h |    4
 6 files changed, 189 insertions(+), 174 deletions(-)

Most of that is copy-paste moving of a few things from discontig.c to
in numa.c.  My point is that it probably isn't too hard (but I guess
that's probably not news).

-i

[*] I'm always happy to run any benchmarks you want on the machines we
have.

-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Received on Sun Sep 5 22:12:12 2004

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:30 EST