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 <[email protected]>
---
 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
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to