Re: [PATCH net v2] net: sched: do not requeue a NULL skb

From: Eric Dumazet
Date: Mon Apr 11 2016 - 19:49:07 EST


On Mon, 2016-04-11 at 16:19 -0700, Cong Wang wrote:

> My point is, for example, in OOM case, we don't know processin
> more SKB would make it better or worse. Maybe we really need to
> check the error code to decide to continue to exit?

Really, given this bug has been there for a long time (v3.18 ???), I
doubt it matters.

Nothing can tell that following packets in the qdisc need any
transformation, and memory allocations.

So I would just fix the bug in the simplest way.

__qdisc_run() has all the checks needed to yield when needed
(if (quota <= 0 || need_resched())) , no need to add more complexity
there.