On Fri, Aug 31, 2012 at 14:50:32 +0800, Daniel Veillard wrote: > A bit of context would be nice > > On Fri, Aug 31, 2012 at 08:23:15AM +0200, Jiri Denemark wrote: > > --- > > src/qemu/qemu_cgroup.c | 29 ++++++++++++++++++++--------- > > 1 file changed, 20 insertions(+), 9 deletions(-) > > > ACK, Thanks, I added the following background story to the commit and pushed: When domain XML contains any of the elements for setting up CPU scheduling parameters (period, quota, emulator_period, or emulator_quota) we need cpu cgroup to enforce the configuration. However, the existing code would just ignore silently such settings if either cgroups were not available at all cpu cgroup was not available. Moreover, APIs for manipulating CPU scheduler parameters were already failing if cpu cgroup was not available. This patch makes cpu cgroup mandatory for all domains that use CPU scheduling elements in their XML. Jirka -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list