[libvirt] [PATCH v2 5/9] vircgroup: introduce virCgroupAddThread

Ján Tomko jtomko at redhat.com
Mon Sep 24 14:55:46 UTC 2018


On Thu, Sep 20, 2018 at 10:54:34AM +0200, Pavel Hrdina wrote:
>Once we introduce cgroup v2 support we need to handle processes and
>threads differently.
>
>Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
>---
>
>Notes:
>    changes in v2:
>        - added comment for new flag
>
> src/libvirt_private.syms |  1 +
> src/qemu/qemu_process.c  |  2 +-
> src/util/vircgroup.c     | 32 ++++++++++++++++++++++++++++++++
> src/util/vircgroup.h     |  1 +
> 4 files changed, 35 insertions(+), 1 deletion(-)
>

AFAICT the nbdpids added in virLXCControllerSetupCgroupLimits also refer
to threads and should use virCgroupAddThread.

But this patch does not make the LXC driver any worse.


Reviewed-by: Ján Tomko <jtomko at redhat.com>

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180924/c7f58767/attachment-0001.sig>


More information about the libvir-list mailing list