[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support.
From: |
Paul Brook |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support. |
Date: |
Thu, 9 Jul 2009 14:48:59 +0100 |
User-agent: |
KMail/1.11.4 (Linux/2.6.29-2-amd64; KDE/4.2.4; x86_64; ; ) |
On Thursday 09 July 2009, Gerd Hoffmann wrote:
> On 07/09/09 15:19, Paul Brook wrote:
> > On Thursday 09 July 2009, Gerd Hoffmann wrote:
> >> Hi,
> >>
> >> This patch series brings driver classes to qdev.
> >
> > I'm not sure this is a good idea. You should be able to figure it out
> > from the device properties.
>
> I don't think this will work out in the general case. Some devices have
> properties which can be used for that, i.e. network cards have a mac
> address (no attribute yet but I expect it will come some day). Not
> every device class has specific properties you can use to identify them
> though. How would you identify a sound card for example?
Ok, put it annother way: Why do you need to identify them? Why would libvirt
care whether a device is (say) a sound card or a VGA adapter? In principle
there's no reason why you shouldn't have many or none of both.
IMHO the only reason we have the current grouping is because it's forced on us
by the various commandline options which mix host and machine configuration.
Paul
- [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/09
- [Qemu-devel] [PATCH 2/5] qdev/class: tag sound, Gerd Hoffmann, 2009/07/09
- [Qemu-devel] [PATCH 4/5] qdev/class: helper function to get a list of drivers., Gerd Hoffmann, 2009/07/09
- [Qemu-devel] [PATCH 3/5] qdev/class: tag network, Gerd Hoffmann, 2009/07/09
- [Qemu-devel] [PATCH 1/5] qdev/class: core, Gerd Hoffmann, 2009/07/09
- [Qemu-devel] [PATCH 5/5] qdev/class: make pci_nic_init() use qdev's device list., Gerd Hoffmann, 2009/07/09
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Paul Brook, 2009/07/09
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/09
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support.,
Paul Brook <=
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/09
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Anthony Liguori, 2009/07/09
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/10
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Paul Brook, 2009/07/10
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/10
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Paul Brook, 2009/07/10
- Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Gerd Hoffmann, 2009/07/10
Re: [Qemu-devel] [PATCH 0/5] qdev: add driver class support., Filip Navara, 2009/07/09