From: Tony Lindgren <tony@atomide.com> To: Felipe Balbi <balbi@ti.com> Cc: Russell King <linux@arm.linux.org.uk>, Linux ARM Kernel Mailing List <linux-arm-kernel@lists.infradead.org>, Linux OMAP Mailing List <linux-omap@vger.kernel.org> Subject: Re: [RFC/NOT FOR MERGING] HACK: add global/private timers for A9 Date: Wed, 3 Jun 2015 13:44:08 -0700 [thread overview] Message-ID: <20150603204407.GX30984@atomide.com> (raw) In-Reply-To: <1433363565-17725-1-git-send-email-balbi@ti.com> * Felipe Balbi <balbi@ti.com> [150603 13:36]: > > AM43xx, even though it's a single processor A9, it still has TWD and global > timer. I was doing some profiling with RT v4.0 and latency is 3.5x lower just > by switching from gptimer to twd/global. > > The only problem is that currently, is_smp() check prevents me from using twd > with AM43xx (that's why it's commented below, for testing purposes). > > In the hopes that we can start a, hopefully, small thread around the subject, > I'm sending this HACK which I used to get TWD and global timer enabled so I > could measure latencies with cyclictest. > > Is it so that TWD shouldn't be available on UP integrations of ARM's Cortex-A > processors ? > --- a/arch/arm/kernel/smp_twd.c > +++ b/arch/arm/kernel/smp_twd.c > @@ -388,8 +389,10 @@ static void __init twd_local_timer_of_register(struct device_node *np) > { > int err; > > +#if 0 > if (!is_smp() || !setup_max_cpus) > return; > +#endif > > twd_ppi = irq_of_parse_and_map(np, 0); > if (!twd_ppi) { Why don't you just check for the c-a9 revision here? If it's the UP processor then allow twd? Similar to what's done in global_timer_of_register? Regards, Tony
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren) To: linux-arm-kernel@lists.infradead.org Subject: [RFC/NOT FOR MERGING] HACK: add global/private timers for A9 Date: Wed, 3 Jun 2015 13:44:08 -0700 [thread overview] Message-ID: <20150603204407.GX30984@atomide.com> (raw) In-Reply-To: <1433363565-17725-1-git-send-email-balbi@ti.com> * Felipe Balbi <balbi@ti.com> [150603 13:36]: > > AM43xx, even though it's a single processor A9, it still has TWD and global > timer. I was doing some profiling with RT v4.0 and latency is 3.5x lower just > by switching from gptimer to twd/global. > > The only problem is that currently, is_smp() check prevents me from using twd > with AM43xx (that's why it's commented below, for testing purposes). > > In the hopes that we can start a, hopefully, small thread around the subject, > I'm sending this HACK which I used to get TWD and global timer enabled so I > could measure latencies with cyclictest. > > Is it so that TWD shouldn't be available on UP integrations of ARM's Cortex-A > processors ? > --- a/arch/arm/kernel/smp_twd.c > +++ b/arch/arm/kernel/smp_twd.c > @@ -388,8 +389,10 @@ static void __init twd_local_timer_of_register(struct device_node *np) > { > int err; > > +#if 0 > if (!is_smp() || !setup_max_cpus) > return; > +#endif > > twd_ppi = irq_of_parse_and_map(np, 0); > if (!twd_ppi) { Why don't you just check for the c-a9 revision here? If it's the UP processor then allow twd? Similar to what's done in global_timer_of_register? Regards, Tony
next prev parent reply other threads:[~2015-06-03 20:44 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-06-03 20:32 [RFC/NOT FOR MERGING] HACK: add global/private timers for A9 Felipe Balbi 2015-06-03 20:32 ` Felipe Balbi 2015-06-03 20:44 ` Tony Lindgren [this message] 2015-06-03 20:44 ` Tony Lindgren 2015-06-03 20:55 ` Arnd Bergmann 2015-06-03 20:55 ` Arnd Bergmann 2015-06-03 21:04 ` Felipe Balbi 2015-06-03 21:04 ` Felipe Balbi 2015-06-03 21:28 ` Felipe Balbi 2015-06-03 21:28 ` Felipe Balbi 2015-06-03 21:41 ` Stephen Boyd 2015-06-03 21:41 ` Stephen Boyd 2015-06-03 21:54 ` Felipe Balbi 2015-06-03 21:54 ` Felipe Balbi 2015-06-04 9:46 ` Mason 2015-06-04 9:46 ` Mason 2015-06-04 19:05 ` Felipe Balbi 2015-06-04 19:05 ` Felipe Balbi 2015-06-04 20:08 ` Felipe Balbi 2015-06-04 20:08 ` Felipe Balbi 2015-06-04 20:18 ` Felipe Balbi 2015-06-04 20:18 ` Felipe Balbi 2015-06-04 20:29 ` Felipe Balbi 2015-06-04 20:29 ` Felipe Balbi 2015-06-04 22:20 ` Stephen Boyd 2015-06-04 22:20 ` Stephen Boyd 2015-06-05 15:42 ` Felipe Balbi 2015-06-05 15:42 ` Felipe Balbi 2015-06-04 20:32 ` Mason 2015-06-04 20:32 ` Mason 2015-06-04 20:37 ` Felipe Balbi 2015-06-04 20:37 ` Felipe Balbi 2015-06-04 21:00 ` Mason 2015-06-04 21:00 ` Mason 2015-06-04 21:46 ` Tony Lindgren 2015-06-04 21:46 ` Tony Lindgren 2015-06-03 22:26 ` Tony Lindgren 2015-06-03 22:26 ` Tony Lindgren 2015-06-04 3:45 ` Felipe Balbi 2015-06-04 3:45 ` Felipe Balbi
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=20150603204407.GX30984@atomide.com \ --to=tony@atomide.com \ --cc=balbi@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ /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: linkBe 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.