On Thu, Nov 17, 2022, 15:42 Tobias Fiebig <tob...@fiebig.nl> wrote: > Heho, > I gave v3 a shot and it performs as expected; For a requested MSS of 1320, > TSO consistently uses a 1308 MSS. So for me, this patch works. Thanks for > fixing this. :-) > > Sadly, I do not have boxes to test with .1q around; If none of you has > either, and that should be tested as well, I can give it a shot in the > coming days, but it might take some time. >
Awesome, thanks for your help! I don't have a .1q setup on hand. Stefan > Side note: I found the 'missing' 12b in 12b of TCP options being added. > :-| So sorry for that noise. > > With best regards, > Tobias > > MTU1500 > RTL8139: +++ C+ mode offloaded task TSO IP data 7272 frame data 7292 > specified MSS=1448 > RTL8139: +++ C+ mode offloaded task TSO IP data 8720 frame data 8740 > specified MSS=1448 > RTL8139: +++ C+ mode offloaded task TSO IP data 8720 frame data 8740 > specified MSS=1448 > RTL8139: +++ C+ mode offloaded task TSO IP data 10168 frame data 10188 > specified MSS=1448 > > MTU1320 > RTL8139: +++ C+ mode offloaded task TSO IP data 2648 frame data 2668 > specified MSS=1308 > RTL8139: +++ C+ mode offloaded task TSO IP data 10496 frame data 10516 > specified MSS=1308 > RTL8139: +++ C+ mode offloaded task TSO IP data 6572 frame data 6592 > specified MSS=1308 > RTL8139: +++ C+ mode offloaded task TSO IP data 6572 frame data 6592 > specified MSS=1308 > >