diff options
author | Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> | 2023-02-15 15:05:58 +0200 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2023-04-10 12:20:35 -0600 |
commit | c90b3b588e369c20087699316259fa5ebbb16f2d (patch) | |
tree | c6746556ab5f4eba378565f77c8c852c3750b527 | |
parent | 22a8be280383812235131dda18a8212a59fadd2d (diff) | |
download | linux-stable-c90b3b588e369c20087699316259fa5ebbb16f2d.tar.gz linux-stable-c90b3b588e369c20087699316259fa5ebbb16f2d.tar.bz2 linux-stable-c90b3b588e369c20087699316259fa5ebbb16f2d.zip |
selftests/resctrl: Move ->setup() call outside of test specific branches
resctrl_val() function is called only by MBM, MBA, and CMT tests which
means the else branch is never used.
Both test branches call param->setup().
Remove the unused else branch and place the ->setup() call outside of
the test specific branches reducing code duplication.
Co-developed-by: Fenghua Yu <fenghua.yu@intel.com>
Signed-off-by: Fenghua Yu <fenghua.yu@intel.com>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
-rw-r--r-- | tools/testing/selftests/resctrl/resctrl_val.c | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/tools/testing/selftests/resctrl/resctrl_val.c b/tools/testing/selftests/resctrl/resctrl_val.c index b32b96356ec7..787546a52849 100644 --- a/tools/testing/selftests/resctrl/resctrl_val.c +++ b/tools/testing/selftests/resctrl/resctrl_val.c @@ -734,29 +734,22 @@ int resctrl_val(char **benchmark_cmd, struct resctrl_val_param *param) /* Test runs until the callback setup() tells the test to stop. */ while (1) { + ret = param->setup(1, param); + if (ret) { + ret = 0; + break; + } + if (!strncmp(resctrl_val, MBM_STR, sizeof(MBM_STR)) || !strncmp(resctrl_val, MBA_STR, sizeof(MBA_STR))) { - ret = param->setup(1, param); - if (ret) { - ret = 0; - break; - } - ret = measure_vals(param, &bw_resc_start); if (ret) break; } else if (!strncmp(resctrl_val, CMT_STR, sizeof(CMT_STR))) { - ret = param->setup(1, param); - if (ret) { - ret = 0; - break; - } sleep(1); ret = measure_cache_vals(param, bm_pid); if (ret) break; - } else { - break; } } |