bus/pci: change IOVA as VA flag name
[dpdk.git] / drivers / net / sfc / base / ef10_nic.c
index 50e23b7..b25ce19 100644 (file)
@@ -10,7 +10,7 @@
 #include "mcdi_mon.h"
 #endif
 
-#if EFSYS_OPT_HUNTINGTON || EFSYS_OPT_MEDFORD || EFSYS_OPT_MEDFORD2
+#if EFX_OPTS_EF10()
 
 #include "ef10_tlv_layout.h"
 
@@ -24,9 +24,7 @@ efx_mcdi_get_port_assignment(
                MC_CMD_GET_PORT_ASSIGNMENT_OUT_LEN);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        req.emr_cmd = MC_CMD_GET_PORT_ASSIGNMENT;
        req.emr_in_buf = payload;
@@ -70,9 +68,7 @@ efx_mcdi_get_port_modes(
                MC_CMD_GET_PORT_MODES_OUT_LEN);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        req.emr_cmd = MC_CMD_GET_PORT_MODES;
        req.emr_in_buf = payload;
@@ -227,7 +223,7 @@ fail1:
        return (rc);
 }
 
-static __checkReturn           efx_rc_t
+       __checkReturn           efx_rc_t
 efx_mcdi_vadaptor_alloc(
        __in                    efx_nic_t *enp,
        __in                    uint32_t port_id)
@@ -237,8 +233,6 @@ efx_mcdi_vadaptor_alloc(
                MC_CMD_VADAPTOR_ALLOC_OUT_LEN);
        efx_rc_t rc;
 
-       EFSYS_ASSERT3U(enp->en_vport_id, ==, EVB_PORT_ID_NULL);
-
        req.emr_cmd = MC_CMD_VADAPTOR_ALLOC;
        req.emr_in_buf = payload;
        req.emr_in_length = MC_CMD_VADAPTOR_ALLOC_IN_LEN;
@@ -265,7 +259,7 @@ fail1:
        return (rc);
 }
 
-static __checkReturn           efx_rc_t
+       __checkReturn           efx_rc_t
 efx_mcdi_vadaptor_free(
        __in                    efx_nic_t *enp,
        __in                    uint32_t port_id)
@@ -308,9 +302,7 @@ efx_mcdi_get_mac_address_pf(
                MC_CMD_GET_MAC_ADDRESSES_OUT_LEN);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        req.emr_cmd = MC_CMD_GET_MAC_ADDRESSES;
        req.emr_in_buf = payload;
@@ -366,9 +358,7 @@ efx_mcdi_get_mac_address_vf(
                MC_CMD_VPORT_GET_MAC_ADDRESSES_OUT_LENMAX);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        req.emr_cmd = MC_CMD_VPORT_GET_MAC_ADDRESSES;
        req.emr_in_buf = payload;
@@ -430,9 +420,7 @@ efx_mcdi_get_clock(
                MC_CMD_GET_CLOCK_OUT_LEN);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        req.emr_cmd = MC_CMD_GET_CLOCK;
        req.emr_in_buf = payload;
