Re: [PATCH v3 03/10] x86/mtrr: replace use_intel() with a local flag

2022-09-28 Thread Juergen Gross
On 12.09.22 11:10, Juergen Gross wrote: On 11.09.22 12:16, Borislav Petkov wrote: On Thu, Sep 08, 2022 at 10:49:07AM +0200, Juergen Gross wrote: diff --git a/arch/x86/include/asm/cacheinfo.h b/arch/x86/include/asm/cacheinfo.h index 86b2e0dcc4bf..1aeafa9888f7 100644 --- a/arch/x86/include/asm/ca

Re: [PATCH v3 03/10] x86/mtrr: replace use_intel() with a local flag

2022-09-19 Thread Borislav Petkov
On Mon, Sep 12, 2022 at 11:10:29AM +0200, Juergen Gross wrote: > In the end this variable doesn't specify which caching types are available, > but the ways to select/control the caching types. > > So what about "memory_caching_select" or "memory_caching_control" instead? _control sounds like the

Re: [PATCH v3 03/10] x86/mtrr: replace use_intel() with a local flag

2022-09-12 Thread Juergen Gross
On 11.09.22 12:16, Borislav Petkov wrote: On Thu, Sep 08, 2022 at 10:49:07AM +0200, Juergen Gross wrote: diff --git a/arch/x86/include/asm/cacheinfo.h b/arch/x86/include/asm/cacheinfo.h index 86b2e0dcc4bf..1aeafa9888f7 100644 --- a/arch/x86/include/asm/cacheinfo.h +++ b/arch/x86/include/asm/cach

[PATCH v3 03/10] x86/mtrr: replace use_intel() with a local flag

2022-09-08 Thread Juergen Gross
In MTRR code use_intel() is only used in one source file, and the relevant use_intel_if member of struct mtrr_ops is set only in generic_mtrr_ops. Replace use_intel() with a single flag in cacheinfo.c, which can be set when assigning generic_mtrr_ops to mtrr_if. This allows to drop use_intel_if fr