summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Gundersen <teg@jklm.no>2015-04-24 17:50:49 +0200
committerAnthony G. Basile <blueness@gentoo.org>2015-05-03 15:27:40 -0400
commit29af2fd71d66c574b4d5d9b9249c854ced30676b (patch)
tree82fe575fc5e0a40311ec1d8b72fa3d9d0f4d2f7a
parent31151da3c4ae7f5331559f6de37b8c77d9acea3e (diff)
udevd: improve logging in SIGCHLD handling
Remove some redundant logging, and reduce the log-level in most cases. The only case that is really critical is if a worker failed while hanlding an event, so keep that at error level.
-rw-r--r--src/udev/udevd.c18
1 files changed, 9 insertions, 9 deletions
diff --git a/src/udev/udevd.c b/src/udev/udevd.c
index 4a9944d494..97e8fa9e4b 100644
--- a/src/udev/udevd.c
+++ b/src/udev/udevd.c
@@ -864,21 +864,21 @@ static void handle_signal(struct udev *udev, int signo) {
if (worker->pid != pid)
continue;
- log_debug("worker ["PID_FMT"] exit", pid);
if (WIFEXITED(status)) {
- if (WEXITSTATUS(status) != 0)
- log_error("worker ["PID_FMT"] exit with return code %i",
- pid, WEXITSTATUS(status));
+ if (WEXITSTATUS(status) == 0)
+ log_debug("worker ["PID_FMT"] exited", pid);
+ else
+ log_warning("worker ["PID_FMT"] exited with return code %i", pid, WEXITSTATUS(status));
} else if (WIFSIGNALED(status)) {
- log_error("worker ["PID_FMT"] terminated by signal %i (%s)",
- pid, WTERMSIG(status), strsignal(WTERMSIG(status)));
+ log_warning("worker ["PID_FMT"] terminated by signal %i (%s)",
+ pid, WTERMSIG(status), strsignal(WTERMSIG(status)));
} else if (WIFSTOPPED(status)) {
- log_error("worker ["PID_FMT"] stopped", pid);
+ log_info("worker ["PID_FMT"] stopped", pid);
} else if (WIFCONTINUED(status)) {
- log_error("worker ["PID_FMT"] continued", pid);
+ log_info("worker ["PID_FMT"] continued", pid);
} else {
- log_error("worker ["PID_FMT"] exit with status 0x%04x", pid, status);
+ log_warning("worker ["PID_FMT"] exit with status 0x%04x", pid, status);
}
if (!WIFEXITED(status) || WEXITSTATUS(status) != 0) {