citrus-it commented on this pull request.


>                       vdev_dbgmsg(vd, "vdev_disk_open: invalid "
                            "vdev_devid '%s'", vd->vdev_devid);
-                       return (SET_ERROR(EINVAL));
+                       dvd->vd_devid = NULL;
+                       dvd->vd_minor = NULL;
                }
        }
 

Right, it is preserved since it is only updated if opened by physical path and 
dvd->vd_devid != NULL.
It may be a good idea to update the stored devid, but this case bites me mostly 
when moving USB drives between Linux and illumos.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/605#discussion_r198786952
------------------------------------------
openzfs: openzfs-developer
Permalink: 
https://openzfs.topicbox.com/groups/developer/Tb273d787e1d7e25e-Mad0b2f391c4eb04e76e2aedd
Delivery options: https://openzfs.topicbox.com/groups

Reply via email to