net: Always poll at least one device in net_rx_action
authorHerbert Xu <herbert@gondor.apana.org.au>
Sat, 20 Dec 2014 20:16:24 +0000 (07:16 +1100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Dec 2014 04:20:21 +0000 (23:20 -0500)
We should only perform the softnet_break check after we have polled
at least one device in net_rx_action.  Otherwise a zero or negative
setting of netdev_budget can lock up the whole system.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index c0cf1293df0605d75551a56d1d03b251ece6918b..b85eba9dfd88a52335fa5bb69e4a36833dfcdbaf 100644 (file)
@@ -4634,16 +4634,15 @@ static void net_rx_action(struct softirq_action *h)
        while (!list_empty(&list)) {
                struct napi_struct *n;
 
+               n = list_first_entry(&list, struct napi_struct, poll_list);
+               budget -= napi_poll(n, &repoll);
+
                /* If softirq window is exhausted then punt.
                 * Allow this to run for 2 jiffies since which will allow
                 * an average latency of 1.5/HZ.
                 */
                if (unlikely(budget <= 0 || time_after_eq(jiffies, time_limit)))
                        goto softnet_break;
-
-
-               n = list_first_entry(&list, struct napi_struct, poll_list);
-               budget -= napi_poll(n, &repoll);
        }
 
        if (!sd_has_rps_ipi_waiting(sd) &&