Re: [libvirt] [PATCH v2 09/12] qemu: Start PR daemons on domain startup

2018-03-07 Thread John Ferlan
On 03/06/2018 12:31 PM, Michal Privoznik wrote: > On 03/02/2018 04:54 PM, John Ferlan wrote: >> >> >> On 02/21/2018 01:11 PM, Michal Privoznik wrote: >>> Before we exec() qemu we have to spawn pr-helper processes for >>> all managed reservations (well, technically there can only one). >> >> "can

Re: [libvirt] [PATCH v2 09/12] qemu: Start PR daemons on domain startup

2018-03-06 Thread Michal Privoznik
On 03/02/2018 04:54 PM, John Ferlan wrote: > > > On 02/21/2018 01:11 PM, Michal Privoznik wrote: >> Before we exec() qemu we have to spawn pr-helper processes for >> all managed reservations (well, technically there can only one). > > "can be only one" > > Since there can only be one why do we

Re: [libvirt] [PATCH v2 09/12] qemu: Start PR daemons on domain startup

2018-03-02 Thread John Ferlan
On 02/21/2018 01:11 PM, Michal Privoznik wrote: > Before we exec() qemu we have to spawn pr-helper processes for > all managed reservations (well, technically there can only one). "can be only one" Since there can only be one why do we bother w/ plurality? > The only caveat there is that we sh

[libvirt] [PATCH v2 09/12] qemu: Start PR daemons on domain startup

2018-02-21 Thread Michal Privoznik
Before we exec() qemu we have to spawn pr-helper processes for all managed reservations (well, technically there can only one). The only caveat there is that we should place the process into the same namespace and cgroup as qemu (so that it shares the same view of the system). But we can do that on