Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Daniel P. Berrange
On Fri, Feb 26, 2016 at 02:46:56PM +0100, Henning Schild wrote: > On Fri, 26 Feb 2016 13:26:36 + > "Daniel P. Berrange" wrote: > > > On Fri, Feb 26, 2016 at 02:17:35PM +0100, Henning Schild wrote: > > > On Fri, 26 Feb 2016 13:00:04 + > > > "Daniel P. Berrange"

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Henning Schild
On Fri, 26 Feb 2016 13:26:36 + "Daniel P. Berrange" wrote: > On Fri, Feb 26, 2016 at 02:17:35PM +0100, Henning Schild wrote: > > On Fri, 26 Feb 2016 13:00:04 + > > "Daniel P. Berrange" wrote: > > > > > On Fri, Feb 26, 2016 at 01:43:05PM

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Daniel P. Berrange
On Fri, Feb 26, 2016 at 02:17:35PM +0100, Henning Schild wrote: > On Fri, 26 Feb 2016 13:00:04 + > "Daniel P. Berrange" wrote: > > > On Fri, Feb 26, 2016 at 01:43:05PM +0100, Henning Schild wrote: > > IIUC, the original problem you wanted to address was that vCPU pids >

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Henning Schild
On Fri, 26 Feb 2016 13:00:04 + "Daniel P. Berrange" wrote: > On Fri, Feb 26, 2016 at 01:43:05PM +0100, Henning Schild wrote: > > On Fri, 26 Feb 2016 12:21:02 + > > "Daniel P. Berrange" wrote: > > > > > On Fri, Feb 26, 2016 at 01:16:15PM

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Daniel P. Berrange
On Fri, Feb 26, 2016 at 01:43:05PM +0100, Henning Schild wrote: > On Fri, 26 Feb 2016 12:21:02 + > "Daniel P. Berrange" wrote: > > > On Fri, Feb 26, 2016 at 01:16:15PM +0100, Henning Schild wrote: > > > On Fri, 26 Feb 2016 11:13:13 + > > > "Daniel P. Berrange"

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Henning Schild
On Fri, 26 Feb 2016 12:21:02 + "Daniel P. Berrange" wrote: > On Fri, Feb 26, 2016 at 01:16:15PM +0100, Henning Schild wrote: > > On Fri, 26 Feb 2016 11:13:13 + > > "Daniel P. Berrange" wrote: > > > > > On Tue, Feb 23, 2016 at 04:58:39PM

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Daniel P. Berrange
On Fri, Feb 26, 2016 at 01:16:15PM +0100, Henning Schild wrote: > On Fri, 26 Feb 2016 11:13:13 + > "Daniel P. Berrange" wrote: > > > On Tue, Feb 23, 2016 at 04:58:39PM +0100, Henning Schild wrote: > > > virCgroupNewMachine used to add the pidleader to the newly created >

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Henning Schild
On Fri, 26 Feb 2016 11:13:13 + "Daniel P. Berrange" wrote: > On Tue, Feb 23, 2016 at 04:58:39PM +0100, Henning Schild wrote: > > virCgroupNewMachine used to add the pidleader to the newly created > > machine cgroup. Do not do this implicit anymore. > > > >

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-26 Thread Daniel P. Berrange
On Tue, Feb 23, 2016 at 04:58:39PM +0100, Henning Schild wrote: > virCgroupNewMachine used to add the pidleader to the newly created > machine cgroup. Do not do this implicit anymore. > > Signed-off-by: Henning Schild > --- > src/lxc/lxc_cgroup.c | 11 +++ >

Re: [libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-25 Thread John Ferlan
On 02/23/2016 10:58 AM, Henning Schild wrote: > virCgroupNewMachine used to add the pidleader to the newly created > machine cgroup. Do not do this implicit anymore. > > Signed-off-by: Henning Schild > --- > src/lxc/lxc_cgroup.c | 11 +++ >

[libvirt] [PATCH 4/9] util: cgroups do not implicitly add task to new machine cgroup

2016-02-23 Thread Henning Schild
virCgroupNewMachine used to add the pidleader to the newly created machine cgroup. Do not do this implicit anymore. Signed-off-by: Henning Schild --- src/lxc/lxc_cgroup.c | 11 +++ src/qemu/qemu_cgroup.c | 11 +++ src/util/vircgroup.c | 22