[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2-RFC 2/2] pci: add standard bridge device
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCHv2-RFC 2/2] pci: add standard bridge device |
Date: |
Mon, 20 Feb 2012 01:44:34 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, Feb 19, 2012 at 04:57:07PM +0200, Michael S. Tsirkin wrote:
> 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.
I just sent the patch making bus id for bridges
follow the value set by the user.
That will make the bridge behave in the same way
as everybody else :)
> --
> MST
[Qemu-devel] [PATCHv2-RFC 1/2] shpc: standard hot plug controller, Michael S. Tsirkin, 2012/02/13
Re: [Qemu-devel] [PATCHv2-RFC 1/2] shpc: standard hot plug controller, Gerd Hoffmann, 2012/02/17
[Qemu-devel] [PATCH RFC] seabios: add OSHP method stub, Michael S. Tsirkin, 2012/02/13