net/sfc: use NIC EVQ descs limits instead of defines
[dpdk.git] / drivers / net / sfc / sfc_rx.c
index eb4875f..faa9758 100644 (file)
@@ -381,15 +381,15 @@ sfc_rxq_info_by_dp_rxq(const struct sfc_dp_rxq *dp_rxq)
 {
        const struct sfc_dp_queue *dpq = &dp_rxq->dpq;
        struct rte_eth_dev *eth_dev;
-       struct sfc_adapter *sa;
+       struct sfc_adapter_shared *sas;
 
        SFC_ASSERT(rte_eth_dev_is_valid_port(dpq->port_id));
        eth_dev = &rte_eth_devices[dpq->port_id];
 
-       sa = eth_dev->data->dev_private;
+       sas = sfc_adapter_shared_by_eth_dev(eth_dev);
 
-       SFC_ASSERT(dpq->queue_id < sa->rxq_count);
-       return &sa->rxq_info[dpq->queue_id];
+       SFC_ASSERT(dpq->queue_id < sas->rxq_count);
+       return &sas->rxq_info[dpq->queue_id];
 }
 
 struct sfc_rxq *
@@ -402,15 +402,16 @@ sfc_rxq_by_dp_rxq(const struct sfc_dp_rxq *dp_rxq)
        SFC_ASSERT(rte_eth_dev_is_valid_port(dpq->port_id));
        eth_dev = &rte_eth_devices[dpq->port_id];
 
-       sa = eth_dev->data->dev_private;
+       sa = sfc_adapter_by_eth_dev(eth_dev);
 
