net/ionic: improve link state handling
[dpdk.git] / drivers / net / ionic / ionic_main.c
index 0b420ee..ce5d113 100644 (file)
  * Copyright(c) 2018-2019 Pensando Systems, Inc. All rights reserved.
  */
 
+#include <stdbool.h>
+
 #include <rte_memzone.h>
 
 #include "ionic.h"
+#include "ionic_ethdev.h"
+#include "ionic_lif.h"
+
+static const char *
+ionic_error_to_str(enum ionic_status_code code)
+{
+       switch (code) {
+       case IONIC_RC_SUCCESS:
+               return "IONIC_RC_SUCCESS";
+       case IONIC_RC_EVERSION:
+               return "IONIC_RC_EVERSION";
+       case IONIC_RC_EOPCODE:
+               return "IONIC_RC_EOPCODE";
+       case IONIC_RC_EIO:
+               return "IONIC_RC_EIO";
+       case IONIC_RC_EPERM:
+               return "IONIC_RC_EPERM";
+       case IONIC_RC_EQID:
+               return "IONIC_RC_EQID";
+       case IONIC_RC_EQTYPE:
+               return "IONIC_RC_EQTYPE";
+       case IONIC_RC_ENOENT:
+               return "IONIC_RC_ENOENT";
+       case IONIC_RC_EINTR:
+               return "IONIC_RC_EINTR";
+       case IONIC_RC_EAGAIN:
+               return "IONIC_RC_EAGAIN";
+       case IONIC_RC_ENOMEM:
+               return "IONIC_RC_ENOMEM";
+       case IONIC_RC_EFAULT:
+               return "IONIC_RC_EFAULT";
+       case IONIC_RC_EBUSY:
+               return "IONIC_RC_EBUSY";
+       case IONIC_RC_EEXIST:
+               return "IONIC_RC_EEXIST";
+       case IONIC_RC_EINVAL:
+               return "IONIC_RC_EINVAL";
+       case IONIC_RC_ENOSPC:
+               return "IONIC_RC_ENOSPC";
+       case IONIC_RC_ERANGE:
+               return "IONIC_RC_ERANGE";
+       case IONIC_RC_BAD_ADDR:
+               return "IONIC_RC_BAD_ADDR";
+       case IONIC_RC_DEV_CMD:
+               return "IONIC_RC_DEV_CMD";
+       case IONIC_RC_ERROR:
+               return "IONIC_RC_ERROR";
+       case IONIC_RC_ERDMA:
+               return "IONIC_RC_ERDMA";
+       default:
+               return "IONIC_RC_UNKNOWN";
+       }
+}
+
+const char *
+ionic_opcode_to_str(enum ionic_cmd_opcode opcode)
+{
+       switch (opcode) {
+       case IONIC_CMD_NOP:
+               return "IONIC_CMD_NOP";
+       case IONIC_CMD_INIT:
+               return "IONIC_CMD_INIT";
+       case IONIC_CMD_RESET:
+               return "IONIC_CMD_RESET";
+       case IONIC_CMD_IDENTIFY:
+               return "IONIC_CMD_IDENTIFY";
+       case IONIC_CMD_GETATTR:
+               return "IONIC_CMD_GETATTR";
+       case IONIC_CMD_SETATTR:
+               return "IONIC_CMD_SETATTR";
+       case IONIC_CMD_PORT_IDENTIFY:
+               return "IONIC_CMD_PORT_IDENTIFY";
+       case IONIC_CMD_PORT_INIT:
+               return "IONIC_CMD_PORT_INIT";
+       case IONIC_CMD_PORT_RESET:
+               return "IONIC_CMD_PORT_RESET";
+       case IONIC_CMD_PORT_GETATTR:
+               return "IONIC_CMD_PORT_GETATTR";
+       case IONIC_CMD_PORT_SETATTR:
+               return "IONIC_CMD_PORT_SETATTR";
+       case IONIC_CMD_LIF_INIT:
+               return "IONIC_CMD_LIF_INIT";
+       case IONIC_CMD_LIF_RESET:
+               return "IONIC_CMD_LIF_RESET";
+       case IONIC_CMD_LIF_IDENTIFY:
+               return "IONIC_CMD_LIF_IDENTIFY";
+       case IONIC_CMD_LIF_SETATTR:
+               return "IONIC_CMD_LIF_SETATTR";
+       case IONIC_CMD_LIF_GETATTR:
+               return "IONIC_CMD_LIF_GETATTR";
+       case IONIC_CMD_RX_MODE_SET:
+               return "IONIC_CMD_RX_MODE_SET";
+       case IONIC_CMD_RX_FILTER_ADD:
+               return "IONIC_CMD_RX_FILTER_ADD";
+       case IONIC_CMD_RX_FILTER_DEL:
+               return "IONIC_CMD_RX_FILTER_DEL";
+       case IONIC_CMD_Q_INIT:
+               return "IONIC_CMD_Q_INIT";
+       case IONIC_CMD_Q_CONTROL:
+               return "IONIC_CMD_Q_CONTROL";
+       case IONIC_CMD_Q_IDENTIFY:
+               return "IONIC_CMD_Q_IDENTIFY";
+       case IONIC_CMD_RDMA_RESET_LIF:
+               return "IONIC_CMD_RDMA_RESET_LIF";
+       case IONIC_CMD_RDMA_CREATE_EQ:
+               return "IONIC_CMD_RDMA_CREATE_EQ";
+       case IONIC_CMD_RDMA_CREATE_CQ:
+               return "IONIC_CMD_RDMA_CREATE_CQ";
+       case IONIC_CMD_RDMA_CREATE_ADMINQ:
+               return "IONIC_CMD_RDMA_CREATE_ADMINQ";
+       default:
+               return "DEVCMD_UNKNOWN";
+       }
+}
+
+int
+ionic_adminq_check_err(struct ionic_admin_ctx *ctx, bool timeout)
+{
+       const char *name;
+       const char *status;
+
+       name = ionic_opcode_to_str(ctx->cmd.cmd.opcode);
+
+       if (ctx->comp.comp.status || timeout) {
+               status = ionic_error_to_str(ctx->comp.comp.status);
+               IONIC_PRINT(ERR, "%s (%d) failed: %s (%d)",
+                       name,
+                       ctx->cmd.cmd.opcode,
+                       timeout ? "TIMEOUT" : status,
+                       timeout ? -1 : ctx->comp.comp.status);
+               return -EIO;
+       }
+
+       IONIC_PRINT(DEBUG, "%s (%d) succeeded", name, ctx->cmd.cmd.opcode);
+
+       return 0;
+}
+
+static int
+ionic_wait_ctx_for_completion(struct ionic_lif *lif, struct ionic_qcq *qcq,
+               struct ionic_admin_ctx *ctx, unsigned long max_wait)
+{
+       unsigned long step_usec = IONIC_DEVCMD_CHECK_PERIOD_US;
+       unsigned long max_wait_usec = max_wait * 1000000L;
+       unsigned long elapsed_usec = 0;
+       int budget = 8;
+
+       while (ctx->pending_work && elapsed_usec < max_wait_usec) {
+               /*
+                * Locking here as adminq is served inline (this could be called
+                * from multiple places)
+                */
+               rte_spinlock_lock(&lif->adminq_service_lock);
+
+               ionic_qcq_service(qcq, budget, ionic_adminq_service, NULL);
+
+               rte_spinlock_unlock(&lif->adminq_service_lock);
+
+               rte_delay_us_block(step_usec);
+               elapsed_usec += step_usec;
+       }
+
+       return (!ctx->pending_work);
+}
+
+int
+ionic_adminq_post_wait(struct ionic_lif *lif, struct ionic_admin_ctx *ctx)
+{
+       struct ionic_qcq *qcq = lif->adminqcq;
+       bool done;
+       int err;
+
+       IONIC_PRINT(DEBUG, "Sending %s (%d) via the admin queue",
+               ionic_opcode_to_str(ctx->cmd.cmd.opcode), ctx->cmd.cmd.opcode);
+
+       err = ionic_adminq_post(lif, ctx);
+       if (err) {
+               IONIC_PRINT(ERR, "Failure posting %d to the admin queue (%d)",
+                       ctx->cmd.cmd.opcode, err);
+               return err;
+       }
+
+       done = ionic_wait_ctx_for_completion(lif, qcq, ctx,
+               IONIC_DEVCMD_TIMEOUT);
+
+       return ionic_adminq_check_err(ctx, !done /* timed out */);
+}
 
 static int
 ionic_dev_cmd_wait(struct ionic_dev *idev, unsigned long max_wait)
 {
-       unsigned long step_msec = 100;
-       unsigned int max_wait_msec = max_wait * 1000;
-       unsigned long elapsed_msec = 0;
+       unsigned long step_usec = IONIC_DEVCMD_CHECK_PERIOD_US;
+       unsigned long max_wait_usec = max_wait * 1000000L;
+       unsigned long elapsed_usec = 0;
        int done;
 
        /* Wait for dev cmd to complete.. but no more than max_wait sec */
@@ -19,20 +208,20 @@ ionic_dev_cmd_wait(struct ionic_dev *idev, unsigned long max_wait)
        do {
                done = ionic_dev_cmd_done(idev);
                if (done) {
-                       IONIC_PRINT(DEBUG, "DEVCMD %d done took %ld msecs",
-                               idev->dev_cmd->cmd.cmd.opcode,
-                               elapsed_msec);
+                       IONIC_PRINT(DEBUG, "DEVCMD %d done took %ld usecs",
+                               ioread8(&idev->dev_cmd->cmd.cmd.opcode),
+                               elapsed_usec);
                        return 0;
                }
 
-               msec_delay(step_msec);
+               rte_delay_us_block(step_usec);
 
-               elapsed_msec += step_msec;
-       } while (elapsed_msec < max_wait_msec);
+               elapsed_usec += step_usec;
+       } while (elapsed_usec < max_wait_usec);
 
