Which udev version did you do this patch against? Ubuntu 12.04 currently has 175-0ubuntu9.4, and it does not have this code at all. In particular, "This is true for Ubuntu 12.04.3 (quantal)" is wrong (12.04.x is precise, not quantal), and 12.04's udev does not have the /ata check at all. This was only introduced later on in saucy (13.10) when we moved to systemd's udev version.
This does have this check, just at a different position: /* * We do not support the ATA transport class, it uses global counters * to name the ata devices which numbers spread across multiple * controllers. * * The real link numbers are not exported. Also, possible chains of ports * behind port multipliers cannot be composed that way. * * Until all that is solved at the kernel level, there are no by-path/ * links for ATA devices. */ if (strstr(name, "/ata") != NULL) { parent = NULL; goto out; } ** Changed in: udev (Ubuntu) Status: New => Incomplete -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1283303 Title: ATA drive /dev/disk/by-path/ incorrect due to kernel change To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/udev/+bug/1283303/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs