qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] fixup! target/arm: Add sve-default-vector-length cpu property
@ 2021-07-26 20:11 Richard Henderson
  2021-07-26 20:22 ` Peter Maydell
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Henderson @ 2021-07-26 20:11 UTC (permalink / raw)
  To: qemu-devel; +Cc: peter.maydell, drjones

---
 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



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] fixup! target/arm: Add sve-default-vector-length cpu property
  2021-07-26 20:11 [PATCH] fixup! target/arm: Add sve-default-vector-length cpu property Richard Henderson
@ 2021-07-26 20:22 ` Peter Maydell
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Maydell @ 2021-07-26 20:22 UTC (permalink / raw)
  To: Richard Henderson; +Cc: Andrew Jones, QEMU Developers

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


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-26 20:23 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-26 20:11 [PATCH] fixup! target/arm: Add sve-default-vector-length cpu property Richard Henderson
2021-07-26 20:22 ` Peter Maydell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).