lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Zy0njoaJGS9310eR@google.com>
Date: Thu, 7 Nov 2024 12:48:14 -0800
From: Sean Christopherson <seanjc@...gle.com>
To: Tejun Heo <tj@...nel.org>
Cc: "Michal Koutný" <mkoutny@...e.com>, Paolo Bonzini <pbonzini@...hat.com>, 
	Luca Boccassi <bluca@...ian.org>, Roman Gushchin <roman.gushchin@...ux.dev>, kvm@...r.kernel.org, 
	cgroups@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: cgroup2 freezer and kvm_vm_worker_thread()

On Thu, Nov 07, 2024, Tejun Heo wrote:
> Hello,
> 
> On Thu, Nov 07, 2024 at 07:05:46PM +0100, Michal Koutný wrote:
> ...
> > I'd first ask why the kvm_vm_worker_thread needs to be in the KVM task's
> > cgroup (and copy its priority at creation time but no later adjustments)?
> > 
> > If it can remain inside root cgroup (like any other good kthread) its
> > job may be even chunked into periodic/deferred workqueue pieces with no
> > kthread per KVM at all.
> 
> That'd be better but I suppose kvm wants them in the same cgroup for a
> reason.

Yes.  The one and only user of kvm_vm_create_worker_thread() does non-trivial
work on behalf of the VM, and so we want all of the CPU time consumed by that
work to be charged to the VM's container, e.g. so that a VM that is generating
a lot of work doesn't negatively impact other VMs on the same host (the amount
of work done is directly affected by how the guest is accessing its memory).

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