summaryrefslogtreecommitdiff
path: root/src/shared/watchdog.c
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2021-09-15 15:52:53 +0200
committerLennart Poettering <lennart@poettering.net>2021-09-15 16:01:02 +0200
commit84b5e291e24d00a18df0a825520a9fe48fa21b72 (patch)
treef380275b0ddd4b345a64b0dd9352f2469cff4bf7 /src/shared/watchdog.c
parenta4588af942af976c55f72869340c24d5017db278 (diff)
downloadsystemd-84b5e291e24d00a18df0a825520a9fe48fa21b72.tar.gz
watchdog: add ", ignoring" to log messages about errors we ignore
As per coding style. Also downgrade relevant log messages to LOG_WARNING.
Diffstat (limited to 'src/shared/watchdog.c')
-rw-r--r--src/shared/watchdog.c16
1 files changed, 8 insertions, 8 deletions
diff --git a/src/shared/watchdog.c b/src/shared/watchdog.c
index b9c816760d..3f6a2f0228 100644
--- a/src/shared/watchdog.c
+++ b/src/shared/watchdog.c
@@ -30,7 +30,7 @@ static int update_timeout(void) {
flags = WDIOS_DISABLECARD;
if (ioctl(watchdog_fd, WDIOC_SETOPTIONS, &flags) < 0)
- return log_warning_errno(errno, "Failed to disable hardware watchdog: %m");
+ return log_warning_errno(errno, "Failed to disable hardware watchdog, ignoring: %m");
} else {
int sec, flags;
usec_t t;
@@ -38,7 +38,7 @@ static int update_timeout(void) {
t = DIV_ROUND_UP(watchdog_timeout, USEC_PER_SEC);
sec = MIN(t, (usec_t) INT_MAX); /* Saturate */
if (ioctl(watchdog_fd, WDIOC_SETTIMEOUT, &sec) < 0)
- return log_warning_errno(errno, "Failed to set timeout to %is: %m", sec);
+ return log_warning_errno(errno, "Failed to set timeout to %is, ignoring: %m", sec);
/* Just in case the driver is buggy */
assert(sec > 0);
@@ -57,7 +57,7 @@ static int update_timeout(void) {
}
if (ioctl(watchdog_fd, WDIOC_KEEPALIVE, 0) < 0)
- return log_warning_errno(errno, "Failed to ping hardware watchdog: %m");
+ return log_warning_errno(errno, "Failed to ping hardware watchdog, ignoring: %m");
watchdog_last_ping = now(clock_boottime_or_monotonic());
}
@@ -75,10 +75,10 @@ static int open_watchdog(void) {
fn = watchdog_device ?: "/dev/watchdog";
watchdog_fd = open(fn, O_WRONLY|O_CLOEXEC);
if (watchdog_fd < 0)
- return log_debug_errno(errno, "Failed to open watchdog device %s: %m", fn);
+ return log_debug_errno(errno, "Failed to open watchdog device %s, ignoring: %m", fn);
if (ioctl(watchdog_fd, WDIOC_GETSUPPORT, &ident) < 0)
- log_debug_errno(errno, "Hardware watchdog %s does not support WDIOC_GETSUPPORT ioctl: %m", fn);
+ log_debug_errno(errno, "Hardware watchdog %s does not support WDIOC_GETSUPPORT ioctl, ignoring: %m", fn);
else
log_info("Using hardware watchdog '%s', version %x, device %s",
ident.identity,
@@ -156,7 +156,7 @@ int watchdog_ping(void) {
}
if (ioctl(watchdog_fd, WDIOC_KEEPALIVE, 0) < 0)
- return log_warning_errno(errno, "Failed to ping hardware watchdog: %m");
+ return log_warning_errno(errno, "Failed to ping hardware watchdog, ignoring: %m");
watchdog_last_ping = ntime;
return 0;
@@ -172,7 +172,7 @@ void watchdog_close(bool disarm) {
/* Explicitly disarm it */
flags = WDIOS_DISABLECARD;
if (ioctl(watchdog_fd, WDIOC_SETOPTIONS, &flags) < 0)
- log_warning_errno(errno, "Failed to disable hardware watchdog: %m");
+ log_warning_errno(errno, "Failed to disable hardware watchdog, ignoring: %m");
/* To be sure, use magic close logic, too */
for (;;) {
@@ -182,7 +182,7 @@ void watchdog_close(bool disarm) {
break;
if (errno != EINTR) {
- log_error_errno(errno, "Failed to disarm watchdog timer: %m");
+ log_warning_errno(errno, "Failed to disarm watchdog timer, ignoring: %m");
break;
}
}