Re: [PATCH v11 35/40] kselftest/arm64: signal: Handle ZA signal context in core code

[Mark Brown <broonie@xxxxxxxxxx>]

On Mon, Feb 07, 2022 at 06:01:49PM -0700, Shuah Khan wrote:
> On 2/7/22 8:21 AM, Mark Brown wrote:

> > +		case ZA_MAGIC:
> > +			if (flags & ZA_CTX)
> > +				*err = "Multiple ZA_MAGIC";
> 
> Is this considered an error?

Yes.

> > @@ -165,6 +198,9 @@ bool validate_reserved(ucontext_t *uc, size_t resv_sz, char **err)
> >   		if (flags & SVE_CTX)
> >   			if (!validate_sve_context(sve, err))
> >   				return false;
> > +		if (flags & ZA_CTX)
> > +			if (!validate_za_context(za, err))

> If validate_za_context() finds an error, the above "Multiple ZA_MAGIC" will
> be overwritten? Is that the intent?

Both are errors.

Attachment: signature.asc
Description: PGP signature