Re: [libvirt] [PATCH] Add missing checks for QEMU domain state in tunables APIs

2011-03-29 Thread Daniel Veillard
On Tue, Mar 29, 2011 at 11:03:21AM +0100, Daniel P. Berrange wrote: > The methods qemuDomain{Get,Set}{Memory,Blkio,Scheduler}Parameters > all forgot to do a check on virDomainIsActive(), resulting in bogus > error messages from later parts of their impl > > * src/qemu/qemu_driver.c: Add missing ch

[libvirt] [PATCH] Add missing checks for QEMU domain state in tunables APIs

2011-03-29 Thread Daniel P. Berrange
The methods qemuDomain{Get,Set}{Memory,Blkio,Scheduler}Parameters all forgot to do a check on virDomainIsActive(), resulting in bogus error messages from later parts of their impl * src/qemu/qemu_driver.c: Add missing checks on virDomainIsActive() --- src/qemu/qemu_driver.c | 38 +++