On Thu, May 01, 2025 at 09:04:42PM -0400, Waiman Long longman@redhat.com wrote:
Modify the test_memcontrol.c to ignore low event in the 3rd child cgroup with memory_recursiveprot on.
The 4th child cgroup has no memory usage and so has an effective low of 0. It has no low event count because the mem_cgroup_below_low() check in shrink_node_memcgs() is skipped as mem_cgroup_below_min() returns true. If we ever change mem_cgroup_below_min() in such a way that it no longer skips the no usage case, we will have to add code to explicitly skip it.
With this patch applied, the test_memcg_low sub-test finishes successfully without failure in most cases. Though both test_memcg_low and test_memcg_min sub-tests may still fail occasionally if the memory.current values fall outside of the expected ranges.
Suggested-by: Michal Koutný mkoutny@suse.com Signed-off-by: Waiman Long longman@redhat.com
Acked-by: Michal Koutný mkoutny@suse.com
(Thank you. Not sure if this can be both with Suggested-by, so either of them alone is fine by me.)