- User Since
- Aug 24 2017, 3:21 PM (68 w, 1 d)
Stephen Hurd, this is what I believe to be the correct fix for this issue. I'm not entirely certain how to reproduce the original setup that triggered the bug, so some help with that would be appreciated.
Thu, Dec 6
I know I'm late commenting on this, but I think the correct fix is to initialize tx_cidx_processed to txr->desc_count -1 when we initialize.
Nov 7 2018
Nov 5 2018
If I fix the driver to enable IP checksums on TSOs even when CSUM_IP isn't set, *and* I enable IFLIB_TSO_INIT_IP, then this patch isn't necessary.
Actually, I think the reason we set NEED_CSUM_ZERO in some hardware is only because they need the IP sum zero'd during TSO, not because they need it for non-TSO packets....
Nov 2 2018
Thanks! This looks great.
So this is a step in the right direction, but isn't a complete fix for the issue. We also need to enable the ip_sum workaround in this case as well. Possibly we can just move that check to after the TX_OFFLOAD4 and TX_OFFLOAD6 checks?
This doesn't seem like the right fix.
May 11 2018
- taskqgroup: make tqg_name a constant
- taskqgroup: mark name paramter of taskqgroup_attach as const
- iflib: mark irq allocation name parameter as constant
Looks like this involves a bit more cleanup to the taskqgroup code (since ultimately the name gets passed there). due to the scope I think I'll split it into two commits.
May 8 2018
Hmmm... I think there were. I'll double check this later today and see what I missed. I have a separate system for sending the reviews because my test system doesn't have direct access, I probably just forgot a few lines..
May 7 2018
May 4 2018
Remove the redundant message in iflib_device_register
Oops didn't mean to upload a new revision..
While we're cleaning this up, I'd either remove this print, or add a matching one for the tx setup, and remove the one in the caller. I realize that you can deduce which failed by how many prints you get, but that seems ... weird..
As mentioned on the other review, maybe remove the print in the caller, since it is now redundant..
Can we MFC this to stable/11? I originally found the bug on my FreeBSD 11.1 system.
May 3 2018
I've got a copy of the github mirror and wasnt sure how to convert the git commits back to SVN revisions, so I referenced the git commits where this was broken in my summary.