[PATCH] fixup! target/arm: Add sve-default-vector-length cpu property

Richard Henderson posted 1 patch 2 years, 9 months ago
Failed in applying to current master (apply log)
docs/system/arm/cpu-features.rst | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
[PATCH] fixup! target/arm: Add sve-default-vector-length cpu property
Posted by Richard Henderson 2 years, 9 months ago
---
 docs/system/arm/cpu-features.rst | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
index 79b87f7c5f..67847a5cc1 100644
--- a/docs/system/arm/cpu-features.rst
+++ b/docs/system/arm/cpu-features.rst
@@ -385,5 +385,6 @@ defined to mirror the Linux kernel parameter file
 is in units of bytes and must be between 16 and 8192.  
 If not specified, the default vector length is 64.
 
-If the default length is larger than the maximum vector length enabled
-with ``sve<N>`` properties, the actual vector length will be reduced.
+If the default length is larger than the maximum vector length enabled,
+the actual vector length will be reduced.  Note that the maximum vector
+length supported by QEMU is 256.
-- 
2.25.1


Re: [PATCH] fixup! target/arm: Add sve-default-vector-length cpu property
Posted by Peter Maydell 2 years, 9 months ago
On Mon, 26 Jul 2021 at 21:11, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> ---
>  docs/system/arm/cpu-features.rst | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/docs/system/arm/cpu-features.rst b/docs/system/arm/cpu-features.rst
> index 79b87f7c5f..67847a5cc1 100644
> --- a/docs/system/arm/cpu-features.rst
> +++ b/docs/system/arm/cpu-features.rst
> @@ -385,5 +385,6 @@ defined to mirror the Linux kernel parameter file
>  is in units of bytes and must be between 16 and 8192.
>  If not specified, the default vector length is 64.
>
> -If the default length is larger than the maximum vector length enabled
> -with ``sve<N>`` properties, the actual vector length will be reduced.
> +If the default length is larger than the maximum vector length enabled,
> +the actual vector length will be reduced.  Note that the maximum vector
> +length supported by QEMU is 256.

Thanks. I've squashed that into the patch in target-arm.next.

I plan to send this as part of a pullreq for rc1 tomorrow probably
midday-ish UK time, unless somebody speaks up now to say they'd
rather have more time for review/testing/whatever. (That wouldn't
mean it missed 6.1, as a bugfix-of-sorts this can go in rc2 if
necessary.)

-- PMM