sfc: Remove dependence on NAPI polling in efx_test_eventq_irq()

We cannot safely assume that the NAPI handler will complete within the
20 ms that we allow for the event self-test.  The handler may be
deferred for longer than this, particularly on realtime kernels.

Instead, check whether either an event has been handled or (as in the
old failure path) whether an interrupt has been received and an event
has been delivered but not yet handled.  Use napi_disable() to
synchronize with the NAPI handler before checking, since it will
clear events before updating eventq_read_ptr.

Remove the test result chan.N.eventq.poll, since it is not an error
if the NAPI handler does not run during the test.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>

diff --git a/drivers/net/ethernet/sfc/ethtool.c b/drivers/net/ethernet/sfc/ethtool.c
index 1a7f7ba27bead1e74ceed32acc8c096a5210df5f..ba901167801340bae724fd79278309676dfd672e 100644 (file)
--- a/drivers/net/ethernet/sfc/ethtool.c
+++ b/drivers/net/ethernet/sfc/ethtool.c
@@ -404,10 +404,6 @@ static int efx_ethtool_fill_self_tests(struct efx_nic *efx,
                              &tests->eventq_int[channel->channel],
                              EFX_CHANNEL_NAME(channel),
                              "eventq.int", NULL);
-               efx_fill_test(n++, strings, data,
-                             &tests->eventq_poll[channel->channel],
-                             EFX_CHANNEL_NAME(channel),
-                             "eventq.poll", NULL);
        }
 
        efx_fill_test(n++, strings, data, &tests->registers,

diff --git a/drivers/net/ethernet/sfc/selftest.c b/drivers/net/ethernet/sfc/selftest.c
index 7718287ad0bc2353e75172e574ecf752a796883c..0f84789244755fff435bf037dff601c9c957fc11 100644 (file)
--- a/drivers/net/ethernet/sfc/selftest.c
+++ b/drivers/net/ethernet/sfc/selftest.c
@@ -161,11 +161,8 @@ static int efx_test_eventq_irq(struct efx_channel *channel,
                               struct efx_self_tests *tests)
 {
        struct efx_nic *efx = channel->efx;
-       unsigned int read_ptr, count;
-
-       tests->eventq_dma[channel->channel] = -1;
-       tests->eventq_int[channel->channel] = -1;
-       tests->eventq_poll[channel->channel] = -1;
+       unsigned int read_ptr;
+       bool napi_ran, dma_seen, int_seen;
 
        read_ptr = channel->eventq_read_ptr;
        channel->efx->last_irq_cpu = -1;
@@ -173,44 +170,48 @@ static int efx_test_eventq_irq(struct efx_channel *channel,
 
        efx_nic_generate_test_event(channel);
 
-       /* Wait for arrival of interrupt */
-       count = 0;
-       do {
-               schedule_timeout_uninterruptible(HZ / 100);
-
-               if (ACCESS_ONCE(channel->eventq_read_ptr) != read_ptr)
-                       goto eventq_ok;
-       } while (++count < 2);
-
-       netif_err(efx, drv, efx->net_dev,
-                 "channel %d timed out waiting for event queue\n",
-                 channel->channel);
-
-       /* See if interrupt arrived */
-       if (channel->efx->last_irq_cpu >= 0) {
-               netif_err(efx, drv, efx->net_dev,
-                         "channel %d saw interrupt on CPU%d "
-                         "during event queue test\n", channel->channel,
-                         raw_smp_processor_id());
-               tests->eventq_int[channel->channel] = 1;
+       /* Wait for arrival of interrupt.  NAPI processing may or may
+        * not complete in time, but we can cope in any case.
+        */
+       msleep(10);
+       napi_disable(&channel->napi_str);
+       if (channel->eventq_read_ptr != read_ptr) {
+               napi_ran = true;
+               dma_seen = true;
+               int_seen = true;
+       } else {
+               napi_ran = false;
+               dma_seen = efx_nic_event_present(channel);
+               int_seen = efx->last_irq_cpu >= 0;
        }
+       napi_enable(&channel->napi_str);
+       efx_nic_eventq_read_ack(channel);
+
+       tests->eventq_dma[channel->channel] = dma_seen ? 1 : -1;
+       tests->eventq_int[channel->channel] = int_seen ? 1 : -1;
 
-       /* Check to see if event was received even if interrupt wasn't */
-       if (efx_nic_event_present(channel)) {
+       if (dma_seen && int_seen) {
+               netif_dbg(efx, drv, efx->net_dev,
+                         "channel %d event queue passed (with%s NAPI)\n",
+                         channel->channel, napi_ran ? "" : "out");
+               return 0;
+       } else {
+               /* Report failure and whether either interrupt or DMA worked */
                netif_err(efx, drv, efx->net_dev,
-                         "channel %d event was generated, but "
-                         "failed to trigger an interrupt\n", channel->channel);
-               tests->eventq_dma[channel->channel] = 1;
+                         "channel %d timed out waiting for event queue\n",
+                         channel->channel);
+               if (int_seen)
+                       netif_err(efx, drv, efx->net_dev,
+                                 "channel %d saw interrupt "
+                                 "during event queue test\n",
+                                 channel->channel);
+               if (dma_seen)
+                       netif_err(efx, drv, efx->net_dev,
+                                 "channel %d event was generated, but "
+                                 "failed to trigger an interrupt\n",
+                                 channel->channel);
+               return -ETIMEDOUT;
        }
-
-       return -ETIMEDOUT;
- eventq_ok:
-       netif_dbg(efx, drv, efx->net_dev, "channel %d event queue passed\n",
-                 channel->channel);
-       tests->eventq_dma[channel->channel] = 1;
-       tests->eventq_int[channel->channel] = 1;
-       tests->eventq_poll[channel->channel] = 1;
-       return 0;
 }
 
 static int efx_test_phy(struct efx_nic *efx, struct efx_self_tests *tests,

diff --git a/drivers/net/ethernet/sfc/selftest.h b/drivers/net/ethernet/sfc/selftest.h
index dba5456e70f354796f57591f8d9cd0eff100273f..87abe2a5384679e02cefe06d24a161ad54513bb3 100644 (file)
--- a/drivers/net/ethernet/sfc/selftest.h
+++ b/drivers/net/ethernet/sfc/selftest.h
@@ -37,7 +37,6 @@ struct efx_self_tests {
        int interrupt;
        int eventq_dma[EFX_MAX_CHANNELS];
        int eventq_int[EFX_MAX_CHANNELS];
-       int eventq_poll[EFX_MAX_CHANNELS];
        /* offline tests */
        int registers;
        int phy_ext[EFX_MAX_PHY_TESTS];