From: johnstul@us.ibm.com (john stultz)
To: linux-arm-kernel@lists.infradead.org
Subject: [patch 00/16] arm: Replace arm sched_clock by clocksource based sched_clock
Date: Mon, 25 Apr 2011 12:10:49 -0700 [thread overview]
Message-ID: <1303758649.2816.57.camel@work-vm> (raw)
In-Reply-To: <BANLkTi=5Uz_TNKOiJKDn7Mh30fxv3aY67Q@mail.gmail.com>
On Sun, 2011-04-24 at 09:27 +0200, Linus Walleij wrote:
> 2011/4/23 Thomas Gleixner <tglx@linutronix.de>:
>
> > Most of ARM sched_clocks is using the timekeeping clocksource as the
> > sched_clock. Only versatile uses a different clock for it, but there
> > is no real reason why it can't use the clocksource as well.
>
> Excellent patch set!
>
> > Another 650 lines gone :)
>
> Any reason why we can't simply move *all* of these to
> drivers/clocksource/* and proceed with consolidation there?
This gives me some pause, as having the clocksources in the platform
code makes it easier to know what platform needs to be tested when a
change is made. The rule so far has been if more then one arch can make
use of it, it should be moved to the drivers/clocksource dir. Otherwise,
leave it close to any arch specific code that manages the hardware.
> It must be a pain having to grep through the tree to find all of
> these (even though I bet you know them all by heart now).
Not sure if grepping backwards through config dependencies to see who
actually uses a specific clocksource would be easier. But I'll admit
that sharing code is likely more important.
thanks
-john
next prev parent reply other threads:[~2011-04-25 19:10 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-23 20:54 [patch 00/16] arm: Replace arm sched_clock by clocksource based sched_clock Thomas Gleixner
2011-04-23 20:54 ` [patch 01/16] time: Provide clocksource based sched_clock() Thomas Gleixner
2011-04-23 21:33 ` Stephen Boyd
2011-04-23 21:40 ` Thomas Gleixner
2011-04-25 7:10 ` Kukjin Kim
2011-04-26 16:36 ` Stephen Boyd
2011-04-23 20:54 ` [patch 02/16] arm: plat-orion: Use clocksource based sched_clock Thomas Gleixner
2011-04-23 20:54 ` [patch 03/16] arm: s5p: " Thomas Gleixner
2011-04-23 21:33 ` Stephen Boyd
2011-04-23 21:41 ` Thomas Gleixner
2011-04-25 7:08 ` Kukjin Kim
2011-04-23 20:54 ` [patch 04/16] arm: davinci: " Thomas Gleixner
2011-04-26 15:40 ` Nori, Sekhar
2011-04-23 20:54 ` [patch 05/16] arm: ixp4xx: " Thomas Gleixner
2011-04-23 20:54 ` [patch 06/16] arm: mmp: " Thomas Gleixner
2011-04-23 20:54 ` [patch 07/16] arm: omap: " Thomas Gleixner
2011-04-29 11:57 ` Tony Lindgren
2011-04-29 12:28 ` Thomas Gleixner
2011-04-29 12:51 ` Tony Lindgren
2011-04-29 14:19 ` Thomas Gleixner
2011-05-02 8:10 ` Tony Lindgren
2011-04-23 20:54 ` [patch 08/16] arm: pxa: " Thomas Gleixner
2011-04-25 16:25 ` Eric
2011-04-26 7:23 ` Sascha Hauer
2011-04-26 7:26 ` Eric Miao
2011-04-23 20:54 ` [patch 09/16] arm: sa1100: " Thomas Gleixner
2011-04-23 20:54 ` [patch 10/16] arm: tegra: " Thomas Gleixner
2011-04-23 20:54 ` [patch 11/16] arm: u300: " Thomas Gleixner
2011-04-24 7:03 ` Linus Walleij
2011-04-23 20:54 ` [patch 12/16] arm: plat-iop: " Thomas Gleixner
2011-04-23 20:54 ` [patch 13/16] arm plat-mxc: " Thomas Gleixner
2011-04-26 7:23 ` Sascha Hauer
2011-04-23 20:54 ` [patch 14/16] arm: nomadik: " Thomas Gleixner
2011-04-24 7:04 ` Linus Walleij
2011-04-23 20:54 ` [patch 15/16] arm: versatile: " Thomas Gleixner
2011-04-23 20:54 ` [patch 16/16] arm: Remove sched_clock code Thomas Gleixner
2011-04-25 7:11 ` Kukjin Kim
2011-04-24 7:27 ` [patch 00/16] arm: Replace arm sched_clock by clocksource based sched_clock Linus Walleij
2011-04-25 19:10 ` john stultz [this message]
2011-04-26 7:45 ` Linus Walleij
2011-04-26 8:50 ` Tony Lindgren
2011-04-26 8:02 ` Thomas Gleixner
2011-04-29 9:46 ` Russell King - ARM Linux
2011-04-29 10:22 ` Thomas Gleixner
2011-04-29 10:32 ` Tony Lindgren
2011-04-29 17:01 ` Thomas Gleixner
2011-04-29 21:53 ` Linus Walleij
2011-04-29 21:57 ` Thomas Gleixner
2011-05-02 8:18 ` Tony Lindgren
2011-05-08 20:34 ` Linus Walleij
2011-06-13 2:32 ` Rob Herring
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=1303758649.2816.57.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.