Re: [PATCH] arm: kernel: utilize hrtimer based broadcast

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From	Thomas Gleixner <tglx@linutronix.de>
Newsgroups	linux.kernel
Subject	Re: [PATCH] arm: kernel: utilize hrtimer based broadcast
Date	Tue, 05 Jan 2016 10:50:02 +0100
Message-ID	<qNwpA-eM-25@gated-at.bofh.it> (permalink)
References	<pN9Ye-18j-25@gated-at.bofh.it> <qo7E7-2QQ-11@gated-at.bofh.it> <qKAzo-7dk-15@gated-at.bofh.it> <qKECZ-1qo-7@gated-at.bofh.it> <qL2YG-1hA-3@gated-at.bofh.it> <qMrLk-6Wu-9@gated-at.bofh.it>
User-Agent	Alpine 2.11 (DEB 23 2013-08-11)
MIME-Version	1.0
Content-Type	TEXT/PLAIN; charset=US-ASCII
X-Linutronix-Spam-Score	-1.0
X-Linutronix-Spam-Level	-
X-Linutronix-Spam-Status	No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	36
Organization	linux.* mail to news gateway
X-Original-Cc	Arnd Bergmann <arnd@arndb.de>, LAK <linux-arm-kernel@lists.infradead.org>, Huan Wang <alison.wang@nxp.com>, "mark.rutland@arm.com" <mark.rutland@arm.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Huan Wang <alison.wang@freescale.com>, Daniel Lezcano <daniel.lezcano@linaro.org>, John Stultz <john.stultz@linaro.org>
X-Original-Date	Tue, 5 Jan 2016 10:42:57 +0100 (CET)
X-Original-Message-ID	<alpine.DEB.2.11.1601051039320.3689@nanos>
X-Original-References	<1437124312-44700-1-git-send-email-b18965@freescale.com> <20151027081834.GB1425@svinekod> <DB3PR04MB07407D83CC5F8F5F19A7207F4FB0@DB3PR04MB074.eurprd04.prod.outlook.com> <4875455.lvoc6XpkZc@wuerfel> <alpine.DEB.2.11.1512291449380.28591@nanos> <20160102102955.GP8644@n2100.arm.linux.org.uk>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1301381

Show key headers only | View raw

On Sat, 2 Jan 2016, Russell King - ARM Linux wrote:
> On Tue, Dec 29, 2015 at 02:54:10PM +0100, Thomas Gleixner wrote:
> > I have no real opinion about that patch. It does no harm to unconditionally
> > setup the hrtimer based broadcast even if it's never used.
> > 
> > Up to the arch maintainer to decide. 
> 
> That's really not fair to keep shovelling these kinds of decisions onto
> architecture maintainers without any kind of explanation about how an
> architecture maintainer should make such a decision.
> 
> Do I roll a 6-face dice, and if it gives an odd number, I apply this
> patch, otherwise I reject it?
> 
> Is there a technical basis for making the decision?  If so, please
> explain what the technical arguments are against having or not having
> this change.

The hrtimer based broadcast device is used when you have per cpu timers which
stop in deeper power states, but you have no other timer hardware on the chip
which can backup the per cpu timer in deep power states. The trick is that it
emulates a timer hardware via a hrtimer and then tells the cpu idle code not
to go into deep power states on the cpu which owns that hrtimer. All other
cpus can go as deep as they want and still get woken up.

The only downside of adding this unconditionally is extra code in case that it
is not needed on a particular platform.

Hope that helps.

     tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] arm: kernel: utilize hrtimer based broadcast Russell King - ARM Linux <linux@arm.linux.org.uk> - 2016-01-02 11:40 +0100
  Re: [PATCH] arm: kernel: utilize hrtimer based broadcast Thomas Gleixner <tglx@linutronix.de> - 2016-01-05 10:50 +0100

csiph-web