Looks good, thanks.

Ethan

On Wed, May 30, 2012 at 5:16 PM, Ben Pfaff <b...@nicira.com> wrote:
> This code, which dates to August 2008, initially sets the packet-in
> scheduler token buckets to 10% full, without any rationale.  I suspect
> that this is just a typo for 100% full, which I think would be more
> conventional, so this commit switches to that.
>
> Signed-off-by: Ben Pfaff <b...@nicira.com>
> ---
>  ofproto/pinsched.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/ofproto/pinsched.c b/ofproto/pinsched.c
> index 41e9c8d..4a939b8 100644
> --- a/ofproto/pinsched.c
> +++ b/ofproto/pinsched.c
> @@ -268,7 +268,7 @@ pinsched_create(int rate_limit, int burst_limit)
>     ps->n_queued = 0;
>     ps->next_txq = NULL;
>     ps->last_fill = time_msec();
> -    ps->tokens = rate_limit * 100;
> +    ps->tokens = rate_limit * 1000;
>     ps->n_txq = 0;
>     ps->n_normal = 0;
>     ps->n_limited = 0;
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to