net/mlx5: support represented port flow action
[dpdk.git] / drivers / net / sfc / sfc_repr.c
index fc3229a..2500b14 100644 (file)
@@ -32,15 +32,21 @@ struct sfc_repr_shared {
        uint16_t                switch_port_id;
 };
 
+struct sfc_repr_queue_stats {
+       union sfc_pkts_bytes            packets_bytes;
+};
+
 struct sfc_repr_rxq {
        /* Datapath members */
        struct rte_ring                 *ring;
+       struct sfc_repr_queue_stats     stats;
 };
 
 struct sfc_repr_txq {
        /* Datapath members */
        struct rte_ring                 *ring;
        efx_mport_id_t                  egress_mport;
+       struct sfc_repr_queue_stats     stats;
 };
 
 /** Primary process representor private data */
@@ -160,10 +166,35 @@ sfc_repr_tx_queue_stop(void *queue)
        rte_ring_reset(txq->ring);
 }
 
+static uint16_t
+sfc_repr_rx_burst(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts)
+{
+       struct sfc_repr_rxq *rxq = rx_queue;
+       void **objs = (void *)&rx_pkts[0];
+       unsigned int n_rx;
+
+       /* mbufs port is already filled correctly by representors proxy */
+       n_rx = rte_ring_sc_dequeue_burst(rxq->ring, objs, nb_pkts, NULL);
+
+       if (n_rx > 0) {
+               unsigned int n_bytes = 0;
+               unsigned int i = 0;
+
+               do {
+                       n_bytes += rx_pkts[i]->pkt_len;
+               } while (++i < n_rx);
+
+               sfc_pkts_bytes_add(&rxq->stats.packets_bytes, n_rx, n_bytes);
+       }
+
+       return n_rx;
+}
+
 static uint16_t
 sfc_repr_tx_burst(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
 {
        struct sfc_repr_txq *txq = tx_queue;
+       unsigned int n_bytes = 0;
        unsigned int n_tx;
        void **objs;
        uint16_t i;
@@ -183,6 +214,7 @@ sfc_repr_tx_burst(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
                m->ol_flags |= sfc_dp_mport_override;
                *RTE_MBUF_DYNFIELD(m, sfc_dp_mport_offset,
                                   efx_mport_id_t *) = txq->egress_mport;
+               n_bytes += tx_pkts[i]->pkt_len;
        }
 
        objs = (void *)&tx_pkts[0];
@@ -192,14 +224,18 @@ sfc_repr_tx_burst(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)
         * Remove m-port override flag from packets that were not enqueued
         * Setting the flag only for enqueued packets after the burst is
         * not possible since the ownership of enqueued packets is
-        * transferred to representor proxy.
+        * transferred to representor proxy. The same logic applies to
+        * counting the enqueued packets' bytes.
         */
        for (i = n_tx; i < nb_pkts; ++i) {
                struct rte_mbuf *m = tx_pkts[i];
 
                m->ol_flags &= ~sfc_dp_mport_override;
+               n_bytes -= m->pkt_len;
        }
 
+       sfc_pkts_bytes_add(&txq->stats.packets_bytes, n_tx, n_bytes);
+
        return n_tx;
 }
 
@@ -803,6 +839,7 @@ sfc_repr_dev_close(struct rte_eth_dev *dev)
 
        (void)sfc_repr_proxy_del_port(srs->pf_port_id, srs->repr_id);
 
+       dev->rx_pkt_burst = NULL;
        dev->tx_pkt_burst = NULL;
        dev->dev_ops = NULL;
 
@@ -816,6 +853,35 @@ sfc_repr_dev_close(struct rte_eth_dev *dev)
        return 0;
 }
 
