net/mlx5: separate aging counter pool range
[dpdk.git] / drivers / net / mlx5 / mlx5_flow_meter.c
index c39e703..bf34687 100644 (file)
@@ -4,13 +4,73 @@
  */
 #include <math.h>
 
+#include <rte_tailq.h>
 #include <rte_malloc.h>
 #include <rte_mtr.h>
 #include <rte_mtr_driver.h>
 
+#include <mlx5_devx_cmds.h>
+#include <mlx5_malloc.h>
+
 #include "mlx5.h"
 #include "mlx5_flow.h"
 
+/**
+ * Create the meter action.
+ *
+ * @param priv
+ *   Pointer to mlx5_priv.
+ * @param[in] fm
+ *   Pointer to flow meter to be converted.
+ *
+ * @return
+ *   Pointer to the meter action on success, NULL otherwise.
+ */
+static void *
+mlx5_flow_meter_action_create(struct mlx5_priv *priv,
+                             struct mlx5_flow_meter *fm)
+{
+#ifdef HAVE_MLX5_DR_CREATE_ACTION_FLOW_METER
+       struct mlx5dv_dr_flow_meter_attr mtr_init;
+       void *attr = fm->mfts->fmp;
+       struct mlx5_flow_meter_srtcm_rfc2697_prm *srtcm =
+                                                    &fm->profile->srtcm_prm;
+
+       fm->mfts->fmp_size = MLX5_ST_SZ_BYTES(flow_meter_parameters);
+       memset(attr, 0, fm->mfts->fmp_size);
+       MLX5_SET(flow_meter_parameters, attr, valid, 1);
+       MLX5_SET(flow_meter_parameters, attr, bucket_overflow, 1);
+       MLX5_SET(flow_meter_parameters, attr,
+                start_color, MLX5_FLOW_COLOR_GREEN);
+       MLX5_SET(flow_meter_parameters, attr, both_buckets_on_green, 0);
+       MLX5_SET(flow_meter_parameters,
+                attr, cbs_exponent, srtcm->cbs_exponent);
+       MLX5_SET(flow_meter_parameters,
+                attr, cbs_mantissa, srtcm->cbs_mantissa);
+       MLX5_SET(flow_meter_parameters,
+                attr, cir_exponent, srtcm->cir_exponent);
+       MLX5_SET(flow_meter_parameters,
+                attr, cir_mantissa, srtcm->cir_mantissa);
+       MLX5_SET(flow_meter_parameters,
+                attr, ebs_exponent, srtcm->ebs_exponent);
+       MLX5_SET(flow_meter_parameters,
+                attr, ebs_mantissa, srtcm->ebs_mantissa);
+       mtr_init.next_table =
+               fm->transfer ? fm->mfts->transfer.tbl->obj :
+                   fm->egress ? fm->mfts->egress.tbl->obj :
+                                      fm->mfts->ingress.tbl->obj;
+       mtr_init.reg_c_index = priv->mtr_color_reg - REG_C_0;
+       mtr_init.flow_meter_parameter = fm->mfts->fmp;
+       mtr_init.flow_meter_parameter_sz = fm->mfts->fmp_size;
+       mtr_init.active = fm->active_state;
+       return mlx5_glue->dv_create_flow_action_meter(&mtr_init);
+#else
+       (void)priv;
+       (void)fm;
+       return NULL;
+#endif
+}
+
 /**
  * Find meter profile by id.
  *
@@ -244,7 +304,7 @@ mlx5_flow_mtr_cap_get(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        memset(cap, 0, sizeof(*cap));
        cap->n_max = 1 << qattr->log_max_flow_meter;
        cap->n_shared_max = cap->n_max;
@@ -290,15 +350,15 @@ mlx5_flow_meter_profile_add(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Check input params. */
        ret = mlx5_flow_meter_profile_validate(dev, meter_profile_id,
                                               profile, error);
        if (ret)
                return ret;
        /* Meter profile memory allocation. */