-       IONIC_PRINT(DEBUG, "DEVCMD %d timeout after %ld msecs",
-               idev->dev_cmd->cmd.cmd.opcode,
-               elapsed_msec);
+       IONIC_PRINT(ERR, "DEVCMD %d timeout after %ld usecs",
+               ioread8(&idev->dev_cmd->cmd.cmd.opcode),
+               elapsed_usec);
 
        return -ETIMEDOUT;
 }
@@ -55,10 +244,12 @@ ionic_dev_cmd_wait_check(struct ionic_dev *idev, unsigned long max_wait)
        int err;
 
        err = ionic_dev_cmd_wait(idev, max_wait);
-       if (err)
-               return err;
 
-       return ionic_dev_cmd_check_error(idev);
+       if (!err)
+               err = ionic_dev_cmd_check_error(idev);
+
+       IONIC_PRINT(DEBUG, "dev_cmd returned %d", err);
+       return err;
 }
 
 int
@@ -113,22 +304,18 @@ int
 ionic_init(struct ionic_adapter *adapter)
 {
        struct ionic_dev *idev = &adapter->idev;
-       int err;
 
        ionic_dev_cmd_init(idev);
-       err = ionic_dev_cmd_wait_check(idev, IONIC_DEVCMD_TIMEOUT);
-       return err;
+       return ionic_dev_cmd_wait_check(idev, IONIC_DEVCMD_TIMEOUT);
 }
 
 int
 ionic_reset(struct ionic_adapter *adapter)
 {
        struct ionic_dev *idev = &adapter->idev;
-       int err;
 
        ionic_dev_cmd_reset(idev);
-       err = ionic_dev_cmd_wait_check(idev, IONIC_DEVCMD_TIMEOUT);
-       return err;
+       return ionic_dev_cmd_wait_check(idev, IONIC_DEVCMD_TIMEOUT);
 }
 
 int
