[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/nvme: fix example serial in documentation
From: |
Klaus Jensen |
Subject: |
Re: [PATCH] hw/nvme: fix example serial in documentation |
Date: |
Mon, 27 Jun 2022 19:11:51 +0200 |
On Jun 27 14:39, Niklas Cassel wrote:
> The serial prop on the controller is actually describing the nvme
> subsystem serial, which has to be identical for all controllers within
> the same nvme subsystem.
>
> This is enforced since commit a859eb9f8f64 ("hw/nvme: enforce common
> serial per subsystem").
>
> Fix the documentation, so that people copying the qemu command line
> example won't get an error on qemu start.
>
> Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
> ---
> docs/system/devices/nvme.rst | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/docs/system/devices/nvme.rst b/docs/system/devices/nvme.rst
> index aba253304e..30f841ef62 100644
> --- a/docs/system/devices/nvme.rst
> +++ b/docs/system/devices/nvme.rst
> @@ -104,8 +104,8 @@ multipath I/O.
> .. code-block:: console
>
> -device nvme-subsys,id=nvme-subsys-0,nqn=subsys0
> - -device nvme,serial=a,subsys=nvme-subsys-0
> - -device nvme,serial=b,subsys=nvme-subsys-0
> + -device nvme,serial=deadbeef,subsys=nvme-subsys-0
> + -device nvme,serial=deadbeef,subsys=nvme-subsys-0
>
> This will create an NVM subsystem with two controllers. Having controllers
> linked to an ``nvme-subsys`` device allows additional ``nvme-ns`` parameters:
> --
> 2.36.1
>
Woops!
Thanks Niklas, applied to nvme-next!
signature.asc
Description: PGP signature