From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2D004A052B; Wed, 29 Jul 2020 20:10:40 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id DCEA837B7; Wed, 29 Jul 2020 20:10:38 +0200 (CEST) Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by dpdk.org (Postfix) with ESMTP id F135710A3 for ; Wed, 29 Jul 2020 20:10:36 +0200 (CEST) Received: from Internal Mail-Server by MTLPINE1 (envelope-from orika@mellanox.com) with SMTP; 29 Jul 2020 21:10:34 +0300 Received: from pegasus04.mtr.labs.mlnx. (pegasus04.mtr.labs.mlnx [10.210.16.126]) by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id 06TIAYEp031107; Wed, 29 Jul 2020 21:10:34 +0300 From: Ori Kam To: jerinj@marvell.com, xiang.w.wang@intel.com, matan@mellanox.com, viacheslavo@mellanox.com, John McNamara , Marko Kovacevic Cc: guyk@marvell.com, dev@dpdk.org, pbhagavatula@marvell.com, shahafs@mellanox.com, hemant.agrawal@nxp.com, opher@mellanox.com, alexr@mellanox.com, dovrat@marvell.com, pkapoor@marvell.com, nipun.gupta@nxp.com, bruce.richardson@intel.com, yang.a.hong@intel.com, harry.chang@intel.com, gu.jian1@zte.com.cn, shanjiangh@chinatelecom.cn, zhangy.yun@chinatelecom.cn, lixingfu@huachentel.com, wushuai@inspur.com, yuyingxia@yxlink.com, fanchenggang@sunyainfo.com, davidfgao@tencent.com, liuzhong1@chinaunicom.cn, zhaoyong11@huawei.com, oc@yunify.com, jim@netgate.com, hongjun.ni@intel.com, deri@ntop.org, fc@napatech.com, arthur.su@lionic.com, thomas@monjalon.net, orika@mellanox.com, rasland@mellanox.com, Yuval Avnery Date: Wed, 29 Jul 2020 18:09:57 +0000 Message-Id: <1596046198-134903-1-git-send-email-orika@mellanox.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1595793496-73205-1-git-send-email-orika@mellanox.com> References: <1595793496-73205-1-git-send-email-orika@mellanox.com> Subject: [dpdk-dev] [PATCH v4] app/test-regex: add RegEx test application X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" From: Yuval Avnery Following the new RegEx class. There is a need to create a dedicated test application in order to validate this class and PMD. Unlike net device this application loads data from a file. This commit introduces the new RegEx test app. The basic app flow: 1. Configure the RegEx device to use one queue, and set the rule database, using precompiled file. 2. Allocate mbufs based on the requested number of jobs, each job will i get one mbuf. 3. Enqueue as much as possible jobs. 4. Dequeue jobs. 5. if the number of dequeue jobs < requested number of jobs job to step Signed-off-by: Ori Kam Signed-off-by: Yuval Avnery --- v4: * Address ML comments. v3: * Fix possible issue in case of error. v2: * Remove rule file and data generation data. * Address ML comments. --- app/Makefile | 1 + app/meson.build | 1 + app/test-regex/Makefile | 17 ++ app/test-regex/main.c | 436 +++++++++++++++++++++++++++++++++++++++++ app/test-regex/meson.build | 5 + doc/guides/tools/index.rst | 1 + doc/guides/tools/testregex.rst | 73 +++++++ 7 files changed, 534 insertions(+) create mode 100644 app/test-regex/Makefile create mode 100644 app/test-regex/main.c create mode 100644 app/test-regex/meson.build create mode 100644 doc/guides/tools/testregex.rst diff --git a/app/Makefile b/app/Makefile index 0392a7d..453c4fe 100644 --- a/app/Makefile +++ b/app/Makefile @@ -13,6 +13,7 @@ DIRS-$(CONFIG_RTE_LIBRTE_FIB) += test-fib DIRS-$(CONFIG_RTE_TEST_FLOW_PERF) += test-flow-perf DIRS-$(CONFIG_RTE_LIBRTE_PIPELINE) += test-pipeline DIRS-$(CONFIG_RTE_LIBRTE_IPSEC) += test-sad +DIRS-$(CONFIG_RTE_LIBRTE_REGEXDEV) += test-regex ifeq ($(CONFIG_RTE_LIBRTE_BBDEV),y) DIRS-$(CONFIG_RTE_TEST_BBDEV) += test-bbdev diff --git a/app/meson.build b/app/meson.build index 585b908..eb74f21 100644 --- a/app/meson.build +++ b/app/meson.build @@ -18,6 +18,7 @@ apps = [ 'test-flow-perf', 'test-pipeline', 'test-pmd', + 'test-regex', 'test-sad'] # for BSD only diff --git a/app/test-regex/Makefile b/app/test-regex/Makefile new file mode 100644 index 0000000..a059a8c --- /dev/null +++ b/app/test-regex/Makefile @@ -0,0 +1,17 @@ +# SPDX-License-Identifier: BSD-3-Clause +# Copyright 2020 Mellanox Technologies, Ltd + +include $(RTE_SDK)/mk/rte.vars.mk + +APP = testregex + +CFLAGS += -O3 +CFLAGS += $(WERROR_FLAGS) +CFLAGS += -DALLOW_EXPERIMENTAL_API + +# +# all source are stored in SRCS-y +# +SRCS-y := main.c + +include $(RTE_SDK)/mk/rte.app.mk diff --git a/app/test-regex/main.c b/app/test-regex/main.c new file mode 100644 index 0000000..9cdf1da --- /dev/null +++ b/app/test-regex/main.c @@ -0,0 +1,436 @@ +/* SPDX-License-Identifier: BSD-3-Clause + * Copyright 2020 Mellanox Technologies, Ltd + */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include +#include +#include + +#define MAX_FILE_NAME 255 +#define MBUF_CACHE_SIZE 256 +#define MBUF_SIZE (1 << 8) +#define START_BURST_SIZE 32u + +enum app_args { + ARG_HELP, + ARG_RULES_FILE_NAME, + ARG_DATA_FILE_NAME, + ARG_NUM_OF_JOBS, + ARG_PERF_MODE, + ARG_NUM_OF_ITERATIONS, + +}; + +static void +usage(const char *prog_name) +{ + printf("%s [EAL options] --\n" + " --rules NAME: precompiled rules file\n" + " --data NAME: data file to use\n" + " --nb_jobs: number of jobs to use\n" + " --perf N: only outputs the performance data\n" + " --nb_iter N: number of iteration to run\n", + prog_name); +} + +static void +args_parse(int argc, char **argv, char *rules_file, char *data_file, + uint32_t *nb_jobs, bool *perf_mode, uint32_t *nb_iterations) +{ + char **argvopt; + int opt; + int opt_idx; + size_t len; + static struct option lgopts[] = { + { "help", 0, 0, ARG_HELP}, + { "rules", 1, 0, ARG_RULES_FILE_NAME}, + /* Rules database file to load. */ + { "data", 1, 0, ARG_DATA_FILE_NAME}, + /* Data file to load. */ + { "nb_jobs", 1, 0, ARG_NUM_OF_JOBS}, + /* Number of jobs to create. */ + { "perf", 0, 0, ARG_PERF_MODE}, + /* Perf test only */ + { "nb_iter", 1, 0, ARG_NUM_OF_ITERATIONS} + /* Number of iterations to run with perf test */ + }; + + argvopt = argv; + while ((opt = getopt_long(argc, argvopt, "", + lgopts, &opt_idx)) != EOF) { + switch (opt) { + case ARG_RULES_FILE_NAME: + len = strnlen(optarg, MAX_FILE_NAME - 1); + if (len == MAX_FILE_NAME) + rte_exit(EXIT_FAILURE, + "Rule file name to long max %d\n", + MAX_FILE_NAME - 1); + strncpy(rules_file, optarg, MAX_FILE_NAME - 1); + break; + case ARG_DATA_FILE_NAME: + len = strnlen(optarg, MAX_FILE_NAME - 1); + if (len == MAX_FILE_NAME) + rte_exit(EXIT_FAILURE, + "Data file name to long max %d\n", + MAX_FILE_NAME - 1); + strncpy(data_file, optarg, MAX_FILE_NAME - 1); + break; + case ARG_NUM_OF_JOBS: + *nb_jobs = atoi(optarg); + break; + case ARG_PERF_MODE: + *perf_mode = true; + break; + case ARG_NUM_OF_ITERATIONS: + *nb_iterations = atoi(optarg); + break; + case ARG_HELP: + usage("RegEx test app"); + break; + default: + fprintf(stderr, "Invalid option: %s\n", argv[optind]); + usage("RegEx test app"); + rte_exit(EXIT_FAILURE, "Invalid option\n"); + break; + } + } + + if (!perf_mode) + *nb_iterations = 1; +} + +static long +read_file(char *file, char **buf) +{ + FILE *fp; + long buf_len = 0; + size_t read_len; + int res = 0; + + fp = fopen(file, "r"); + if (!fp) + return -EIO; + if (fseek(fp, 0L, SEEK_END) == 0) { + buf_len = ftell(fp); + if (buf_len == -1) { + res = EIO; + goto error; + } + *buf = rte_malloc(NULL, sizeof(char) * (buf_len + 1), 4096); + if (!*buf) { + res = ENOMEM; + goto error; + } + if (fseek(fp, 0L, SEEK_SET) != 0) { + res = EIO; + goto error; + } + read_len = fread(*buf, sizeof(char), buf_len, fp); + if (read_len != (unsigned long)buf_len) { + res = EIO; + goto error; + } + } + fclose(fp); + return buf_len; +error: + printf("Error, can't open file %s\n, err = %d", file, res); + if (fp) + fclose(fp); + if (*buf) + rte_free(*buf); + return -res; +} + +static int +init_port(struct rte_mempool **mbuf_mp, uint32_t nb_jobs, + uint16_t *nb_max_payload, char *rules_file, uint8_t *nb_max_matches) +{ + uint16_t id; + uint16_t num_devs; + char *rules = NULL; + long rules_len; + struct rte_regexdev_info info; + struct rte_regexdev_config dev_conf = { + .nb_queue_pairs = 1, + .nb_groups = 1, + }; + struct rte_regexdev_qp_conf qp_conf = { + .nb_desc = 1024, + .qp_conf_flags = RTE_REGEX_QUEUE_PAIR_CFG_OOS_F, + }; + int res = 0; + + num_devs = rte_regexdev_count(); + if (num_devs == 0) { + printf("Error, no devices detected.\n"); + return -EINVAL; + } + + *mbuf_mp = rte_pktmbuf_pool_create("mbuf_pool", nb_jobs, 0, + 0, MBUF_SIZE, rte_socket_id()); + if (*mbuf_mp == NULL) { + printf("Error, can't create memory pool\n"); + res = -ENOMEM; + goto error; + } + + rules_len = read_file(rules_file, &rules); + if (rules_len < 0) { + printf("Error, can't read rules files.\n"); + res = -EIO; + goto error; + } + + for (id = 0; id < num_devs; id++) { + res = rte_regexdev_info_get(id, &info); + if (res != 0) { + printf("Error, can't get device info.\n"); + goto error; + } + printf(":: initializing dev: %d\n", id); + *nb_max_matches = info.max_matches; + *nb_max_payload = info.max_payload_size; + if (info.regexdev_capa & RTE_REGEXDEV_SUPP_MATCH_AS_END_F) + dev_conf.dev_cfg_flags |= RTE_REGEXDEV_CFG_MATCH_AS_END_F; + dev_conf.nb_max_matches = info.max_matches; + dev_conf.nb_rules_per_group = info.max_rules_per_group; + dev_conf.rule_db_len = rules_len; + dev_conf.rule_db = rules; + res = rte_regexdev_configure(id, &dev_conf); + if (res < 0) { + printf("Error, can't configure device %d.\n", id); + goto error; + } + res = rte_regexdev_queue_pair_setup(id, 0, &qp_conf); + if (res < 0) { + printf("Error, can't setup queue pair for device %d.\n", + id); + goto error; + } + printf(":: initializing device: %d done\n", id); + } + rte_free(rules); + return 0; +error: + if (rules) + rte_free(rules); + if (*mbuf_mp) + rte_mempool_free(*mbuf_mp); + return res; + +} + +static void +extbuf_free_cb(void *addr __rte_unused, void *fcb_opaque __rte_unused) +{ + +} + +static int +run_regex(struct rte_mempool *mbuf_mp, uint32_t nb_jobs, + uint16_t nb_max_payload, bool perf_mode, uint32_t nb_iterations, + char *data_file, uint8_t nb_max_matches) +{ + char *buf = NULL; + long buf_len; + long job_len; + uint32_t actual_jobs = 0; + uint32_t i; + struct rte_regex_ops **ops; + uint16_t dev_id = 0; + uint16_t qp_id = 0; + uint8_t nb_matches; + struct rte_regexdev_match *match; + long pos = 0; + unsigned long d_ind = 0; + struct rte_mbuf_ext_shared_info shinfo; + uint32_t total_enqueue = 0; + uint32_t total_dequeue = 0; + uint32_t total_matches = 0; + int res = 0; + time_t start; + time_t end; + double time; + + shinfo.free_cb = extbuf_free_cb; + + ops = rte_malloc(NULL, sizeof(*ops) * nb_jobs, 0); + if (!ops) { + printf("Error, can't allocate memory for ops.\n"); + return -ENOMEM; + } + + /* Allocate the jobs and assign each job with an mbuf. */ + for (i = 0; i < nb_jobs; i++) { + ops[i] = rte_malloc(NULL, sizeof(*ops[0]) + nb_max_matches * + sizeof(struct rte_regexdev_match), 0); + if (!ops[i]) { + printf("Error, can't allocate memory for op.\n"); + res = -ENOMEM; + goto end; + } + ops[i]->mbuf = rte_pktmbuf_alloc(mbuf_mp); + if (!ops[i]->mbuf) { + printf("Error, can't attach mbuf.\n"); + res = -ENOMEM; + goto end; + } + } + + buf_len = read_file(data_file, &buf); + if (buf_len <= 0) { + printf("Error, can't read file, or file is empty.\n"); + res = -EXIT_FAILURE; + goto end; + } + + job_len = buf_len / nb_jobs; + if (job_len == 0) { + printf("Error, To many jobs, for the given input.\n"); + res = -EXIT_FAILURE; + goto end; + } + + if (job_len > nb_max_payload) { + printf("Error, not enough jobs to cover input.\n"); + res = -EXIT_FAILURE; + goto end; + } + + /* Assign each mbuf with the data to handle. */ + for (i = 0; (pos < buf_len) && (i < nb_jobs) ; i++) { + long act_job_len = RTE_MIN(job_len, buf_len - pos); + rte_pktmbuf_attach_extbuf(ops[i]->mbuf, &buf[pos], 0, + act_job_len, &shinfo); + ops[i]->mbuf->data_len = job_len; + ops[i]->mbuf->pkt_len = act_job_len; + ops[i]->user_id = i; + ops[i]->group_id0 = 1; + pos += act_job_len; + actual_jobs++; + } + + start = clock(); + for (i = 0; i < nb_iterations; i++) { + total_enqueue = 0; + total_dequeue = 0; + while (total_dequeue < actual_jobs) { + struct rte_regex_ops **cur_ops_to_enqueue = ops + + total_enqueue; + struct rte_regex_ops **cur_ops_to_dequeue = ops + + total_dequeue; + + if (actual_jobs - total_enqueue) + total_enqueue += rte_regexdev_enqueue_burst + (dev_id, qp_id, cur_ops_to_enqueue, + actual_jobs - total_enqueue); + + total_dequeue += rte_regexdev_dequeue_burst + (dev_id, qp_id, cur_ops_to_dequeue, + total_enqueue - total_dequeue); + } + } + end = clock(); + time = ((double)end - start) / CLOCKS_PER_SEC; + printf("Job len = %ld Bytes\n", job_len); + printf("Time = %lf sec\n", time); + printf("Perf = %lf Gbps\n", + (((double)actual_jobs * job_len * nb_iterations * 8) / time) / + 1000000000.0); + + if (!perf_mode) { + /* Log results per job. */ + for (d_ind = 0; d_ind < total_dequeue; d_ind++) { + nb_matches = ops[d_ind % actual_jobs]->nb_matches; + printf("Job id %"PRIu64" number of matches = %d\n", + ops[d_ind]->user_id, nb_matches); + total_matches += nb_matches; + match = ops[d_ind % actual_jobs]->matches; + for (i = 0; i < nb_matches; i++) { + printf("match %d, rule = %d, start = %d,len = %d\n", + i, match->rule_id, match->start_offset, + match->len); + match++; + } + } + printf("Total matches = %d\n", total_matches); + printf("All Matches:\n"); + + /* Log absolute results. */ + for (d_ind = 0; d_ind < total_dequeue; d_ind++) { + nb_matches = ops[d_ind % actual_jobs]->nb_matches; + total_matches += nb_matches; + match = ops[d_ind % actual_jobs]->matches; + for (i = 0; i < nb_matches; i++) { + printf("start = %ld, len = %d, rule = %d\n", + match->start_offset + d_ind * job_len, + match->len, match->rule_id); + match++; + } + } + } +end: + for (i = 0; i < actual_jobs; i++) { + if (ops[i]) { + if (ops[i]->mbuf) + rte_pktmbuf_free(ops[i]->mbuf); + rte_free(ops[i]); + } + } + rte_free(ops); + if (buf) + rte_free(buf); + return res; +} + +int +main(int argc, char **argv) +{ + char rules_file[MAX_FILE_NAME]; + char data_file[MAX_FILE_NAME]; + struct rte_mempool *mbuf_mp = NULL; + uint32_t nb_jobs = 0; + uint16_t nb_max_payload = 0; + bool perf_mode = 0; + uint32_t nb_iterations = 0; + uint8_t nb_max_matches = 0; + int ret; + + ret = rte_eal_init(argc, argv); + if (ret < 0) + rte_exit(EXIT_FAILURE, "EAL init failed\n"); + argc -= ret; + argv += ret; + if (argc > 1) + args_parse(argc, argv, rules_file, data_file, &nb_jobs, + &perf_mode, &nb_iterations); + + ret = init_port(&mbuf_mp, nb_jobs, &nb_max_payload, rules_file, + &nb_max_matches); + if (ret < 0) + rte_exit(EXIT_FAILURE, "init port failed\n"); + ret = run_regex(mbuf_mp, nb_jobs, nb_max_payload, perf_mode, + nb_iterations, data_file, nb_max_matches); + if (ret < 0) { + rte_mempool_free(mbuf_mp); + rte_exit(EXIT_FAILURE, "RegEx function failed\n"); + } + rte_mempool_free(mbuf_mp); + return 0; +} diff --git a/app/test-regex/meson.build b/app/test-regex/meson.build new file mode 100644 index 0000000..472677f --- /dev/null +++ b/app/test-regex/meson.build @@ -0,0 +1,5 @@ +# SPDX-License-Identifier: BSD-3-Clause +# Copyright 2020 Mellanox Technologies, Ltd + +sources = files('main.c') +deps = ['regexdev'] diff --git a/doc/guides/tools/index.rst b/doc/guides/tools/index.rst index 4840cf47..c721943 100644 --- a/doc/guides/tools/index.rst +++ b/doc/guides/tools/index.rst @@ -17,3 +17,4 @@ DPDK Tools User Guides cryptoperf comp_perf testeventdev + testregex diff --git a/doc/guides/tools/testregex.rst b/doc/guides/tools/testregex.rst new file mode 100644 index 0000000..347d607 --- /dev/null +++ b/doc/guides/tools/testregex.rst @@ -0,0 +1,73 @@ +.. SPDX-License-Identifier: BSD-3-Clause + Copyright 2020 Mellanox Technologies, Ltd + +dpdk-test-regex Tool +==================== + +The ``dpdk-test-regex`` tool is a Data Plane Development Kit (DPDK) +application that allows functional testing and performance measurement for +the RegEx PMDs. +The test supports only one core and one PMD. +It is based on precompiled rule file, and an input file, both of them can +be selected using command-line options. + +In general case, each PMD has its own rule file. + +The test outputs the following data: + +* Performance, in gigabit per second. + +* Matching results (rule id, position, length), for each job. + +* Matching results in absolute location (rule id, position , length), + relative to the start of the input data. + + +Limitations +~~~~~~~~~~~ + +* Only one queue is supported. + +* Supports only precompiled rules. + + +Application Options +~~~~~~~~~~~~~~~~~~~ + +``--rules NAME`` + precompiled rule file + +``--data NAME`` + data file to use + +``--nb_jobs N`` + number of jobs to use + +``--perf N`` + only outputs the performance data + +``--nb_iter N`` + number of iteration to run + +``--help`` + prints this help + + +Running the Tool +---------------- + +**Step 1: Compile a rule file** + +In order for the RegEx to work it must have a precompiled rule file. +to generate this file there is a need to use a RegEx compiler that matches the +RegEx PMD. + +**Step 2: Generate a data file** + +The data file, will be used as a source data for the RegEx to work on. + +**Step 3: Run the tool** + +The tool has a number of command line options. Here is the sample command line:: + + ./dpdk-test-regex -w 83:00.0 -- --rules rule_file.rof2 --data data_file.txt --job 100 -- 1.8.3.1