diff options
author | Steinar H. Gunderson <sesse@google.com> | 2007-09-29 14:21:47 +0000 |
---|---|---|
committer | Steinar H. Gunderson <sesse@google.com> | 2007-09-29 14:21:47 +0000 |
commit | c788efffd4c1b00494813639549f3fde36761c8f (patch) | |
tree | 9aed7308214f566cfae229d39c380675d3bab4b8 | |
parent | c1a475e70870d8b1ecff2e72452fcd0aec277e51 (diff) |
Read and process as many packets as possible in read_udp_packets, to avoid having to run the entire event loop once per packet. (Patch from the Google tree.)
-rw-r--r-- | ares/ares_process.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/ares/ares_process.c b/ares/ares_process.c index f3ead0a22..9e03e4034 100644 --- a/ares/ares_process.c +++ b/ares/ares_process.c @@ -399,13 +399,17 @@ static void read_udp_packets(ares_channel channel, fd_set *read_fds, * extra system calls and confusion. */ FD_CLR(server->udp_socket, read_fds); - count = sread(server->udp_socket, buf, sizeof(buf)); - if (count == -1 && try_again(SOCKERRNO)) - continue; - else if (count <= 0) - handle_error(channel, i, now); - - process_answer(channel, buf, (int)count, i, 0, now); + /* To reduce event loop overhead, read and process as many + * packets as we can. */ + do { + count = sread(server->udp_socket, buf, sizeof(buf)); + if (count == -1 && try_again(SOCKERRNO)) + continue; + else if (count <= 0) + handle_error(channel, i, now); + else + process_answer(channel, buf, (int)count, i, 0, now); + } while (count > 0); } } |