Commit 149ff729 authored by Ilpo Järvinen's avatar Ilpo Järvinen Committed by Shuah Khan
Browse files

selftests/resctrl: Remove ben_count variable 



ben_count is only used to write the terminator for the list. It is
enough to use i from the loop so no need for another variable.

Remove ben_count variable as it is not needed.

Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Tested-by: default avatarShaopeng Tan <tan.shaopeng@jp.fujitsu.com>
Reviewed-by: default avatarReinette Chatre <reinette.chatre@intel.com>
Reviewed-by: default avatarShaopeng Tan <tan.shaopeng@jp.fujitsu.com>
Reviewed-by: default avatar"Wieczor-Retman, Maciej" <maciej.wieczor-retman@intel.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent e33cb570

tools/testing/selftests/resctrl/resctrl_tests.c

+2 −3
Original line number Diff line number Diff line
@@ -172,7 +172,7 @@ int main(int argc, char **argv) 
	bool has_ben = false, mbm_test = true, mba_test = true, cmt_test = true;

	int c, cpu_no = 1, argc_new = argc, i, no_of_bits = 0;

	const char *benchmark_cmd[BENCHMARK_ARGS];

	int ben_ind, ben_count, tests = 0;

	int ben_ind, tests = 0;

	char *span_str = NULL;

	bool cat_test = true;

	int ret;
@@ -180,7 +180,6 @@ int main(int argc, char **argv) 
	for (i = 0; i < argc; i++) {

		if (strcmp(argv[i], "-b") == 0) {

			ben_ind = i + 1;

			ben_count = argc - ben_ind;

			argc_new = ben_ind - 1;

			has_ben = true;

			break;
@@ -265,7 +264,7 @@ int main(int argc, char **argv) 
		/* Extract benchmark command from command line. */

		for (i = 0; i < argc - ben_ind; i++)

			benchmark_cmd[i] = argv[i + ben_ind];

		benchmark_cmd[ben_count] = NULL;

		benchmark_cmd[i] = NULL;

	} else {

		/* If no benchmark is given by "-b" argument, use fill_buf. */

		benchmark_cmd[0] = "fill_buf";