qemu-devel
[Top][All Lists]
Advanced

[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: Sun, 19 Feb 2012 16:57:07 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]