-       fmp = rte_calloc(__func__, 1, sizeof(struct mlx5_flow_meter_profile),
-                        RTE_CACHE_LINE_SIZE);
+       fmp = mlx5_malloc(MLX5_MEM_ZERO, sizeof(struct mlx5_flow_meter_profile),
+                        RTE_CACHE_LINE_SIZE, SOCKET_ID_ANY);
        if (fmp == NULL)
                return -rte_mtr_error_set(error, ENOMEM,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED,
@@ -315,7 +375,7 @@ mlx5_flow_meter_profile_add(struct rte_eth_dev *dev,
        TAILQ_INSERT_TAIL(fmps, fmp, next);
        return 0;
 error:
-       rte_free(fmp);
+       mlx5_free(fmp);
        return ret;
 }
 
@@ -343,22 +403,22 @@ mlx5_flow_meter_profile_delete(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter profile must exist. */
        fmp = mlx5_flow_meter_profile_find(priv, meter_profile_id);
        if (fmp == NULL)
                return -rte_mtr_error_set(error, ENOENT,
                                          RTE_MTR_ERROR_TYPE_METER_PROFILE_ID,
                                          &meter_profile_id,
-                                         "Meter profile id invalid.");
+                                         "Meter profile id is invalid.");
        /* Check profile is unused. */
        if (fmp->ref_cnt)
                return -rte_mtr_error_set(error, EBUSY,
                                          RTE_MTR_ERROR_TYPE_METER_PROFILE_ID,
-                                         NULL, "Meter profile in use.");
+                                         NULL, "Meter profile is in use.");
        /* Remove from list. */
        TAILQ_REMOVE(&priv->flow_meter_profiles, fmp, next);
-       rte_free(fmp);
+       mlx5_free(fmp);
        return 0;
 }
 
@@ -572,11 +632,12 @@ mlx5_flow_meter_create(struct rte_eth_dev *dev, uint32_t meter_id,
                        };
        int ret;
        unsigned int i;
+       uint32_t idx = 0;
 
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Validate the parameters. */
        ret = mlx5_flow_meter_validate(priv, meter_id, params, error);
        if (ret)
@@ -588,16 +649,18 @@ mlx5_flow_meter_create(struct rte_eth_dev *dev, uint32_t meter_id,
                                          RTE_MTR_ERROR_TYPE_METER_PROFILE_ID,
                                          NULL, "Meter profile id not valid.");
        /* Allocate the flow meter memory. */
-       fm = rte_calloc(__func__, 1,
-                       sizeof(struct mlx5_flow_meter), RTE_CACHE_LINE_SIZE);
+       fm = mlx5_ipool_zmalloc(priv->sh->ipool[MLX5_IPOOL_MTR], &idx);
        if (fm == NULL)
                return -rte_mtr_error_set(error, ENOMEM,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
                                          "Memory alloc failed for meter.");
+       fm->idx = idx;
        /* Fill the flow meter parameters. */
        fm->meter_id = meter_id;
        fm->profile = fmp;
-       fm->params = *params;
+       memcpy(fm->action, params->action, sizeof(params->action));
+       fm->stats_mask = params->stats_mask;
+
        /* Alloc policer counters. */
        for (i = 0; i < RTE_DIM(fm->policer_stats.cnt); i++) {
                fm->policer_stats.cnt[i] = mlx5_counter_alloc(dev);
@@ -624,7 +687,7 @@ error:
        for (i = 0; i < RTE_DIM(fm->policer_stats.cnt); i++)
                if (fm->policer_stats.cnt[i])
                        mlx5_counter_free(dev, fm->policer_stats.cnt[i]);
-       rte_free(fm);
+       mlx5_ipool_free(priv->sh->ipool[MLX5_IPOOL_MTR], idx);
        return -rte_mtr_error_set(error, -ret,
                                  RTE_MTR_ERROR_TYPE_UNSPECIFIED,
                                  NULL, "Failed to create devx meter.");
@@ -661,7 +724,7 @@ mlx5_flow_meter_destroy(struct rte_eth_dev *dev, uint32_t meter_id,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter object must exist. */
        fm = mlx5_flow_meter_find(priv, meter_id);
        if (fm == NULL)
@@ -675,7 +738,7 @@ mlx5_flow_meter_destroy(struct rte_eth_dev *dev, uint32_t meter_id,
                                          NULL, "Meter object is being used.");
        /* Get the meter profile. */
        fmp = fm->profile;
-       RTE_ASSERT(fmp);
+       MLX5_ASSERT(fmp);
        /* Update dependencies. */
        fmp->ref_cnt--;
        /* Remove from the flow meter list. */
@@ -687,7 +750,7 @@ mlx5_flow_meter_destroy(struct rte_eth_dev *dev, uint32_t meter_id,
        /* Free meter flow table */
        mlx5_flow_destroy_policer_rules(dev, fm, &attr);
        mlx5_flow_destroy_mtr_tbls(dev, fm->mfts);
-       rte_free(fm);
+       mlx5_ipool_free(priv->sh->ipool[MLX5_IPOOL_MTR], fm->idx);
        return 0;
 }
 
@@ -766,7 +829,7 @@ mlx5_flow_meter_enable(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter object must exist. */
        fm = mlx5_flow_meter_find(priv, meter_id);
        if (fm == NULL)
@@ -807,7 +870,7 @@ mlx5_flow_meter_disable(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter object must exist. */
        fm = mlx5_flow_meter_find(priv, meter_id);
        if (fm == NULL)
@@ -855,7 +918,7 @@ mlx5_flow_meter_profile_update(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter profile must exist. */
        fmp = mlx5_flow_meter_profile_find(priv, meter_profile_id);
        if (fmp == NULL)
@@ -918,7 +981,7 @@ mlx5_flow_meter_stats_update(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter object must exist. */
        fm = mlx5_flow_meter_find(priv, meter_id);
        if (fm == NULL)
@@ -975,7 +1038,7 @@ mlx5_flow_meter_stats_read(struct rte_eth_dev *dev,
        if (!priv->mtr_en)
                return -rte_mtr_error_set(error, ENOTSUP,
                                          RTE_MTR_ERROR_TYPE_UNSPECIFIED, NULL,
-                                         "Meter is not support");
+                                         "Meter is not supported");
        /* Meter object must exist. */
        fm = mlx5_flow_meter_find(priv, meter_id);
        if (fm == NULL)
@@ -990,7 +1053,7 @@ mlx5_flow_meter_stats_read(struct rte_eth_dev *dev,
                                                 &bytes);
                        if (ret)
                                goto error;
-                       if (fm->params.action[i] == MTR_POLICER_ACTION_DROP) {
+                       if (fm->action[i] == MTR_POLICER_ACTION_DROP) {
                                pkts_dropped += pkts;
                                bytes_dropped += bytes;
                        }
@@ -1081,3 +1144,150 @@ mlx5_flow_meter_find(struct mlx5_priv *priv, uint32_t meter_id)
                        return fm;
        return NULL;
 }
+
+/**
+ * Attach meter to flow.
+ * Unidirectional Meter creation can only be done
+ * when flow direction is known, i.e. when calling meter_attach.
+ *
+ * @param [in] priv
+ *  Pointer to mlx5 private data.
+ * @param [in] meter_id
+ *  Flow meter id.
+ * @param [in] attr
+ *  Pointer to flow attributes.
+ * @param [out] error
+ *  Pointer to error structure.
+ *
+ * @return the flow meter pointer, NULL otherwise.
+ */
+struct mlx5_flow_meter *
+mlx5_flow_meter_attach(struct mlx5_priv *priv, uint32_t meter_id,
+                      const struct rte_flow_attr *attr,
+                      struct rte_flow_error *error)
+{
+       struct mlx5_flow_meter *fm;
+
+       fm = mlx5_flow_meter_find(priv, meter_id);
+       if (fm == NULL) {
+               rte_flow_error_set(error, ENOENT,
+                                  RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
+                                  "Meter object id not valid");
+               goto error;
+       }
+       if (!fm->shared && fm->ref_cnt) {
+               DRV_LOG(ERR, "Cannot share a non-shared meter.");
+               rte_flow_error_set(error, EINVAL,
+                                 RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
+                                 "Meter can't be shared");
+               goto error;
+       }
+       if (!fm->ref_cnt++) {
+               MLX5_ASSERT(!fm->mfts->meter_action);
+               fm->ingress = attr->ingress;
+               fm->egress = attr->egress;
+               fm->transfer = attr->transfer;
+               /* This also creates the meter object. */
+               fm->mfts->meter_action = mlx5_flow_meter_action_create(priv,
+                                                                      fm);
+               if (!fm->mfts->meter_action)
+                       goto error_detach;
+       } else {
+               MLX5_ASSERT(fm->mfts->meter_action);
+               if (attr->transfer != fm->transfer ||
+                   attr->ingress != fm->ingress ||
+                   attr->egress != fm->egress) {
+                       DRV_LOG(ERR, "meter I/O attributes do not "
+                               "match flow I/O attributes.");
+                       goto error_detach;
+               }
+       }
+       return fm;
+error_detach:
+       mlx5_flow_meter_detach(fm);
+       rte_flow_error_set(error, EINVAL, RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL,
+                         fm->mfts->meter_action ? "Meter attr not match" :
+                         "Meter action create failed");
+error:
+       return NULL;
+}
+
+/**
+ * Detach meter from flow.
+ *
+ * @param [in] fm
+ *  Pointer to flow meter.
+ */
+void
+mlx5_flow_meter_detach(struct mlx5_flow_meter *fm)
+{
+       MLX5_ASSERT(fm->ref_cnt);
+       if (--fm->ref_cnt)
+               return;
+       if (fm->mfts->meter_action)
+               mlx5_glue->destroy_flow_action(fm->mfts->meter_action);
+       fm->mfts->meter_action = NULL;
+       fm->ingress = 0;
+       fm->egress = 0;
+       fm->transfer = 0;
+}
+
+/**
+ * Flush meter configuration.
+ *
+ * @param[in] dev
+ *   Pointer to Ethernet device.
+ * @param[out] error
+ *   Pointer to rte meter error structure.
+ *
+ * @return
+ *   0 on success, a negative errno value otherwise and rte_errno is set.
+ */
+int
+mlx5_flow_meter_flush(struct rte_eth_dev *dev, struct rte_mtr_error *error)
+{
+       struct mlx5_priv *priv = dev->data->dev_private;
+       struct mlx5_flow_meters *fms = &priv->flow_meters;
+       struct mlx5_mtr_profiles *fmps = &priv->flow_meter_profiles;
+       struct mlx5_flow_meter_profile *fmp;
+       struct mlx5_flow_meter *fm;
+       const struct rte_flow_attr attr = {
+                               .ingress = 1,
+                               .egress = 1,
+                               .transfer = priv->config.dv_esw_en ? 1 : 0,
+                       };
+       void *tmp;
+       uint32_t i;
+
+       TAILQ_FOREACH_SAFE(fm, fms, next, tmp) {
+               /* Meter object must not have any owner. */
+               MLX5_ASSERT(!fm->ref_cnt);
+               /* Get meter profile. */
+               fmp = fm->profile;
+               if (fmp == NULL)
+                       return -rte_mtr_error_set(error, EINVAL,
+                               RTE_MTR_ERROR_TYPE_METER_PROFILE_ID,
+                               NULL, "MTR object meter profile invalid.");
+               /* Update dependencies. */
+               fmp->ref_cnt--;
+               /* Remove from list. */
+               TAILQ_REMOVE(fms, fm, next);
+               /* Free policer counters. */
+               for (i = 0; i < RTE_DIM(fm->policer_stats.cnt); i++)
+                       if (fm->policer_stats.cnt[i])
+                               mlx5_counter_free(dev,
+                                                 fm->policer_stats.cnt[i]);
+               /* Free meter flow table. */
+               mlx5_flow_destroy_policer_rules(dev, fm, &attr);
+               mlx5_flow_destroy_mtr_tbls(dev, fm->mfts);
+               mlx5_ipool_free(priv->sh->ipool[MLX5_IPOOL_MTR], fm->idx);
+       }
+       TAILQ_FOREACH_SAFE(fmp, fmps, next, tmp) {
+               /* Check unused. */
+               MLX5_ASSERT(!fmp->ref_cnt);
+               /* Remove from list. */
+               TAILQ_REMOVE(&priv->flow_meter_profiles, fmp, next);
+               mlx5_free(fmp);
+       }
+       return 0;
+}