Re: [PATCH] icmp: Restore resistence to abnormal messages

On Wed, Nov 16, 2016 at 2:14 AM, Florian Westphal [off-list ref] wrote:

Vicente Jiménez [off-list ref] wrote:

quoted

1- add warning with pr_warn_ratelimited. I like this idea. I also
though about adding some message but I have no kernel experience and I
preferred to have just a working solution.

I added this only to show whats happening.

I don't like such printks because end users can't do anything about it.

What about using net_dbg_ratelimited macro? it only adds messages if
debug is enabled.

quoted

Finally, both patches decrement current packet by a value: Mine by 2
and Florian's by 8 bytes. Both arbitrary values. Personally I prefer
to go by small steps. If the small step fails, it just iterate again
and with 4 iterations, my patch also decrement the original value by 8
bytes (4x2).
Basically they are the same but my patch take smaller steps and miss
the warning message.

IIRC I chose 8 because connection recovered faster in my case.

I have not experienced this issue again (I dropped the patch from
my kernel at some point and the connection stalls did not reappear so
this got fixed elsewhere).

My issue is permanent for now in various locations. We have to
decrease MTU manually on all devices with newer kernels. We don't have
direct access to those abnormal routers because they are managed by a
communication provider that think the network had no problem because
all their Windows machines apparently work perfectly.

-- 
cheers
vicente