From: marex@denx.de (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: MX28 poweroff issue
Date: Wed, 4 Jul 2012 02:13:04 +0200 [thread overview]
Message-ID: <201207040213.04758.marex@denx.de> (raw)
In-Reply-To: <20120703224609.GC26355@n2100.arm.linux.org.uk>
Dear Russell King - ARM Linux,
> On Wed, Jul 04, 2012 at 12:30:36AM +0200, Marek Vasut wrote:
> > I recently confirmed this issue when doing "poweroff". The issue was
> > initially reported by Detlev Zundel.
>
> My guess is it's because we don't disable interrupts on halt - because
> the kernel used to allow halt followed by another ctrl-alt-reboot to
> cause the thing to reboot.
This I consider correct.
> That's long since gone, so we might as well make the CPU spin endlessly
> waiting for someone to pull the power on halt...
The backtrace signals the problem being caused by timer interrupt. Maybe the
struct sys_timer shall be extended by .exit callback to shut down the timer?
Well, the other option is to disable interrupts, indeed.
Russell, you're the wiser one, therefore shall I submit a patch-attempt that
disables interrupts before the spinning?
Best regards,
Marek Vasut
next prev parent reply other threads:[~2012-07-04 0:13 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-03 22:30 MX28 poweroff issue Marek Vasut
2012-07-03 22:46 ` Russell King - ARM Linux
2012-07-04 0:13 ` Marek Vasut [this message]
2012-07-04 1:10 ` Fabio Estevam
2012-07-04 2:12 ` Marek Vasut
2012-07-04 3:14 ` Fabio Estevam
2012-07-04 3:47 ` Marek Vasut
2012-07-04 7:07 ` Uwe Kleine-König
2012-07-04 8:20 ` Russell King - ARM Linux
2012-07-04 8:45 ` Uwe Kleine-König
2012-07-04 14:30 ` Marek Vasut
2012-07-04 7:38 ` Attila Kinali
2012-07-04 14:31 ` Marek Vasut
2012-07-04 14:53 ` Shawn Guo
2012-07-04 15:19 ` Russell King - ARM Linux
2012-07-05 16:08 ` Shawn Guo
2012-07-05 16:23 ` Marek Vasut
2012-07-05 20:32 ` Fabio Estevam
2012-07-05 20:59 ` Fabio Estevam
2012-07-05 20:10 ` Russell King - ARM Linux
2012-07-06 4:49 ` Shawn Guo
2012-07-06 7:32 ` Lothar Waßmann
2012-07-19 17:04 ` Marek Vasut
2012-07-20 0:48 ` Shawn Guo
2012-07-20 1:53 ` Marek Vasut
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=201207040213.04758.marex@denx.de \
--to=marex@denx.de \
--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.