From 956212c9a226d9192e4b3c085e917fe59f6d5cb9 Mon Sep 17 00:00:00 2001

From: Maurizio Lombardi <mlombard@redhat.com>

Date: Tue, 2 Jun 2015 17:26:25 +0200

Subject: [PATCH] udev: also create old sas paths

RHEL-only

Resolves: #957112

---

 rules/60-persistent-storage.rules |  2 ++

 src/udev/udev-builtin-path_id.c   | 33 ++++++++++++++++++++++++-------

 2 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/rules/60-persistent-storage.rules b/rules/60-persistent-storage.rules

index 25b44a55cb..71ab974844 100644

--- a/rules/60-persistent-storage.rules

+++ b/rules/60-persistent-storage.rules

@@ -54,7 +54,9 @@ KERNEL=="mspblk[0-9]p[0-9]", ENV{ID_NAME}=="?*", ENV{ID_SERIAL}=="?*", SYMLINK+=

 # by-path (parent device path)

 ENV{DEVTYPE}=="disk", DEVPATH!="*/virtual/*", IMPORT{builtin}="path_id"

 ENV{DEVTYPE}=="disk", ENV{ID_PATH}=="?*", SYMLINK+="disk/by-path/$env{ID_PATH}"

+ENV{DEVTYPE}=="disk", ENV{ID_SAS_PATH}=="?*", SYMLINK+="disk/by-path/$env{ID_SAS_PATH}"

 ENV{DEVTYPE}=="partition", ENV{ID_PATH}=="?*", SYMLINK+="disk/by-path/$env{ID_PATH}-part%n"

+ENV{DEVTYPE}=="partition", ENV{ID_SAS_PATH}=="?*", SYMLINK+="disk/by-path/$env{ID_SAS_PATH}-part%n"

 # skip unpartitioned removable media devices from drivers which do not send "change" events

 ENV{DEVTYPE}=="disk", KERNEL!="sd*|sr*", ATTR{removable}=="1", GOTO="persistent_storage_end"

diff --git a/src/udev/udev-builtin-path_id.c b/src/udev/udev-builtin-path_id.c

index bb0a6242ac..025392df5b 100644

--- a/src/udev/udev-builtin-path_id.c

+++ b/src/udev/udev-builtin-path_id.c

@@ -154,7 +154,7 @@ out:

         return parent;

 }

-static struct udev_device *handle_scsi_sas(struct udev_device *parent, char **path)

+static struct udev_device *handle_scsi_sas(struct udev_device *parent, bool enable_new_sas_path, char **path, bool *new_sas_path)

 {

         struct udev *udev  = udev_device_get_udev(parent);

         struct udev_device *targetdev;

@@ -169,6 +169,8 @@ static struct udev_device *handle_scsi_sas(struct udev_device *parent, char **pa

         const char *phy_count;

         char *lun = NULL;

+        *new_sas_path = false;

+

         targetdev = udev_device_get_parent_with_subsystem_devtype(parent, "scsi", "scsi_target");

         if (targetdev == NULL)

                 return NULL;

@@ -201,7 +203,7 @@ static struct udev_device *handle_scsi_sas(struct udev_device *parent, char **pa

         }

         /* Check if we are simple disk */

-        if (strncmp(phy_count, "1", 2) != 0) {

+        if (strncmp(phy_count, "1", 2) != 0 || !enable_new_sas_path) {

                  parent = handle_scsi_sas_wide_port(parent, path);

                  goto out;

         }

@@ -241,6 +243,8 @@ static struct udev_device *handle_scsi_sas(struct udev_device *parent, char **pa

         if (lun)

                 free(lun);

+

+        *new_sas_path = true;

 out:

         udev_device_unref(target_sasdev);

         udev_device_unref(expander_sasdev);

@@ -466,7 +470,7 @@ out:

         return hostdev;

 }

-static struct udev_device *handle_scsi(struct udev_device *parent, char **path, bool *supported_parent) {

+static struct udev_device *handle_scsi(struct udev_device *parent, bool enable_new_sas_path, char **path, bool *supported_parent, bool *new_sas_path) {

         const char *devtype;

         const char *name;

         const char *id;

@@ -494,7 +498,7 @@ static struct udev_device *handle_scsi(struct udev_device *parent, char **path,

         }

         if (strstr(name, "/end_device-") != NULL) {

-                parent = handle_scsi_sas(parent, path);

+                parent = handle_scsi_sas(parent, enable_new_sas_path, path, new_sas_path);

                 *supported_parent = true;

                 goto out;

         }

@@ -610,6 +614,8 @@ static int builtin_path_id(struct udev_device *dev, int argc, char *argv[], bool

         char *path = NULL;

         bool supported_transport = false;

         bool supported_parent = false;

+        bool new_sas_path = false;

+        bool enable_new_sas_path = true;

         /* S390 ccw bus */

         parent = udev_device_get_parent_with_subsystem_devtype(dev, "ccw", NULL);

@@ -618,6 +624,8 @@ static int builtin_path_id(struct udev_device *dev, int argc, char *argv[], bool

                 goto out;

         }

+restart:

+        ;

         /* walk up the chain of devices and compose path */

         parent = dev;

         while (parent != NULL) {

@@ -629,7 +637,7 @@ static int builtin_path_id(struct udev_device *dev, int argc, char *argv[], bool

                 } else if (streq(subsys, "scsi_tape")) {

                         handle_scsi_tape(parent, &path);

                 } else if (streq(subsys, "scsi")) {

-                        parent = handle_scsi(parent, &path, &supported_parent);

+                        parent = handle_scsi(parent, enable_new_sas_path, &path, &supported_parent, &new_sas_path);

                         supported_transport = true;

                 } else if (streq(subsys, "cciss")) {

                         parent = handle_cciss(parent, &path);

@@ -721,9 +729,20 @@ out:

                         i--;

                 tag[i] = '\0';

-                udev_builtin_add_property(dev, test, "ID_PATH", path);

-                udev_builtin_add_property(dev, test, "ID_PATH_TAG", tag);

+                if (new_sas_path) {

+                        udev_builtin_add_property(dev, test, "ID_SAS_PATH", path);

+                } else {

+                        udev_builtin_add_property(dev, test, "ID_PATH", path);

+                        udev_builtin_add_property(dev, test, "ID_PATH_TAG", tag);

+                }

+

                 free(path);

+

+                if (new_sas_path) {

+                        enable_new_sas_path = false;

+                        goto restart;

+                }

+

                 return EXIT_SUCCESS;

         }

         return EXIT_FAILURE;