diff options
| author | Niklas Cassel <niklas.cassel@wdc.com> | 2022-06-28 14:22:09 +0200 |
|---|---|---|
| committer | Klaus Jensen <k.jensen@samsung.com> | 2022-07-15 10:40:33 +0200 |
| commit | dfa82ac201af28c451271d3f1dc6827b431cd827 (patch) | |
| tree | 1f676461b3446f986483de5e265b8aadec71f08b /hw/nvme/ns.c | |
| parent | 146b5fa505fc21baa129c7538936fbdd9875b6ed (diff) | |
| download | qemu-dfa82ac201af28c451271d3f1dc6827b431cd827.zip qemu-dfa82ac201af28c451271d3f1dc6827b431cd827.tar.gz qemu-dfa82ac201af28c451271d3f1dc6827b431cd827.tar.bz2 | |
hw/nvme: force nvme-ns param 'shared' to false if no nvme-subsys node
Since commit 916b0f0b5264 ("hw/nvme: change nvme-ns 'shared' default")
the default value of nvme-ns param 'shared' is set to true, regardless
if there is a nvme-subsys node or not.
On a system without a nvme-subsys node, a namespace will never be able
to be attached to more than one controller, so for this configuration,
it is counterintuitive for this parameter to be set by default.
Force the nvme-ns param 'shared' to false for configurations where
there is no nvme-subsys node, as the namespace will never be able to
attach to more than one controller anyway.
Signed-off-by: Niklas Cassel <niklas.cassel@wdc.com>
Reviewed-by: Klaus Jensen <k.jensen@samsung.com>
Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
Diffstat (limited to 'hw/nvme/ns.c')
| -rw-r--r-- | hw/nvme/ns.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/hw/nvme/ns.c b/hw/nvme/ns.c index 870c3ca..62a1f97 100644 --- a/hw/nvme/ns.c +++ b/hw/nvme/ns.c @@ -546,6 +546,8 @@ static void nvme_ns_realize(DeviceState *dev, Error **errp) int i; if (!n->subsys) { + /* If no subsys, the ns cannot be attached to more than one ctrl. */ + ns->params.shared = false; if (ns->params.detached) { error_setg(errp, "detached requires that the nvme device is " "linked to an nvme-subsys device"); |