diff options
| author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-07 23:59:20 +0000 |
|---|---|---|
| committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-07 23:59:20 +0000 |
| commit | 4f8a16a92c24d7992c34a6870804da6734a50930 (patch) | |
| tree | 3471830147c04bb3d94ece3ffb3bac69878fdab4 /net | |
| parent | 078822c1b98bb7d074962961494ed556cbca1382 (diff) | |
| download | chromium_src-4f8a16a92c24d7992c34a6870804da6734a50930.zip chromium_src-4f8a16a92c24d7992c34a6870804da6734a50930.tar.gz chromium_src-4f8a16a92c24d7992c34a6870804da6734a50930.tar.bz2 | |
Reduce parallel DNS resolutions to 6
Some routers black-hole resolutions beyond 6 in
parallel, and this should avoid the problem until
we craft a more adaptive implementation.
r=eroman
bug=122529
Review URL: http://codereview.chromium.org/10025005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131298 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net')
| -rw-r--r-- | net/base/host_resolver_impl.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/net/base/host_resolver_impl.cc b/net/base/host_resolver_impl.cc index a6a5a22..534f3a4 100644 --- a/net/base/host_resolver_impl.cc +++ b/net/base/host_resolver_impl.cc @@ -63,11 +63,12 @@ const unsigned kCacheEntryTTLSeconds = 60; // Default TTL for unsuccessful resolutions with ProcTask. const unsigned kNegativeCacheEntryTTLSeconds = 0; -// Maximum of 8 concurrent resolver threads (excluding retries). +// Maximum of 6 concurrent resolver threads (excluding retries). // Some routers (or resolvers) appear to start to provide host-not-found if // too many simultaneous resolutions are pending. This number needs to be -// further optimized, but 8 is what FF currently does. -static const size_t kDefaultMaxProcTasks = 8u; +// further optimized, but 8 is what FF currently does. We found some routers +// that limit this to 6, so we're temporarily holding it at that level. +static const size_t kDefaultMaxProcTasks = 6u; // Helper to mutate the linked list contained by AddressList to the given // port. Note that in general this is dangerous since the AddressList's |