Re: [libvirt] PATCH: Include OS driver name in device information

[Daniel Veillard <veillard redhat com>]

On Fri, May 29, 2009 at 02:39:09PM +0100, Daniel P. Berrange wrote:
> This patch extends the node device XML from
> 
> $ virsh nodedev-dumpxml pci_8086_27d6
> <device>
>   <name>pci_8086_27d6</name>
>   <parent>computer</parent>
>   <capability type='pci'>
>     <domain>0</domain>
>     <bus>0</bus>
>     <slot>28</slot>
>     <function>3</function>
>     <product id='0x27d6'>82801G (ICH7 Family) PCI Express Port 4</product>
>     <vendor id='0x8086'>Intel Corporation</vendor>
>   </capability>
> </device>
> 
> To, also include the operating system driver name. 
> 
> $ virsh nodedev-dumpxml pci_8086_27d6
> <device>
>   <name>pci_8086_27d6</name>
>   <parent>computer</parent>
>   <driver>
>     <name>pcieport-driver</name>
>   </driver>
>   <capability type='pci'>
>     <domain>0</domain>
>     <bus>0</bus>
>     <slot>28</slot>
>     <function>3</function>
>     <product id='0x27d6'>82801G (ICH7 Family) PCI Express Port 4</product>
>     <vendor id='0x8086'>Intel Corporation</vendor>
>   </capability>
> </device>
> 
> 
> We're not defining any particular semantics for the driver name, it is
> just a opaque string following rules of the OS in question.

  Looks fine but let's wait after 0.6.4 release to commit,

   thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/