[libvirt] [PATCH 09/34] Specify bus/unit instead of index for disks with QEMU

Daniel Veillard veillard at redhat.com
Fri Jan 15 13:34:21 UTC 2010


On Fri, Jan 08, 2010 at 05:23:05PM +0000, Daniel P. Berrange wrote:
> The current code for using -drive simply sets the -drive 'index'
> parameter. QEMU internally converts this to bus/unit depending
> on the type of drive. This does not give us precise control over
> the bus/unit assignment though. This change switches over to make
> libvirt explicitly calculate the bus/unit number.
> 
> In addition bus/unit/index are actually irrelevant for VirtIO
> disks, since each virtio disk is a separate PCI device. No disk
> controller is involved.
> 
> Doing the conversion to bus/unit in libvirt allows us to correctly
> attach SCSI controllers when required.
> 
> * src/qemu/qemu_conf.c: Specify bus/unit instead of index for
>   disks
> * tests/qemuxml2argvdata/qemuxml2argv-disk*.args: Switch over from
>   using index=NNNN, to bus=NN, unit=NN for SCSI/IDE/Floppy disks

  Hum, that's an importabt change, but I'm a bit surprized to not
see any checking from the version or help message to detect support.
Was full addressing implemented so long ago that we shouldn't check for
it and try to detect at startup if it's missing ? Just wondering,
not a blocker !

  ACK,

Daniel

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




More information about the libvir-list mailing list