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@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to