[PATCH 0/5] hw/nvme: fix namespace identifiers

Klaus Jensen posted 5 patches 2 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220419121039.1259477-1-its@irrelevant.dk
Maintainers: Keith Busch <kbusch@kernel.org>, Klaus Jensen <its@irrelevant.dk>
There is a newer version of this series
docs/system/devices/nvme.rst |  6 ++++--
hw/nvme/ctrl.c               | 21 ++++++++++-----------
hw/nvme/ns.c                 | 14 +++++++++-----
hw/nvme/nvme.h               |  4 ++++
hw/nvme/subsys.c             |  7 +++++++
5 files changed, 34 insertions(+), 18 deletions(-)
[PATCH 0/5] hw/nvme: fix namespace identifiers
Posted by Klaus Jensen 2 years ago
From: Klaus Jensen <k.jensen@samsung.com>

The namespace identifiers reported by the controller is kind of a mess.
See [1,2].

This series should fix this for both the `-device nvme,drive=...` and
`-device nvme-ns,...` cases.

  [1]: https://lore.kernel.org/linux-nvme/20220224192845.1097602-1-hch@lst.de/
  [2]: https://lore.kernel.org/linux-nvme/20220413044905.376785-1-hch@lst.de/

Klaus Jensen (5):
  hw/nvme: enforce common serial per subsystem
  hw/nvme: always set eui64
  hw/nvme: do not report null uuid
  hw/nvme: do not auto-generate uuid
  hw/nvme: bump firmware revision

 docs/system/devices/nvme.rst |  6 ++++--
 hw/nvme/ctrl.c               | 21 ++++++++++-----------
 hw/nvme/ns.c                 | 14 +++++++++-----
 hw/nvme/nvme.h               |  4 ++++
 hw/nvme/subsys.c             |  7 +++++++
 5 files changed, 34 insertions(+), 18 deletions(-)

-- 
2.35.1

Re: [PATCH 0/5] hw/nvme: fix namespace identifiers
Posted by Keith Busch 2 years ago
On Tue, Apr 19, 2022 at 02:10:34PM +0200, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> The namespace identifiers reported by the controller is kind of a mess.
> See [1,2].
> 
> This series should fix this for both the `-device nvme,drive=...` and
> `-device nvme-ns,...` cases.

Series looks good.

Reviewed-by: Keith Busch <kbusch@kernel.org>