From: Ady Agbarih <adypodo...@gmail.com>

Remove the set/query regexp register commands from devx.
Remove functions that used these commands.
Remove manual rules programming.

Signed-off-by: Ady Agbarih <adypodo...@gmail.com>
---
 drivers/regex/mlx5/mlx5_regex.c      |  28 --
 drivers/regex/mlx5/mlx5_regex.h      |   4 -
 drivers/regex/mlx5/mlx5_regex_devx.c |  48 --
 drivers/regex/mlx5/mlx5_rxp.c        | 708 +--------------------------
 4 files changed, 1 insertion(+), 787 deletions(-)

diff --git a/drivers/regex/mlx5/mlx5_regex.c b/drivers/regex/mlx5/mlx5_regex.c
index 4be36e40c5..f9eb3a2fab 100644
--- a/drivers/regex/mlx5/mlx5_regex.c
+++ b/drivers/regex/mlx5/mlx5_regex.c
@@ -68,29 +68,6 @@ mlx5_regex_close(struct rte_regexdev *dev __rte_unused)
        return 0;
 }
 
-static int
-mlx5_regex_engines_status(struct ibv_context *ctx, int num_engines)
-{
-       uint32_t fpga_ident = 0;
-       int err;
-       int i;
-
-       for (i = 0; i < num_engines; i++) {
-               err = mlx5_devx_regex_register_read(ctx, i,
-                                                   MLX5_RXP_CSR_IDENTIFIER,
-                                                   &fpga_ident);
-               fpga_ident = (fpga_ident & (0x0000FFFF));
-               if (err || fpga_ident != MLX5_RXP_IDENTIFIER) {
-                       DRV_LOG(ERR, "Failed setup RXP %d err %d database "
-                               "memory 0x%x", i, err, fpga_ident);
-                       if (!err)
-                               err = EINVAL;
-                       return err;
-               }
-       }
-       return 0;
-}
-
 static void
 mlx5_regex_get_name(char *name, struct rte_device *dev)
 {
@@ -163,11 +140,6 @@ mlx5_regex_dev_probe(struct rte_device *rte_dev)
                rte_errno = ENOTSUP;
                goto dev_error;
        }