+static int
+sfc_repr_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats)
+{
+       union sfc_pkts_bytes queue_stats;
+       uint16_t i;
+
+       for (i = 0; i < dev->data->nb_rx_queues; i++) {
+               struct sfc_repr_rxq *rxq = dev->data->rx_queues[i];
+
+               sfc_pkts_bytes_get(&rxq->stats.packets_bytes,
+                                  &queue_stats);
+
+               stats->ipackets += queue_stats.pkts;
+               stats->ibytes += queue_stats.bytes;
+       }
+
+       for (i = 0; i < dev->data->nb_tx_queues; i++) {
+               struct sfc_repr_txq *txq = dev->data->tx_queues[i];
+
+               sfc_pkts_bytes_get(&txq->stats.packets_bytes,
+                                  &queue_stats);
+
+               stats->opackets += queue_stats.pkts;
+               stats->obytes += queue_stats.bytes;
+       }
+
+       return 0;
+}
+
 static const struct eth_dev_ops sfc_repr_dev_ops = {
        .dev_configure                  = sfc_repr_dev_configure,
        .dev_start                      = sfc_repr_dev_start,
@@ -823,6 +889,7 @@ static const struct eth_dev_ops sfc_repr_dev_ops = {
        .dev_close                      = sfc_repr_dev_close,
        .dev_infos_get                  = sfc_repr_dev_infos_get,
        .link_update                    = sfc_repr_dev_link_update,
+       .stats_get                      = sfc_repr_stats_get,
        .rx_queue_setup                 = sfc_repr_rx_queue_setup,
        .rx_queue_release               = sfc_repr_rx_queue_release,
        .tx_queue_setup                 = sfc_repr_tx_queue_setup,
@@ -832,9 +899,11 @@ static const struct eth_dev_ops sfc_repr_dev_ops = {
 
 struct sfc_repr_init_data {
        uint16_t                pf_port_id;
-       uint16_t                repr_id;
        uint16_t                switch_domain_id;
        efx_mport_sel_t         mport_sel;
+       efx_pcie_interface_t    intf;
+       uint16_t                pf;
+       uint16_t                vf;
 };
 
 static int
@@ -872,6 +941,9 @@ sfc_repr_eth_dev_init(struct rte_eth_dev *dev, void *init_params)
        switch_port_request.ethdev_mportp = &ethdev_mport_sel;
        switch_port_request.entity_mportp = &repr_data->mport_sel;
        switch_port_request.ethdev_port_id = dev->data->port_id;
+       switch_port_request.port_data.repr.intf = repr_data->intf;
+       switch_port_request.port_data.repr.pf = repr_data->pf;
+       switch_port_request.port_data.repr.vf = repr_data->vf;
 
        ret = sfc_repr_assign_mae_switch_port(repr_data->switch_domain_id,
                                              &switch_port_request,
@@ -884,7 +956,7 @@ sfc_repr_eth_dev_init(struct rte_eth_dev *dev, void *init_params)
        }
 
        ret = sfc_repr_proxy_add_port(repr_data->pf_port_id,
-                                     repr_data->repr_id,
+                                     srs->switch_port_id,
                                      dev->data->port_id,
                                      &repr_data->mport_sel);
        if (ret != 0) {
@@ -911,7 +983,7 @@ sfc_repr_eth_dev_init(struct rte_eth_dev *dev, void *init_params)
        dev->process_private = sr;
 
        srs->pf_port_id = repr_data->pf_port_id;
-       srs->repr_id = repr_data->repr_id;
+       srs->repr_id = srs->switch_port_id;
        srs->switch_domain_id = repr_data->switch_domain_id;
 
        dev->data->dev_flags |= RTE_ETH_DEV_REPRESENTOR;
@@ -924,6 +996,7 @@ sfc_repr_eth_dev_init(struct rte_eth_dev *dev, void *init_params)
                goto fail_mac_addrs;
        }
 
+       dev->rx_pkt_burst = sfc_repr_rx_burst;
        dev->tx_pkt_burst = sfc_repr_tx_burst;
        dev->dev_ops = &sfc_repr_dev_ops;
 
@@ -938,7 +1011,7 @@ fail_mac_addrs:
 
 fail_alloc_sr:
        (void)sfc_repr_proxy_del_port(repr_data->pf_port_id,
-                                     repr_data->repr_id);
+                                     srs->switch_port_id);
 
 fail_create_port:
 fail_mae_assign_switch_port:
@@ -947,17 +1020,42 @@ fail_mae_assign_switch_port:
 }
 
 int
-sfc_repr_create(struct rte_eth_dev *parent, uint16_t representor_id,
-               uint16_t switch_domain_id, const efx_mport_sel_t *mport_sel)
+sfc_repr_create(struct rte_eth_dev *parent,
+               struct sfc_repr_entity_info *entity,
+               uint16_t switch_domain_id,
+               const efx_mport_sel_t *mport_sel)
 {
        struct sfc_repr_init_data repr_data;
        char name[RTE_ETH_NAME_MAX_LEN];
+       int controller;
        int ret;
+       int rc;
        struct rte_eth_dev *dev;
 
-       if (snprintf(name, sizeof(name), "net_%s_representor_%u",
-                    parent->device->name, representor_id) >=
-                       (int)sizeof(name)) {
+       controller = -1;
+       rc = sfc_mae_switch_domain_get_controller(switch_domain_id,
+                                                 entity->intf, &controller);
+       if (rc != 0) {
+               SFC_GENERIC_LOG(ERR, "%s() failed to get DPDK controller for %d",
+                               __func__, entity->intf);
+               return -rc;
+       }
+
+       switch (entity->type) {
+       case RTE_ETH_REPRESENTOR_VF:
+               ret = snprintf(name, sizeof(name), "net_%s_representor_c%upf%uvf%u",
+                              parent->device->name, controller, entity->pf,
+                              entity->vf);
+               break;
+       case RTE_ETH_REPRESENTOR_PF:
+               ret = snprintf(name, sizeof(name), "net_%s_representor_c%upf%u",
+                              parent->device->name, controller, entity->pf);
+               break;
+       default:
+               return -ENOTSUP;
+       }
+
+       if (ret >= (int)sizeof(name)) {
                SFC_GENERIC_LOG(ERR, "%s() failed name too long", __func__);
                return -ENAMETOOLONG;
        }
@@ -966,9 +1064,11 @@ sfc_repr_create(struct rte_eth_dev *parent, uint16_t representor_id,
        if (dev == NULL) {
                memset(&repr_data, 0, sizeof(repr_data));
                repr_data.pf_port_id = parent->data->port_id;
-               repr_data.repr_id = representor_id;
                repr_data.switch_domain_id = switch_domain_id;
                repr_data.mport_sel = *mport_sel;
+               repr_data.intf = entity->intf;
+               repr_data.pf = entity->pf;
+               repr_data.vf = entity->vf;
 
                ret = rte_eth_dev_create(parent->device, name,
                                         sizeof(struct sfc_repr_shared),