Add support to skip running a dma-perf test-case.
Signed-off-by: Amit Prakash Shukla <amitpraka...@marvell.com>
Acked-by: Anoob Joseph <ano...@marvell.com>
Acked-by: Chengwen Feng <fengcheng...@huawei.com>
---
v11:
- config file formatting
app/test-dma-perf/config.ini | 2 ++
app/test-dma-perf/main.c | 48 ++++++++++++++++++++++--------------
app/test-dma-perf/main.h | 1 +
3 files changed, 32 insertions(+), 19 deletions(-)
diff --git a/app/test-dma-perf/config.ini b/app/test-dma-perf/config.ini
index b550f4b23f..bb0b1aa11a 100644
--- a/app/test-dma-perf/config.ini
+++ b/app/test-dma-perf/config.ini
@@ -30,6 +30,8 @@
; If you have already set the "-l" and "-a" parameters using EAL,
; make sure that the value of "lcore" falls within their range of values.
+; "skip" To skip a test-case set skip to 1.
+
; To specify a configuration file, use the "--config" flag followed by the
path to the file.
; To specify a result file, use the "--result" flag followed by the path to
the file.
diff --git a/app/test-dma-perf/main.c b/app/test-dma-perf/main.c
index 544784df50..e9e40e72e7 100644
--- a/app/test-dma-perf/main.c
+++ b/app/test-dma-perf/main.c
@@ -86,6 +86,19 @@ output_header(uint32_t case_id, struct test_configure
*case_cfg)
output_csv(true);
}
+static int
+open_output_csv(const char *rst_path_ptr)
+{
+ fd = fopen(rst_path_ptr, "a");
+ if (!fd) {
+ printf("Open output CSV file error.\n");
+ return 1;
+ }
+ output_csv(true);
+ fclose(fd);
+ return 0;
+}
+
static void
run_test_case(struct test_configure *case_cfg)
{
@@ -322,6 +335,7 @@ load_configs(const char *path)
const char *case_type;
const char *lcore_dma;
const char *mem_size_str, *buf_size_str, *ring_size_str,
*kick_batch_str;
+ const char *skip;
int args_nr, nb_vp;
bool is_dma;
@@ -341,6 +355,13 @@ load_configs(const char *path)
for (i = 0; i < nb_sections; i++) {
snprintf(section_name, CFG_NAME_LEN, "case%d", i + 1);
test_case = &test_cases[i];
+
+ skip = rte_cfgfile_get_entry(cfgfile, section_name, "skip");
+ if (skip && (atoi(skip) == 1)) {
+ test_case->is_skip = true;
+ continue;
+ }
+
case_type = rte_cfgfile_get_entry(cfgfile, section_name,
"type");
if (case_type == NULL) {
printf("Error: No case type in case %d, the test will
be finished here.\n",
@@ -525,31 +546,20 @@ main(int argc, char *argv[])
printf("Running cases...\n");
for (i = 0; i < case_nb; i++) {
- if (!test_cases[i].is_valid) {
- printf("Invalid test case %d.\n\n", i + 1);
- snprintf(output_str[0], MAX_OUTPUT_STR_LEN, "Invalid
case %d\n", i + 1);
-
- fd = fopen(rst_path_ptr, "a");
- if (!fd) {
- printf("Open output CSV file error.\n");
+ if (test_cases[i].is_skip) {
+ printf("Test case %d configured to be skipped.\n\n", i
+ 1);
+ snprintf(output_str[0], MAX_OUTPUT_STR_LEN, "Skip the
test-case %d\n",
+ i + 1);
+ if (open_output_csv(rst_path_ptr))
return 0;
- }
- output_csv(true);
- fclose(fd);
continue;
}
- if (test_cases[i].test_type == TEST_TYPE_NONE) {
- printf("No valid test type in test case %d.\n\n", i +
1);
+ if (!test_cases[i].is_valid) {
+ printf("Invalid test case %d.\n\n", i + 1);
snprintf(output_str[0], MAX_OUTPUT_STR_LEN, "Invalid
case %d\n", i + 1);
-
- fd = fopen(rst_path_ptr, "a");
- if (!fd) {
- printf("Open output CSV file error.\n");
+ if (open_output_csv(rst_path_ptr))
return 0;
- }
- output_csv(true);
- fclose(fd);
continue;
}
diff --git a/app/test-dma-perf/main.h b/app/test-dma-perf/main.h
index 62085e6e8f..32670151af 100644
--- a/app/test-dma-perf/main.h
+++ b/app/test-dma-perf/main.h
@@ -40,6 +40,7 @@ struct lcore_dma_map_t {
struct test_configure {
bool is_valid;
+ bool is_skip;
uint8_t test_type;
const char *test_type_str;
uint16_t src_numa_node;
--
2.25.1
