[libvirt] [PATCH] qemu: check if domain is really active when do setvcpus with --live

lhuang lhuang at redhat.com
Mon Mar 23 02:07:29 UTC 2015


On 03/20/2015 10:52 PM, Pavel Hrdina wrote:
> On Fri, Mar 20, 2015 at 03:07:09PM +0800, Luyao Huang wrote:
>> https://bugzilla.redhat.com/show_bug.cgi?id=1204006
>>
>> Commit e105dc9 fix setting vcpus for offline domain, but forget check
>> if vm is active when pass VIR_DOMAIN_AFFECT_LIVE flags.
>>
>>   # virsh setvcpus test3 4 --live
>>   error: Failed to create controller cpu for group: No such file or directory
>>
>> add a check if we pass VIR_DOMAIN_AFFECT_LIVE flags.
>>
>> Signed-off-by: Luyao Huang <lhuang at redhat.com>
>> ---
>>   src/qemu/qemu_driver.c | 6 ++++++
>>   1 file changed, 6 insertions(+)
>>
> I've updated the patch to error out for every case we are trying to update live
> domain either by using VIR_DOMAIN_AFFECT_LIVE or VIR_DOMAIN_AFFECT_CURRENT.
>
> See <https://www.redhat.com/archives/libvir-list/2015-March/msg01072.html>.

Good, thanks for your help and review.

> Pavel

Luyao




More information about the libvir-list mailing list