diff options
| author | Michal Privoznik <mprivozn@redhat.com> | 2022-04-01 14:30:05 +0200 |
|---|---|---|
| committer | Michal Privoznik <mprivozn@redhat.com> | 2022-04-07 12:12:11 +0200 |
| commit | 629282d8845407c1aff9a26f5dc026e15121f8cd (patch) | |
| tree | 167c70899474842551c02c8b959e9a7180479900 /src/lxc/lxc_controller.c | |
| parent | 5c6622eff771767d37631675fb8b81ba95f0cc14 (diff) | |
| download | libvirt-629282d8845407c1aff9a26f5dc026e15121f8cd.tar.gz | |
lib: Set up cpuset controller for restrictive numatune
The aim of 'restrictive' numatune mode is to rely solely on
CGroups to have QEMU running on configured NUMA nodes. However,
we were never setting the cpuset controller when a domain was
starting up. We are doing so only when
virDomainSetNumaParameters() is called (aka live pinning).
This is obviously wrong. Fortunately, fix is simple as
'restrictive' is similar to 'strict' - every location where
VIR_DOMAIN_NUMATUNE_MEM_STRICT occurs can be audited and
VIR_DOMAIN_NUMATUNE_MEM_RESTRICTIVE case can be added.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2070380
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Diffstat (limited to 'src/lxc/lxc_controller.c')
| -rw-r--r-- | src/lxc/lxc_controller.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/lxc/lxc_controller.c b/src/lxc/lxc_controller.c index 677fa5a4fb..d936f34793 100644 --- a/src/lxc/lxc_controller.c +++ b/src/lxc/lxc_controller.c @@ -811,7 +811,8 @@ static int virLXCControllerSetupResourceLimits(virLXCController *ctrl) virDomainNumatuneMemMode mode; if (virDomainNumatuneGetMode(ctrl->def->numa, -1, &mode) == 0) { - if (mode == VIR_DOMAIN_NUMATUNE_MEM_STRICT && + if ((mode == VIR_DOMAIN_NUMATUNE_MEM_STRICT || + mode == VIR_DOMAIN_NUMATUNE_MEM_RESTRICTIVE) && virCgroupControllerAvailable(VIR_CGROUP_CONTROLLER_CPUSET)) { /* Use virNuma* API iff necessary. Once set and child is exec()-ed, * there's no way for us to change it. Rely on cgroups (if available |