-       if (mlx5_regex_engines_status(ctx, 2)) {
-               DRV_LOG(ERR, "RegEx engine error.");
-               rte_errno = ENOMEM;
-               goto dev_error;
-       }
        priv = rte_zmalloc("mlx5 regex device private", sizeof(*priv),
                           RTE_CACHE_LINE_SIZE);
        if (!priv) {
diff --git a/drivers/regex/mlx5/mlx5_regex.h b/drivers/regex/mlx5/mlx5_regex.h
index c9586ae714..09b360a1ab 100644
--- a/drivers/regex/mlx5/mlx5_regex.h
+++ b/drivers/regex/mlx5/mlx5_regex.h
@@ -113,10 +113,6 @@ int mlx5_regex_rules_db_import(struct rte_regexdev *dev,
                               const char *rule_db, uint32_t rule_db_len);
 
 /* mlx5_regex_devx.c */
-int mlx5_devx_regex_register_write(struct ibv_context *ctx, int engine_id,
-                                  uint32_t addr, uint32_t data);
-int mlx5_devx_regex_register_read(struct ibv_context *ctx, int engine_id,
-                                 uint32_t addr, uint32_t *data);
 int mlx5_devx_regex_database_stop(void *ctx, uint8_t engine);
 int mlx5_devx_regex_database_resume(void *ctx, uint8_t engine);
 int mlx5_devx_regex_rules_program(void *ctx, uint8_t engine, uint32_t rof_mkey,
diff --git a/drivers/regex/mlx5/mlx5_regex_devx.c 
b/drivers/regex/mlx5/mlx5_regex_devx.c
index d8515768c3..262f301121 100644
--- a/drivers/regex/mlx5/mlx5_regex_devx.c
+++ b/drivers/regex/mlx5/mlx5_regex_devx.c
@@ -12,54 +12,6 @@
 #include "mlx5_regex.h"
 #include "mlx5_regex_utils.h"
 
-int
-mlx5_devx_regex_register_write(struct ibv_context *ctx, int engine_id,
-                              uint32_t addr, uint32_t data)
-{
-       uint32_t out[MLX5_ST_SZ_DW(set_regexp_register_out)] = {0};
-       uint32_t in[MLX5_ST_SZ_DW(set_regexp_register_in)] = {0};
-       int ret;
-
-       MLX5_SET(set_regexp_register_in, in, opcode,
-                MLX5_CMD_SET_REGEX_REGISTERS);
-       MLX5_SET(set_regexp_register_in, in, engine_id, engine_id);
-       MLX5_SET(set_regexp_register_in, in, register_address, addr);
-       MLX5_SET(set_regexp_register_in, in, register_data, data);
-
-       ret = mlx5_glue->devx_general_cmd(ctx, in, sizeof(in), out,
-                                         sizeof(out));
-       if (ret) {
-               DRV_LOG(ERR, "Set regexp register failed %d", ret);
-               rte_errno = errno;
-               return -errno;
-       }
-       return 0;
-}
-
-int
-mlx5_devx_regex_register_read(struct ibv_context *ctx, int engine_id,
-                             uint32_t addr, uint32_t *data)
-{
-       uint32_t out[MLX5_ST_SZ_DW(query_regexp_register_out)] = {0};
-       uint32_t in[MLX5_ST_SZ_DW(query_regexp_register_in)] = {0};
-       int ret;
-
-       MLX5_SET(query_regexp_register_in, in, opcode,
-                MLX5_CMD_QUERY_REGEX_REGISTERS);
-       MLX5_SET(query_regexp_register_in, in, engine_id, engine_id);
-       MLX5_SET(query_regexp_register_in, in, register_address, addr);
-
-       ret = mlx5_glue->devx_general_cmd(ctx, in, sizeof(in), out,
-                                         sizeof(out));
-       if (ret) {
-               DRV_LOG(ERR, "Query regexp register failed %d", ret);
-               rte_errno = errno;
-               return -errno;
-       }
-       *data = MLX5_GET(query_regexp_register_out, out, register_data);
-       return 0;
-}
-
 int
 mlx5_devx_regex_database_stop(void *ctx, uint8_t engine)
 {
diff --git a/drivers/regex/mlx5/mlx5_rxp.c b/drivers/regex/mlx5/mlx5_rxp.c
index 79f063a127..8f54ab018e 100644
--- a/drivers/regex/mlx5/mlx5_rxp.c
+++ b/drivers/regex/mlx5/mlx5_rxp.c
@@ -28,71 +28,6 @@
 #define MLX5_REGEX_RXP_ROF2_LINE_LEN 34
 
 /* Private Declarations */
-static int
-rxp_poll_csr_for_value(struct ibv_context *ctx, uint32_t *value,
-                      uint32_t address, uint32_t expected_value,
-                      uint32_t expected_mask, uint32_t timeout_ms, uint8_t id);
-static int
-mlnx_resume_database(struct mlx5_regex_priv *priv, uint8_t id);
-static int
-mlnx_update_database(struct mlx5_regex_priv *priv, uint8_t id);
-static int
-program_rxp_rules(struct mlx5_regex_priv *priv, const char *buf, uint32_t len,
-                 uint8_t id);
-static int
-rxp_init_eng(struct mlx5_regex_priv *priv, uint8_t id);
-static int
-rxp_db_setup(struct mlx5_regex_priv *priv);
-static void
-rxp_dump_csrs(struct ibv_context *ctx, uint8_t id);
-static int
-rxp_start_engine(struct ibv_context *ctx, uint8_t id);
-static int
-rxp_stop_engine(struct ibv_context *ctx, uint8_t id);
-
-static void __rte_unused
-rxp_dump_csrs(struct ibv_context *ctx __rte_unused, uint8_t id __rte_unused)
-{
-       uint32_t reg, i;
-
-       /* Main CSRs*/
-       for (i = 0; i < MLX5_RXP_CSR_NUM_ENTRIES; i++) {
-               if (mlx5_devx_regex_register_read(ctx, id,
-                                                 (MLX5_RXP_CSR_WIDTH * i) +
-                                                 MLX5_RXP_CSR_BASE_ADDRESS,
-                                                 &reg)) {
-                       DRV_LOG(ERR, "Failed to read Main CSRs Engine %d!", id);
-                       return;
-               }
-               DRV_LOG(DEBUG, "RXP Main CSRs (Eng%d) register (%d): %08x",
-                       id, i, reg);
-       }
-       /* RTRU CSRs*/
-       for (i = 0; i < MLX5_RXP_CSR_NUM_ENTRIES; i++) {
-               if (mlx5_devx_regex_register_read(ctx, id,
-                                                 (MLX5_RXP_CSR_WIDTH * i) +
-                                                MLX5_RXP_RTRU_CSR_BASE_ADDRESS,
-                                                 &reg)) {
-                       DRV_LOG(ERR, "Failed to read RTRU CSRs Engine %d!", id);
-                       return;
-               }
-               DRV_LOG(DEBUG, "RXP RTRU CSRs (Eng%d) register (%d): %08x",
-                       id, i, reg);
-       }
-       /* STAT CSRs */
-       for (i = 0; i < MLX5_RXP_CSR_NUM_ENTRIES; i++) {
-               if (mlx5_devx_regex_register_read(ctx, id,
-                                                 (MLX5_RXP_CSR_WIDTH * i) +
-                                               MLX5_RXP_STATS_CSR_BASE_ADDRESS,
-                                                 &reg)) {
-                       DRV_LOG(ERR, "Failed to read STAT CSRs Engine %d!", id);
-                       return;
-               }
-               DRV_LOG(DEBUG, "RXP STAT CSRs (Eng%d) register (%d): %08x",
-                       id, i, reg);
-       }
-}
-
 int
 mlx5_regex_info_get(struct rte_regexdev *dev __rte_unused,
                    struct rte_regexdev_info *info)
@@ -108,614 +43,6 @@ mlx5_regex_info_get(struct rte_regexdev *dev __rte_unused,
        return 0;
 }
 
-static int
-rxp_poll_csr_for_value(struct ibv_context *ctx, uint32_t *value,
-                      uint32_t address, uint32_t expected_value,
-                      uint32_t expected_mask, uint32_t timeout_ms, uint8_t id)
-{
-       unsigned int i;
-       int ret;
-
-       ret = -EBUSY;
-       for (i = 0; i < timeout_ms; i++) {
-               if (mlx5_devx_regex_register_read(ctx, id, address, value))
-                       return -1;
-               if ((*value & expected_mask) == expected_value) {
-                       ret = 0;
-                       break;
-               }
-               rte_delay_us(1000);
-       }
-       return ret;
-}
-
-static int
-rxp_start_engine(struct ibv_context *ctx, uint8_t id)
-{
-       uint32_t ctrl;
-       int ret;
-
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_CSR_CTRL, &ctrl);
-       if (ret)
-               return ret;
-       ctrl |= MLX5_RXP_CSR_CTRL_GO;
-       ctrl |= MLX5_RXP_CSR_CTRL_DISABLE_L2C;
-       ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL, ctrl);
-       return ret;
-}
-
-static int
-rxp_stop_engine(struct ibv_context *ctx, uint8_t id)
-{
-       uint32_t ctrl;
-       int ret;
-
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_CSR_CTRL, &ctrl);
-       if (ret)
-               return ret;
-       ctrl &= ~MLX5_RXP_CSR_CTRL_GO;
-       ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL, ctrl);
-       return ret;
-}
-
-static int
-rxp_init_rtru(struct mlx5_regex_priv *priv, uint8_t id, uint32_t init_bits)
-{
-       uint32_t ctrl_value;
-       uint32_t poll_value;
-       uint32_t expected_value;
-       uint32_t expected_mask;
-       struct ibv_context *ctx = priv->ctx;
-       int ret = 0;
-
-       /* Read the rtru ctrl CSR. */
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                           &ctrl_value);
-       if (ret)
-               return -1;
-       /* Clear any previous init modes. */
-       ctrl_value &= ~(MLX5_RXP_RTRU_CSR_CTRL_INIT_MODE_MASK);
-       if (ctrl_value & MLX5_RXP_RTRU_CSR_CTRL_INIT) {
-               ctrl_value &= ~(MLX5_RXP_RTRU_CSR_CTRL_INIT);
-               mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                              ctrl_value);
-       }
-       /* Set the init_mode bits in the rtru ctrl CSR. */
-       ctrl_value |= init_bits;
-       mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                      ctrl_value);
-       /* Need to sleep for a short period after pulsing the rtru init bit. */
-       rte_delay_us(20000);
-       /* Poll the rtru status CSR until all the init done bits are set. */
-       DRV_LOG(DEBUG, "waiting for RXP rule memory to complete init");
-       /* Set the init bit in the rtru ctrl CSR. */
-       ctrl_value |= MLX5_RXP_RTRU_CSR_CTRL_INIT;
-       mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                      ctrl_value);
-       /* Clear the init bit in the rtru ctrl CSR */
-       ctrl_value &= ~MLX5_RXP_RTRU_CSR_CTRL_INIT;
-       mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                      ctrl_value);
-       /* Check that the following bits are set in the RTRU_CSR. */
-       if (init_bits == MLX5_RXP_RTRU_CSR_CTRL_INIT_MODE_L1_L2) {
-               /* Must be incremental mode */
-               expected_value = MLX5_RXP_RTRU_CSR_STATUS_L1C_INIT_DONE;
-       } else {
-               expected_value = MLX5_RXP_RTRU_CSR_STATUS_IM_INIT_DONE |
-                       MLX5_RXP_RTRU_CSR_STATUS_L1C_INIT_DONE;
-       }
-       if (priv->is_bf2)
-               expected_value |= MLX5_RXP_RTRU_CSR_STATUS_L2C_INIT_DONE;
-
-
-       expected_mask = expected_value;
-       ret = rxp_poll_csr_for_value(ctx, &poll_value,
-                                    MLX5_RXP_RTRU_CSR_STATUS,
-                                    expected_value, expected_mask,
-                                    MLX5_RXP_CSR_STATUS_TRIAL_TIMEOUT, id);
-       if (ret)
-               return ret;
-       DRV_LOG(DEBUG, "rule memory initialise: 0x%08X", poll_value);
-       /* Clear the init bit in the rtru ctrl CSR */
-       ctrl_value &= ~(MLX5_RXP_RTRU_CSR_CTRL_INIT);
-       mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                      ctrl_value);
-       return 0;
-}
-
-static int
-rxp_parse_line(char *line, uint32_t *type, uint32_t *address, uint64_t *value)
-{
-       char *cur_pos;
-
-       if (*line == '\0' || *line == '#')
-               return  1;
-       *type = strtoul(line, &cur_pos, 10);
-       if (*cur_pos != ',' && *cur_pos != '\0')
-               return -1;
-       *address = strtoul(cur_pos+1, &cur_pos, 16);
-       if (*cur_pos != ',' && *cur_pos != '\0')
-               return -1;
-       *value = strtoul(cur_pos+1, &cur_pos, 16);
-       if (*cur_pos != ',' && *cur_pos != '\0')
-               return -1;
-       return 0;
-}
-
-static uint32_t
-rxp_get_reg_address(uint32_t address)
-{
-       uint32_t block;
-       uint32_t reg;
-
-       block = (address >> 16) & 0xFFFF;
-       if (block == 0)
-               reg = MLX5_RXP_CSR_BASE_ADDRESS;
-       else if (block == 1)
-               reg = MLX5_RXP_RTRU_CSR_BASE_ADDRESS;
-       else {
-               DRV_LOG(ERR, "Invalid ROF register 0x%08X!", address);
-                       return UINT32_MAX;
-       }
-       reg += (address & 0xFFFF) * MLX5_RXP_CSR_WIDTH;
-       return reg;
-}
-
-#define MLX5_RXP_NUM_LINES_PER_BLOCK 8
-
-static int
-rxp_program_rof(struct mlx5_regex_priv *priv, const char *buf, uint32_t len,
-               uint8_t id)
-{
-       static const char del[] = "\n\r";
-       char *line;
-       char *tmp;
-       uint32_t type = 0;
-       uint32_t address;
-       uint64_t val;
-       uint32_t reg_val;
-       int ret;
-       int skip = -1;
-       int last = 0;
-       uint32_t temp;
-       uint32_t tmp_addr;
-       uint32_t rof_rule_addr;
-       uint64_t tmp_write_swap[4];
-       struct mlx5_rxp_rof_entry rules[8];
-       int i;
-       int db_free;
-       int j;
-
-       tmp = rte_malloc("", len, 0);
-       if (!tmp)
-               return -ENOMEM;
-       memcpy(tmp, buf, len);
-       db_free = mlnx_update_database(priv, id);
-       if (db_free < 0) {
-               DRV_LOG(ERR, "Failed to setup db memory!");
-               rte_free(tmp);
-               return db_free;
-       }
-       for (line = strtok(tmp, del), j = 0; line; line = strtok(NULL, del),
-            j++, last = type) {
-               ret = rxp_parse_line(line, &type, &address, &val);
-               if (ret != 0) {
-                       if (ret < 0)
-                               goto parse_error;
-                       continue;
-               }
-               switch (type) {
-               case MLX5_RXP_ROF_ENTRY_EQ:
-                       if (skip == 0 && address == 0)
-                               skip = 1;
-                       tmp_addr = rxp_get_reg_address(address);
-                       if (tmp_addr == UINT32_MAX)
-                               goto parse_error;
-                       ret = mlx5_devx_regex_register_read(priv->ctx, id,
-                                                           tmp_addr, &reg_val);
-                       if (ret)
-                               goto parse_error;
-                       if (skip == -1 && address == 0) {
-                               if (val == reg_val) {
-                                       skip = 0;
-                                       continue;
-                               }
-                       } else if (skip == 0) {
-                               if (val != reg_val) {
-                                       DRV_LOG(ERR,
-                                               "got %08X expected == %" PRIx64,
-                                               reg_val, val);
-                                       goto parse_error;
-                               }
-                       }
-                       break;
-               case MLX5_RXP_ROF_ENTRY_GTE:
-                       if (skip == 0 && address == 0)
-                               skip = 1;
-                       tmp_addr = rxp_get_reg_address(address);
-                       if (tmp_addr == UINT32_MAX)
-                               goto parse_error;
-                       ret = mlx5_devx_regex_register_read(priv->ctx, id,
-                                                           tmp_addr, &reg_val);
-                       if (ret)
-                               goto parse_error;
-                       if (skip == -1 && address == 0) {
-                               if (reg_val >= val) {
-                                       skip = 0;
-                                       continue;
-                               }
-                       } else if (skip == 0) {
-                               if (reg_val < val) {
-                                       DRV_LOG(ERR,
-                                               "got %08X expected >= %" PRIx64,
-                                               reg_val, val);
-                                       goto parse_error;
-                               }
-                       }
-                       break;
-               case MLX5_RXP_ROF_ENTRY_LTE:
-                       tmp_addr = rxp_get_reg_address(address);
-                       if (tmp_addr == UINT32_MAX)
-                               goto parse_error;
-                       ret = mlx5_devx_regex_register_read(priv->ctx, id,
-                                                           tmp_addr, &reg_val);
-                       if (ret)
-                               goto parse_error;
-                       if (skip == 0 && address == 0 &&
-                           last != MLX5_RXP_ROF_ENTRY_GTE) {
-                               skip = 1;
-                       } else if (skip == 0 && address == 0 &&
-                                  last == MLX5_RXP_ROF_ENTRY_GTE) {
-                               if (reg_val > val)
-                                       skip = -1;
-                               continue;
-                       }
-                       if (skip == -1 && address == 0) {
-                               if (reg_val <= val) {
-                                       skip = 0;
-                                       continue;
-                               }
-                       } else if (skip == 0) {
-                               if (reg_val > val) {
-                                       DRV_LOG(ERR,
-                                               "got %08X expected <= %" PRIx64,
-                                               reg_val, val);
-                                       goto parse_error;
-                               }
-                       }
-                       break;
-               case MLX5_RXP_ROF_ENTRY_CHECKSUM:
-                       break;
-               case MLX5_RXP_ROF_ENTRY_CHECKSUM_EX_EM:
-                       if (skip)
-                               continue;
-                       tmp_addr = rxp_get_reg_address(address);
-                       if (tmp_addr == UINT32_MAX)
-                               goto parse_error;
-
-                       ret = mlx5_devx_regex_register_read(priv->ctx, id,
-                                                           tmp_addr, &reg_val);
-                       if (ret) {
-                               DRV_LOG(ERR, "RXP CSR read failed!");
-                               return ret;
-                       }
-                       if (reg_val != val) {
-                               DRV_LOG(ERR, "got %08X expected <= %" PRIx64,
-                                       reg_val, val);
-                               goto parse_error;
-                       }
-                       break;
-               case MLX5_RXP_ROF_ENTRY_IM:
-                       if (skip)
-                               continue;
-                       /*
-                        * NOTE: All rules written to RXP must be carried out in
-                        * triplets of: 2xData + 1xAddr.
-                        * No optimisation is currently allowed in this
-                        * sequence to perform less writes.
-                        */
-                       temp = val;
-                       ret |= mlx5_devx_regex_register_write
-                                       (priv->ctx, id,
-                                        MLX5_RXP_RTRU_CSR_DATA_0, temp);
-                       temp = (uint32_t)(val >> 32);
-                       ret |= mlx5_devx_regex_register_write
-                                       (priv->ctx, id,
-                                        MLX5_RXP_RTRU_CSR_DATA_0 +
-                                        MLX5_RXP_CSR_WIDTH, temp);
-                       temp = address;
-                       ret |= mlx5_devx_regex_register_write
-                                       (priv->ctx, id, MLX5_RXP_RTRU_CSR_ADDR,
-                                        temp);
-                       if (ret) {
-                               DRV_LOG(ERR,
-                                       "Failed to copy instructions to RXP.");
-                               goto parse_error;
-                       }
-                       break;
-               case MLX5_RXP_ROF_ENTRY_EM:
-                       if (skip)
-                               continue;
-                       for (i = 0; i < MLX5_RXP_NUM_LINES_PER_BLOCK; i++) {
-                               ret = rxp_parse_line(line, &type,
-                                                    &rules[i].addr,
-                                                    &rules[i].value);
-                               if (ret != 0)
-                                       goto parse_error;
-                               if (i < (MLX5_RXP_NUM_LINES_PER_BLOCK - 1)) {
-                                       line = strtok(NULL, del);
-                                       if (!line)
-                                               goto parse_error;
-                               }
-                       }
-                       if ((uint8_t *)((uint8_t *)
-                                       priv->db[id].ptr +
-                                       ((rules[7].addr <<
-                                        MLX5_RXP_INST_OFFSET))) >=
-                                       ((uint8_t *)((uint8_t *)
-                                       priv->db[id].ptr + MLX5_MAX_DB_SIZE))) {
-                               DRV_LOG(ERR, "DB exceeded memory!");
-                               goto parse_error;
-                       }
-                       /*
-                        * Rule address Offset to align with RXP
-                        * external instruction offset.
-                        */
-                       rof_rule_addr = (rules[0].addr << MLX5_RXP_INST_OFFSET);
-                       /* 32 byte instruction swap (sw work around)! */
-                       tmp_write_swap[0] = le64toh(rules[4].value);
-                       tmp_write_swap[1] = le64toh(rules[5].value);
-                       tmp_write_swap[2] = le64toh(rules[6].value);
-                       tmp_write_swap[3] = le64toh(rules[7].value);
-                       /* Write only 4 of the 8 instructions. */
-                       memcpy((uint8_t *)((uint8_t *)
-                               priv->db[id].ptr + rof_rule_addr),
-                               &tmp_write_swap, (sizeof(uint64_t) * 4));
-                       /* Write 1st 4 rules of block after last 4. */
-                       rof_rule_addr = (rules[4].addr << MLX5_RXP_INST_OFFSET);
-                       tmp_write_swap[0] = le64toh(rules[0].value);
-                       tmp_write_swap[1] = le64toh(rules[1].value);
-                       tmp_write_swap[2] = le64toh(rules[2].value);
-                       tmp_write_swap[3] = le64toh(rules[3].value);
-                       memcpy((uint8_t *)((uint8_t *)
-                               priv->db[id].ptr + rof_rule_addr),
-                               &tmp_write_swap, (sizeof(uint64_t) * 4));
-                       break;
-               default:
-                       break;
-               }
-
-       }
-       rte_free(tmp);
-       return 0;
-parse_error:
-       rte_free(tmp);
-       return ret;
-}
-
-static int
-mlnx_resume_database(struct mlx5_regex_priv *priv, uint8_t id)
-{
-       mlx5_devx_regex_database_resume(priv->ctx, id);
-       return 0;
-}
-
-/*
- * Assign db memory for RXP programming.
- */
-static int
-mlnx_update_database(struct mlx5_regex_priv *priv, uint8_t id)
-{
-       unsigned int i;
-       uint8_t db_free = MLX5_RXP_DB_NOT_ASSIGNED;
-       uint8_t eng_assigned = MLX5_RXP_DB_NOT_ASSIGNED;
-
-       /* Check which database rxp_eng is currently located if any? */
-       for (i = 0; i < (priv->nb_engines + MLX5_RXP_EM_COUNT);
-            i++) {
-               if (priv->db[i].db_assigned_to_eng_num == id) {
-                       eng_assigned = i;
-                       break;
-               }
-       }
-       /*
-        * If private mode then, we can keep the same db ptr as RXP will be
-        * programming EM itself if necessary, however need to see if
-        * programmed yet.
-        */
-       if ((priv->prog_mode == MLX5_RXP_PRIVATE_PROG_MODE) &&
-           (eng_assigned != MLX5_RXP_DB_NOT_ASSIGNED))
-               return eng_assigned;
-       /* Check for inactive db memory to use. */
-       for (i = 0; i < (priv->nb_engines + MLX5_RXP_EM_COUNT);
-            i++) {
-               if (priv->db[i].active == true)
-                       continue; /* Already in use, so skip db. */
-               /* Set this db to active now as free to use. */
-               priv->db[i].active = true;
-               /* Now unassign last db index in use by RXP Eng. */
-               if (eng_assigned != MLX5_RXP_DB_NOT_ASSIGNED) {
-                       priv->db[eng_assigned].active = false;
-                       priv->db[eng_assigned].db_assigned_to_eng_num =
-                               MLX5_RXP_DB_NOT_ASSIGNED;
-
-                       /* Set all DB memory to 0's before setting up DB. */
-                       memset(priv->db[i].ptr, 0x00, MLX5_MAX_DB_SIZE);
-               }
-               /* Now reassign new db index with RXP Engine. */
-               priv->db[i].db_assigned_to_eng_num = id;
-               db_free = i;
-               break;
-       }
-       if (db_free == MLX5_RXP_DB_NOT_ASSIGNED)
-               return -1;
-       return db_free;
-}
-
-/*
- * Program RXP instruction db to RXP engine/s.
- */
-static int
-program_rxp_rules(struct mlx5_regex_priv *priv, const char *buf, uint32_t len,
-                 uint8_t id)
-{
-       int ret;
-       uint32_t val;
-
-       ret = rxp_init_eng(priv, id);
-       if (ret < 0)
-               return ret;
-       /* Confirm the RXP is initialised. */
-       if (mlx5_devx_regex_register_read(priv->ctx, id,
-                                           MLX5_RXP_CSR_STATUS, &val)) {
-               DRV_LOG(ERR, "Failed to read from RXP!");
-               return -ENODEV;
-       }
-       if (!(val & MLX5_RXP_CSR_STATUS_INIT_DONE)) {
-               DRV_LOG(ERR, "RXP not initialised...");
-               return -EBUSY;
-       }
-       ret = mlx5_devx_regex_register_read(priv->ctx, id,
-                                           MLX5_RXP_RTRU_CSR_CTRL, &val);
-       if (ret) {
-               DRV_LOG(ERR, "CSR read failed!");
-               return -1;
-       }
-       val |= MLX5_RXP_RTRU_CSR_CTRL_GO;
-       ret = mlx5_devx_regex_register_write(priv->ctx, id,
-                                            MLX5_RXP_RTRU_CSR_CTRL, val);
-       if (ret) {
-               DRV_LOG(ERR, "Can't program rof file!");
-               return -1;
-       }
-       ret = rxp_program_rof(priv, buf, len, id);
-       if (ret) {
-               DRV_LOG(ERR, "Can't program rof file!");
-               return -1;
-       }
-       if (priv->is_bf2) {
-               ret = rxp_poll_csr_for_value
-                       (priv->ctx, &val, MLX5_RXP_RTRU_CSR_STATUS,
-                        MLX5_RXP_RTRU_CSR_STATUS_UPDATE_DONE,
-                        MLX5_RXP_RTRU_CSR_STATUS_UPDATE_DONE,
-                        MLX5_RXP_POLL_CSR_FOR_VALUE_TIMEOUT, id);
-               if (ret < 0) {
-                       DRV_LOG(ERR, "Rules update timeout: 0x%08X", val);
-                       return ret;
-               }
-               DRV_LOG(DEBUG, "Rules update took %d cycles", ret);
-       }
-       if (mlx5_devx_regex_register_read(priv->ctx, id, MLX5_RXP_RTRU_CSR_CTRL,
-                                         &val)) {
-               DRV_LOG(ERR, "CSR read failed!");
-               return -1;
-       }
-       val &= ~(MLX5_RXP_RTRU_CSR_CTRL_GO);
-       if (mlx5_devx_regex_register_write(priv->ctx, id,
-                                          MLX5_RXP_RTRU_CSR_CTRL, val)) {
-               DRV_LOG(ERR, "CSR write failed!");
-               return -1;
-       }
-       ret = mlx5_devx_regex_register_read(priv->ctx, id, MLX5_RXP_CSR_CTRL,
-                                           &val);
-       if (ret)
-               return ret;
-       val &= ~MLX5_RXP_CSR_CTRL_INIT;
-       ret = mlx5_devx_regex_register_write(priv->ctx, id, MLX5_RXP_CSR_CTRL,
-                                            val);
-       if (ret)
-               return ret;
-       rxp_init_rtru(priv, id, MLX5_RXP_RTRU_CSR_CTRL_INIT_MODE_L1_L2);
-       if (priv->is_bf2) {
-               ret = rxp_poll_csr_for_value(priv->ctx, &val,
-                                            MLX5_RXP_CSR_STATUS,
-                                            MLX5_RXP_CSR_STATUS_INIT_DONE,
-                                            MLX5_RXP_CSR_STATUS_INIT_DONE,
-                                            MLX5_RXP_CSR_STATUS_TRIAL_TIMEOUT,
-                                            id);
-               if (ret) {
-                       DRV_LOG(ERR, "Device init failed!");
-                       return ret;
-               }
-       }
-       ret = mlnx_resume_database(priv, id);
-       if (ret < 0) {
-               DRV_LOG(ERR, "Failed to resume engine!");
-               return ret;
-       }
-
-       return ret;
-
-}
-
-static int
-rxp_init_eng(struct mlx5_regex_priv *priv, uint8_t id)
-{
-       uint32_t ctrl;
-       uint32_t reg;
-       struct ibv_context *ctx = priv->ctx;
-       int ret;
-
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_CSR_CTRL, &ctrl);
-       if (ret)
-               return ret;
-       if (ctrl & MLX5_RXP_CSR_CTRL_INIT) {
-               ctrl &= ~MLX5_RXP_CSR_CTRL_INIT;
-               ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL,
-                                                    ctrl);
-               if (ret)
-                       return ret;
-       }
-       ctrl |= MLX5_RXP_CSR_CTRL_INIT;
-       ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL, ctrl);
-       if (ret)
-               return ret;
-       ctrl &= ~MLX5_RXP_CSR_CTRL_INIT;
-       ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL, ctrl);
-       if (ret)
-               return ret;
-       rte_delay_us(20000);
-       ret = rxp_poll_csr_for_value(ctx, &ctrl, MLX5_RXP_CSR_STATUS,
-                                    MLX5_RXP_CSR_STATUS_INIT_DONE,
-                                    MLX5_RXP_CSR_STATUS_INIT_DONE,
-                                    MLX5_RXP_CSR_STATUS_TRIAL_TIMEOUT, id);
-       if (ret)
-               return ret;
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_CSR_CTRL, &ctrl);
-       if (ret)
-               return ret;
-       ctrl &= ~MLX5_RXP_CSR_CTRL_INIT;
-       ret = mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_CTRL,
-                                            ctrl);
-       if (ret)
-               return ret;
-       ret = rxp_init_rtru(priv, id,
-                           MLX5_RXP_RTRU_CSR_CTRL_INIT_MODE_IM_L1_L2);
-       if (ret)
-               return ret;
-       ret = mlx5_devx_regex_register_read(ctx, id, MLX5_RXP_CSR_CAPABILITY_5,
-                                           &reg);
-       if (ret)
-               return ret;
-       DRV_LOG(DEBUG, "max matches: %d, DDOS threshold: %d", reg >> 16,
-               reg & 0xffff);
-       if ((reg >> 16) >= priv->nb_max_matches)
-               ret = mlx5_devx_regex_register_write(ctx, id,
-                                                    MLX5_RXP_CSR_MAX_MATCH,
-                                                    priv->nb_max_matches);
-       else
-               ret = mlx5_devx_regex_register_write(ctx, id,
-                                                    MLX5_RXP_CSR_MAX_MATCH,
-                                                    (reg >> 16));
-       ret |= mlx5_devx_regex_register_write(ctx, id, MLX5_RXP_CSR_MAX_PREFIX,
-                                        (reg & 0xFFFF));
-       ret |= mlx5_devx_regex_register_write(ctx, id,
-                                             MLX5_RXP_CSR_MAX_LATENCY, 0);
-       ret |= mlx5_devx_regex_register_write(ctx, id,
-                                             MLX5_RXP_CSR_MAX_PRI_THREAD, 0);
-       return ret;
-}
-
 static int
 rxp_db_setup(struct mlx5_regex_priv *priv)
 {
@@ -762,10 +89,6 @@ mlx5_regex_rules_db_import(struct rte_regexdev *dev,
                     const char *rule_db, uint32_t rule_db_len)
 {
        struct mlx5_regex_priv *priv = dev->data->dev_private;
-       struct mlx5_rxp_ctl_rules_pgm *rules = NULL;
-       uint32_t id;
-       int ret;
-       uint32_t ver;
 
        if (priv->prog_mode == MLX5_RXP_MODE_NOT_DEFINED) {
                DRV_LOG(ERR, "RXP programming mode not set!");
@@ -777,37 +100,8 @@ mlx5_regex_rules_db_import(struct rte_regexdev *dev,
        }
        if (rule_db_len == 0)
                return -EINVAL;
-       if (mlx5_devx_regex_register_read(priv->ctx, 0,
-                                         MLX5_RXP_CSR_BASE_ADDRESS, &ver)) {
-               DRV_LOG(ERR, "Failed to read Main CSRs Engine 0!");
-               return -1;
-       }
-       /* Need to ensure RXP not busy before stop! */
-       for (id = 0; id < priv->nb_engines; id++) {
-               ret = rxp_stop_engine(priv->ctx, id);
-               if (ret) {
-                       DRV_LOG(ERR, "Can't stop engine.");
-                       ret = -ENODEV;
-                       goto tidyup_error;
-               }
-               ret = program_rxp_rules(priv, rule_db, rule_db_len, id);
-               if (ret < 0) {
-                       DRV_LOG(ERR, "Failed to program rxp rules.");
-                       ret = -ENODEV;
-                       goto tidyup_error;
-               }
-               ret = rxp_start_engine(priv->ctx, id);
-               if (ret) {
-                       DRV_LOG(ERR, "Can't start engine.");
-                       ret = -ENODEV;
-                       goto tidyup_error;
-               }
-       }
-       rte_free(rules);
+
        return 0;
-tidyup_error:
-       rte_free(rules);
-       return ret;
 }
 
 int
-- 
2.25.1

Reply via email to