From: Daniel Jordan <daniel.m.jordan@oracle.com>
To: Tejun Heo <tj@kernel.org>
Cc: Daniel Jordan <daniel.m.jordan@oracle.com>,
hannes@cmpxchg.org, jiangshanlai@gmail.com, lizefan@huawei.com,
bsd@redhat.com, dan.j.williams@intel.com, dave.hansen@intel.com,
juri.lelli@redhat.com, mhocko@kernel.org, peterz@infradead.org,
steven.sistare@oracle.com, tglx@linutronix.de,
tom.hromatka@oracle.com, vdavydov.dev@gmail.com,
cgroups@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mm@kvack.org, shakeelb@google.com
Subject: Re: [RFC v2 0/5] cgroup-aware unbound workqueues
Date: Wed, 12 Jun 2019 18:29:34 -0400 [thread overview]
Message-ID: <20190612222934.y74wxy3aju6eqs4r@ca-dmjordan1.us.oracle.com> (raw)
In-Reply-To: <20190611195549.GL3341036@devbig004.ftw2.facebook.com>
On Tue, Jun 11, 2019 at 12:55:49PM -0700, Tejun Heo wrote:
> > > CPU doesn't have a backcharging mechanism yet and depending on the use
> > > case, we *might* need to put kthreads in different cgroups. However,
> > > such use cases might not be that abundant and there may be gotaches
> > > which require them to be force-executed and back-charged (e.g. fs
> > > compression from global reclaim).
> >
> > The CPU-intensiveness of these works is one of the reasons for actually putting
> > the workers through the migration path. I don't know of a way to get the
> > workers to respect the cpu controller (and even cpuset for that matter) without
> > doing that.
>
> So, I still think it'd likely be better to go back-charging route than
> actually putting kworkers in non-root cgroups. That's gonna be way
> cheaper, simpler and makes avoiding inadvertent priority inversions
> trivial.
Ok, I'll experiment with backcharging in the cpu controller. Initial plan is
to smooth out resource usage by backcharging after each chunk of work that each
helper thread does rather than do one giant backcharge after the multithreaded
job is over. May turn out better performance-wise to do it less often than
this.
I'll also experiment with getting workqueue workers to respect cpuset without
migrating. Seems to make sense to use the intersection of an unbound worker's
cpumask and the cpuset's cpumask, and make some compromises if the result is
empty.
Daniel
next prev parent reply other threads:[~2019-06-12 22:32 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-05 13:36 Daniel Jordan
2019-06-05 13:36 ` [RFC v2 1/5] cgroup: add cgroup v2 interfaces to migrate kernel threads Daniel Jordan
2019-06-05 13:36 ` [RFC v2 2/5] workqueue, cgroup: add cgroup-aware workqueues Daniel Jordan
2019-06-05 13:36 ` [RFC v2 3/5] workqueue, memcontrol: make memcg throttle workqueue workers Daniel Jordan
2019-06-05 13:36 ` [RFC v2 4/5] workqueue, cgroup: add test module Daniel Jordan
2019-06-05 13:36 ` [RFC v2 5/5] ktask, cgroup: attach helper threads to the master thread's cgroup Daniel Jordan
2019-06-05 13:53 ` [RFC v2 0/5] cgroup-aware unbound workqueues Tejun Heo
2019-06-05 15:32 ` Daniel Jordan
2019-06-11 19:55 ` Tejun Heo
2019-06-12 22:29 ` Daniel Jordan [this message]
2019-06-06 6:15 ` Mike Rapoport
2019-06-11 19:52 ` Tejun Heo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190612222934.y74wxy3aju6eqs4r@ca-dmjordan1.us.oracle.com \
--to=daniel.m.jordan@oracle.com \
--cc=bsd@redhat.com \
--cc=cgroups@vger.kernel.org \
--cc=dan.j.williams@intel.com \
--cc=dave.hansen@intel.com \
--cc=hannes@cmpxchg.org \
--cc=jiangshanlai@gmail.com \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lizefan@huawei.com \
--cc=mhocko@kernel.org \
--cc=peterz@infradead.org \
--cc=shakeelb@google.com \
--cc=steven.sistare@oracle.com \
--cc=tglx@linutronix.de \
--cc=tj@kernel.org \
--cc=tom.hromatka@oracle.com \
--cc=vdavydov.dev@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox