Re: [PATCH v4 8/8] memcg: use proper type for mod_memcg_state

Roman Gushchin
2024-05-01T23:24:02Z

On Wed, May 01, 2024 at 10:26:17AM -0700, Shakeel Butt wrote:
> The memcg stats update functions can take arbitrary integer but the
> only input which make sense is enum memcg_stat_item and we don't
> want these functions to be called with arbitrary integer, so replace
> the parameter type with enum memcg_stat_item and compiler will be able
> to warn if memcg stat update functions are called with incorrect index
> value.
>
> Signed-off-by: Shakeel Butt <shakeel.butt@linux.dev>
> Reviewed-by: T.J. Mercier <tjmercier@google.com>

Reviewed-by: Roman Gushchin <roman.gushchin@linux.dev>

Thanks!