Hi Jean-Christophe, On Sat, 21 Feb 2015 00:33:17 +0800 Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx> wrote: > > > On Feb 20, 2015, at 3:51 PM, Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx> wrote: > > > > Hi Jean-Christophe, > > > > On Fri, 20 Feb 2015 15:48:22 +0800 > > Jean-Christophe PLAGNIOL-VILLARD <plagnioj@xxxxxxxxxxxx> wrote: > > > >> > >>> On Feb 18, 2015, at 8:57 PM, Timo Kokkonen <timo.kokkonen@xxxxxxxxxx> wrote: > >>> > >>> By default the driver will start a kernel timer which keeps on kicking > >>> the watchdog HW until user space has opened the watchdog > >>> device. Usually this is desirable as the watchdog HW is running by > >>> default and the user space may not have any watchdog daemon running at > >>> all. > >>> > >>> However, on production systems it may be mandatory that also early > >>> crashes and lockups will lead to a watchdog reset, even if they happen > >>> before the user space has opened the watchdog device. > >>> > >>> To resolve the issue, add a new device tree property > >>> "early-timeout-sec" which will let the kernel timer to ping the > >>> watchdog HW only as long as the specified timeout permits. The default > >>> is still to use kernel timer, but more strict behavior can be enabled > >>> via the device tree property. > >>> > >>> Signed-off-by: Timo Kokkonen <timo.kokkonen@xxxxxxxxxx> > >>> --- > >>> Documentation/devicetree/bindings/watchdog/watchdog.txt | 7 +++++++ > >>> drivers/watchdog/at91sam9_wdt.c | 9 ++++++++- > >> > >> This should not be handled by the driver but the kernel in a generic way > > > > Could you detail a bit more what you have in mind ? > > move this timeout on the linux thread that keep alive the watchdog not in the driver AFAIK there's no such thing (if there is, could you point me to the source file where this thread is defined ?), and each driver are registering their own timer (if they need one). If you're suggesting to add such common logic to watchdog core, why don't you propose something ? Timo's need is quite generic, but nobody seemed to bother with that before. Moreover, using an at91 specific implementation does not prevent migrating to a more generic implementation when it's available. Actually, it's rather difficult to design a generic infrastructure until you have dealt with several devices requiring the same feature, and that's obviously not the case here. Best Regards, Boris -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
