aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhil Blundell <pb@reciva.com>2009-01-13 11:51:04 +0000
committerPhil Blundell <pb@reciva.com>2009-01-13 11:51:04 +0000
commit35d8fb05017af9ee8514cb317cb37fc2ce5185a9 (patch)
tree9e5165c13033b6700865b7e5026e92aff50314e9
parent87c71953bee0760d106b12ef4f11c09ddfdeffbd (diff)
- ares__send_query() now varies the retry timeout pseudo-randomly to avoid
packet storms when several queries were started at the same time.
-rw-r--r--ares/CHANGES6
-rw-r--r--ares/ares_process.c5
2 files changed, 9 insertions, 2 deletions
diff --git a/ares/CHANGES b/ares/CHANGES
index 1c6b6070b..5bb02f744 100644
--- a/ares/CHANGES
+++ b/ares/CHANGES
@@ -1,6 +1,10 @@
Changelog for the c-ares project
-* January 11 2008 (Daniel Stenberg)
+* January 13 2009 (Phil Blundell)
+- ares__send_query() now varies the retry timeout pseudo-randomly to avoid
+ packet storms when several queries were started at the same time.
+
+* January 11 2009 (Daniel Stenberg)
- Phil Blundell added the internal function ares__expand_name_for_response()
that is now used by the ares_parse_*_reply() functions instead of the
ares_expand_name() simply to easier return ARES_EBADRESP for the cases where
diff --git a/ares/ares_process.c b/ares/ares_process.c
index 8c25b11da..3aa75cdae 100644
--- a/ares/ares_process.c
+++ b/ares/ares_process.c
@@ -715,6 +715,7 @@ void ares__send_query(ares_channel channel, struct query *query,
{
struct send_request *sendreq;
struct server_state *server;
+ int timeplus;
server = &channel->servers[query->server];
if (query->using_tcp)
@@ -778,9 +779,11 @@ void ares__send_query(ares_channel channel, struct query *query,
return;
}
}
+ timeplus = channel->timeout << (query->try / channel->nservers);
+ timeplus = (timeplus * (9 + (rand () & 7))) / 16;
query->timeout = *now;
ares__timeadd(&query->timeout,
- channel->timeout << (query->try / channel->nservers));
+ timeplus);
/* Keep track of queries bucketed by timeout, so we can process
* timeout events quickly.
*/