[Devel] [PATCH hci-8.1 1/2] ms/selftests: cgroup: skip test_memcg_sock under cgroup.memory=nosocket

[no-reply at sashiko.virtuozzo.com]

> With cgroup.memory=nosocket the memory controller does not charge
> socket memory, so the "sock " entry of memory.stat stays 0 and
> memory.current moves only by the kernel objects accept() allocates.
> tcp_client() then passes on 0 == 0 without having checked any
> accounting, or fails if those objects took a fresh charge batch.
> 
> Look for nosocket among the cgroup.memory= options on the kernel command
> line and skip the test there.
> 
> Assisted-by: LLM
> Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>
> Signed-off-by: Tejun Heo <tj at kernel.org>
> (cherry picked from commit 216a983640bce942e25c62e9d2f901dfba888cca)
> 
> Conflict: cgroup_util.h here still calls the buffer size PAGE_SIZE, the
> rename to BUF_SIZE in 6e9f5c2eecd1 ("selftests/cgroup: rename PAGE_SIZE
> to BUF_SIZE in cgroup_util") is not in the tree, so
> memcg_sockets_disabled() sizes its buffer with PAGE_SIZE. The value is
> the same.
> 
> https://virtuozzo.atlassian.net/browse/VSTOR-149854
> 
> Feature: fix selftests
> Signed-off-by: Konstantin Khorenko <khorenko at virtuozzo.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.vzint.dev/#/patchset/20261019-b4-prep-khorenko-vstor-149854-selftests-test_memcg_sock-v1-0-efbfa45dc967@virtuozzo.com?part=1