[PATCH v2 4/5] powerpc: pseries: only store the device node basename in full_name

[Rob Herring]

With dependencies on full_name containing the entire device node path
removed, stop storing the full_name in nodes created by
dlpar_configure_connector() and pSeries_reconfig_add_node().

Signed-off-by: Rob Herring <r...@kernel.org>
Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org>
Cc: Paul Mackerras <pau...@samba.org>
Cc: Michael Ellerman <m...@ellerman.id.au>
Cc: linuxppc-dev@lists.ozlabs.org
---
v2:
- Rework dlpar_parse_cc_node() to a single allocation and strcpy instead
  of kasprintf
  
 arch/powerpc/platforms/pseries/dlpar.c    | 30 +++++++-----------------------
 arch/powerpc/platforms/pseries/reconfig.c |  2 +-
 2 files changed, 8 insertions(+), 24 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/dlpar.c 
b/arch/powerpc/platforms/pseries/dlpar.c
index 783f36364690..31a0615aeea1 100644
--- a/arch/powerpc/platforms/pseries/dlpar.c
+++ b/arch/powerpc/platforms/pseries/dlpar.c
@@ -75,28 +75,17 @@ static struct property *dlpar_parse_cc_property(struct 
cc_workarea *ccwa)
        return prop;
 }
 
-static struct device_node *dlpar_parse_cc_node(struct cc_workarea *ccwa,
-                                              const char *path)
+static struct device_node *(dlpar_parse_cc_node)(struct cc_workarea *ccwa)
 {
        struct device_node *dn;
-       char *name;
-
-       /* If parent node path is "/" advance path to NULL terminator to
-        * prevent double leading slashs in full_name.
-        */
-       if (!path[1])
-               path++;
+       const char *name = (const char *)ccwa + be32_to_cpu(ccwa->name_offset);
 
-       dn = kzalloc(sizeof(*dn), GFP_KERNEL);
+       dn = kzalloc(sizeof(*dn) + strlen(name) + 1, GFP_KERNEL);
        if (!dn)
                return NULL;
 
-       name = (char *)ccwa + be32_to_cpu(ccwa->name_offset);
-       dn->full_name = kasprintf(GFP_KERNEL, "%s/%s", path, name);
-       if (!dn->full_name) {
-               kfree(dn);
-               return NULL;
-       }
+       dn->full_name = (char *)(dn + 1);
+       strcpy((char *)dn->full_name, name);
 
        of_node_set_flag(dn, OF_DYNAMIC);
        of_node_init(dn);
@@ -148,7 +137,6 @@ struct device_node *dlpar_configure_connector(__be32 
drc_index,
        struct property *last_property = NULL;
        struct cc_workarea *ccwa;
        char *data_buf;
-       const char *parent_path = parent->full_name;
        int cc_token;
        int rc = -1;
 
@@ -182,7 +170,7 @@ struct device_node *dlpar_configure_connector(__be32 
drc_index,
                        break;
 
                case NEXT_SIBLING:
-                       dn = dlpar_parse_cc_node(ccwa, parent_path);
+                       dn = dlpar_parse_cc_node(ccwa);
                        if (!dn)
                                goto cc_error;
 
@@ -192,10 +180,7 @@ struct device_node *dlpar_configure_connector(__be32 
drc_index,
                        break;
 
                case NEXT_CHILD:
-                       if (first_dn)
-                               parent_path = last_dn->full_name;
-
-                       dn = dlpar_parse_cc_node(ccwa, parent_path);
+                       dn = dlpar_parse_cc_node(ccwa);
                        if (!dn)
                                goto cc_error;
 
@@ -226,7 +211,6 @@ struct device_node *dlpar_configure_connector(__be32 
drc_index,
 
                case PREV_PARENT:
                        last_dn = last_dn->parent;
-                       parent_path = last_dn->parent->full_name;
                        break;
 
                case CALL_AGAIN:
diff --git a/arch/powerpc/platforms/pseries/reconfig.c 
b/arch/powerpc/platforms/pseries/reconfig.c
index 296c188fd5ca..f24d8159c9e1 100644
--- a/arch/powerpc/platforms/pseries/reconfig.c
+++ b/arch/powerpc/platforms/pseries/reconfig.c
@@ -33,7 +33,7 @@ static int pSeries_reconfig_add_node(const char *path, struct 
property *proplist
        if (!np)
                goto out_err;
 
-       np->full_name = kstrdup(path, GFP_KERNEL);
+       np->full_name = kstrdup(kbasename(path), GFP_KERNEL);
        if (!np->full_name)
                goto out_err;
 
-- 
2.11.0