From mboxrd@z Thu Jan 1 00:00:00 1970 From: marc.zyngier@arm.com (Marc Zyngier) Date: Mon, 12 Dec 2011 17:05:29 +0000 Subject: [PATCH v4] ARM: sched_clock: allow sched_clock to be selected at runtime In-Reply-To: <4EE57BC3.4080306@gmail.com> References: <1323444833-10587-1-git-send-email-marc.zyngier@arm.com> <4EE57BC3.4080306@gmail.com> Message-ID: <4EE63459.1030105@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 12/12/11 03:57, Rob Herring wrote: > On 12/09/2011 09:33 AM, Marc Zyngier wrote: >> sched_clock() is yet another blocker on the road to the single >> image. This patch implements an idea by Russell King: >> >> http://www.spinics.net/lists/linux-omap/msg49561.html >> >> Instead of asking the platform to implement both sched_clock() >> itself and the rollover callback, simply register a read() >> function, and let the ARM code care about sched_clock() itself, >> the conversion to ns and the rollover. sched_clock() uses >> this read() function as an indirection to the platform code. >> If the platform doesn't provide a read(), the code falls back >> to the jiffy counter (just like the default sched_clock). >> >> This allow some simplifications and possibly some footprint gain >> when multiple platforms are compiled in. Among the drawbacks, >> the removal of the *_fixed_sched_clock optimization which could >> negatively impact some platforms (sa1100, tegra, versatile >> and omap). >> >> Tested on 11MPCore, OMAP4 and Tegra. >> >> Cc: Imre Kaloz >> Cc: Krzysztof Halasa >> Cc: Tony Lindgren >> Cc: Eric Miao >> Cc: Colin Cross >> Cc: Erik Gilling >> Cc: Olof Johansson >> Cc: Linus Walleij >> Cc: Sascha Hauer >> Cc: Alessandro Rubini >> Cc: STEricsson >> Cc: Lennert Buytenhek >> Cc: Nicolas Pitre >> Cc: Ben Dooks >> Cc: Kukjin Kim >> Cc: Jamie Iles >> Tested-by: Jamie Iles >> Acked-by: Linus Walleij >> Signed-off-by: Marc Zyngier >> --- >> I'm reposting this patch with the hope that it (or Russell's >> alternative patch http://patchwork.ozlabs.org/patch/115961/) >> will get merged in 3.3. It doesn't look intrusive enough to be >> left on the side of the road for yet another cycle... >> > > Please! I've been holding off on (re)doing and submitting sched_clock > support for highbank until this goes in. Queued as patch 7205/1. M. -- Jazz is not dead. It just smells funny...