qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more
@ 2025-02-13  8:42 Kashyap Chamarthy
  2025-02-17 13:23 ` Peter Maydell
  0 siblings, 1 reply; 5+ messages in thread
From: Kashyap Chamarthy @ 2025-02-13  8:42 UTC (permalink / raw)
  To: qemu-devel
  Cc: maz, Alex Bennée, Steven Lee, Yi Liu, Cédric Le Goater,
	Andrew Jeffery, Alistair Francis, Tyrone Ting, Jamin Lin,
	Troy Lee, Peter Maydell, qemu-arm, Alexandre Iooss, Joel Stanley,
	Edgar E. Iglesias, Zhenzhong Duan, Hao Wu, sebott, Eric Auger,
	Ninad Palsule, Kashyap Chamarthy

In v2:

  - Add live-migration context to the PAuth docs (Marc Zyngier)

  - Fix the Arm capitlalization (Peter Maydell)
    - Context here:
      (https://lists.gnu.org/archive/html/qemu-devel/2025-01/msg05137.html)

    * * *

v1 cover letter:

One is a trivial, mechanical change to consistenlty use "vCPU".  The
other updates some details about the "PAuth" (Pointer Authentication)
feature.

I replaced the "TCG vCPU Features" heading with "PAuth" because of this:
before this change, the section says, it is about "CPU features that are
specific to TCG".  But it has only PAuth-related parameters under it.
Since PAuth is relevant to both KVM and TCG, I moved them under a
separate PAuth section, instead of duplicating it.

But now we have a small inconsistency - there's a KVM-only CPU features
section, but no TCG-only section.  I thought when there are more
TCG-only CPU features, that section can be added back in.  Or I can add
that back in, if anyone feels strongly about it.

Kashyap Chamarthy (3):
  docs/cpu-features: Consistently use vCPU instead of VCPU
  docs/cpu-features: Update "PAuth" (Pointer Authentication) details
  docs: Fix "Arm" capitaliaztion

 docs/devel/testing/qgraph.rst        |  8 ++--
 docs/devel/vfio-iommufd.rst          |  2 +-
 docs/specs/fsi.rst                   |  2 +-
 docs/system/arm/aspeed.rst           |  6 +--
 docs/system/arm/b-l475e-iot01a.rst   |  2 +-
 docs/system/arm/cpu-features.rst     | 60 +++++++++++++++++++++++-----
 docs/system/arm/nrf.rst              |  4 +-
 docs/system/arm/nuvoton.rst          |  4 +-
 docs/system/arm/stm32.rst            | 12 +++---
 docs/system/arm/xlnx-versal-virt.rst | 12 +++---
 docs/system/arm/xlnx-zynq.rst        |  2 +-
 docs/system/guest-loader.rst         |  2 +-
 12 files changed, 77 insertions(+), 39 deletions(-)

-- 
2.48.1



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

* [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more
@ 2025-02-13 13:50 Kashyap Chamarthy
  0 siblings, 0 replies; 5+ messages in thread
From: Kashyap Chamarthy @ 2025-02-13 13:50 UTC (permalink / raw)
  To: qemu-devel
  Cc: maz, Joel Stanley, Ninad Palsule, qemu-arm, Andrew Jeffery,
	Peter Maydell, Alexandre Iooss, Jamin Lin, Cédric Le Goater,
	Edgar E. Iglesias, Eric Auger, Yi Liu, Hao Wu, Tyrone Ting,
	sebott, Steven Lee, Zhenzhong Duan, Alex Bennée, Troy Lee,
	Alistair Francis, Kashyap Chamarthy

In v2:

  - Add live-migration context to the PAuth docs (Marc Zyngier)

  - Fix the Arm capitlalization (Peter Maydell)
    (See:
    https://lists.gnu.org/archive/html/qemu-devel/2025-01/msg05137.html)

    * * *

v1 cover letter:

One is a trivial, mechanical change to consistenlty use "vCPU".  The
other updates some details about the "PAuth" (Pointer Authentication)
feature.

I replaced the "TCG vCPU Features" heading with "PAuth" because of this:
before this change, the section says, it is about "CPU features that are
specific to TCG".  But it has only PAuth-related parameters under it.
Since PAuth is relevant to both KVM and TCG, I moved them under a
separate PAuth section, instead of duplicating it.

But now we have a small inconsistency - there's a KVM-only CPU features
section, but no TCG-only section.  I thought when there are more
TCG-only CPU features, that section can be added back in.  Or I can add
that back in, if anyone feels strongly about it.

Kashyap Chamarthy (3):
  docs/cpu-features: Consistently use vCPU instead of VCPU
  docs/cpu-features: Update "PAuth" (Pointer Authentication) details
  docs: Fix "Arm" capitaliaztion

 docs/devel/testing/qgraph.rst        |  8 ++--
 docs/devel/vfio-iommufd.rst          |  2 +-
 docs/specs/fsi.rst                   |  2 +-
 docs/system/arm/aspeed.rst           |  6 +--
 docs/system/arm/b-l475e-iot01a.rst   |  2 +-
 docs/system/arm/cpu-features.rst     | 60 +++++++++++++++++++++++-----
 docs/system/arm/nrf.rst              |  4 +-
 docs/system/arm/nuvoton.rst          |  4 +-
 docs/system/arm/stm32.rst            | 12 +++---
 docs/system/arm/xlnx-versal-virt.rst | 12 +++---
 docs/system/arm/xlnx-zynq.rst        |  2 +-
 docs/system/guest-loader.rst         |  2 +-
 12 files changed, 77 insertions(+), 39 deletions(-)

-- 
2.48.1



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

* Re: [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more
  2025-02-13  8:42 [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more Kashyap Chamarthy
@ 2025-02-17 13:23 ` Peter Maydell
  2025-02-17 16:41   ` Kashyap Chamarthy
  0 siblings, 1 reply; 5+ messages in thread
From: Peter Maydell @ 2025-02-17 13:23 UTC (permalink / raw)
  To: Kashyap Chamarthy
  Cc: qemu-devel, maz, Alex Bennée, Steven Lee, Yi Liu,
	Cédric Le Goater, Andrew Jeffery, Alistair Francis,
	Tyrone Ting, Jamin Lin, Troy Lee, qemu-arm, Alexandre Iooss,
	Joel Stanley, Edgar E. Iglesias, Zhenzhong Duan, Hao Wu, sebott,
	Eric Auger, Ninad Palsule

On Thu, 13 Feb 2025 at 08:44, Kashyap Chamarthy <kchamart@redhat.com> wrote:
>
> In v2:
>
>   - Add live-migration context to the PAuth docs (Marc Zyngier)
>
>   - Fix the Arm capitlalization (Peter Maydell)
>     - Context here:
>       (https://lists.gnu.org/archive/html/qemu-devel/2025-01/msg05137.html)


> Kashyap Chamarthy (3):

Hi -- it looks like only patches 1 and 2 ever made it to
the list. Would you mind resending, please?

>   docs/cpu-features: Consistently use vCPU instead of VCPU
>   docs/cpu-features: Update "PAuth" (Pointer Authentication) details
>   docs: Fix "Arm" capitaliaztion

(nit: "capitalization")

thanks
-- PMM


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

* [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more
@ 2025-02-17 16:37 Kashyap Chamarthy
  0 siblings, 0 replies; 5+ messages in thread
From: Kashyap Chamarthy @ 2025-02-17 16:37 UTC (permalink / raw)
  To: qemu-devel
  Cc: Ninad Palsule, sebott, maz, Andrew Jeffery, Alistair Francis,
	Edgar E. Iglesias, Tyrone Ting, Hao Wu, Zhenzhong Duan,
	Alex Bennée, Peter Maydell, Cédric Le Goater,
	Steven Lee, Troy Lee, Joel Stanley, Eric Auger, Jamin Lin, Yi Liu,
	qemu-arm, Alexandre Iooss, Kashyap Chamarthy

[
    Resending v2, Peter pointed out that only patches 1 and 2 made it to
    the list; so I'm re-sending with a commit message touch-up:
    s/capitalization/capitaliaztion.

    Alex Bennée reviewed the first two patches that did show up:
    - https://lists.nongnu.org/archive/html/qemu-devel/2025-02/msg02838.html
    - https://lists.nongnu.org/archive/html/qemu-devel/2025-02/msg02837.html
]

In v2:

  - Add live-migration context to the PAuth docs (Marc Zyngier)

  - Fix the Arm capitlalization (Peter Maydell)
    (See:
    https://lists.gnu.org/archive/html/qemu-devel/2025-01/msg05137.html)

    * * *

v1 cover letter:

One is a trivial, mechanical change to consistenlty use "vCPU".  The
other updates some details about the "PAuth" (Pointer Authentication)
feature.

I replaced the "TCG vCPU Features" heading with "PAuth" because of this:
before this change, the section says, it is about "CPU features that are
specific to TCG".  But it has only PAuth-related parameters under it.
Since PAuth is relevant to both KVM and TCG, I moved them under a
separate PAuth section, instead of duplicating it.

But now we have a small inconsistency - there's a KVM-only CPU features
section, but no TCG-only section.  I thought when there are more
TCG-only CPU features, that section can be added back in.  Or I can add
that back in, if anyone feels strongly about it.

Kashyap Chamarthy (3):
  docs/cpu-features: Consistently use vCPU instead of VCPU
  docs/cpu-features: Update "PAuth" (Pointer Authentication) details
  docs: Fix "Arm" capitalization

 docs/devel/testing/qgraph.rst        |  8 ++--
 docs/devel/vfio-iommufd.rst          |  2 +-
 docs/specs/fsi.rst                   |  2 +-
 docs/system/arm/aspeed.rst           |  6 +--
 docs/system/arm/b-l475e-iot01a.rst   |  2 +-
 docs/system/arm/cpu-features.rst     | 60 +++++++++++++++++++++++-----
 docs/system/arm/nrf.rst              |  4 +-
 docs/system/arm/nuvoton.rst          |  4 +-
 docs/system/arm/stm32.rst            | 12 +++---
 docs/system/arm/xlnx-versal-virt.rst | 12 +++---
 docs/system/arm/xlnx-zynq.rst        |  2 +-
 docs/system/guest-loader.rst         |  2 +-
 12 files changed, 77 insertions(+), 39 deletions(-)

-- 
2.48.1



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

* Re: [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more
  2025-02-17 13:23 ` Peter Maydell
