| Message ID | 20230315-arm64-override-sve-sme-v1-1-3514ad657548@kernel.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64/idreg: Don't disable SME when disabling SME | expand |
Hi!
You have a (funny) typo in the subject/commit message:
arm64/idreg: Don't disable SME when disabling SME
I guess the second 'SME' is meant to be 'SVE'!
On Thu, Mar 16, 2023 at 03:19:20PM +0000, Mark Brown wrote:
> SVE and SME are separate features which can be implemented without each
> other but currently if the user specifies arm64.nosve then we disable SME
> as well as SVE. There is already a separate override for SME so remove the
> implicit disablement from the SVE override.
>
> One usecase for this would be testing SME only support on a system which
> implements both SVE and SME.
>
> Signed-off-by: Mark Brown <broonie@kernel.org>
> ---
> arch/arm64/kernel/idreg-override.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/kernel/idreg-override.c b/arch/arm64/kernel/idreg-override.c
> index d833d78a7f31..9d8edeb6aa46 100644
> --- a/arch/arm64/kernel/idreg-override.c
> +++ b/arch/arm64/kernel/idreg-override.c
> @@ -167,7 +167,7 @@ static const struct {
> } aliases[] __initconst = {
> { "kvm-arm.mode=nvhe", "id_aa64mmfr1.vh=0" },
> { "kvm-arm.mode=protected", "id_aa64mmfr1.vh=0" },
> - { "arm64.nosve", "id_aa64pfr0.sve=0 id_aa64pfr1.sme=0" },
> + { "arm64.nosve", "id_aa64pfr0.sve=0" },
> { "arm64.nosme", "id_aa64pfr1.sme=0" },
> { "arm64.nobti", "id_aa64pfr1.bt=0" },
> { "arm64.nopauth",
>
> ---
> base-commit: eeac8ede17557680855031c6f305ece2378af326
> change-id: 20230315-arm64-override-sve-sme-b65f8dbbbcbe
>
> Best regards,
> --
> Mark Brown <broonie@kernel.org>
>
Thanks,
Joey
diff --git a/arch/arm64/kernel/idreg-override.c b/arch/arm64/kernel/idreg-override.c index d833d78a7f31..9d8edeb6aa46 100644 --- a/arch/arm64/kernel/idreg-override.c +++ b/arch/arm64/kernel/idreg-override.c @@ -167,7 +167,7 @@ static const struct { } aliases[] __initconst = { { "kvm-arm.mode=nvhe", "id_aa64mmfr1.vh=0" }, { "kvm-arm.mode=protected", "id_aa64mmfr1.vh=0" }, - { "arm64.nosve", "id_aa64pfr0.sve=0 id_aa64pfr1.sme=0" }, + { "arm64.nosve", "id_aa64pfr0.sve=0" }, { "arm64.nosme", "id_aa64pfr1.sme=0" }, { "arm64.nobti", "id_aa64pfr1.bt=0" }, { "arm64.nopauth",
SVE and SME are separate features which can be implemented without each other but currently if the user specifies arm64.nosve then we disable SME as well as SVE. There is already a separate override for SME so remove the implicit disablement from the SVE override. One usecase for this would be testing SME only support on a system which implements both SVE and SME. Signed-off-by: Mark Brown <broonie@kernel.org> --- arch/arm64/kernel/idreg-override.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- base-commit: eeac8ede17557680855031c6f305ece2378af326 change-id: 20230315-arm64-override-sve-sme-b65f8dbbbcbe Best regards,