Re: [PATCH 2/2] target/arm: Add sve-default-vector-length cpu property

[Richard Henderson]

On 7/15/21 11:14 PM, Peter Maydell wrote:
> > +#ifdef CONFIG_USER_ONLY
> > +/* Mirror linux /proc/sys/abi/sve_default_vector_length. */
>
> /proc/sys/abi/sve_default_vector_length wants a vector length in
> bytes, and it looks like we take a length in bits. I assume that's
> to match other places where the user can specify vector lengths,
> but we should mention the units we expect and that it's not what
> the kernel uses.

Oops, that wasn't intentional.

> We also don't support the kernel's (undocumented) "-1 means set
> to the maximum" behaviour -- do we need it, or is that more reasonably
> achievable by the user via other properties ?

I didn't notice that one either, possibly because it's undocumented.  Might as well 
support that too.


r~