Re: [PATCH 5/8] qemu: support attachment of disk device with boot index

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

 



On Mon, Jan 05, 2015 at 03:29:44PM +0800, Wang Rui wrote:
> When we attach a disk to a running VM with boot index, we can get a
> successful result. But in fact the boot index won't take effect. QEMU
> supported to set device's boot index online recently(since QEMU 2.2.0).
> 

It seems older QEMU silently ignores the bootindex specified with
device_add.

Checking for the support upfront lets libvirt error out earlier.
(Even in the case of updating the boot-index, there would be no need
to enter the monitor to report the error).
Can that be done by looking at some qom property, or can it only
be checked by comparing the QEMU version?

> After this patch, the boot index will take effect after
> virDomainAttachDevice(Flags) API returning success. If new disk is
> attached successfully but boot index is set failed, we'll remove the
> new disk to restore.

The bootindex can be specified as a part of the device_add command,
removing the need to detach the device on failure (and the previous code
movement patch).

Also, can the qom-set really fail? It seems the property has been
around for a while, it was just ignored.

Jan

> 
> Signed-off-by: Wang Rui <moon.wangrui@xxxxxxxxxx>
> Signed-off-by: Zhou Yimin <zhouyimin@xxxxxxxxxx>
> ---
>  src/qemu/qemu_hotplug.c | 88 +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 88 insertions(+)
> 

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]