Thread (3 messages) 3 messages, 3 authors, 2017-02-22

Re: [PATCH v2 1/3] power: reset: Add Renesas reset driver

From: Guenter Roeck <linux@roeck-us.net>
Date: 2017-02-22 15:28:15
Also in: linux-pm, linux-renesas-soc, linux-watchdog

On Wed, Feb 22, 2017 at 01:35:12PM +0000, Chris Brandt wrote:
Hello Geert and Guenter,

On Thursday, February 16, 2017, Guenter Roeck wrote:
quoted
FWIW, the watchdog subsystem should support that easily, even with 125 ms
hardware timeout. We added that capability for that very purpose. That
would only fail if the system is stuck with interrupts disabled for more
than 125 ms, which seems unlikely. I think the gpio watchdog on some
systems has a similar low hardware timeout.

While I'm going to try that for the RZ/A1, I have a question for you guys
(or anyone else that has an opinion about end applications)


If I were going to make a request to the chip designers to make the timeout longer
for the next RZ/A chip, what would be a good max timeout for common Linux applications?

Looking through the drivers in the watchdog directly, I see default timeouts of 20,
30, 60, and 120 seconds.
30 and 60 are pretty common for default timeouts, though I personally think
they are a bit long. If you want direct HW support, a maximum of 120 seconds
should be sufficient though not really necessary anymore since the core
supports virtual timeouts. A maximum of at least 30 seconds would be needed
if the watchdog is supposed to run at boot time (ie if it is enabled by
ROMMON/BIOS and kept running by the kernel).

Hope this helps,
Guenter
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help