All of lore.kernel.org
 help / color / mirror / Atom feed
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(-)

             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.