Re: [PATCH v16 11/11] docs/s390x/cpu topology: document s390x cpu topology

[Pierre Morel]

On 3/1/23 16:52, Nina Schoetterl-Glausch wrote:
> On Wed, 2023-02-22 at 15:21 +0100, Pierre Morel wrote:
> > Add some basic examples for the definition of cpu topology
> > in s390x.
> >
> > Signed-off-by: Pierre Morel <pmorel@linux.ibm.com>
> > ---
> >   docs/system/s390x/cpu-topology.rst | 378 +++++++++++++++++++++++++++++
> >   docs/system/target-s390x.rst       |   1 +
> >   2 files changed, 379 insertions(+)
> >   create mode 100644 docs/system/s390x/cpu-topology.rst
> >
> > diff --git a/docs/system/s390x/cpu-topology.rst 
> > b/docs/system/s390x/cpu-topology.rst
> > new file mode 100644
> > index 0000000000..d470e28b97
> > --- /dev/null
> > +++ b/docs/system/s390x/cpu-topology.rst
> > @@ -0,0 +1,378 @@
> [...]
> > +
> > +set-cpu-topology
> > +++++++++++++++++
> > +
> > +The command set-cpu-topology allows the admin to modify the topology
> > +tree or the topology modifiers of a vCPU in the configuration.
> > +
> > +.. code-block:: QMP
> > +
> > + -> { "execute": "set-cpu-topology",
> > +      "arguments": {
> > +         "core-id": 11,
> > +         "socket-id": 0,
> > +         "book-id": 0,
> > +         "drawer-id": 0,
> > +         "entitlement": low,
> > +         "dedicated": false
> > +      }
> > +    }
> > + <- {"return": {}}
> This fails when building the documenation.
> You need to get rid of the arrows and need "" around the low.
>
>
> [...]


right, thanks


Pierre