[LTP] [PATCH] memcontrol03: Account for process size in cgroup allocation

[Martin Doucha]

Hi!

On 20. 05. 25 19:57, ALOK TIWARI wrote:
> Error log without this commit:
> ===============================================================
> I was seeing error on 64K image aarch64 (failure can occur randomly):
> <snip>
> memcontrol03.c:212: TFAIL: Expect: (A/B/C memory.current=21168128) ~= 
> 34603008
> <snip>
> 
> LTP test PASSED with commit:
> ===============================================================
> here my observation for arrch64 64K page Image with this commit:
> 
> <snip>
> memcontrol03.c:224: TPASS: Expect: (A/B/C memory.current=21299200) ~= 
> 34603008
> <snip>

This failure is unrelated to the patch and happens randomly.

> Is there any case where this LTP test depends on the upstream commit 
> 1bc542c6a0d ('mm/vmscan: wake up flushers conditionally to avoid cgroup 
> OOM')?

The test predates the kernel commit by several years. And while the 
mm/vmscan change might help prevent the OOM this patch is trying to fix, 
the OOM still looks valid to me and should be dealt with in the test code.

-- 
Martin Doucha   mdoucha@suse.cz
SW Quality Engineer
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic