selftests/resctrl: Change a few printed messages
commit e75074781f
upstream.
Change a few printed messages to report test progress more clearly.
Add a missing "\n" at the end of one printed message.
Suggested-by: Shuah Khan <shuah@kernel.org>
Signed-off-by: Fenghua Yu <fenghua.yu@intel.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Signed-off-by: Chen Zhuo <sagazchen@tencent.com>
Signed-off-by: Xinghui Li <korantli@tencent.com>
This commit is contained in:
parent
7b68dc2468
commit
f705dcd5d3
|
@ -301,7 +301,7 @@ int show_cache_info(unsigned long sum_llc_val, int no_of_bits,
|
|||
ret = platform && abs((int)diff_percent) > max_diff_percent &&
|
||||
(cmt ? (abs(avg_diff) > max_diff) : true);
|
||||
|
||||
ksft_print_msg("%s cache miss rate within %d%%\n",
|
||||
ksft_print_msg("%s Check cache miss rate within %d%%\n",
|
||||
ret ? "Fail:" : "Pass:", max_diff_percent);
|
||||
|
||||
ksft_print_msg("Percent diff=%d\n", abs((int)diff_percent));
|
||||
|
|
|
@ -80,7 +80,7 @@ static void show_mba_info(unsigned long *bw_imc, unsigned long *bw_resc)
|
|||
avg_diff = (float)labs(avg_bw_resc - avg_bw_imc) / avg_bw_imc;
|
||||
avg_diff_per = (int)(avg_diff * 100);
|
||||
|
||||
ksft_print_msg("%s MBA: diff within %d%% for schemata %u\n",
|
||||
ksft_print_msg("%s Check MBA diff within %d%% for schemata %u\n",
|
||||
avg_diff_per > MAX_DIFF_PERCENT ?
|
||||
"Fail:" : "Pass:",
|
||||
MAX_DIFF_PERCENT,
|
||||
|
@ -93,10 +93,10 @@ static void show_mba_info(unsigned long *bw_imc, unsigned long *bw_resc)
|
|||
failed = true;
|
||||
}
|
||||
|
||||
ksft_print_msg("%s schemata change using MBA\n",
|
||||
ksft_print_msg("%s Check schemata change using MBA\n",
|
||||
failed ? "Fail:" : "Pass:");
|
||||
if (failed)
|
||||
ksft_print_msg("At least one test failed");
|
||||
ksft_print_msg("At least one test failed\n");
|
||||
}
|
||||
|
||||
static int check_results(void)
|
||||
|
|
|
@ -37,7 +37,7 @@ show_bw_info(unsigned long *bw_imc, unsigned long *bw_resc, int span)
|
|||
avg_diff_per = (int)(avg_diff * 100);
|
||||
|
||||
ret = avg_diff_per > MAX_DIFF_PERCENT;
|
||||
ksft_print_msg("%s MBM: diff within %d%%\n",
|
||||
ksft_print_msg("%s Check MBM diff within %d%%\n",
|
||||
ret ? "Fail:" : "Pass:", MAX_DIFF_PERCENT);
|
||||
ksft_print_msg("avg_diff_per: %d%%\n", avg_diff_per);
|
||||
ksft_print_msg("Span (MB): %d\n", span);
|
||||
|
|
|
@ -570,14 +570,14 @@ bool check_resctrlfs_support(void)
|
|||
|
||||
fclose(inf);
|
||||
|
||||
ksft_print_msg("%s kernel supports resctrl filesystem\n",
|
||||
ksft_print_msg("%s Check kernel supports resctrl filesystem\n",
|
||||
ret ? "Pass:" : "Fail:");
|
||||
|
||||
if (!ret)
|
||||
return ret;
|
||||
|
||||
dp = opendir(RESCTRL_PATH);
|
||||
ksft_print_msg("%s resctrl mountpoint \"%s\" exists\n",
|
||||
ksft_print_msg("%s Check resctrl mountpoint \"%s\" exists\n",
|
||||
dp ? "Pass:" : "Fail:", RESCTRL_PATH);
|
||||
if (dp)
|
||||
closedir(dp);
|
||||
|
|
Loading…
Reference in New Issue