@@ -892,9 +880,7 @@ ef10_nic_pio_alloc(
        uint32_t buf, blk;
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
        EFSYS_ASSERT(bufnump);
        EFSYS_ASSERT(handlep);
        EFSYS_ASSERT(blknump);
@@ -1122,6 +1108,12 @@ ef10_get_datapath_caps(
        else
                encp->enc_datapath_cap_evb = B_FALSE;
 
+       /* Check if the firmware supports vport reconfiguration */
+       if (CAP_FLAGS1(req, VPORT_RECONFIGURE))
+               encp->enc_vport_reconfigure_supported = B_TRUE;
+       else
+               encp->enc_vport_reconfigure_supported = B_FALSE;
+
        /* Check if the firmware supports VLAN insertion */
        if (CAP_FLAGS1(req, TX_VLAN_INSERTION))
                encp->enc_hw_tx_insert_vlan_enabled = B_TRUE;
@@ -1197,6 +1189,24 @@ ef10_get_datapath_caps(
        else
                encp->enc_init_evq_v2_supported = B_FALSE;
 
+       /*
+        * Check if the NO_CONT_EV mode for RX events is supported.
+        */
+       if (CAP_FLAGS2(req, INIT_RXQ_NO_CONT_EV))
+               encp->enc_no_cont_ev_mode_supported = B_TRUE;
+       else
+               encp->enc_no_cont_ev_mode_supported = B_FALSE;
+
+       /*
+        * Check if buffer size may and must be specified on INIT_RXQ.
+        * It may be always specified to efx_rx_qcreate(), but will be
+        * just kept libefx internal if MCDI does not support it.
+        */
+       if (CAP_FLAGS2(req, INIT_RXQ_WITH_BUFFER_SIZE))
+               encp->enc_init_rxq_with_buffer_size = B_TRUE;
+       else
+               encp->enc_init_rxq_with_buffer_size = B_FALSE;
+
        /*
         * Check if firmware-verified NVRAM updates must be used.
         *
@@ -1210,6 +1220,19 @@ ef10_get_datapath_caps(
        else
                encp->enc_nvram_update_verify_result_supported = B_FALSE;
 
+       if (CAP_FLAGS2(req, NVRAM_UPDATE_POLL_VERIFY_RESULT))
+               encp->enc_nvram_update_poll_verify_result_supported = B_TRUE;
+       else
+               encp->enc_nvram_update_poll_verify_result_supported = B_FALSE;
+
+       /*
+        * Check if firmware update via the BUNDLE partition is supported
+        */
+       if (CAP_FLAGS2(req, BUNDLE_UPDATE))
+               encp->enc_nvram_bundle_update_supported = B_TRUE;
+       else
+               encp->enc_nvram_bundle_update_supported = B_FALSE;
+
        /*
         * Check if firmware provides packet memory and Rx datapath
         * counters.
@@ -1441,6 +1464,9 @@ fail1:
 }
 
 
+#define        EFX_EXT_PORT_MAX        4
+#define        EFX_EXT_PORT_NA         0xFF
+
 /*
  * Table of mapping schemes from port number to external number.
  *
@@ -1454,7 +1480,7 @@ fail1:
  *   port mapping (n:1)
  *     |
  *     v
- * External port number (normally 1-based)
+ * External port number (1-based)
  *     |
  *   fixed (1:1) or cable assembly (1:m)
  *     |
@@ -1466,9 +1492,8 @@ fail1:
  * how to determine which external cage/magjack corresponds to the port
  * numbers used by the driver.
  *
- * The count of adjacent port numbers that map to each external number,
- * and the offset in the numbering, is determined by the chip family and
- * current port mode.
+ * The count of consecutive port numbers that map to each external number,
+ * is determined by the chip family and the current port mode.
  *
  * For the Huntington family, the current port mode cannot be discovered,
  * but a single mapping is used by all modes for a given chip variant,
@@ -1479,8 +1504,7 @@ fail1:
 static struct ef10_external_port_map_s {
        efx_family_t    family;
        uint32_t        modes_mask;
-       int32_t         count;
-       int32_t         offset;
+       uint8_t         base_port[EFX_EXT_PORT_MAX];
 }      __ef10_external_port_mappings[] = {
        /*
         * Modes used by Huntington family controllers where each port
@@ -1499,8 +1523,7 @@ static struct ef10_external_port_map_s {
                (1U << TLV_PORT_MODE_10G) |                     /* mode 0 */
                (1U << TLV_PORT_MODE_10G_10G) |                 /* mode 2 */
                (1U << TLV_PORT_MODE_10G_10G_10G_10G),          /* mode 4 */
-               1,      /* ports per cage */
-               1       /* first cage */
+               { 0, 1, 2, 3 }
        },
        /*
         * Modes which for Huntington identify a chip variant where 2
@@ -1517,8 +1540,7 @@ static struct ef10_external_port_map_s {
                (1U << TLV_PORT_MODE_40G_40G) |                 /* mode 3 */
                (1U << TLV_PORT_MODE_40G_10G_10G) |             /* mode 6 */
                (1U << TLV_PORT_MODE_10G_10G_40G),              /* mode 7 */
-               2,      /* ports per cage */
-               1       /* first cage */
+               { 0, 2, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
         * Modes that on Medford allocate each port number to a separate
@@ -1531,9 +1553,9 @@ static struct ef10_external_port_map_s {
        {
                EFX_FAMILY_MEDFORD,
                (1U << TLV_PORT_MODE_1x1_NA) |                  /* mode 0 */
+               (1U << TLV_PORT_MODE_1x4_NA) |                  /* mode 1 */
                (1U << TLV_PORT_MODE_1x1_1x1),                  /* mode 2 */
-               1,      /* ports per cage */
-               1       /* first cage */
+               { 0, 1, 2, 3 }
        },
        /*
         * Modes that on Medford allocate 2 adjacent port numbers to each
@@ -1545,18 +1567,17 @@ static struct ef10_external_port_map_s {
         */
        {
                EFX_FAMILY_MEDFORD,
-               (1U << TLV_PORT_MODE_1x4_NA) |                  /* mode 1 */
                (1U << TLV_PORT_MODE_1x4_1x4) |                 /* mode 3 */
+               (1U << TLV_PORT_MODE_2x1_2x1) |                 /* mode 5 */
                (1U << TLV_PORT_MODE_1x4_2x1) |                 /* mode 6 */
                (1U << TLV_PORT_MODE_2x1_1x4) |                 /* mode 7 */
                /* Do not use 10G_10G_10G_10G_Q1_Q2 (see bug63270) */
                (1U << TLV_PORT_MODE_10G_10G_10G_10G_Q1_Q2),    /* mode 9 */
-               2,      /* ports per cage */
-               1       /* first cage */
+               { 0, 2, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
-        * Modes that on Medford allocate 4 adjacent port numbers to each
-        * connector, starting on cage 1.
+        * Modes that on Medford allocate 4 adjacent port numbers to
+        * cage 1.
         *      port 0 -> cage 1
         *      port 1 -> cage 1
         *      port 2 -> cage 1
@@ -1564,15 +1585,13 @@ static struct ef10_external_port_map_s {
         */
        {
                EFX_FAMILY_MEDFORD,
-               (1U << TLV_PORT_MODE_2x1_2x1) |                 /* mode 5 */
                /* Do not use 10G_10G_10G_10G_Q1 (see bug63270) */
                (1U << TLV_PORT_MODE_4x1_NA),                   /* mode 4 */
-               4,      /* ports per cage */
-               1       /* first cage */
+               { 0, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
-        * Modes that on Medford allocate 4 adjacent port numbers to each
-        * connector, starting on cage 2.
+        * Modes that on Medford allocate 4 adjacent port numbers to
+        * cage 2.
         *      port 0 -> cage 2
         *      port 1 -> cage 2
         *      port 2 -> cage 2
@@ -1581,8 +1600,7 @@ static struct ef10_external_port_map_s {
        {
                EFX_FAMILY_MEDFORD,
                (1U << TLV_PORT_MODE_NA_4x1),                   /* mode 8 */
-               4,      /* ports per cage */
-               2       /* first cage */
+               { EFX_EXT_PORT_NA, 0, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
         * Modes that on Medford2 allocate each port number to a separate
@@ -1597,23 +1615,29 @@ static struct ef10_external_port_map_s {
                (1U << TLV_PORT_MODE_1x1_NA) |                  /* mode 0 */
                (1U << TLV_PORT_MODE_1x4_NA) |                  /* mode 1 */
                (1U << TLV_PORT_MODE_1x1_1x1) |                 /* mode 2 */
+               (1U << TLV_PORT_MODE_1x4_1x4) |                 /* mode 3 */
                (1U << TLV_PORT_MODE_1x2_NA) |                  /* mode 10 */
                (1U << TLV_PORT_MODE_1x2_1x2) |                 /* mode 12 */
                (1U << TLV_PORT_MODE_1x4_1x2) |                 /* mode 15 */
                (1U << TLV_PORT_MODE_1x2_1x4),                  /* mode 16 */
-               1,      /* ports per cage */
-               1       /* first cage */
+               { 0, 1, 2, 3 }
        },
        /*
-        * FIXME: Some port modes are not representable in this mapping:
-        *  - TLV_PORT_MODE_1x2_2x1 (mode 17):
+        * Modes that on Medford2 allocate 1 port to cage 1 and the rest
+        * to cage 2.
         *      port 0 -> cage 1
         *      port 1 -> cage 2
         *      port 2 -> cage 2
         */
+       {
+               EFX_FAMILY_MEDFORD2,
+               (1U << TLV_PORT_MODE_1x2_2x1) |                 /* mode 17 */
+               (1U << TLV_PORT_MODE_1x4_2x1),                  /* mode 6 */
+               { 0, 1, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
+       },
        /*
-        * Modes that on Medford2 allocate 2 adjacent port numbers to each
-        * cage, starting on cage 1.
+        * Modes that on Medford2 allocate 2 adjacent port numbers to cage 1
+        * and the rest to cage 2.
         *      port 0 -> cage 1
         *      port 1 -> cage 1
         *      port 2 -> cage 2
@@ -1621,30 +1645,15 @@ static struct ef10_external_port_map_s {
         */
        {
                EFX_FAMILY_MEDFORD2,
-               (1U << TLV_PORT_MODE_1x4_1x4) |                 /* mode 3 */
                (1U << TLV_PORT_MODE_2x1_2x1) |                 /* mode 4 */
-               (1U << TLV_PORT_MODE_1x4_2x1) |                 /* mode 6 */
                (1U << TLV_PORT_MODE_2x1_1x4) |                 /* mode 7 */
                (1U << TLV_PORT_MODE_2x2_NA) |                  /* mode 13 */
                (1U << TLV_PORT_MODE_2x1_1x2),                  /* mode 18 */
-               2,      /* ports per cage */
-               1       /* first cage */
+               { 0, 2, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
-        * Modes that on Medford2 allocate 2 adjacent port numbers to each
-        * cage, starting on cage 2.
-        *      port 0 -> cage 2
-        *      port 1 -> cage 2
-        */
-       {
-               EFX_FAMILY_MEDFORD2,
-               (1U << TLV_PORT_MODE_NA_2x2),                   /* mode 14 */
-               2,      /* ports per cage */
-               2       /* first cage */
-       },
-       /*
-        * Modes that on Medford2 allocate 4 adjacent port numbers to each
-        * connector, starting on cage 1.
+        * Modes that on Medford2 allocate up to 4 adjacent port numbers
+        * to cage 1.
         *      port 0 -> cage 1
         *      port 1 -> cage 1
         *      port 2 -> cage 1
@@ -1653,12 +1662,11 @@ static struct ef10_external_port_map_s {
        {
                EFX_FAMILY_MEDFORD2,
                (1U << TLV_PORT_MODE_4x1_NA),                   /* mode 5 */
-               4,      /* ports per cage */
-               1       /* first cage */
+               { 0, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
        /*
-        * Modes that on Medford2 allocate 4 adjacent port numbers to each
-        * connector, starting on cage 2.
+        * Modes that on Medford2 allocate up to 4 adjacent port numbers
+        * to cage 2.
         *      port 0 -> cage 2
         *      port 1 -> cage 2
         *      port 2 -> cage 2
@@ -1667,9 +1675,9 @@ static struct ef10_external_port_map_s {
        {
                EFX_FAMILY_MEDFORD2,
                (1U << TLV_PORT_MODE_NA_4x1) |                  /* mode 8 */
-               (1U << TLV_PORT_MODE_NA_1x2)                  /* mode 11 */
-               4,      /* ports per cage */
-               2       /* first cage */
+               (1U << TLV_PORT_MODE_NA_1x2) |                  /* mode 11 */
+               (1U << TLV_PORT_MODE_NA_2x2),                   /* mode 14 */
+               { EFX_EXT_PORT_NA, 0, EFX_EXT_PORT_NA, EFX_EXT_PORT_NA }
        },
 };
 
@@ -1684,8 +1692,8 @@ ef10_external_port_mapping(
        uint32_t port_modes;
        uint32_t matches;
        uint32_t current;
-       int32_t count = 1; /* Default 1-1 mapping */
-       int32_t offset = 1; /* Default starting external port number */
+       struct ef10_external_port_map_s *mapp = NULL;
+       int ext_index = port; /* Default 1-1 mapping */
 
        if ((rc = efx_mcdi_get_port_modes(enp, &port_modes, &current,
                    NULL)) != 0) {
@@ -1722,8 +1730,7 @@ ef10_external_port_mapping(
                         * there will be multiple matches. The mapping on the
                         * last match is used.
                         */
-                       count = eepmp->count;
-                       offset = eepmp->offset;
+                       mapp = eepmp;
                        port_modes &= ~matches;
                }
        }
@@ -1735,11 +1742,75 @@ ef10_external_port_mapping(
        }
 
 out:
+       if (mapp != NULL) {
+               /*
+                * External ports are assigned a sequence of consecutive
+                * port numbers, so find the one with the closest base_port.
+                */
+               uint32_t delta = EFX_EXT_PORT_NA;
+
+               for (i = 0; i < EFX_EXT_PORT_MAX; i++) {
+                       uint32_t base = mapp->base_port[i];
+                       if ((base != EFX_EXT_PORT_NA) && (base <= port)) {
+                               if ((port - base) < delta) {
+                                       delta = (port - base);
+                                       ext_index = i;
+                               }
+                       }
+               }
+       }
+       *external_portp = (uint8_t)(ext_index + 1);
+
+       return (0);
+
+fail1:
+       EFSYS_PROBE1(fail1, efx_rc_t, rc);
+
+       return (rc);
+}
+
+static __checkReturn   efx_rc_t
+ef10_set_workaround_bug26807(
+       __in            efx_nic_t *enp)
+{
+       efx_nic_cfg_t *encp = &(enp->en_nic_cfg);
+       uint32_t flags;
+       efx_rc_t rc;
+
        /*
-        * Scale as required by last matched mode and then convert to
-        * correctly offset numbering
+        * If the bug26807 workaround is enabled, then firmware has enabled
+        * support for chained multicast filters. Firmware will reset (FLR)
+        * functions which have filters in the hardware filter table when the
+        * workaround is enabled/disabled.
+        *
+        * We must recheck if the workaround is enabled after inserting the
+        * first hardware filter, in case it has been changed since this check.
         */
-       *external_portp = (uint8_t)((port / count) + offset);
+       rc = efx_mcdi_set_workaround(enp, MC_CMD_WORKAROUND_BUG26807,
+           B_TRUE, &flags);
+       if (rc == 0) {
+               encp->enc_bug26807_workaround = B_TRUE;
+               if (flags & (1 << MC_CMD_WORKAROUND_EXT_OUT_FLR_DONE_LBN)) {
+                       /*
+                        * Other functions had installed filters before the
+                        * workaround was enabled, and they have been reset
+                        * by firmware.
+                        */
+                       EFSYS_PROBE(bug26807_workaround_flr_done);
+                       /* FIXME: bump MC warm boot count ? */
+               }
+       } else if (rc == EACCES) {
+               /*
+                * Unprivileged functions cannot enable the workaround in older
+                * firmware.
+                */
+               encp->enc_bug26807_workaround = B_FALSE;
+       } else if ((rc == ENOTSUP) || (rc == ENOENT)) {
+               encp->enc_bug26807_workaround = B_FALSE;
+       } else {
+               goto fail1;
+       }
+
        return (0);
 
 fail1:
@@ -1773,6 +1844,8 @@ ef10_nic_board_cfg(
        /* EFX MCDI interface uses one-based port numbers */
        emip->emi_port = port + 1;
 
+       encp->enc_assigned_port = port;
+
        if ((rc = ef10_external_port_mapping(enp, port,
                    &encp->enc_external_port)) != 0)
                goto fail2;
@@ -1882,7 +1955,7 @@ ef10_nic_board_cfg(
        encp->enc_rxq_limit = EFX_RXQ_LIMIT_TARGET;
        encp->enc_txq_limit = EFX_TXQ_LIMIT_TARGET;
 
-       encp->enc_buftbl_limit = 0xFFFFFFFF;
+       encp->enc_buftbl_limit = UINT32_MAX;
 
        /* Get interrupt vector limits */
        if ((rc = efx_mcdi_get_vector_cfg(enp, &base, &nvec, NULL)) != 0) {
@@ -1906,13 +1979,18 @@ ef10_nic_board_cfg(
                goto fail10;
        encp->enc_privilege_mask = mask;
 
+       if ((rc = ef10_set_workaround_bug26807(enp)) != 0)
+               goto fail11;
+
        /* Get remaining controller-specific board config */
        if ((rc = enop->eno_board_cfg(enp)) != 0)
                if (rc != EACCES)
-                       goto fail11;
+                       goto fail12;
 
        return (0);
 
+fail12:
+       EFSYS_PROBE(fail12);
 fail11:
        EFSYS_PROBE(fail11);
 fail10:
@@ -1947,9 +2025,7 @@ ef10_nic_probe(
        efx_drv_cfg_t *edcp = &(enp->en_drv_cfg);
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        /* Read and clear any assertion state */
        if ((rc = efx_mcdi_read_assertion(enp)) != 0)
@@ -2148,6 +2224,58 @@ fail1:
        return (rc);
 }
 
+static __checkReturn   efx_rc_t
+ef10_upstream_port_vadaptor_alloc(
+       __in            efx_nic_t *enp)
+{
+       uint32_t retry;
+       uint32_t delay_us;
+       efx_rc_t rc;
+
+       /*
+        * On a VF, this may fail with MC_CMD_ERR_NO_EVB_PORT (ENOENT) if the PF
+        * driver has yet to bring up the EVB port. See bug 56147. In this case,
+        * retry the request several times after waiting a while. The wait time
+        * between retries starts small (10ms) and exponentially increases.
+        * Total wait time is a little over two seconds. Retry logic in the
+        * client driver may mean this whole loop is repeated if it continues to
+        * fail.
+        */
+       retry = 0;
+       delay_us = 10000;
+       while ((rc = efx_mcdi_vadaptor_alloc(enp, EVB_PORT_ID_ASSIGNED)) != 0) {
+               if (EFX_PCI_FUNCTION_IS_PF(&enp->en_nic_cfg) ||
+                   (rc != ENOENT)) {
+                       /*
+                        * Do not retry alloc for PF, or for other errors on
+                        * a VF.
+                        */
+                       goto fail1;
+               }
+
+               /* VF startup before PF is ready. Retry allocation. */
+               if (retry > 5) {
+                       /* Too many attempts */
+                       rc = EINVAL;
+                       goto fail2;
+               }
+               EFSYS_PROBE1(mcdi_no_evb_port_retry, int, retry);
+               EFSYS_SLEEP(delay_us);
+               retry++;
+               if (delay_us < 500000)
+                       delay_us <<= 2;
+       }
+
+       return (0);
+
+fail2:
+       EFSYS_PROBE(fail2);
+fail1:
+       EFSYS_PROBE1(fail1, efx_rc_t, rc);
+
+       return (rc);
+}
+
        __checkReturn   efx_rc_t
 ef10_nic_init(
        __in            efx_nic_t *enp)
@@ -2156,10 +2284,9 @@ ef10_nic_init(
        uint32_t min_vi_count, max_vi_count;
        uint32_t vi_count, vi_base, vi_shift;
        uint32_t i;
-       uint32_t retry;
-       uint32_t delay_us;
        uint32_t vi_window_size;
        efx_rc_t rc;
+       boolean_t alloc_vadaptor = B_TRUE;
 
        EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
            enp->en_family == EFX_FAMILY_MEDFORD ||
@@ -2260,48 +2387,29 @@ ef10_nic_init(
        }
 
        /*
-        * Allocate a vAdaptor attached to our upstream vPort/pPort.
-        *
-        * On a VF, this may fail with MC_CMD_ERR_NO_EVB_PORT (ENOENT) if the PF
-        * driver has yet to bring up the EVB port. See bug 56147. In this case,
-        * retry the request several times after waiting a while. The wait time
-        * between retries starts small (10ms) and exponentially increases.
-        * Total wait time is a little over two seconds. Retry logic in the
-        * client driver may mean this whole loop is repeated if it continues to
-        * fail.
+        * For SR-IOV use case, vAdaptor is allocated for PF and associated VFs
+        * during NIC initialization when vSwitch is created and vports are
+        * allocated. Hence, skip vAdaptor allocation for EVB and update vport
+        * id in NIC structure with the one allocated for PF.
         */
-       retry = 0;
-       delay_us = 10000;
-       while ((rc = efx_mcdi_vadaptor_alloc(enp, EVB_PORT_ID_ASSIGNED)) != 0) {
-               if (EFX_PCI_FUNCTION_IS_PF(&enp->en_nic_cfg) ||
-                   (rc != ENOENT)) {
-                       /*
-                        * Do not retry alloc for PF, or for other errors on
-                        * a VF.
-                        */
-                       goto fail5;
-               }
-
-               /* VF startup before PF is ready. Retry allocation. */
-               if (retry > 5) {
-                       /* Too many attempts */
-                       rc = EINVAL;
-                       goto fail6;
-               }
-               EFSYS_PROBE1(mcdi_no_evb_port_retry, int, retry);
-               EFSYS_SLEEP(delay_us);
-               retry++;
-               if (delay_us < 500000)
-                       delay_us <<= 2;
-       }
 
        enp->en_vport_id = EVB_PORT_ID_ASSIGNED;
+#if EFSYS_OPT_EVB
+       if ((enp->en_vswitchp != NULL) && (enp->en_vswitchp->ev_evcp != NULL)) {
+               /* For EVB use vport allocated on vswitch */
+               enp->en_vport_id = enp->en_vswitchp->ev_evcp->evc_vport_id;
+               alloc_vadaptor = B_FALSE;
+       }
+#endif
+       if (alloc_vadaptor != B_FALSE) {
+               /* Allocate a vAdaptor attached to our upstream vPort/pPort */
+               if ((rc = ef10_upstream_port_vadaptor_alloc(enp)) != 0)
+                       goto fail5;
+       }
        enp->en_nic_cfg.enc_mcdi_max_payload_length = MCDI_CTL_SDU_LEN_MAX_V2;
 
        return (0);
 
-fail6:
-       EFSYS_PROBE(fail6);
 fail5:
        EFSYS_PROBE(fail5);
 fail4:
@@ -2324,9 +2432,7 @@ ef10_nic_get_vi_pool(
        __in            efx_nic_t *enp,
        __out           uint32_t *vi_countp)
 {
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        /*
         * Report VIs that the client driver can use.
@@ -2346,9 +2452,7 @@ ef10_nic_get_bar_region(
 {
        efx_rc_t rc;
 
-       EFSYS_ASSERT(enp->en_family == EFX_FAMILY_HUNTINGTON ||
-           enp->en_family == EFX_FAMILY_MEDFORD ||
-           enp->en_family == EFX_FAMILY_MEDFORD2);
+       EFSYS_ASSERT(EFX_FAMILY_IS_EF10(enp));
 
        /*
         * TODO: Specify host memory mapping alignment and granularity
@@ -2417,9 +2521,21 @@ ef10_nic_fini(
 {
        uint32_t i;
        efx_rc_t rc;
+       boolean_t do_vadaptor_free = B_TRUE;
 
-       (void) efx_mcdi_vadaptor_free(enp, enp->en_vport_id);
-       enp->en_vport_id = 0;
+#if EFSYS_OPT_EVB
+       if (enp->en_vswitchp != NULL) {
+               /*
+                * For SR-IOV the vAdaptor is freed with the vswitch,
+                * so do not free it here.
+                */
+               do_vadaptor_free = B_FALSE;
+       }
+#endif
+       if (do_vadaptor_free != B_FALSE) {
+               (void) efx_mcdi_vadaptor_free(enp, enp->en_vport_id);
+               enp->en_vport_id = EVB_PORT_ID_NULL;
+       }
 
        /* Unlink piobufs from extra VIs in WC mapping */
        if (enp->en_arch.ef10.ena_piobuf_count > 0) {
@@ -2555,4 +2671,4 @@ fail1:
 
 #endif /* EFSYS_OPT_FW_SUBVARIANT_AWARE */
 
-#endif /* EFSYS_OPT_HUNTINGTON || EFSYS_OPT_MEDFORD || EFSYS_OPT_MEDFORD2 */
+#endif /* EFX_OPTS_EF10() */