Re: Disabling watchdog when closing /dev/watchdog

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 10/09/2013 04:47 PM, Markus Mayer wrote:
Currently, the watchdog framework works such that upon closing
/dev/watchdog it will attempt to disable the watchdog timer and,
failing that, will poke the watchdog (via the watchdog_release()
function).

I am working on a new watchdog driver for a Broadcom SoC, and for test
purposes at least, it would be helpful if the watchdog could be left
alone when the device is closed (i.e. the watchdog is neither stopped
nor poked). That way I could use a command line utility to issue ioctl
calls and observe the effect of these calls.

Right now, using my line tool to check the "time left" will return the
proper amount of time left, but it will also poke the watchdog,
therefore changing the time that's actually left until the watchdog
expires (the command line tool opens /dev/watchdog, issues an ioctl
and closes /dev/watchdog), turning a "read-only" operation into one
that actually changes the watchdog's status.

Automatically stopping or poking the watchdog upon close only makes
sense in the case of a continuously running daemon that keeps
/dev/watchdog open at all times. It doesn't work for case where an
application would open the watchdog device only for brief amounts of
time.

Would it make sense to add a flag like "WDOG_LEAVE_ON" to, say, struct
watchdog_device's "status" values, which the driver could set,
allowing watchdog_release() to leave the watchdog alone upon closing
the device? Or is there a different way to achieve this? A daemon is
not practicable for me at this point, and even later on, we may not
want to go with a daemon solution, but may want to rely on the
watchdog continuing to run even if the device is closed.


It might make more sense to use sysfs attributes to provide this
functionality (name, status, ping, configured timeout, time left,
reboot reason, ...). I think that would be much cleaner and accomplish
the same. Another option might be debugfs if this is supposed
to be used for debugging only.

Thanks,
Guenter

--
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




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux