Re: [PATCH next] r8169: add support for Byte Queue Limits
From: Tom Herbert <hidden>
Date: 2014-10-01 04:57:02
On Tue, Sep 30, 2014 at 12:42 AM, Florian Westphal [off-list ref] wrote:
Tom Herbert [off-list ref] wrote:quoted
Watch inflight and limit in the byte_queue_limits for the queue. inflight must always go back to zero when link goes idle.Yes, inflight goes to 0 when link is idle. Output of while true; do for n in inflight limit; do echo -n $n\ ; cat $n; done; sleep 1; done during netperf run, 100mbit peer: inflight 0 limit 3028 inflight 6056 limit 4542 [ no changes during test ] limit 4542 inflight 3028 limit 6122 inflight 0 limit 6122 [ changed cable to 1gbit peer, restart netperf ] inflight 37850 limit 36336 inflight 33308 limit 31794 inflight 33308 limit 31794 inflight 27252 limit 25738 [ no changes during test ] inflight 27252 limit 25738 inflight 0 limit 28766 [ change cable to 100mbit peer, restart netperf ] limit 28766 inflight 27370 limit 28766 inflight 4542 limit 5990 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 6056 limit 4542 inflight 0 [ end of test ] I think thats what its supposed to look like :-)
Looks good! Acked-by: Tom Herbert <redacted>