Re: Softirq priority inversion from "softirq: reduce latencies"

Eric Dumazet
2016-02-29T18:25:47Z

On lun., 2016-02-29 at 10:05 -0800, Peter Hurley wrote:

> While I appreciate the attempt, that's not the problem.
>
> Just to be clear
>
> if (time_before(jiffies, end) && !need_resched() &&
> --max_restart)
> goto restart;
>
> aborts softirq *even if 0ns have elapsed*, if NET_RX has woken a process.

Sure, now remove the 1st and 2nd condition.

You would still 'abort' (ie wakeup ksoftirqd really) when --max_restart
becomes 0

So, instead of some subtle load dependent bug, you know have a reliable
trigger.

The fact it took 3 years for someone to complain about this change
should tell us something really.

The only way for your bug to hide would be to remove all the 'break
infinite loop' logic.

And this is not going to happen.