Thread (12 messages) 12 messages, 5 authors, 2007-08-31

Re: [PATCH 2.6.23] ibmebus: Prevent bus_id collisions

flat view

From: jschopp <hidden>
Date: 2007-08-29 18:33:41
Also in: lkml

quoted hunk ↗ jump to hunk
+	len = strlen(dn->full_name + 1);
+	bus_len = min(len, BUS_ID_SIZE - 1);
+	memcpy(dev->ofdev.dev.bus_id, dn->full_name + 1
+	       + (len - bus_len), bus_len);
+	for (i = 0; i < bus_len; i++)
+		if (dev->ofdev.dev.bus_id[i] == '/')
+			dev->ofdev.dev.bus_id[i] = '_';
 
 	/* Register with generic device framework. */
 	if (ibmebus_register_device_common(dev, dn->name) != 0) {
What happens when the full name is > 31 characters?  It looks to me that it will be truncated, which takes away the uniqueness guarantee.

There must be an individual property that is guaranteed to be unique and less than 32 characters.  How about "ibm,my-drc-index"?  That looks like a good candidate.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help