Hi Ilpo, On 5/20/24 5:30 AM, Ilpo Järvinen wrote:
initialize_mem_bw_resctrl() and set_mbm_path() contain complicated set of conditions, each yielding different file to be opened to measure memory bandwidth through resctrl FS. In practice, only two of them are used. For MBA test, ctrlgrp is always provided, and for MBM test both ctrlgrp and mongrp are set. The file used differ between MBA/MBM test, however, MBM test unnecessarily create monitor group because resctrl FS already provides monitoring interface underneath any ctrlgrp too, which is what the MBA selftest uses. Consolidate memory bandwidth file used to the one used by the MBA selftest. Remove all unused branches opening other files to simplify the code. Suggested-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> ---
fyi ... if the write_bm_pid_to_resctrl() fix in patch #16 was closer to this change it would have made this series easier to review. That fix is clearly needed here and its omission makes this patch as well as later patches #14 and #15 harder to review. No need to rework the series at this point. It is just some comment on how a simple patch ordering change can make a series easier to understand. Reviewed-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> Reinette
