Re: [PATCH 9/9] qemu_cgroup: assert threading cgroup layout for machine cgroup

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 26 Feb 2016 11:22:07 +0000
"Daniel P. Berrange" <berrange@xxxxxxxxxx> wrote:

> On Tue, Feb 23, 2016 at 04:58:44PM +0100, Henning Schild wrote:
> > Make sure the thread related controls of the machine cgroup never
> > get any tasks assigned.
> > 
> > Signed-off-by: Henning Schild <henning.schild@xxxxxxxxxxx>
> > ---
> >  src/qemu/qemu_cgroup.c | 14 ++++++++++++--
> >  1 file changed, 12 insertions(+), 2 deletions(-)  
> 
> NACK This also won't work for same reason as previous patch

Having that in place can still be useful after we have sorted out the
random result of what systemd gave us.
Is the general idea of such an assertion a good idea, and should i
adopt it according to comments?
At the moment i just used the assertion mask in the only code-path that
adds tasks within libvirt. If we have to deal with manipulation from
the outside, it might be a good idea to introduce more assertions based
on the mask.

Henning

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]