Re: [PATCH v5 2/5] selftests/resctrl: Detect vendor to select memory controller sysfs name

[Reinette Chatre]

Hi Swapnil,

On 10/22/26 6:25 AM, Swapnil Sapkal wrote:

> diff --git a/tools/testing/selftests/resctrl/resctrl_val.c b/tools/testing/selftests/resctrl/resctrl_val.c
> index c9cac957726b..fdbbe2155cea 100644
> --- a/tools/testing/selftests/resctrl/resctrl_val.c
> +++ b/tools/testing/selftests/resctrl/resctrl_val.c
> @@ -10,7 +10,7 @@
>   */
>  #include "resctrl.h"
>  
> -#define UNCORE_IMC		"uncore_imc"
> +#define UNCORE_IMC		"uncore_imc_"

When considering the next patch introduces "AMD_UMC", could 
"UNCORE_IMC" be renamed to "INTEL_IMC" to better support how
this series ends up managing the architectural differences?

>  #define READ_FILE_NAME		"cas_count_read"
>  #define DYN_PMU_PATH		"/sys/bus/event_source/devices"
>  #define SCALE			0.00006103515625
Reinette