Re: [PATCH v3 5/5] selftests: memcg: Factor out common parts of memory.{low,min} tests

From: Roman Gushchin
Date: Tue May 24 2022 - 22:28:03 EST


On Tue, May 24, 2022 at 06:29:55PM +0200, Michal Koutny wrote:
> The memory protection test setup and runtime is almost equal for
> memory.low and memory.min cases.
> It makes modification of the common parts prone to mistakes, since the
> protections are similar not only in setup but also in principle, factor
> the common part out.
>
> Past exceptions between the tests:
> - missing memory.min is fine (kept),
> - test_memcg_low protected orphaned pagecache (adapted like
> test_memcg_min and we keep the processes of protected memory running).
>
> The evaluation in two tests is different (OOM of allocator vs low events
> of protégés), this is kept different.
>
> Signed-off-by: Michal Koutný <mkoutny@xxxxxxxx>

Acked-by: Roman Gushchin <roman.gushchin@xxxxxxxxx>