@@ -153,12 +340,12 @@ ionic_port_identify(struct ionic_adapter *adapter)
                                ioread32(&idev->dev_cmd->data[i]);
        }
 
-       IONIC_PRINT(INFO, "speed %d ", ident->port.config.speed);
-       IONIC_PRINT(INFO, "mtu %d ", ident->port.config.mtu);
-       IONIC_PRINT(INFO, "state %d ", ident->port.config.state);
-       IONIC_PRINT(INFO, "an_enable %d ", ident->port.config.an_enable);
-       IONIC_PRINT(INFO, "fec_type %d ", ident->port.config.fec_type);
-       IONIC_PRINT(INFO, "pause_type %d ", ident->port.config.pause_type);
+       IONIC_PRINT(INFO, "speed %d", ident->port.config.speed);
+       IONIC_PRINT(INFO, "mtu %d", ident->port.config.mtu);
+       IONIC_PRINT(INFO, "state %d", ident->port.config.state);
+       IONIC_PRINT(INFO, "an_enable %d", ident->port.config.an_enable);
+       IONIC_PRINT(INFO, "fec_type %d", ident->port.config.fec_type);
+       IONIC_PRINT(INFO, "pause_type %d", ident->port.config.pause_type);
        IONIC_PRINT(INFO, "loopback_mode %d",
                ident->port.config.loopback_mode);
 
@@ -199,8 +386,7 @@ ionic_port_init(struct ionic_adapter *adapter)
        idev->port_info_sz = RTE_ALIGN(sizeof(*idev->port_info), PAGE_SIZE);
 
        snprintf(z_name, sizeof(z_name), "%s_port_%s_info",
-               IONIC_DRV_NAME,
-               adapter->pci_dev->device.name);
+               IONIC_DRV_NAME, adapter->name);
 
        idev->port_info_z = ionic_memzone_reserve(z_name, idev->port_info_sz,
                SOCKET_ID_ANY);