On Mon, Feb 20, 2017 at 11:57:59AM -0600, Benjamin Marzinski wrote: > > + > > + snprintf(pp->vendor_id, SCSI_VENDOR_SIZE, "NVME"); > > + snprintf(pp->product_id, SCSI_PRODUCT_SIZE, "%s", udev_device_get_sysattr_value(nvme, "model")); > > + snprintf(pp->serial, SERIAL_SIZE, "%s", udev_device_get_sysattr_value(nvme, "serial")); > > + snprintf(pp->rev, SCSI_REV_SIZE, "%s", udev_device_get_sysattr_value(nvme, "firmware_rev")); > > + snprintf(pp->wwid, WWID_SIZE, "%s", udev_device_get_sysattr_value(pp->udev, "wwid")); > > With your udev rules change, you shouldn't need to set the wwid here. It > should just get dealt with by the get_uid call in pathinfo (or with > the new uevent merging code, before multipathd ever calls pathinfo). Oh right, you are correct. I can spin this with that removed. -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
