[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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 veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]