* Andre Przywara <[email protected]> wrote:
> On 10/24/2012 12:46 PM, Ingo Molnar wrote:
> >
> >* Andre Przywara <[email protected]> wrote:
> >
> >>The WAF may hurt the performance of some workloads, caused by
> >>aliasing issues in the L1 cache.
> >>Disable it on the affected CPUs.
> >>
> >>Signed-off-by: Andre Przywara <[email protected]>
> >>---
> >> arch/x86/kernel/cpu/amd.c | 14 ++++++++++++++
> >> 1 file changed, 14 insertions(+)
> >>
> >>diff --git a/arch/x86/kernel/cpu/amd.c b/arch/x86/kernel/cpu/amd.c
> >>index f7e98a2..1b7d165 100644
> >>--- a/arch/x86/kernel/cpu/amd.c
> >>+++ b/arch/x86/kernel/cpu/amd.c
> >>@@ -631,6 +631,20 @@ static void __cpuinit init_amd(struct cpuinfo_x86 *c)
> >> }
> >> }
> >>
> >>+ /*
> >>+ * The way access filter has a performance penalty on some workloads.
> >>+ * Disable it on the affected CPUs.
> >>+ */
> >>+ if ((c->x86 == 0x15) &&
> >>+ (c->x86_model >= 0x02) && (c->x86_model < 0x20)) {
> >>+ u64 val;
> >>+
> >>+ if (!rdmsrl_safe(0xc0011021, &val) && !(val & 0x1E)) {
> >>+ val |= 0x1E;
> >>+ wrmsrl_safe(0xc0011021, val);
> >>+ }
> >>+ }
> >
> >Would be nice to hear more about the background of this change,
> >about the amount of 'penalty' and the nature of the workloads.
>
> The issue is similar to that one of last year:
> http://lkml.indiana.edu/hypermail/linux/kernel/1107.3/00041.html
> That new patch does not replace the old one, we just need another
> quirk for newer CPUs.
>
> The penalty depends on the circumstances, if you need a number, I
> can only tell you the 3% from last year.
>
> The workloads would be those that access code from the same physical
> page under different virtual addresses, so different processes using
> the same libraries with ASLR or multiple instances of PIE-binaries.
> The code needs to be accessed simultaneously from both cores of the
> same compute unit.
>
> More details here:
> http://developer.amd.com/Assets/SharedL1InstructionCacheonAMD15hCPU.pdf
>
> >Also, it would be useful to know how the [0x02..0x1f] model
> >range was chosen.
>
> That is everything that has the core known as Piledriver.
> That includes the AMD A-Series (aka Trinity) and the just released
> new CPUs of the FX-Series (aka Vishera).
> The model numbering is a bit odd here: FX CPUs have model 2,
> A-Series has model 10h, with possible extensions to 1Fh. Hence the
> range of model ids.
> Model 2 CPUs should have the fix in BIOS, but I guess there are
> quite some boxes out there which will not be updated. Better safe
> than sorry.
Most of this info makes a perfect changelog entry - mind sending
an updated v2 patch with this information included?
Thanks,
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
