Re: [PATCH] - Improve SN2 TLB flushing algorithms

From: Jack Steiner <>
Date: 2004-01-29 12:13:42
On Thu, Jan 29, 2004 at 10:57:49AM +1100, Peter Chubb wrote:
> >>>>> "Jack" == Jack Steiner <> writes:
> Jack> On Wed, Jan 28, 2004 at 09:17:59PM +0000, Christoph Hellwig
> Jack> wrote:
> >> On Wed, Jan 28, 2004 at 02:59:12PM -0600, Jack Steiner wrote: >
> >> +#ifdef CONFIG_NUMA > + cpus_clear(mm->cpu_vm_mask); > +#endif
> >> 
> >> I really hate this ifdefs all over the place.  Does this really
> >> hurt that much on non-NUMA systems?  Also SN2 seems to use this
> >> code always so CONFIG_NUMA looks like the wrong ifdef to me.
> Jack> Are you suggesting that we remove the #ifdef OR hide the code in
> Jack> a function that, depending on config options, may or may not do
> Jack> anything?

I dont have a strong opinion. Either is acceptible to me.
David suggested removing the #ifdef iff we can move cpu_vm_mask
closer to the mm_context_t. I'll pursue this....

> Jack> The code is needed on kernels built for SN2. That includes both
> Jack> CONFIG_IA64_GENERIC & CONFIG_IA64_SGI_SN2. I agree that
> Jack> CONFIG_NUMA is not a great choice but nothing else seemed
> Jack> appropriate.
> As CONFIG_IA64_GENERIC is a superset of CONFIG_IA64_SGI_SN2
> why not make CONFIG_IA64_GENERIC turn on CONFIG_IA64_SGI_SN2   and
> then only test for the latter.

This wont work with the current usage of the CONFIG options. For 
	# if defined (CONFIG_IA64_HP_SIM)
	#  include <asm/machvec_hpsim.h>
	# elif defined (CONFIG_IA64_DIG)
	#  include <asm/machvec_dig.h>
	# elif defined (CONFIG_IA64_HP_ZX1)
	#  include <asm/machvec_hpzx1.h>
	# elif defined (CONFIG_IA64_SGI_SN2)
	#  include <asm/machvec_sn2.h>
	# elif defined (CONFIG_IA64_GENERIC)

(It sounded like a good idea though. Most but not all places that check
CONFIG_IA64_SGI_SN2 also check for CONFIG_IA64_GENERIC).

> And yes, using a function that disappears or does nothing on platforms
> for which it is inappropriate is a better approach than littering
> #ifdefs though the code.
> --
> Dr Peter Chubb  peterc AT
> The technical we do immediately,  the political takes *forever*


Jack Steiner (          651-683-5302
Principal Engineer                      SGI - Silicon Graphics, Inc.

To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to
More majordomo info at
Received on Wed Jan 28 20:17:37 2004

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