aboutsummaryrefslogtreecommitdiffstats
path: root/net/core
diff options
context:
space:
mode:
authorPaolo Valente <paolo.valente@unimore.it>2012-11-05 20:29:24 +0000
committerDavid S. Miller <davem@davemloft.net>2012-11-07 15:37:04 -0500
commit3015f3d2a3cd9614294025849d3ed89fd2f3a7f5 (patch)
treee6190a5287b9df1e823f86f46d45ee1ffc5276da /net/core
parentcacb6ba0f36ab14a507f4ee7697e8332899015d2 (diff)
downloadkernel_goldelico_gta04-3015f3d2a3cd9614294025849d3ed89fd2f3a7f5.zip
kernel_goldelico_gta04-3015f3d2a3cd9614294025849d3ed89fd2f3a7f5.tar.gz
kernel_goldelico_gta04-3015f3d2a3cd9614294025849d3ed89fd2f3a7f5.tar.bz2
pkt_sched: enable QFQ to support TSO/GSO
If the max packet size for some class (configured through tc) is violated by the actual size of the packets of that class, then QFQ would not schedule classes correctly, and the data structures implementing the bucket lists may get corrupted. This problem occurs with TSO/GSO even if the max packet size is set to the MTU, and is, e.g., the cause of the failure reported in [1]. Two patches have been proposed to solve this problem in [2], one of them is a preliminary version of this patch. This patch addresses the above issues by: 1) setting QFQ parameters to proper values for supporting TSO/GSO (in particular, setting the maximum possible packet size to 64KB), 2) automatically increasing the max packet size for a class, lmax, when a packet with a larger size than the current value of lmax arrives. The drawback of the first point is that the maximum weight for a class is now limited to 4096, which is equal to 1/16 of the maximum weight sum. Finally, this patch also forcibly caps the timestamps of a class if they are too high to be stored in the bucket list. This capping, taken from QFQ+ [3], handles the unfrequent case described in the comment to the function slot_insert. [1] http://marc.info/?l=linux-netdev&m=134968777902077&w=2 [2] http://marc.info/?l=linux-netdev&m=135096573507936&w=2 [3] http://marc.info/?l=linux-netdev&m=134902691421670&w=2 Signed-off-by: Paolo Valente <paolo.valente@unimore.it> Tested-by: Cong Wang <amwang@redhat.com> Acked-by: Stephen Hemminger <shemminger@vyatta.com> Acked-by: Stephen Hemminger <shemminger@vyatta.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
0 files changed, 0 insertions, 0 deletions