Re: [PATCH 16/19] nvme-multipath: add nvme_mpath_{add,delete}_ns()

[John Garry]

On 02/03/2026 12:48, Nilay Shroff wrote:
+void nvme_mpath_add_ns(struct nvme_ns *ns)
+{
+    struct nvme_ns_head *head = ns->head;
+    struct mpath_disk *mpath_disk = head->mpath_disk;
+    struct mpath_head *mpath_head;
+
+    if (!mpath_disk)
+        return;
+
+    mpath_head = mpath_disk->mpath_head;
+
+    ns->mpath_device.disk = ns->disk;
+    mpath_add_device(mpath_head, &ns->mpath_device);
+}

As we have now reference to mpath_device from struct nvme_ns
then why do we still maintain reference to ns->disk? We may
want to directly access path device disk using ns->mpath_device.disk,
makes sense?

For !CONFIG_NVME_MULTIPATH or !multipath mod param set, mpath_device is 
not properly maintained, so I would rather not use it in the core code.

Thanks,
John