* 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. 
Also, it would be useful to know how the [0x02..0x1f] model 
range was chosen.

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/

Reply via email to