On Thu, Mar 15, 2012 at 6:08 PM, Matthew Toseland
<toad at amphibian.dyndns.org> wrote:
> On Thursday 15 Mar 2012 20:24:10 Evan Daniel wrote:
>> On Thu, Mar 15, 2012 at 10:22 AM, Matthew Toseland
>> <toad at amphibian.dyndns.org> wrote:
>> > commit cb21f7d4bc5e1940b6acdb49004b912f8fb705e5
>> > Author: Matthew Toseland <toad at amphibian.dyndns.org>
>> > Date: ? Thu Mar 15 14:18:10 2012 +0000
>> >
>> > ? ?Tweak peers limit by bandwidth SCALING_CONSTANT: Lots of peers are on 
>> > the high end of the peers limit. Reduce the multiplier, so it takes more 
>> > bandwidth to have 40 peers; this should improve the average bandwidth per 
>> > connection.
>> >
>> > This is actually from evanbd:
>> >
>> > [17:11:04] <toad_> evanbd: what was your new proposed formula for 
>> > bandwidth/peers limit?
>> > [17:11:19] <toad_> evanbd: we should try that out in 1406 if there aren't 
>> > any other major changes, and in 1407 if there are
>> > [17:11:43] <evanbd> Currently, it's peers = sqrt(12*limit), with limit in 
>> > KiB/s.
>> > [17:11:47] <toad_> ok
>> > [17:11:54] <evanbd> I'm proposing changing the 12 to 6 or so.
>> > [17:12:07] <toad_> ahhh
>> > [17:12:26] <toad_> meaning high bandwidth nodes have the same number of 
>> > peers but middle ones have less => higher bandwidth per peer
>> > [17:12:34] <evanbd> Exactly.
>> > [17:13:03] <toad_> okay, we should do that soon
>> >
>> > This should be in 1406 or 1407.
>>
>> Oops, had that in my branch, looks like I forgot to do a pull request.
>> Anyway, thanks!
>
> This should be merged soon after 1406 IMHO. We want to have one network level 
> change per build, and there are some minor things in 1406 (deadlock fixes and 
> the location thingy).

I like that plan.

Evan

Reply via email to