]> xenbits.xensource.com Git - libvirt.git/commit
Avoid spamming logs with cgroups warnings
authorDaniel P. Berrange <berrange@redhat.com>
Wed, 27 Feb 2013 16:51:04 +0000 (16:51 +0000)
committerDaniel P. Berrange <berrange@redhat.com>
Wed, 27 Feb 2013 22:51:24 +0000 (22:51 +0000)
commit279336c5d8c44f28956b3427ab2bf207d06878e2
treebed4c350e67349f1aa6b3a68b5e238252295d724
parentb4a124efc328ac221ff4e8a6fde3a1a0c0202d68
Avoid spamming logs with cgroups warnings

The code for putting the emulator threads in a separate cgroup
would spam the logs with warnings

2013-02-27 16:08:26.731+0000: 29624: warning : virCgroupMoveTask:887 : no vm cgroup in controller 3
2013-02-27 16:08:26.731+0000: 29624: warning : virCgroupMoveTask:887 : no vm cgroup in controller 4
2013-02-27 16:08:26.732+0000: 29624: warning : virCgroupMoveTask:887 : no vm cgroup in controller 6

This is because it has only created child cgroups for 3 of the
controllers, but was trying to move the processes from all the
controllers. The fix is to only try to move threads in the
controllers we actually created. Also remove the warning and
make it return a hard error to avoid such lazy callers in the
future.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
src/qemu/qemu_cgroup.c
src/util/vircgroup.c