On 21 Aug 2016, at 19:18, Andrey V. Elsukov <a...@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
_______________________________________________
freebsd-net@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-net
To unsubscribe, send any mail to "freebsd-net-unsubscr...@freebsd.org"

Reply via email to