Hi!

> +     if (irqs2 & TWL4030_ACCHGOV) {
>               dev_crit(bci->dev, "Ac charger overvoltage\n");
> +             power_off = true;
> +     }
> +
> +     /* Try to shutdown the system */
> +     if (power_off)
> +             orderly_poweroff(true);
>  
>       return IRQ_HANDLED;
>  }

Userland can make orderly_poweroff take long time, or even forever. (Think 
disconnected
network with nfsroot).

Should we do something more forceful here? Userland has to handle sudden 
poweroffs, anyway...

(You could invent new function "give userland 5 seconds to shut system down" if 
you
really wanted...)

                                                                                
Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to