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

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(a)redhat.com&gt; > --- > 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>;.

Pavel