[lkml]   [2014]   [Mar]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] ARM: machine_power_off should not return
On Tue, Mar 25, 2014 at 07:45:55PM -0300, Ezequiel Garcia wrote:
> Let's Cc: LAKML, and To: Russell.
> Russell, any comments on this?
> Without this patch we got the heartbeat's reboot_notifier called twice while
> testing the recent hibernation patches, which was unexpected and produced a
> kernel panic:

I don't see why we should make this change. kernel/reboot.c handles
this function returning, so other places should do too.

Even on x86, this function can return:

void machine_power_off(void)

.power_off = native_machine_power_off,

static void native_machine_power_off(void)
if (pm_power_off) {
if (!reboot_force)
/* A fallback in case there is no PM info available */

void tboot_shutdown(u32 shutdown_type)
void (*shutdown)(void);

if (!tboot_enabled())

Therefore, I'd say... it's a bug in the hibernation code - or we probably
have many buggy architectures. I'd suggest fixing the hibernation code
rather than stuffing some workaround like an endless loop into every

FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly
improving, and getting towards what was expected from it.

 \ /
  Last update: 2014-03-26 02:41    [W:0.092 / U:0.280 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site