Re: [PATCH][retry 2] Conform L3 Cache Index Disable toLinuxstandards From: Eric Lammerts <eric@lammerts.org>

From: Ingo Molnar
Date: Mon Mar 02 2009 - 15:02:43 EST



* Langsdorf, Mark <mark.langsdorf@xxxxxxx> wrote:

> > > > Add ABI Documentation entry and fix some /sys directory formating
> > > > issues with the L3 Cache Index Disable feature for future AMD
> > > > processors. __Add a check to disable it for family 0x10 models
> > > > that do not support it yet.
> > >
> > > x86_64 allnoconfig:
> > >
> > > arch/x86/kernel/cpu/intel_cacheinfo.c:556: warning:
> > 'free_cache_attributes' defined but not used
> > > arch/x86/kernel/cpu/intel_cacheinfo.c:596: warning:
> > 'detect_cache_attributes' defined but not used
>
> My code doesn't modify either of these functions, so
> I'm not causing the regression.

The more serious problem is what i reported, that your patch
triggers this build failure:

arch/x86/kernel/built-in.o: In function `store_cache_disable':
intel_cacheinfo.c:(.text+0xd322): undefined reference to `k8_northbridges'
arch/x86/kernel/built-in.o: In function `show_cache_disable':
intel_cacheinfo.c:(.text+0xd52c): undefined reference to `k8_northbridges'

so i've excluded it from tip:master for now. Please resubmit a
fixed patch.

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