@ 2025-02-17 16:41   ` Kashyap Chamarthy
  0 siblings, 0 replies; 5+ messages in thread
From: Kashyap Chamarthy @ 2025-02-17 16:41 UTC (permalink / raw)
  To: Peter Maydell
  Cc: qemu-devel, maz, Alex Bennée, Steven Lee, Yi Liu,
	Cédric Le Goater, Andrew Jeffery, Alistair Francis,
	Tyrone Ting, Jamin Lin, Troy Lee, qemu-arm, Alexandre Iooss,
	Joel Stanley, Edgar E. Iglesias, Zhenzhong Duan, Hao Wu, sebott,
	Eric Auger, Ninad Palsule

On Mon, Feb 17, 2025 at 01:23:51PM +0000, Peter Maydell wrote:
> On Thu, 13 Feb 2025 at 08:44, Kashyap Chamarthy <kchamart@redhat.com> wrote:
> >
> > In v2:
> >
> >   - Add live-migration context to the PAuth docs (Marc Zyngier)
> >
> >   - Fix the Arm capitlalization (Peter Maydell)
> >     - Context here:
> >       (https://lists.gnu.org/archive/html/qemu-devel/2025-01/msg05137.html)
> 
> 
> > Kashyap Chamarthy (3):
> 
> Hi -- it looks like only patches 1 and 2 ever made it to
> the list. Would you mind resending, please?

It's strange, but as you pointed out on IRC: occasionally the list "eats
an email for no clear reason".

Now sent a v2, with a note at the top of the cover-letter.

> >   docs/cpu-features: Consistently use vCPU instead of VCPU
> >   docs/cpu-features: Update "PAuth" (Pointer Authentication) details
> >   docs: Fix "Arm" capitaliaztion
> 
> (nit: "capitalization")

Fixed in the re-sent v2.

-- 
/kashyap



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

end of thread, other threads:[~2025-02-17 16:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-13  8:42 [PATCH v2 0/3] docs: Small changes to system/arm/cpu-features and more Kashyap Chamarthy
2025-02-17 13:23 ` Peter Maydell
2025-02-17 16:41   ` Kashyap Chamarthy
  -- strict thread matches above, loose matches on Subject: below --
2025-02-13 13:50 Kashyap Chamarthy
2025-02-17 16:37 Kashyap Chamarthy

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