X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=lib%2Fpoll-loop.c;h=7e328bc24ce486b2243cdfce1929c5464f513b02;hb=a088a1ffeba3d113c1be918969085a5c0de0d518;hp=de080b4c9e5d28c49fa4997fc92f5f1cf68f426a;hpb=1e276d1a10539a8cd97d2ad63c073a9a43f0f1ef;p=openvswitch diff --git a/lib/poll-loop.c b/lib/poll-loop.c index de080b4c..7e328bc2 100644 --- a/lib/poll-loop.c +++ b/lib/poll-loop.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2008, 2009, 2010, 2011 Nicira Networks. + * Copyright (c) 2008, 2009, 2010, 2011, 2012 Nicira, Inc. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -55,22 +55,14 @@ struct poll_waiter { /* All active poll waiters. */ static struct list waiters = LIST_INITIALIZER(&waiters); -/* Max time to wait in next call to poll_block(), in milliseconds, or -1 to +/* Time at which to wake up the next call to poll_block(), in milliseconds as + * returned by time_msec(), LLONG_MIN to wake up immediately, or LLONG_MAX to * wait forever. */ -static int timeout = -1; +static long long int timeout_when = LLONG_MAX; /* Location where waiter created. */ static const char *timeout_where; -/* Array of file descriptors from last run of poll_block(). */ -static struct pollfd *pollfds; - -/* Allocated size of pollfds. */ -static size_t max_pollfds; - -/* Current number of elements in pollfds. */ -static int n_pollfds; - static struct poll_waiter *new_waiter(int fd, short int events, const char *where); @@ -91,16 +83,6 @@ poll_fd_wait(int fd, short int events, const char *where) return new_waiter(fd, events, where); } -/* The caller must ensure that 'msec' is not negative. */ -static void -poll_timer_wait__(int msec, const char *where) -{ - if (timeout < 0 || msec < timeout) { - timeout = msec; - timeout_where = where; - } -} - /* Causes the following call to poll_block() to block for no more than 'msec' * milliseconds. If 'msec' is nonpositive, the following call to poll_block() * will not block at all. @@ -114,14 +96,25 @@ poll_timer_wait__(int msec, const char *where) void poll_timer_wait(long long int msec, const char *where) { - poll_timer_wait__((msec < 0 ? 0 - : msec > INT_MAX ? INT_MAX - : msec), - where); + long long int now = time_msec(); + long long int when; + + if (msec <= 0) { + /* Wake up immediately. */ + when = LLONG_MIN; + } else if ((unsigned long long int) now + msec <= LLONG_MAX) { + /* Normal case. */ + when = now + msec; + } else { + /* now + msec would overflow. */ + when = LLONG_MAX; + } + + poll_timer_wait_until(when, where); } /* Causes the following call to poll_block() to wake up when the current time, - * as returned by time_msec(), reaches 'msec' or later. If 'msec' is earlier + * as returned by time_msec(), reaches 'when' or later. If 'when' is earlier * than the current time, the following call to poll_block() will not block at * all. * @@ -132,13 +125,12 @@ poll_timer_wait(long long int msec, const char *where) * Ordinarily the 'where' argument is supplied automatically; see poll-loop.h * for more information. */ void -poll_timer_wait_until(long long int msec, const char *where) +poll_timer_wait_until(long long int when, const char *where) { - long long int now = time_msec(); - poll_timer_wait__((msec <= now ? 0 - : msec < now + INT_MAX ? msec - now - : INT_MAX), - where); + if (when < timeout_when) { + timeout_when = when; + timeout_where = where; + } } /* Causes the following call to poll_block() to wake up immediately, without @@ -165,7 +157,8 @@ poll_immediate_wake(const char *where) static void log_wakeup(const char *where, const struct pollfd *pollfd, int timeout) { - static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(120, 120); + static struct vlog_rate_limit trace_rl = VLOG_RATE_LIMIT_INIT(1, 1); + static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(10, 10); enum vlog_level level; int cpu_usage; struct ds s; @@ -208,6 +201,11 @@ log_wakeup(const char *where, const struct pollfd *pollfd, int timeout) } if (cpu_usage >= 0) { ds_put_format(&s, " (%d%% CPU usage)", cpu_usage); + + if (!vlog_should_drop(THIS_MODULE, level, &trace_rl)) { + ds_put_char(&s, '\n'); + format_backtraces(&s, 2); + } } VLOG(level, "%s", ds_cstr(&s)); ds_destroy(&s); @@ -219,8 +217,12 @@ log_wakeup(const char *where, const struct pollfd *pollfd, int timeout) void poll_block(void) { + static struct pollfd *pollfds; + static size_t max_pollfds; + struct poll_waiter *pw, *next; - int n_waiters; + int n_waiters, n_pollfds; + int elapsed; int retval; /* Register fatal signal events before actually doing any real work for @@ -242,15 +244,15 @@ poll_block(void) n_pollfds++; } - if (!timeout) { + if (timeout_when == LLONG_MIN) { COVERAGE_INC(poll_zero_timeout); } - retval = time_poll(pollfds, n_pollfds, timeout); + retval = time_poll(pollfds, n_pollfds, timeout_when, &elapsed); if (retval < 0) { static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5); VLOG_ERR_RL(&rl, "poll: %s", strerror(-retval)); } else if (!retval) { - log_wakeup(timeout_where, NULL, timeout); + log_wakeup(timeout_where, NULL, elapsed); } LIST_FOR_EACH_SAFE (pw, next, node, &waiters) { @@ -260,7 +262,7 @@ poll_block(void) poll_cancel(pw); } - timeout = -1; + timeout_when = LLONG_MAX; timeout_where = NULL; /* Handle any pending signals before doing anything else. */ @@ -281,25 +283,6 @@ poll_cancel(struct poll_waiter *pw) free(pw); } } - -/* Checks whether the given file descriptor caused the poll loop to wake up - * in the previous iteration. If it did, returns a bitmask of the events - * that caused the wakeup. Otherwise returns 0; - */ -short int -poll_fd_woke(int fd) -{ - int i; - short int events = 0; - - for (i = 0; i < n_pollfds; i++) { - if (pollfds[i].fd == fd) { - events |= pollfds[i].revents; - } - } - - return events; -} /* Creates and returns a new poll_waiter for 'fd' and 'events'. */ static struct poll_waiter *