Hi Ilpo,

On 5/20/24 5:30 AM, Ilpo Järvinen wrote:
Nothing during MBA test uses mongrp even if it has been defined ever
since the introduction of the MBA test in the commit 01fee6b4d1f9
("selftests/resctrl: Add MBA test").

Remove the mongrp from MBA test.

Signed-off-by: Ilpo Järvinen <[email protected]>
---
  tools/testing/selftests/resctrl/mba_test.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/tools/testing/selftests/resctrl/mba_test.c 
b/tools/testing/selftests/resctrl/mba_test.c
index 9c9a4f22e529..5e0b1e794295 100644
--- a/tools/testing/selftests/resctrl/mba_test.c
+++ b/tools/testing/selftests/resctrl/mba_test.c
@@ -166,7 +166,6 @@ static int mba_run_test(const struct resctrl_test *test, 
const struct user_param
        struct resctrl_val_param param = {
                .resctrl_val    = MBA_STR,
                .ctrlgrp        = "c1",
-               .mongrp         = "m1",
                .filename       = RESULT_FILE_NAME,
                .init           = mba_init,
                .setup          = mba_setup,

This may explain the unexpected checks that are removed in final patch?

Reviewed-by: Reinette Chatre <[email protected]>

Reinette

Reply via email to