On Fri, Feb 17, 2012 at 02:25:56PM +0100, Gerd Hoffmann wrote: > Hi, > > > + /* If we don't specify the name, the bus will be addressed as <id>.0, where > > + * id is the parent id. But it seems more natural to address the bus using > > + * the parent device name. */ > > + if (dev->qdev.id && *dev->qdev.id) { > > + br->bus_name = dev->qdev.id; > > + } > > That makes the bridge behave different than everybody else. > Not a good idea IMHO. > > cheers, > Gerd Everybody else has names built up according to an undocumented scheme which no one can figure out without reading code, so no one uses them. We need to fix that, but there is, generally, no need for these names so it stayed low priority. With the bridge people must use the id to connect devices to it, so name must be a sane one. -- MST -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html