Beckhoff / BBAPI

Beckhoff BIOS API Linux Kernel Driver
Other
25 stars 9 forks source link

"WATCHDOG_NOWAYOUT" feature request #1

Closed TroelsTH closed 8 years ago

TroelsTH commented 8 years ago

It seems like the watchdog driver doesn’t check for kernel define WATCHDOG_NOWAYOUT like e.g. the “sch311x_wdt”. Please add check for the define in the wd_stop function so that the power controller watchdog isn't disabled. This feature is needed to secure that the power controller watchdog catches e.g. kernel panics during shutdown, which otherwise could lead to hung systems.

pbruenn commented 8 years ago

Hi Troels, can you confirm, that commit 403aa85eeccba03a2df5fbbb6849ca2c4f2c188c does what you need? Regards, Patrick

TroelsTH commented 8 years ago

Hi Patrick I have compiled and verified that your changes work as expected... THANKS :-) In particular thanks for the quick response

Best regards Troels

On Thu, Jun 23, 2016 at 4:00 PM, Patrick Brünn notifications@github.com wrote:

Hi Troels, can you confirm, that commit 403aa85 https://github.com/Beckhoff/BBAPI/commit/403aa85eeccba03a2df5fbbb6849ca2c4f2c188c does what you need? Regards, Patrick

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Beckhoff/BBAPI/issues/1#issuecomment-228058948, or mute the thread https://github.com/notifications/unsubscribe/AGqsKz5qLgUXejLgGXzzxi6lXye2zZcnks5qOpFkgaJpZM4I8vOS .