All of lore.kernel.org
 help / color / mirror / Atom feed
From: johnstul@us.ibm.com (john stultz)
To: linux-arm-kernel@lists.infradead.org
Subject: [BUG] 2.6.37-rc3 massive interactivity regression on ARM
Date: Wed, 08 Dec 2010 12:42:21 -0800	[thread overview]
Message-ID: <1291840941.2909.40.camel@work-vm> (raw)
In-Reply-To: <1291819459.28378.64.camel@laptop>

On Wed, 2010-12-08 at 15:44 +0100, Peter Zijlstra wrote:
> On Wed, 2010-12-08 at 14:28 +0000, Russell King - ARM Linux wrote:
> > So, what I'm saying is that if wrapping in 4 seconds is a problem,
> > then maybe we shouldn't be providing sched_clock() at all.
> 
> 4 seconds should be perfectly fine, we call it at least every scheduler
> tick (HZ) and NO_HZ will either have to limit the max sleep period or
> provide its own sleep duration (using a more expensive clock) so we can
> recover (much like GTOD already requires).
> 
> > Also, if wrapping below 64-bits is also a problem, again, maybe we
> > shouldn't be providing it there either.  Eg:
> 
> That's mostly due to hysterical raisins and we should just fix that, the
> simplest way is to use something like kernel/sched_clock.c to use
> sched_clock() deltas to make a running u64 value.
> 
> Like said, John Stultz was already looking at doing something like that
> because there's a number of architectures suffering this same problem
> and they're all already using part of the clocksource infrastructure to
> implement the sched_clock() interface simply because they only have a
> single hardware resource.

I'm not actively working on it right now, but trying to rework the
sched_clock code so its more like the generic timekeeping code is on my
list (I'm Looking to see if I can bump it up to the front in the near
future).

thanks
-john

WARNING: multiple messages have this Message-ID (diff)
From: john stultz <johnstul@us.ibm.com>
To: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Mikael Pettersson <mikpe@it.uu.se>,
	Venkatesh Pallipadi <venki@google.com>,
	Ingo Molnar <mingo@elte.hu>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [BUG] 2.6.37-rc3 massive interactivity regression on ARM
Date: Wed, 08 Dec 2010 12:42:21 -0800	[thread overview]
Message-ID: <1291840941.2909.40.camel@work-vm> (raw)
In-Reply-To: <1291819459.28378.64.camel@laptop>

On Wed, 2010-12-08 at 15:44 +0100, Peter Zijlstra wrote:
> On Wed, 2010-12-08 at 14:28 +0000, Russell King - ARM Linux wrote:
> > So, what I'm saying is that if wrapping in 4 seconds is a problem,
> > then maybe we shouldn't be providing sched_clock() at all.
> 
> 4 seconds should be perfectly fine, we call it at least every scheduler
> tick (HZ) and NO_HZ will either have to limit the max sleep period or
> provide its own sleep duration (using a more expensive clock) so we can
> recover (much like GTOD already requires).
> 
> > Also, if wrapping below 64-bits is also a problem, again, maybe we
> > shouldn't be providing it there either.  Eg:
> 
> That's mostly due to hysterical raisins and we should just fix that, the
> simplest way is to use something like kernel/sched_clock.c to use
> sched_clock() deltas to make a running u64 value.
> 
> Like said, John Stultz was already looking at doing something like that
> because there's a number of architectures suffering this same problem
> and they're all already using part of the clocksource infrastructure to
> implement the sched_clock() interface simply because they only have a
> single hardware resource.

I'm not actively working on it right now, but trying to rework the
sched_clock code so its more like the generic timekeeping code is on my
list (I'm Looking to see if I can bump it up to the front in the near
future).

thanks
-john



  parent reply	other threads:[~2010-12-08 20:42 UTC|newest]

Thread overview: 102+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-27 15:16 [BUG] 2.6.37-rc3 massive interactivity regression on ARM Mikael Pettersson
2010-11-27 15:16 ` Mikael Pettersson
2010-12-05 12:32 ` Mikael Pettersson
2010-12-05 12:32   ` Mikael Pettersson
2010-12-05 13:17   ` Russell King - ARM Linux
2010-12-05 13:17     ` Russell King - ARM Linux
2010-12-05 14:19     ` Russell King - ARM Linux
2010-12-05 14:19       ` Russell King - ARM Linux
2010-12-05 16:07       ` Mikael Pettersson
2010-12-05 16:07         ` Mikael Pettersson
2010-12-05 16:21         ` Russell King - ARM Linux
2010-12-05 16:21           ` Russell King - ARM Linux
2010-12-08 12:40           ` Peter Zijlstra
2010-12-08 12:40             ` Peter Zijlstra
2010-12-08 12:55             ` Russell King - ARM Linux
2010-12-08 12:55               ` Russell King - ARM Linux
2010-12-08 14:04               ` Peter Zijlstra
2010-12-08 14:04                 ` Peter Zijlstra
2010-12-08 14:28                 ` Russell King - ARM Linux
2010-12-08 14:28                   ` Russell King - ARM Linux
2010-12-08 14:44                   ` Peter Zijlstra
2010-12-08 14:44                     ` Peter Zijlstra
2010-12-08 15:05                     ` Russell King - ARM Linux
2010-12-08 15:05                       ` Russell King - ARM Linux
2010-12-08 15:43                     ` Linus Walleij
2010-12-08 15:43                       ` Linus Walleij
2010-12-08 20:42                     ` john stultz [this message]
2010-12-08 20:42                       ` john stultz
2010-12-08 23:31                   ` Venkatesh Pallipadi
2010-12-08 23:31                     ` Venkatesh Pallipadi
2010-12-09 12:52                     ` Peter Zijlstra
2010-12-09 12:52                       ` Peter Zijlstra
2010-12-09 17:43                       ` Venkatesh Pallipadi
2010-12-09 17:43                         ` Venkatesh Pallipadi
2010-12-09 17:55                         ` Peter Zijlstra
2010-12-09 17:55                           ` Peter Zijlstra
2010-12-09 18:11                           ` Venkatesh Pallipadi
2010-12-09 18:11                             ` Venkatesh Pallipadi
2010-12-09 18:55                             ` Peter Zijlstra
2010-12-09 18:55                               ` Peter Zijlstra
2010-12-09 22:21                               ` Venkatesh Pallipadi
2010-12-09 22:21                                 ` Venkatesh Pallipadi
2010-12-09 23:16                                 ` Peter Zijlstra
2010-12-09 23:16                                   ` Peter Zijlstra
2010-12-09 23:35                                   ` Venkatesh Pallipadi
2010-12-09 23:35                                     ` Venkatesh Pallipadi
2010-12-10 10:08                                     ` Peter Zijlstra
2010-12-10 10:08                                       ` Peter Zijlstra
2010-12-10 13:17                                       ` Peter Zijlstra
2010-12-10 13:17                                         ` Peter Zijlstra
2010-12-10 13:27                                         ` Peter Zijlstra
2010-12-10 13:27                                           ` Peter Zijlstra
2010-12-10 13:47                                           ` Peter Zijlstra
2010-12-10 13:47                                             ` Peter Zijlstra
2010-12-10 16:50                                             ` Russell King - ARM Linux
2010-12-10 16:50                                               ` Russell King - ARM Linux
2010-12-10 16:54                                               ` Peter Zijlstra
2010-12-10 16:54                                                 ` Peter Zijlstra
2010-12-10 17:18                                             ` Eric Dumazet
2010-12-10 17:18                                               ` Eric Dumazet
2010-12-10 17:49                                               ` Peter Zijlstra
2010-12-10 17:49                                                 ` Peter Zijlstra
2010-12-10 18:14                                                 ` Eric Dumazet
2010-12-10 18:14                                                   ` Eric Dumazet
2010-12-10 18:39                                                   ` Christoph Lameter
2010-12-10 18:39                                                     ` Christoph Lameter
2010-12-10 18:46                                                     ` Peter Zijlstra
2010-12-10 18:46                                                       ` Peter Zijlstra
2010-12-10 19:51                                                       ` Christoph Lameter
2010-12-10 19:51                                                         ` Christoph Lameter
2010-12-10 20:07                                                         ` Peter Zijlstra
2010-12-10 20:07                                                           ` Peter Zijlstra
2010-12-10 20:23                                                           ` Christoph Lameter
2010-12-10 20:23                                                             ` Christoph Lameter
2010-12-10 20:32                                                             ` Peter Zijlstra
2010-12-10 20:32                                                               ` Peter Zijlstra
2010-12-10 20:39                                                             ` Eric Dumazet
2010-12-10 20:39                                                               ` Eric Dumazet
2010-12-10 20:49                                                               ` Eric Dumazet
2010-12-10 20:49                                                                 ` Eric Dumazet
2010-12-10 21:09                                                                 ` Christoph Lameter
2010-12-10 21:09                                                                   ` Christoph Lameter
2010-12-10 21:22                                                                   ` Eric Dumazet
2010-12-10 21:22                                                                     ` Eric Dumazet
2010-12-10 21:45                                                                     ` Christoph Lameter
2010-12-10 21:45                                                                       ` Christoph Lameter
2010-12-10 17:56                                             ` Russell King - ARM Linux
2010-12-10 17:56                                               ` Russell King - ARM Linux
2010-12-10 18:10                                               ` Peter Zijlstra
2010-12-10 18:10                                                 ` Peter Zijlstra
2010-12-10 18:43                                                 ` Peter Zijlstra
2010-12-10 18:43                                                   ` Peter Zijlstra
2010-12-10 19:17                                                 ` Russell King - ARM Linux
2010-12-10 19:17                                                   ` Russell King - ARM Linux
2010-12-10 19:37                                                   ` Peter Zijlstra
2010-12-10 19:37                                                     ` Peter Zijlstra
2010-12-10 19:25                                                 ` Peter Zijlstra
2010-12-10 19:25                                                   ` Peter Zijlstra
2010-12-13 14:33                             ` Jack Daniel
2010-12-13 14:33                               ` Jack Daniel
2010-12-06 21:29       ` Venkatesh Pallipadi
2010-12-06 21:29         ` Venkatesh Pallipadi

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=1291840941.2909.40.camel@work-vm \
    --to=johnstul@us.ibm.com \
    --cc=linux-arm-kernel@lists.infradead.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.