[systemd-devel] [PATCH] journald: ignore failure to watch hostname_fd on older kernels

Dave Reisner dreisner at archlinux.org
Fri Feb 21 08:22:50 PST 2014


Prior to 3.2, /proc/sys/kernel/hostname isn't a pollable file and
sd_event_add_io will return EPERM. Ignore this failure, since it isn't
critical to journald operation.
---
Reported and tested by user sraue on IRC.

 src/journal/journald-server.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/journal/journald-server.c b/src/journal/journald-server.c
index 5872e91..add6cf5 100644
--- a/src/journal/journald-server.c
+++ b/src/journal/journald-server.c
@@ -1430,6 +1430,14 @@ static int server_open_hostname(Server *s) {
 
         r = sd_event_add_io(s->event, &s->hostname_event_source, s->hostname_fd, 0, dispatch_hostname_change, s);
         if (r < 0) {
+                /* kernels prior to 3.2 don't support polling this file.
+                 * quietly ignore the failure. */
+                if (r == -EPERM) {
+                        close_nointr_nofail(s->hostname_fd);
+                        s->hostname_fd = -1;
+                        return 0;
+                }
+
                 log_error("Failed to register hostname fd in event loop: %s", strerror(-r));
                 return r;
         }
-- 
1.9.0



More information about the systemd-devel mailing list