[PATCH] arm64/idreg: Don't disable SME when disabling SME

Joey Gouly joey.gouly at arm.com
Thu Mar 16 08:32:36 PDT 2023


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 at 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 at kernel.org>
> 

Thanks,
Joey



More information about the linux-arm-kernel mailing list