Re: [PATCH v2 6/9] selftests/resctrl: Change initialize_llc_perf() return type to void

From: Reinette Chatre
Date: Wed Mar 15 2023 - 20:05:13 EST


Hi Ilpo,

On 2/15/2023 5:06 AM, Ilpo Järvinen wrote:
> initialize_llc_perf() unconditionally returns 0.
>
> initialize_llc_perf() performs only memory initialization, none of
> which can fail.
>
> Change the return type from int to void to accurately reflect that its
> return value doesn't need to be checked. Remove the error checking from
> the only callsite.
>
> Co-developed-by: Fenghua Yu <fenghua.yu@xxxxxxxxx>
> Signed-off-by: Fenghua Yu <fenghua.yu@xxxxxxxxx>
> Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
> ---

Thank you

Reviewed-by: Reinette Chatre <reinette.chatre@xxxxxxxxx>

Reinette