| Message ID | 1526638022-4137-4-git-send-email-vladimir.murzin@arm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, May 18, 2018 at 11:07:02AM +0100, Vladimir Murzin wrote: > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index 11fc28e..8f59d47 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -2636,6 +2636,10 @@ > > noclflush [BUGS=X86] Don't use the CLFLUSH instruction > > + nocnp [ARM64] > + Disable CNP (Common not Private translations) > + even if it is supported by processor. > + > nodelayacct [KNL] Disable per-task delay accounting > > nodsp [SH] Disable hardware DSP at boot time. Do we actually have a use-case for this command line option? I'm not considering hardware errata as these are handled separately in the kernel.
On 23/05/18 18:17, Catalin Marinas wrote: > On Fri, May 18, 2018 at 11:07:02AM +0100, Vladimir Murzin wrote: >> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt >> index 11fc28e..8f59d47 100644 >> --- a/Documentation/admin-guide/kernel-parameters.txt >> +++ b/Documentation/admin-guide/kernel-parameters.txt >> @@ -2636,6 +2636,10 @@ >> >> noclflush [BUGS=X86] Don't use the CLFLUSH instruction >> >> + nocnp [ARM64] >> + Disable CNP (Common not Private translations) >> + even if it is supported by processor. >> + >> nodelayacct [KNL] Disable per-task delay accounting >> >> nodsp [SH] Disable hardware DSP at boot time. > > Do we actually have a use-case for this command line option? I'm not > considering hardware errata as these are handled separately in the > kernel. > Well, I cannot count all cases, yet we might see CnP support advertised by CPU via ID register (where CPU meant to be part of bL) but not really doing optimisations in hardware. Probably, some userspace (benchmarks) might not benefit of CnP; otoh maybe better way for such case would be user-space asking kernel to {dis,en}able CnP... I have no strong opinion on patch, so I'm fine to drop it and come back when/if we get results from real hardware. Cheers Vladimir
Hi Vladimir, Catalin, On 24/05/18 09:20, Vladimir Murzin wrote: > On 23/05/18 18:17, Catalin Marinas wrote: >> On Fri, May 18, 2018 at 11:07:02AM +0100, Vladimir Murzin wrote: >>> diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt >>> index 11fc28e..8f59d47 100644 >>> --- a/Documentation/admin-guide/kernel-parameters.txt >>> +++ b/Documentation/admin-guide/kernel-parameters.txt >>> @@ -2636,6 +2636,10 @@ >>> + nocnp [ARM64] >>> + Disable CNP (Common not Private translations) >>> + even if it is supported by processor. >> Do we actually have a use-case for this command line option? I'm not >> considering hardware errata as these are handled separately in the >> kernel. > Well, I cannot count all cases, yet we might see CnP support advertised > by CPU via ID register (where CPU meant to be part of bL) but not really > doing optimisations in hardware. > > Probably, some userspace (benchmarks) might not benefit of CnP; otoh maybe > better way for such case would be user-space asking kernel to {dis,en}able > CnP... > > I have no strong opinion on patch, so I'm fine to drop it and come back > when/if we get results from real hardware. We may want to disable CNP without rebuilding the kernel, which would also have the affect of code-layout changes... Thanks, James
diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index 11fc28e..8f59d47 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt @@ -2636,6 +2636,10 @@ noclflush [BUGS=X86] Don't use the CLFLUSH instruction + nocnp [ARM64] + Disable CNP (Common not Private translations) + even if it is supported by processor. + nodelayacct [KNL] Disable per-task delay accounting nodsp [SH] Disable hardware DSP at boot time. diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c index 199e9dd..eee6a31 100644 --- a/arch/arm64/kernel/cpufeature.c +++ b/arch/arm64/kernel/cpufeature.c @@ -860,6 +860,15 @@ static bool has_cache_dic(const struct arm64_cpu_capabilities *entry, return read_sanitised_ftr_reg(SYS_CTR_EL0) & BIT(CTR_DIC_SHIFT); } +static bool nocnp; + +static int __init early_nocnp(char *p) +{ + nocnp = true; + return 0; +} +early_param("nocnp", early_nocnp); + static bool __maybe_unused has_useable_cnp(const struct arm64_cpu_capabilities *entry, int scope) { @@ -867,7 +876,7 @@ has_useable_cnp(const struct arm64_cpu_capabilities *entry, int scope) if (elfcorehdr_size) return false; #endif - return has_cpuid_feature(entry, scope); + return has_cpuid_feature(entry, scope) && !nocnp; } #ifdef CONFIG_UNMAP_KERNEL_AT_EL0
There are cases when activating of Common Not Private (CNP) feature might not be desirable; this patch allows to forcefully disable CNP even it is supported by hardware. Signed-off-by: Vladimir Murzin <vladimir.murzin@arm.com> --- Documentation/admin-guide/kernel-parameters.txt | 4 ++++ arch/arm64/kernel/cpufeature.c | 11 ++++++++++- 2 files changed, 14 insertions(+), 1 deletion(-)