From 39c155ea0d8b24895017fd5cf48508924ce2016d Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Thu, 4 Apr 2013 20:07:48 +0200 Subject: journal: add sd_journal_get_timeout() call to public API Let's do the wake-up logic on NFS internally, making things simpler for users. --- src/journal/journal-internal.h | 2 ++ src/journal/libsystemd-journal.sym | 1 + src/journal/sd-journal.c | 43 +++++++++++++++++++++++++++++++++----- src/systemd/sd-journal.h | 3 ++- 4 files changed, 43 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/journal/journal-internal.h b/src/journal/journal-internal.h index bc9e44d42d..3accf14c05 100644 --- a/src/journal/journal-internal.h +++ b/src/journal/journal-internal.h @@ -126,6 +126,8 @@ struct sd_journal { size_t data_threshold; Set *errors; + + usec_t last_process_usec; }; char *journal_make_match_string(sd_journal *j); diff --git a/src/journal/libsystemd-journal.sym b/src/journal/libsystemd-journal.sym index e241318cb0..cdebf10ded 100644 --- a/src/journal/libsystemd-journal.sym +++ b/src/journal/libsystemd-journal.sym @@ -97,4 +97,5 @@ global: LIBSYSTEMD_JOURNAL_201 { global: sd_journal_get_events; + sd_journal_get_timeout; } LIBSYSTEMD_JOURNAL_198; diff --git a/src/journal/sd-journal.c b/src/journal/sd-journal.c index 3eba4cd0d1..c1f69827e9 100644 --- a/src/journal/sd-journal.c +++ b/src/journal/sd-journal.c @@ -1994,6 +1994,30 @@ _public_ int sd_journal_get_events(sd_journal *j) { return POLLIN; } +_public_ int sd_journal_get_timeout(sd_journal *j, uint64_t *timeout_usec) { + int fd; + + if (!j) + return -EINVAL; + if (!timeout_usec) + return -EINVAL; + + fd = sd_journal_get_fd(j); + if (fd < 0) + return fd; + + if (!j->on_network) { + *timeout_usec = (uint64_t) -1; + return 0; + } + + /* If we are on the network we need to regularly check for + * changes manually */ + + *timeout_usec = j->last_process_usec + JOURNAL_FILES_RECHECK_USEC; + return 1; +} + static void process_inotify_event(sd_journal *j, struct inotify_event *e) { Directory *d; int r; @@ -2076,6 +2100,8 @@ _public_ int sd_journal_process(sd_journal *j) { if (!j) return -EINVAL; + j->last_process_usec = now(CLOCK_MONOTONIC); + for (;;) { struct inotify_event *e; ssize_t l; @@ -2109,6 +2135,7 @@ _public_ int sd_journal_process(sd_journal *j) { _public_ int sd_journal_wait(sd_journal *j, uint64_t timeout_usec) { int r; + uint64_t t; assert(j); @@ -2127,12 +2154,18 @@ _public_ int sd_journal_wait(sd_journal *j, uint64_t timeout_usec) { return determine_change(j); } - if (j->on_network) { - /* If we are on the network we need to regularly check - * for changes manually */ + r = sd_journal_get_timeout(j, &t); + if (r < 0) + return r; + + if (t != (uint64_t) -1) { + usec_t n; + + n = now(CLOCK_MONOTONIC); + t = t > n ? t - n : 0; - if (timeout_usec == (uint64_t) -1 || timeout_usec > JOURNAL_FILES_RECHECK_USEC) - timeout_usec = JOURNAL_FILES_RECHECK_USEC; + if (timeout_usec == (uint64_t) -1 || timeout_usec > t) + timeout_usec = t; } do { diff --git a/src/systemd/sd-journal.h b/src/systemd/sd-journal.h index aa7693af70..afafee2d82 100644 --- a/src/systemd/sd-journal.h +++ b/src/systemd/sd-journal.h @@ -128,9 +128,10 @@ void sd_journal_restart_unique(sd_journal *j); int sd_journal_get_fd(sd_journal *j); int sd_journal_get_events(sd_journal *j); -int sd_journal_reliable_fd(sd_journal *j); +int sd_journal_get_timeout(sd_journal *j, uint64_t *timeout_usec); int sd_journal_process(sd_journal *j); int sd_journal_wait(sd_journal *j, uint64_t timeout_usec); +int sd_journal_reliable_fd(sd_journal *j); int sd_journal_get_catalog(sd_journal *j, char **text); int sd_journal_get_catalog_for_message_id(sd_id128_t id, char **ret); -- cgit v1.2.3-54-g00ecf