* [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
@ 2021-01-04 11:38 ` Shannon Zhao
0 siblings, 0 replies; 8+ messages in thread
From: Shannon Zhao @ 2021-01-04 11:38 UTC (permalink / raw)
To: kvmarm, maz, will, anshuman.khandual, suzuki.poulose; +Cc: linux-arm-kernel
Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST") deletes
CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
Just remove CONFIG_KVM_ARM_HOST here.
Signed-off-by: Shannon Zhao <shannon.zhao@linux.alibaba.com>
---
arch/arm64/kernel/cpufeature.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c
index 7ffb5f1..e99edde 100644
--- a/arch/arm64/kernel/cpufeature.c
+++ b/arch/arm64/kernel/cpufeature.c
@@ -2568,7 +2568,7 @@ static void verify_hyp_capabilities(void)
int parange, ipa_max;
unsigned int safe_vmid_bits, vmid_bits;
- if (!IS_ENABLED(CONFIG_KVM) || !IS_ENABLED(CONFIG_KVM_ARM_HOST))
+ if (!IS_ENABLED(CONFIG_KVM))
return;
safe_mmfr1 = read_sanitised_ftr_reg(SYS_ID_AA64MMFR1_EL1);
--
1.8.3.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
2021-01-04 11:38 ` Shannon Zhao
@ 2021-01-04 12:40 ` Marc Zyngier
-1 siblings, 0 replies; 8+ messages in thread
From: Marc Zyngier @ 2021-01-04 12:40 UTC (permalink / raw)
To: Shannon Zhao; +Cc: will, kvmarm, linux-arm-kernel, anshuman.khandual
On 2021-01-04 11:38, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST")
> deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
>
> Signed-off-by: Shannon Zhao <shannon.zhao@linux.alibaba.com>
> ---
> arch/arm64/kernel/cpufeature.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/kernel/cpufeature.c
> b/arch/arm64/kernel/cpufeature.c
> index 7ffb5f1..e99edde 100644
> --- a/arch/arm64/kernel/cpufeature.c
> +++ b/arch/arm64/kernel/cpufeature.c
> @@ -2568,7 +2568,7 @@ static void verify_hyp_capabilities(void)
> int parange, ipa_max;
> unsigned int safe_vmid_bits, vmid_bits;
>
> - if (!IS_ENABLED(CONFIG_KVM) || !IS_ENABLED(CONFIG_KVM_ARM_HOST))
> + if (!IS_ENABLED(CONFIG_KVM))
> return;
>
> safe_mmfr1 = read_sanitised_ftr_reg(SYS_ID_AA64MMFR1_EL1);
Well spotted. It means this function has never run since v5.8.
It only performs a sanity check, but still...
Thanks,
M.
--
Jazz is not dead. It just smells funny...
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
@ 2021-01-04 12:40 ` Marc Zyngier
0 siblings, 0 replies; 8+ messages in thread
From: Marc Zyngier @ 2021-01-04 12:40 UTC (permalink / raw)
To: Shannon Zhao
Cc: suzuki.poulose, will, kvmarm, linux-arm-kernel, anshuman.khandual
On 2021-01-04 11:38, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST")
> deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
>
> Signed-off-by: Shannon Zhao <shannon.zhao@linux.alibaba.com>
> ---
> arch/arm64/kernel/cpufeature.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/kernel/cpufeature.c
> b/arch/arm64/kernel/cpufeature.c
> index 7ffb5f1..e99edde 100644
> --- a/arch/arm64/kernel/cpufeature.c
> +++ b/arch/arm64/kernel/cpufeature.c
> @@ -2568,7 +2568,7 @@ static void verify_hyp_capabilities(void)
> int parange, ipa_max;
> unsigned int safe_vmid_bits, vmid_bits;
>
> - if (!IS_ENABLED(CONFIG_KVM) || !IS_ENABLED(CONFIG_KVM_ARM_HOST))
> + if (!IS_ENABLED(CONFIG_KVM))
> return;
>
> safe_mmfr1 = read_sanitised_ftr_reg(SYS_ID_AA64MMFR1_EL1);
Well spotted. It means this function has never run since v5.8.
It only performs a sanity check, but still...
Thanks,
M.
--
Jazz is not dead. It just smells funny...
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
2021-01-04 11:38 ` Shannon Zhao
@ 2021-01-05 12:48 ` Catalin Marinas
-1 siblings, 0 replies; 8+ messages in thread
From: Catalin Marinas @ 2021-01-05 12:48 UTC (permalink / raw)
To: Shannon Zhao; +Cc: maz, anshuman.khandual, will, kvmarm, linux-arm-kernel
On Mon, Jan 04, 2021 at 07:38:44PM +0800, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST") deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
>
> Signed-off-by: Shannon Zhao <shannon.zhao@linux.alibaba.com>
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
@ 2021-01-05 12:48 ` Catalin Marinas
0 siblings, 0 replies; 8+ messages in thread
From: Catalin Marinas @ 2021-01-05 12:48 UTC (permalink / raw)
To: Shannon Zhao
Cc: suzuki.poulose, maz, anshuman.khandual, will, kvmarm,
linux-arm-kernel
On Mon, Jan 04, 2021 at 07:38:44PM +0800, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST") deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
>
> Signed-off-by: Shannon Zhao <shannon.zhao@linux.alibaba.com>
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
2021-01-04 11:38 ` Shannon Zhao
@ 2021-01-05 13:23 ` Marc Zyngier
-1 siblings, 0 replies; 8+ messages in thread
From: Marc Zyngier @ 2021-01-05 13:23 UTC (permalink / raw)
To: will, anshuman.khandual, suzuki.poulose, kvmarm, Shannon Zhao
Cc: linux-arm-kernel
On Mon, 4 Jan 2021 19:38:44 +0800, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST") deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
Applied to next, thanks!
[1/1] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
commit: 45ba7b195a369f35cb39094fdb32efe5908b34ad
Cheers,
M.
--
Without deviation from the norm, progress is not possible.
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
@ 2021-01-05 13:23 ` Marc Zyngier
0 siblings, 0 replies; 8+ messages in thread
From: Marc Zyngier @ 2021-01-05 13:23 UTC (permalink / raw)
To: will, anshuman.khandual, suzuki.poulose, kvmarm, Shannon Zhao
Cc: linux-arm-kernel
On Mon, 4 Jan 2021 19:38:44 +0800, Shannon Zhao wrote:
> Commit d82755b2e781 ("KVM: arm64: Kill off CONFIG_KVM_ARM_HOST") deletes
> CONFIG_KVM_ARM_HOST option, it should use CONFIG_KVM instead.
>
> Just remove CONFIG_KVM_ARM_HOST here.
Applied to next, thanks!
[1/1] arm64: cpufeature: remove non-exist CONFIG_KVM_ARM_HOST
commit: 45ba7b195a369f35cb39094fdb32efe5908b34ad
Cheers,
M.
--
Without deviation from the norm, progress is not possible.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread