svn commit: r304313 - head/sys/net

[Robert Watson]

On 21 Aug 2016, at 19:18, Andrey V. Elsukov <ae at FreeBSD.org> wrote:

> On 21.08.16 20:33, Adrian Chadd wrote:
>>> Does this make sense?
>> 
>> Right. Let me go and look into it a little more. I think we may want 
>> to revert the change (which just landed to -11, so maybe revert that 
>> too) so I can test both of them out for correctness.
>> 
>> Andrey, I'm sorry for suggesting we back it out, but I'd like to
>> make sure we don't break networking on 11. :) Is that okay? I will
>> look at this tonight/tomorrow.
> 
> Hi,
> 
> I already wanted to merge it into releneg/11.0.
> 
> ...
> I think we can keep this in the release, but if you have an idea how to
> improve it - just do it and then do MFC.

I tend to take the view that crashing is worse than not crashing, so we should ensure something appears in releng/11.0 that is suitable to ship in a release — at the very least this patch. However, we do need to be careful about packet reordering, as it can be extremely hard to diagnose while leading to observable performance degradation, which is clearly not the goal of enabling higher netisr thread counts. The change required to do this properly should be extremely small, and it’s worth getting it sorted out and merged for the release, if we can.

Robert