[ovs-dev] [PATCH 1/3] bond: Change rebalance ratio to be more fair
zoltan.kiss at citrix.com
Mon Jan 7 22:23:25 UTC 2013
On 07/01/13 22:12, Ethan Jackson wrote:
> This seems fine to me. My only comment is that we should probably add a
> bit more explanation. It took me 20 minutes just to figure out what
> this code is intended to do.
> > + /* Ideal ratio is 1, move it if the new ratio is
> closer to it. */
> Perhaps we could expand this comment a bit? Something like . . .
> /* We're aiming for an ideal ratio of 1, meaning both the 'from' and
> 'to' slave have the same load. Therefore, we only move an entry if it
> decreases the load on from, and brings us closer to equal traffic load. */
> Perhaps the commit message could be expanded in a similar manner as
> well? I don't feel as strongly about that though.
Expanding comment and commit messages sounds good to me.
More information about the dev