On 06/03/2018 10:16 PM, Sarah Newman wrote:
On 03/06/2018 08:13 AM, Tariq Toukan wrote:
I have a general question about the process.
I don't totally get what branch this patch is targeted to.
It touches critical areas in datapath and should go through regression tests
before it is accepted to any branch.
This one is against 4.9 -
I assume you'd be interested in applying a fix to all currently maintained
stable branches between 3.6 to 4.11, per
I know of at least one other person whose workaround has been to disable GRO,
but I don't understand the networking code well enough to guarantee that
disabling GRO means the problem will never occur under all possible error
conditions. If it's guaranteed that disabling GRO will definitely mitigate
the bug, probably it's better to stop supporting GRO for these versions of the
driver instead of trying to implement some other fix.
I don't think we have a guarantee, it just happens to work.
Also, disabling GRO will cause dramatic performance degradation.
We should fix this, not WA.