There are netif_napi_add() and napi_enable() in for_each_possible_cpu loop in function cvm_oct_rx_initialize(). So it's logical to add napi_disable() in for_each_possible_cpu loop in function cvm_oct_rx_shutdown().
Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Marina Makienko <[email protected]> --- drivers/staging/octeon/ethernet-rx.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/staging/octeon/ethernet-rx.c b/drivers/staging/octeon/ethernet-rx.c index 34afc16..e5b62fc 100644 --- a/drivers/staging/octeon/ethernet-rx.c +++ b/drivers/staging/octeon/ethernet-rx.c @@ -558,6 +558,8 @@ void cvm_oct_rx_shutdown(void) { int i; /* Shutdown all of the NAPIs */ - for_each_possible_cpu(i) + for_each_possible_cpu(i) { netif_napi_del(&cvm_oct_napi[i].napi); + napi_disable(&cvm_oct_napi[i].napi); + } } -- 1.7.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

