Re: [libvirt] [PATCH v4 07/14] qemu_cgroup: Allow /dev/mapper/control for PR

2018-04-17 Thread Michal Privoznik
On 04/17/2018 02:25 PM, John Ferlan wrote: > > > On 04/16/2018 10:56 AM, Michal Privoznik wrote: >> On 04/14/2018 02:55 PM, John Ferlan wrote: >>> >>> >>> On 04/10/2018 10:58 AM, Michal Privoznik wrote: Just like in previous commit, qemu-pr-helper might want to open /dev/mapper/control

Re: [libvirt] [PATCH v4 07/14] qemu_cgroup: Allow /dev/mapper/control for PR

2018-04-17 Thread John Ferlan
On 04/16/2018 10:56 AM, Michal Privoznik wrote: > On 04/14/2018 02:55 PM, John Ferlan wrote: >> >> >> On 04/10/2018 10:58 AM, Michal Privoznik wrote: >>> Just like in previous commit, qemu-pr-helper might want to open >>> /dev/mapper/control under certain circumstances. Therefore we >>> have to

Re: [libvirt] [PATCH v4 07/14] qemu_cgroup: Allow /dev/mapper/control for PR

2018-04-16 Thread Michal Privoznik
On 04/14/2018 02:55 PM, John Ferlan wrote: > > > On 04/10/2018 10:58 AM, Michal Privoznik wrote: >> Just like in previous commit, qemu-pr-helper might want to open >> /dev/mapper/control under certain circumstances. Therefore we >> have to allow it in cgroups. > > Perhaps the two patches should

Re: [libvirt] [PATCH v4 07/14] qemu_cgroup: Allow /dev/mapper/control for PR

2018-04-15 Thread John Ferlan
On 04/10/2018 10:58 AM, Michal Privoznik wrote: > Just like in previous commit, qemu-pr-helper might want to open > /dev/mapper/control under certain circumstances. Therefore we > have to allow it in cgroups. Perhaps the two patches should be combined then... yes, I know cgroups is different

[libvirt] [PATCH v4 07/14] qemu_cgroup: Allow /dev/mapper/control for PR

2018-04-10 Thread Michal Privoznik
Just like in previous commit, qemu-pr-helper might want to open /dev/mapper/control under certain circumstances. Therefore we have to allow it in cgroups. Signed-off-by: Michal Privoznik --- src/qemu/qemu_cgroup.c | 33 ++---