From: Frederic Weisbecker <fweisbec@gmail.com>
To: Oleg Nesterov <oleg@redhat.com>, Peter Zijlstra <peterz@infradead.org>
Cc: LKML <linux-kernel@vger.kernel.org>,
Frederic Weisbecker <fweisbec@gmail.com>,
Fernando Luis Vazquez Cao <fernando_b1@lab.ntt.co.jp>,
Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@kernel.org>,
Andrew Morton <akpm@linux-foundation.org>,
Arjan van de Ven <arjan@linux.intel.com>
Subject: [RFC PATCH 0/5] nohz: Fix racy sleeptime stats v2
Date: Sat, 19 Oct 2013 17:17:16 +0200 [thread overview]
Message-ID: <1382195841-6558-1-git-send-email-fweisbec@gmail.com> (raw)
Hi,
So this is a new iteration with some more changes on top of the latest discussions
we had.
Unfortunately it's not yet a real viable solution. Check out patch 4/5 for details,
I think that's too much overhead. Also it changes enough the semantics to break the
idle sleeptime accounting along the way. This is because moving the accounting to
io_schedule() makes it account _all_ IO sleeptime, even if the CPU is still busy,
not just when it is idle like before. As a result, the idle sleeptime, that relies on
a diff against the io sleeptime, is broken.
Anyway, my hope with this new take is mostly to launch a discussion with a fresher
base and see if we can find better ideas.
Thanks!
git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
sched/iowait
Thanks,
Frederic
---
Frederic Weisbecker (5):
nohz: Convert a few places to use local per cpu accesses
nohz: Only update sleeptime stats locally
timer: Change idle/iowait accounting semantics
sched: Refactor iowait accounting
nohz: Synchronize sleep time stats with seqlock
include/linux/ktime.h | 7 +++
include/linux/sched.h | 1 +
include/linux/tick.h | 11 ++--
kernel/sched/core.c | 68 ++++++++++++++++++---
kernel/sched/cputime.c | 2 +-
kernel/sched/sched.h | 5 +-
kernel/softirq.c | 4 +-
kernel/time/tick-broadcast.c | 6 +-
kernel/time/tick-internal.h | 4 +-
kernel/time/tick-sched.c | 138 ++++++++++++++-----------------------------
kernel/time/timer_list.c | 3 +-
11 files changed, 130 insertions(+), 119 deletions(-)
next reply other threads:[~2013-10-19 15:17 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-19 15:17 Frederic Weisbecker [this message]
2013-10-19 15:17 ` [PATCH 1/5] nohz: Convert a few places to use local per cpu accesses Frederic Weisbecker
2013-10-19 15:17 ` [PATCH 2/5] nohz: Only update sleeptime stats locally Frederic Weisbecker
2013-10-19 15:17 ` [PATCH 3/5] timer: Change idle/iowait accounting semantics Frederic Weisbecker
2013-10-20 7:34 ` Andreas Mohr
2013-11-04 16:22 ` Frederic Weisbecker
2013-10-19 15:17 ` [PATCH 4/5] sched: Refactor iowait accounting Frederic Weisbecker
2013-10-19 15:35 ` Peter Zijlstra
2013-10-19 16:02 ` Frederic Weisbecker
2013-10-19 16:05 ` Peter Zijlstra
2013-10-19 16:20 ` Frederic Weisbecker
2013-10-20 11:10 ` Andreas Mohr
2013-11-04 17:34 ` Frederic Weisbecker
2013-11-04 18:38 ` Andreas Mohr
2013-10-19 15:17 ` [PATCH 5/5] nohz: Synchronize sleep time stats with seqlock Frederic Weisbecker
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=1382195841-6558-1-git-send-email-fweisbec@gmail.com \
--to=fweisbec@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=arjan@linux.intel.com \
--cc=fernando_b1@lab.ntt.co.jp \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@kernel.org \
--cc=oleg@redhat.com \
--cc=penguin-kernel@I-love.SAKURA.ne.jp \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
/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.