From: Mike Galbraith <umgwanakikbuti@gmail.com>
To: Johannes Weiner <hannes@cmpxchg.org>,
Peter Zijlstra <peterz@infradead.org>
Cc: Tejun Heo <tj@kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
Andrew Morton <akpm@linux-foundation.org>,
Li Zefan <lizefan@huawei.com>, Paul Turner <pjt@google.com>,
Ingo Molnar <mingo@redhat.com>,
linux-kernel@vger.kernel.org, cgroups@vger.kernel.org,
linux-api@vger.kernel.org, kernel-team@fb.com
Subject: Re: [Documentation] State of CPU controller in cgroup v2
Date: Wed, 17 Aug 2016 11:33:23 +0200 [thread overview]
Message-ID: <1471426403.4110.188.camel@gmail.com> (raw)
In-Reply-To: <20160816163045.GA16427@cmpxchg.org>
On Tue, 2016-08-16 at 12:30 -0400, Johannes Weiner wrote:
> On Tue, Aug 16, 2016 at 04:07:38PM +0200, Peter Zijlstra wrote:
> > Also, the argument there seems unfair at best, you don't need cpu-v2 for
> > buffered write control, you only need memcg and block co-mounted.
>
> Yes, memcg and block agreeing is enough for that case. But I mentioned
> a whole bunch of these examples, to make the broader case for a common
> controller model.
The core issue I have with that model is that it defines context=mm,
and declares context=task to be invalid, while in reality, both views
are perfectly valid, useful, and in use. That redefinition of context
is demonstrably harmful when applied to scheduler related controllers,
rendering a substantial portion of to be managed objects completely
unmanageable. You (collectively) know that full well.
AFAIKT, there is only one viable option, and that is to continue to
allow both. Whether you like the duality or not (who would), it's
deeply embedded in what's under the controllers, and won't go away.
I'll now go try a little harder while you ponder (or pop) this thought
bubble, see if I can set a new personal best at the art of ignoring.
(CC did not help btw, your bad if you don't like bubble content)
-Mike
next prev parent reply other threads:[~2016-08-17 9:33 UTC|newest]
Thread overview: 87+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-05 17:07 [Documentation] State of CPU controller in cgroup v2 Tejun Heo
2016-08-05 17:07 ` Tejun Heo
2016-08-05 17:09 ` [PATCH 1/2] sched: Misc preps for cgroup unified hierarchy interface Tejun Heo
2016-08-05 17:09 ` Tejun Heo
2016-08-05 17:09 ` [PATCH 2/2] sched: Implement interface for cgroup unified hierarchy Tejun Heo
2016-08-05 17:09 ` Tejun Heo
2016-08-06 9:04 ` [Documentation] State of CPU controller in cgroup v2 Mike Galbraith
2016-08-06 9:04 ` Mike Galbraith
2016-08-10 22:09 ` Johannes Weiner
2016-08-10 22:09 ` Johannes Weiner
2016-08-11 6:25 ` Mike Galbraith
2016-08-11 6:25 ` Mike Galbraith
2016-08-12 22:17 ` Johannes Weiner
2016-08-12 22:17 ` Johannes Weiner
2016-08-13 5:08 ` Mike Galbraith
2016-08-13 5:08 ` Mike Galbraith
2016-08-16 14:07 ` Peter Zijlstra
2016-08-16 14:07 ` Peter Zijlstra
2016-08-16 14:58 ` Chris Mason
2016-08-16 14:58 ` Chris Mason
2016-08-16 16:30 ` Johannes Weiner
2016-08-16 16:30 ` Johannes Weiner
2016-08-17 9:33 ` Mike Galbraith [this message]
2016-08-16 21:59 ` Tejun Heo
2016-08-16 21:59 ` Tejun Heo
2016-08-17 20:18 ` Andy Lutomirski
2016-08-20 15:56 ` Tejun Heo
2016-08-20 15:56 ` Tejun Heo
2016-08-20 18:45 ` Andy Lutomirski
2016-08-29 22:20 ` Tejun Heo
2016-08-29 22:20 ` Tejun Heo
2016-08-31 3:42 ` Andy Lutomirski
2016-08-31 3:42 ` Andy Lutomirski
2016-08-31 17:32 ` Tejun Heo
2016-08-31 19:11 ` Andy Lutomirski
2016-08-31 19:11 ` Andy Lutomirski
2016-08-31 21:07 ` Tejun Heo
2016-08-31 21:07 ` Tejun Heo
2016-08-31 21:46 ` Andy Lutomirski
2016-09-03 22:05 ` Tejun Heo
2016-09-03 22:05 ` Tejun Heo
2016-09-05 17:37 ` Andy Lutomirski
2016-09-06 10:29 ` Peter Zijlstra
2016-09-06 10:29 ` Peter Zijlstra
2016-10-04 14:47 ` Tejun Heo
2016-10-05 8:07 ` Peter Zijlstra
2016-10-05 8:07 ` Peter Zijlstra
2016-09-09 22:57 ` Tejun Heo
2016-09-10 8:54 ` Mike Galbraith
2016-09-10 8:54 ` Mike Galbraith
2016-09-10 10:08 ` Mike Galbraith
2016-09-10 10:08 ` Mike Galbraith
2016-09-30 9:06 ` Tejun Heo
2016-09-30 9:06 ` Tejun Heo
2016-09-30 14:53 ` Mike Galbraith
2016-09-30 14:53 ` Mike Galbraith
2016-09-12 15:20 ` Austin S. Hemmelgarn
2016-09-12 15:20 ` Austin S. Hemmelgarn
2016-09-19 21:34 ` Tejun Heo
2016-09-19 21:34 ` Tejun Heo
[not found] ` <CALCETrUhpPQdyZ-6WRjdB+iLbpGBduRZMWXQtCuS+R7Cq7rygg@mail.gmail.com>
2016-09-14 20:00 ` Tejun Heo
2016-09-15 20:08 ` Andy Lutomirski
2016-09-15 20:08 ` Andy Lutomirski
2016-09-16 7:51 ` Peter Zijlstra
2016-09-16 7:51 ` Peter Zijlstra
2016-09-16 15:12 ` Andy Lutomirski
2016-09-16 15:12 ` Andy Lutomirski
2016-09-16 16:19 ` Peter Zijlstra
2016-09-16 16:19 ` Peter Zijlstra
2016-09-16 16:29 ` Andy Lutomirski
2016-09-16 16:29 ` Andy Lutomirski
2016-09-16 16:50 ` Peter Zijlstra
2016-09-16 16:50 ` Peter Zijlstra
2016-09-16 18:19 ` Andy Lutomirski
2016-09-16 18:19 ` Andy Lutomirski
2016-09-17 1:47 ` Peter Zijlstra
2016-09-17 1:47 ` Peter Zijlstra
2016-09-19 21:53 ` Tejun Heo
2016-09-19 21:53 ` Tejun Heo
2016-08-31 19:57 ` Andy Lutomirski
2016-08-31 19:57 ` Andy Lutomirski
2016-08-22 10:12 ` Mike Galbraith
2016-08-22 10:12 ` Mike Galbraith
2016-08-21 5:34 ` James Bottomley
2016-08-21 5:34 ` James Bottomley
2016-08-29 22:35 ` Tejun Heo
2016-08-29 22:35 ` 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=1471426403.4110.188.camel@gmail.com \
--to=umgwanakikbuti@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=cgroups@vger.kernel.org \
--cc=hannes@cmpxchg.org \
--cc=kernel-team@fb.com \
--cc=linux-api@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lizefan@huawei.com \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=pjt@google.com \
--cc=tj@kernel.org \
--cc=torvalds@linux-foundation.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.