-       SFC_ASSERT(dpq->queue_id < sa->rxq_count);
+       SFC_ASSERT(dpq->queue_id < sfc_sa2shared(sa)->rxq_count);
        return &sa->rxq_ctrl[dpq->queue_id];
 }
 
 static sfc_dp_rx_qsize_up_rings_t sfc_efx_rx_qsize_up_rings;
 static int
 sfc_efx_rx_qsize_up_rings(uint16_t nb_rx_desc,
+                         __rte_unused struct sfc_dp_rx_hw_limits *limits,
                          __rte_unused struct rte_mempool *mb_pool,
                          unsigned int *rxq_entries,
                          unsigned int *evq_entries,
@@ -567,7 +568,7 @@ sfc_rx_qflush(struct sfc_adapter *sa, unsigned int sw_index)
        unsigned int wait_count;
        int rc;
 
-       rxq_info = &sa->rxq_info[sw_index];
+       rxq_info = &sfc_sa2shared(sa)->rxq_info[sw_index];
        SFC_ASSERT(rxq_info->state & SFC_RXQ_STARTED);
 
        rxq = &sa->rxq_ctrl[sw_index];
@@ -618,7 +619,7 @@ sfc_rx_qflush(struct sfc_adapter *sa, unsigned int sw_index)
 static int
 sfc_rx_default_rxq_set_filter(struct sfc_adapter *sa, struct sfc_rxq *rxq)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        boolean_t need_rss = (rss->channels > 0) ? B_TRUE : B_FALSE;
        struct sfc_port *port = &sa->port;
        int rc;
@@ -669,7 +670,6 @@ retry:
 int
 sfc_rx_qstart(struct sfc_adapter *sa, unsigned int sw_index)
 {
-       struct sfc_port *port = &sa->port;
        struct sfc_rxq_info *rxq_info;
        struct sfc_rxq *rxq;
        struct sfc_evq *evq;
@@ -677,9 +677,9 @@ sfc_rx_qstart(struct sfc_adapter *sa, unsigned int sw_index)
 
        sfc_log_init(sa, "sw_index=%u", sw_index);
 
-       SFC_ASSERT(sw_index < sa->rxq_count);
+       SFC_ASSERT(sw_index < sfc_sa2shared(sa)->rxq_count);
 
-       rxq_info = &sa->rxq_info[sw_index];
+       rxq_info = &sfc_sa2shared(sa)->rxq_info[sw_index];
        SFC_ASSERT(rxq_info->state == SFC_RXQ_INITIALIZED);
 
        rxq = &sa->rxq_ctrl[sw_index];
@@ -731,7 +731,7 @@ sfc_rx_qstart(struct sfc_adapter *sa, unsigned int sw_index)
 
        rxq_info->state |= SFC_RXQ_STARTED;
 
-       if ((sw_index == 0) && !port->isolated) {
+       if (sw_index == 0 && !sfc_sa2shared(sa)->isolated) {
                rc = sfc_rx_default_rxq_set_filter(sa, rxq);
                if (rc != 0)
                        goto fail_mac_filter_default_rxq_set;
@@ -766,9 +766,9 @@ sfc_rx_qstop(struct sfc_adapter *sa, unsigned int sw_index)
 
        sfc_log_init(sa, "sw_index=%u", sw_index);
 
-       SFC_ASSERT(sw_index < sa->rxq_count);
+       SFC_ASSERT(sw_index < sfc_sa2shared(sa)->rxq_count);
 
-       rxq_info = &sa->rxq_info[sw_index];
+       rxq_info = &sfc_sa2shared(sa)->rxq_info[sw_index];
 
        if (rxq_info->state == SFC_RXQ_INITIALIZED)
                return;
@@ -961,7 +961,7 @@ sfc_rx_qinit(struct sfc_adapter *sa, unsigned int sw_index,
             struct rte_mempool *mb_pool)
 {
        const efx_nic_cfg_t *encp = efx_nic_cfg_get(sa->nic);
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        int rc;
        unsigned int rxq_entries;
        unsigned int evq_entries;
@@ -972,13 +972,21 @@ sfc_rx_qinit(struct sfc_adapter *sa, unsigned int sw_index,
        struct sfc_evq *evq;
        struct sfc_rxq *rxq;
        struct sfc_dp_rx_qcreate_info info;
+       struct sfc_dp_rx_hw_limits hw_limits;
 
-       rc = sa->priv.dp_rx->qsize_up_rings(nb_rx_desc, mb_pool, &rxq_entries,
-                                           &evq_entries, &rxq_max_fill_level);
+       memset(&hw_limits, 0, sizeof(hw_limits));
+       hw_limits.rxq_max_entries = sa->rxq_max_entries;
+       hw_limits.rxq_min_entries = sa->rxq_min_entries;
+       hw_limits.evq_max_entries = sa->evq_max_entries;
+       hw_limits.evq_min_entries = sa->evq_min_entries;
+
+       rc = sa->priv.dp_rx->qsize_up_rings(nb_rx_desc, &hw_limits, mb_pool,
+                                           &rxq_entries, &evq_entries,
+                                           &rxq_max_fill_level);
        if (rc != 0)
                goto fail_size_up_rings;
-       SFC_ASSERT(rxq_entries >= EFX_RXQ_MINNDESCS);
-       SFC_ASSERT(rxq_entries <= EFX_RXQ_MAXNDESCS);
+       SFC_ASSERT(rxq_entries >= sa->rxq_min_entries);
+       SFC_ASSERT(rxq_entries <= sa->rxq_max_entries);
        SFC_ASSERT(rxq_max_fill_level <= nb_rx_desc);
 
        offloads = rx_conf->offloads |
@@ -1007,8 +1015,8 @@ sfc_rx_qinit(struct sfc_adapter *sa, unsigned int sw_index,
                goto fail_bad_conf;
        }
 
-       SFC_ASSERT(sw_index < sa->rxq_count);
-       rxq_info = &sa->rxq_info[sw_index];
+       SFC_ASSERT(sw_index < sfc_sa2shared(sa)->rxq_count);
+       rxq_info = &sfc_sa2shared(sa)->rxq_info[sw_index];
 
        SFC_ASSERT(rxq_entries <= rxq_info->max_entries);
        rxq_info->entries = rxq_entries;
@@ -1098,10 +1106,10 @@ sfc_rx_qfini(struct sfc_adapter *sa, unsigned int sw_index)
        struct sfc_rxq_info *rxq_info;
        struct sfc_rxq *rxq;
 
-       SFC_ASSERT(sw_index < sa->rxq_count);
+       SFC_ASSERT(sw_index < sfc_sa2shared(sa)->rxq_count);
        sa->eth_dev->data->rx_queues[sw_index] = NULL;
 
-       rxq_info = &sa->rxq_info[sw_index];
+       rxq_info = &sfc_sa2shared(sa)->rxq_info[sw_index];
 
        SFC_ASSERT(rxq_info->state == SFC_RXQ_INITIALIZED);
 
@@ -1176,7 +1184,7 @@ sfc_rx_hash_types_mask_supp(efx_rx_hash_type_t hash_type,
 int
 sfc_rx_hash_init(struct sfc_adapter *sa)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        const efx_nic_cfg_t *encp = efx_nic_cfg_get(sa->nic);
        uint32_t alg_mask = encp->enc_rx_scale_hash_alg_mask;
        efx_rx_hash_alg_t alg;
@@ -1232,7 +1240,7 @@ sfc_rx_hash_init(struct sfc_adapter *sa)
 void
 sfc_rx_hash_fini(struct sfc_adapter *sa)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
 
        rte_free(rss->hf_map);
 }
@@ -1241,7 +1249,7 @@ int
 sfc_rx_hf_rte_to_efx(struct sfc_adapter *sa, uint64_t rte,
                     efx_rx_hash_type_t *efx)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        efx_rx_hash_type_t hash_types = 0;
        unsigned int i;
 
@@ -1265,9 +1273,8 @@ sfc_rx_hf_rte_to_efx(struct sfc_adapter *sa, uint64_t rte,
 }
 
 uint64_t
-sfc_rx_hf_efx_to_rte(struct sfc_adapter *sa, efx_rx_hash_type_t efx)
+sfc_rx_hf_efx_to_rte(struct sfc_rss *rss, efx_rx_hash_type_t efx)
 {
-       struct sfc_rss *rss = &sa->rss;
        uint64_t rte = 0;
        unsigned int i;
 
@@ -1285,9 +1292,9 @@ static int
 sfc_rx_process_adv_conf_rss(struct sfc_adapter *sa,
                            struct rte_eth_rss_conf *conf)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        efx_rx_hash_type_t efx_hash_types = rss->hash_types;
-       uint64_t rss_hf = sfc_rx_hf_efx_to_rte(sa, efx_hash_types);
+       uint64_t rss_hf = sfc_rx_hf_efx_to_rte(rss, efx_hash_types);
        int rc;
 
        if (rss->context_type != EFX_RX_SCALE_EXCLUSIVE) {
@@ -1319,7 +1326,7 @@ sfc_rx_process_adv_conf_rss(struct sfc_adapter *sa,
 static int
 sfc_rx_rss_config(struct sfc_adapter *sa)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
        int rc = 0;
 
        if (rss->channels > 0) {
@@ -1345,10 +1352,11 @@ finish:
 int
 sfc_rx_start(struct sfc_adapter *sa)
 {
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
        unsigned int sw_index;
        int rc;
 
-       sfc_log_init(sa, "rxq_count=%u", sa->rxq_count);
+       sfc_log_init(sa, "rxq_count=%u", sas->rxq_count);
 
        rc = efx_rx_init(sa->nic);
        if (rc != 0)
@@ -1358,10 +1366,10 @@ sfc_rx_start(struct sfc_adapter *sa)
        if (rc != 0)
                goto fail_rss_config;
 
-       for (sw_index = 0; sw_index < sa->rxq_count; ++sw_index) {
-               if (sa->rxq_info[sw_index].state == SFC_RXQ_INITIALIZED &&
-                   (!sa->rxq_info[sw_index].deferred_start ||
-                    sa->rxq_info[sw_index].deferred_started)) {
+       for (sw_index = 0; sw_index < sas->rxq_count; ++sw_index) {
+               if (sas->rxq_info[sw_index].state == SFC_RXQ_INITIALIZED &&
+                   (!sas->rxq_info[sw_index].deferred_start ||
+                    sas->rxq_info[sw_index].deferred_started)) {
                        rc = sfc_rx_qstart(sa, sw_index);
                        if (rc != 0)
                                goto fail_rx_qstart;
@@ -1385,13 +1393,14 @@ fail_rx_init:
 void
 sfc_rx_stop(struct sfc_adapter *sa)
 {
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
        unsigned int sw_index;
 
-       sfc_log_init(sa, "rxq_count=%u", sa->rxq_count);
+       sfc_log_init(sa, "rxq_count=%u", sas->rxq_count);
 
-       sw_index = sa->rxq_count;
+       sw_index = sas->rxq_count;
        while (sw_index-- > 0) {
-               if (sa->rxq_info[sw_index].state & SFC_RXQ_STARTED)
+               if (sas->rxq_info[sw_index].state & SFC_RXQ_STARTED)
                        sfc_rx_qstop(sa, sw_index);
        }
 
@@ -1401,10 +1410,12 @@ sfc_rx_stop(struct sfc_adapter *sa)
 static int
 sfc_rx_qinit_info(struct sfc_adapter *sa, unsigned int sw_index)
 {
-       struct sfc_rxq_info *rxq_info = &sa->rxq_info[sw_index];
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
+       struct sfc_rxq_info *rxq_info = &sas->rxq_info[sw_index];
+       const efx_nic_cfg_t *encp = efx_nic_cfg_get(sa->nic);
        unsigned int max_entries;
 
-       max_entries = EFX_RXQ_MAXNDESCS;
+       max_entries = encp->enc_rxq_max_ndescs;
        SFC_ASSERT(rte_is_power_of_2(max_entries));
 
        rxq_info->max_entries = max_entries;
@@ -1415,9 +1426,10 @@ sfc_rx_qinit_info(struct sfc_adapter *sa, unsigned int sw_index)
 static int
 sfc_rx_check_mode(struct sfc_adapter *sa, struct rte_eth_rxmode *rxmode)
 {
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
        uint64_t offloads_supported = sfc_rx_get_dev_offload_caps(sa) |
                                      sfc_rx_get_queue_offload_caps(sa);
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sas->rss;
        int rc = 0;
 
        switch (rxmode->mq_mode) {
@@ -1463,17 +1475,18 @@ sfc_rx_check_mode(struct sfc_adapter *sa, struct rte_eth_rxmode *rxmode)
 static void
 sfc_rx_fini_queues(struct sfc_adapter *sa, unsigned int nb_rx_queues)
 {
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
        int sw_index;
 
-       SFC_ASSERT(nb_rx_queues <= sa->rxq_count);
+       SFC_ASSERT(nb_rx_queues <= sas->rxq_count);
 
-       sw_index = sa->rxq_count;
+       sw_index = sas->rxq_count;
        while (--sw_index >= (int)nb_rx_queues) {
-               if (sa->rxq_info[sw_index].state & SFC_RXQ_INITIALIZED)
+               if (sas->rxq_info[sw_index].state & SFC_RXQ_INITIALIZED)
                        sfc_rx_qfini(sa, sw_index);
        }
 
-       sa->rxq_count = nb_rx_queues;
+       sas->rxq_count = nb_rx_queues;
 }
 
 /**
@@ -1487,27 +1500,28 @@ sfc_rx_fini_queues(struct sfc_adapter *sa, unsigned int nb_rx_queues)
 int
 sfc_rx_configure(struct sfc_adapter *sa)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_adapter_shared * const sas = sfc_sa2shared(sa);
+       struct sfc_rss *rss = &sas->rss;
        struct rte_eth_conf *dev_conf = &sa->eth_dev->data->dev_conf;
        const unsigned int nb_rx_queues = sa->eth_dev->data->nb_rx_queues;
        int rc;
 
        sfc_log_init(sa, "nb_rx_queues=%u (old %u)",
-                    nb_rx_queues, sa->rxq_count);
+                    nb_rx_queues, sas->rxq_count);
 
        rc = sfc_rx_check_mode(sa, &dev_conf->rxmode);
        if (rc != 0)
                goto fail_check_mode;
 
-       if (nb_rx_queues == sa->rxq_count)
+       if (nb_rx_queues == sas->rxq_count)
                goto configure_rss;
 
-       if (sa->rxq_info == NULL) {
+       if (sas->rxq_info == NULL) {
                rc = ENOMEM;
-               sa->rxq_info = rte_calloc_socket("sfc-rxqs", nb_rx_queues,
-                                                sizeof(sa->rxq_info[0]), 0,
-                                                sa->socket_id);
-               if (sa->rxq_info == NULL)
+               sas->rxq_info = rte_calloc_socket("sfc-rxqs", nb_rx_queues,
+                                                 sizeof(sas->rxq_info[0]), 0,
+                                                 sa->socket_id);
+               if (sas->rxq_info == NULL)
                        goto fail_rxqs_alloc;
 
                /*
@@ -1522,13 +1536,13 @@ sfc_rx_configure(struct sfc_adapter *sa)
                struct sfc_rxq_info *new_rxq_info;
                struct sfc_rxq *new_rxq_ctrl;
 
-               if (nb_rx_queues < sa->rxq_count)
+               if (nb_rx_queues < sas->rxq_count)
                        sfc_rx_fini_queues(sa, nb_rx_queues);
 
                rc = ENOMEM;
                new_rxq_info =
-                       rte_realloc(sa->rxq_info,
-                                   nb_rx_queues * sizeof(sa->rxq_info[0]), 0);
+                       rte_realloc(sas->rxq_info,
+                                   nb_rx_queues * sizeof(sas->rxq_info[0]), 0);
                if (new_rxq_info == NULL && nb_rx_queues > 0)
                        goto fail_rxqs_realloc;
 
@@ -1538,29 +1552,29 @@ sfc_rx_configure(struct sfc_adapter *sa)
                if (new_rxq_ctrl == NULL && nb_rx_queues > 0)
                        goto fail_rxqs_ctrl_realloc;
 
-               sa->rxq_info = new_rxq_info;
+               sas->rxq_info = new_rxq_info;
                sa->rxq_ctrl = new_rxq_ctrl;
-               if (nb_rx_queues > sa->rxq_count) {
-                       memset(&sa->rxq_info[sa->rxq_count], 0,
-                              (nb_rx_queues - sa->rxq_count) *
-                              sizeof(sa->rxq_info[0]));
-                       memset(&sa->rxq_ctrl[sa->rxq_count], 0,
-                              (nb_rx_queues - sa->rxq_count) *
+               if (nb_rx_queues > sas->rxq_count) {
+                       memset(&sas->rxq_info[sas->rxq_count], 0,
+                              (nb_rx_queues - sas->rxq_count) *
+                              sizeof(sas->rxq_info[0]));
+                       memset(&sa->rxq_ctrl[sas->rxq_count], 0,
+                              (nb_rx_queues - sas->rxq_count) *
                               sizeof(sa->rxq_ctrl[0]));
                }
        }
 
-       while (sa->rxq_count < nb_rx_queues) {
-               rc = sfc_rx_qinit_info(sa, sa->rxq_count);
+       while (sas->rxq_count < nb_rx_queues) {
+               rc = sfc_rx_qinit_info(sa, sas->rxq_count);
                if (rc != 0)
                        goto fail_rx_qinit_info;
 
-               sa->rxq_count++;
+               sas->rxq_count++;
        }
 
 configure_rss:
        rss->channels = (dev_conf->rxmode.mq_mode == ETH_MQ_RX_RSS) ?
-                        MIN(sa->rxq_count, EFX_MAXRSS) : 0;
+                        MIN(sas->rxq_count, EFX_MAXRSS) : 0;
 
        if (rss->channels > 0) {
                struct rte_eth_rss_conf *adv_conf_rss;
@@ -1598,7 +1612,7 @@ fail_check_mode:
 void
 sfc_rx_close(struct sfc_adapter *sa)
 {
-       struct sfc_rss *rss = &sa->rss;
+       struct sfc_rss *rss = &sfc_sa2shared(sa)->rss;
 
        sfc_rx_fini_queues(sa, 0);
 
@@ -1607,6 +1621,6 @@ sfc_rx_close(struct sfc_adapter *sa)
        free(sa->rxq_ctrl);
        sa->rxq_ctrl = NULL;
 
-       rte_free(sa->rxq_info);
-       sa->rxq_info = NULL;
+       rte_free(sfc_sa2shared(sa)->rxq_info);
+       sfc_sa2shared(sa)->rxq_info = NULL;
 }