shaba/systemd
1
0
Fork 0
mirror of https://github.com/systemd/systemd.git synced 2024-11-07 01:27:11 +03:00
Code

bus: make sure sd_bus_get_timeout() returns a 0 timeout of there are already read but not dispatched messages

Browse Source
This commit is contained in:
Lennart Poettering 2013-10-16 06:01:46 +02:00
parent 6014597dd6
commit 8efd638179
1 changed files with 21 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/libsystemd-bus/sd-bus.c
View File

@ -1589,6 +1589,11 @@ int sd_bus_get_timeout(sd_bus *bus, uint64_t *timeout_usec) {
return 0; return 0;
} }
if (bus->rqueue_size > 0) {
*timeout_usec = 0;
return 1;
}
c = prioq_peek(bus->reply_callbacks_prioq); c = prioq_peek(bus->reply_callbacks_prioq);
if (!c) { if (!c) {
*timeout_usec = (uint64_t) -1; *timeout_usec = (uint64_t) -1;
@ -1924,7 +1929,7 @@ static int bus_poll(sd_bus *bus, bool need_more, uint64_t timeout_usec) {
struct pollfd p[2] = {}; struct pollfd p[2] = {};
int r, e, n; int r, e, n;
struct timespec ts; struct timespec ts;
usec_t until, m; usec_t m = (usec_t) -1;
assert(bus); assert(bus);
assert_return(BUS_IS_OPEN(bus->state), -ENOTCONN); assert_return(BUS_IS_OPEN(bus->state), -ENOTCONN);
@ -1934,18 +1939,24 @@ static int bus_poll(sd_bus *bus, bool need_more, uint64_t timeout_usec) {
return e; return e;
if (need_more) if (need_more)
/* The caller really needs some more data, he doesn't
* care about what's already read, or any timeouts
* except its own.*/
e |= POLLIN; e |= POLLIN;
else {
usec_t until;
/* The caller wants to process if there's something to
* process, but doesn't care otherwise */
r = sd_bus_get_timeout(bus, &until); r = sd_bus_get_timeout(bus, &until);
if (r < 0) if (r < 0)
return r; return r;
if (r == 0) if (r > 0) {
m = (uint64_t) -1;
else {
usec_t nw; usec_t nw;
nw = now(CLOCK_MONOTONIC); nw = now(CLOCK_MONOTONIC);
m = until > nw ? until - nw : 0; m = until > nw ? until - nw : 0;
} }
}
if (timeout_usec != (uint64_t) -1 && (m == (uint64_t) -1 || timeout_usec < m)) if (timeout_usec != (uint64_t) -1 && (m == (uint64_t) -1 || timeout_usec < m))
m = timeout_usec; m = timeout_usec;