DPDK patches and discussions
 help / color / mirror / Atom feed
From: Francis Kelly <fkelly@nvidia.com>
To: <tmonjalon@nvidia.com>, Ori Kam <orika@nvidia.com>
Cc: <jamhunter@nvidia.com>, <aagbarih@nvidia.com>, <dev@dpdk.org>,
	Ady Agbarih <adypodoman@gmail.com>
Subject: [dpdk-dev] [PATCH 04/10] regex/mlx5: remove regexp register read/write
Date: Fri, 22 Oct 2021 15:45:54 +0000	[thread overview]
Message-ID: <20211022154600.2180938-4-fkelly@nvidia.com> (raw)
In-Reply-To: <20211022154600.2180938-1-fkelly@nvidia.com>

From: Ady Agbarih <adypodoman@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 <adypodoman@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


  parent reply	other threads:[~2021-10-22 15:48 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-22 15:45 [dpdk-dev] [PATCH 01/10] common/mlx5: update PRM definitions for regex availability Francis Kelly
2021-10-22 15:45 ` [dpdk-dev] [PATCH 02/10] regex/mlx5: add cleanup code Francis Kelly
2021-10-22 15:45 ` [dpdk-dev] [PATCH 03/10] common/mlx5: update regex DevX commands Francis Kelly
2021-10-24 13:39   ` Ori Kam
2021-10-22 15:45 ` Francis Kelly [this message]
2021-10-24 13:40   ` [dpdk-dev] [PATCH 04/10] regex/mlx5: remove regexp register read/write Ori Kam
2021-10-22 15:45 ` [dpdk-dev] [PATCH 05/10] regex/mlx5: move RXP to CrSpace Francis Kelly
2021-10-24 13:41   ` Ori Kam
2021-10-22 15:45 ` [dpdk-dev] [PATCH 06/10] regex/mlx5: remove start/stop engine API Francis Kelly
2021-10-22 15:45 ` [dpdk-dev] [PATCH 07/10] regex/mlx5: removed redundant rxp csr file Francis Kelly
2021-10-24 13:42   ` Ori Kam
2021-10-22 15:45 ` [dpdk-dev] [PATCH 08/10] regex/mlx5: fix uninitialized QP destroy Francis Kelly
2021-10-24 13:42   ` Ori Kam
2021-10-22 15:45 ` [dpdk-dev] [PATCH 09/10] regex/mlx5: prevent QP double setup Francis Kelly
2021-10-24 13:43   ` Ori Kam
2021-10-22 15:46 ` [dpdk-dev] [PATCH 10/10] doc: updated release notes and mlx5 file Francis Kelly
2021-10-24 13:43   ` Ori Kam
2021-11-03 22:13     ` Thomas Monjalon
2021-10-24 13:38 ` [dpdk-dev] [PATCH 01/10] common/mlx5: update PRM definitions for regex availability Ori Kam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211022154600.2180938-4-fkelly@nvidia.com \
    --to=fkelly@nvidia.com \
    --cc=aagbarih@nvidia.com \
    --cc=adypodoman@gmail.com \
    --cc=dev@dpdk.org \
    --cc=jamhunter@nvidia.com \
    --cc=orika@nvidia.com \
    --cc=tmonjalon@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).