]> git.baikalelectronics.ru Git - kernel.git/commit
gro_cells: mark napi struct as not busy poll candidates
authorEric Dumazet <edumazet@google.com>
Tue, 15 Nov 2016 00:28:42 +0000 (16:28 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Nov 2016 03:27:27 +0000 (22:27 -0500)
commit35ec025598c54c32fafb1c19339f7edf14e88f78
treebb89a8330ce7936f26ef120d829e23b9a9de2527
parent29d989ace455e19e849526e6c9bfd2adaac35508
gro_cells: mark napi struct as not busy poll candidates

Rolf Neugebauer reported very long delays at netns dismantle.

Eric W. Biederman was kind enough to look at this problem
and noticed synchronize_net() occurring from netif_napi_del() that was
added in linux-4.5

Busy polling makes no sense for tunnels NAPI.
If busy poll is used for sessions over tunnels, the poller will need to
poll the physical device queue anyway.

netif_tx_napi_add() could be used here, but function name is misleading,
and renaming it is not stable material, so set NAPI_STATE_NO_BUSY_POLL
bit directly.

This will avoid inserting gro_cells napi structures in napi_hash[]
and avoid the problematic synchronize_net() (per possible cpu) that
Rolf reported.

Fixes: 86afab6074af ("net: provide generic busy polling to all NAPI drivers")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Reported-by: Rolf Neugebauer <rolf.neugebauer@docker.com>
Reported-by: Eric W. Biederman <ebiederm@xmission.com>
Acked-by: Cong Wang <xiyou.wangcong@gmail.com>
Tested-by: Rolf Neugebauer <rolf.neugebauer@docker.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/gro_cells.h