[libvirt] [PATCH v2 1/9] fix bug in qemuSetupCgroupForEmulator
Eric Blake
eblake at redhat.com
Fri Sep 7 04:56:02 UTC 2012
On 09/06/2012 04:13 AM, Hu Tao wrote:
> Should not return 0 when failed to setup cgroup.
> ---
> src/qemu/qemu_cgroup.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
ACK with one nit.
>
> diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c
> index 5b42793..c95cc77 100644
> --- a/src/qemu/qemu_cgroup.c
> +++ b/src/qemu/qemu_cgroup.c
> @@ -691,13 +691,15 @@ int qemuSetupCgroupForEmulator(struct qemud_driver *driver,
> }
>
> if (def->cputune.emulatorpin &&
> - qemuCgroupControllerActive(driver, VIR_CGROUP_CONTROLLER_CPUSET) &&
> - qemuSetupCgroupEmulatorPin(cgroup_emulator, def->cputune.emulatorpin) < 0)
> + qemuCgroupControllerActive(driver, VIR_CGROUP_CONTROLLER_CPUSET)) {
> + rc = qemuSetupCgroupEmulatorPin(cgroup_emulator, def->cputune.emulatorpin);
> + if (rc < 0)
> goto cleanup;
Indentation is now off.
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 617 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120906/198a5116/attachment-0001.sig>
More information about the libvir-list
mailing list