[libvirt] [PATCH 2/2] vircgroupv2: fix virCgroupV2ValidateMachineGroup
Pavel Hrdina
phrdina at redhat.com
Fri Dec 14 03:35:10 UTC 2018
On Thu, Dec 13, 2018 at 04:28:30PM +0100, Michal Privoznik wrote:
> On 12/12/18 11:17 AM, Pavel Hrdina wrote:
> > When libvirt is reconnecting to running domain that uses cgroup v2
> > the QEMU process reports cgroup for the emulator directory because the
> > main thread is in that cgroup. We need to remove the "/emulator" part
> > in order to match with the root cgroup directory name for that domain.
> >
> > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > ---
> > src/util/vircgroupv2.c | 6 ++++++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/src/util/vircgroupv2.c b/src/util/vircgroupv2.c
> > index dc2b2a65bc..b9925967d7 100644
> > --- a/src/util/vircgroupv2.c
> > +++ b/src/util/vircgroupv2.c
> > @@ -118,6 +118,12 @@ virCgroupV2ValidateMachineGroup(virCgroupPtr group,
> > if (virCgroupPartitionEscape(&scopename) < 0)
> > return false;
> >
> > + if (!(tmp = strrchr(group->unified.placement, '/')))
> > + return false;
> > +
> > + if (STREQ(tmp, "/emulator"))
> > + *tmp = '\0';
> > +
>
> How about:
>
> if (STREQ(tmp, "/emulator")) {
> *tmp = '\0';
>
> if (!(tmp = strrchr(group->unified.placement, '/')))
> return false;
> }
Right, that is better.
> ACK regardless.
Thanks, pushed now.
Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20181214/4d781ae4/attachment-0001.sig>
More information about the libvir-list
mailing list