http://bugzilla.suse.com/show_bug.cgi?id=1048679
http://bugzilla.suse.com/show_bug.cgi?id=1048679#c34
--- Comment #34 from Thomas Blume
(In reply to Franck Bui from comment #28)
1. /sys/bus/nvme is assumed which doesn't exist
Hmm, I don't see where this one comes from.
I mean the 'ENV{UD_BUS}="nvme"' in
KERNEL=="nvme*", ENV{DEVTYPE}=="disk", ENV{ID_SERIAL}!="?*", IMPORT{program}="scsi_id --export --whitelisted -d $tempnode", ENV{ID_BUS}="nvme"
2. ID_SERIAL is used which is set to something different in line 23 of the persistent storage rules. So not getting the actual ID_SERIAL.
I think this is addressed by the patch, no ?
Yes, both these issues are addressed by your patch. I just had to explain everything for Thomas again why/how your patch fixes this.
Hm, the patch makes sure that ID_SERIAL is used, but this looks like a workaround. Shouldn't we rather find out why ID_SERIAL gets overwritten by ID_SERIAL_SHORT? I don't see any reason therefore in the initial upstream request at: https://github.com/systemd/systemd/issues/1453 Why should nvme device treated differently from other block devices (e.g. scsi or cciss)? Hannes, according to the header of 60-persistent-storage.rules you are the creator of the Linux persistent device names scheme. Could you give a statement about the usage of ID_SERIAL_SHORT versus ID_SERIAL? -- You are receiving this mail because: You are on the CC list for the bug.