[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220718124227.17b3c611@gandalf.local.home>
Date: Mon, 18 Jul 2022 12:42:27 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Peter Zijlstra <peterz@...radead.org>
Cc: LKML <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>,
Juri Lelli <juri.lelli@...hat.com>,
Vincent Guittot <vincent.guittot@...aro.org>,
Dietmar Eggemann <dietmar.eggemann@....com>,
Ben Segall <bsegall@...gle.com>, Mel Gorman <mgorman@...e.de>,
Daniel Bristot de Oliveira <bristot@...hat.com>,
Valentin Schneider <vschneid@...hat.com>,
Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: schedstat false counting of domain load_balance() tried to move
one or more tasks failed
On Mon, 18 Jul 2022 11:51:26 +0200
Peter Zijlstra <peterz@...radead.org> wrote:
> > Do we care? Should it be fixed? Should it be documented?
>
> *shrug*, I suppose we can fix. People using this stuff are the sort that
> are likely to read documentation instead of code.
Yep.
>
> At the same time; I suspect it's been 'broken' like forever, so who
> knows what people are actually expecting today.
As you stated, it's used by people that read documentation more than the
code. My expectation is that they are making wrong decisions because what
they expect those numbers to mean are not what is actually happening.
I think it's better to make the functionality match the documentation, and
if people complain, we can ask them what exactly they expected and why. And
perhaps they might be complaining that a benchmark isn't behaving as
expected because they were interpreting the results incorrectly.
I'll go write up a fix.
Thanks!
-- Steve
Powered by blists - more mailing lists