From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f169.google.com (mail-qk1-f169.google.com [209.85.222.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id C4D843D4133 for ; Sat, 28 Feb 2026 22:19:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772317156; cv=none; b=c1YRDKSO84qmxhNWswl5HR3yDGATPsZN/BpEj9XujwpiWPzwIzsd65PPuBBwf9bAyMUHPEkdPgKgE3oJgKzToakYfdbp1WBX3jnInYMyX1jqzcAiK+7Hw8LHnYXuDPTfJWJ1w3zzsR2kekxJAlGDuc8aqMIK1twIfvBT3k6ptEY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772317156; c=relaxed/simple; bh=yDsqQH5BbiO+PW6zzUX2oUUyXOo83ndu0xMlDz0uCHM=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=jnpcy+B0R/r32c1DKWEHodVISkiGSbF0FgGrRbMBKBvB0X0XJVdppnU5zWlV1FeUZaAbyzhCZ2Q7P8tURw5wdZMlDpbTPBOaogeDQrUHzIiqFXzZHn1qt6rO3M5AfN7+oSEQC6f8tGfuueCsj7TLMw0RC9RlD+DFkHSwVfwJBzw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=etsalapatis.com; spf=pass smtp.mailfrom=etsalapatis.com; dkim=pass (2048-bit key) header.d=etsalapatis-com.20230601.gappssmtp.com header.i=@etsalapatis-com.20230601.gappssmtp.com header.b=gNjVDkLa; arc=none smtp.client-ip=209.85.222.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=etsalapatis.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=etsalapatis.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=etsalapatis-com.20230601.gappssmtp.com header.i=@etsalapatis-com.20230601.gappssmtp.com header.b="gNjVDkLa" Received: by mail-qk1-f169.google.com with SMTP id af79cd13be357-8cb3bae8d3eso299466985a.1 for ; Sat, 28 Feb 2026 14:19:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=etsalapatis-com.20230601.gappssmtp.com; s=20230601; t=1772317154; x=1772921954; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=JgUvB7vchkbEExheWs0hP4qVVuHRRrf4mV32RUQg+58=; b=gNjVDkLaskYBiaTKcWpjprc4lBdhxWBwQZzPik/wDvijrAOW+BVffXTsgGDbT9wW+N r867rQOe4152DbAPl+n5ZP22Yz33YpunpyJSxDY8X65clixVQfv9yzi97Vf6iXP8WSFj PUJvxLhlPM4oW/gNSwFm9BR/X5POrh/NETBbadh9ib3tF02ogq+AJ1nYQ5wawCsnP9vS RwsGQxKqkMl/6Q2Q9E2GIQbY0dyGKYF/8jZ3e8lqK3jRA24DFJxHrnBLHfbramnUB+uG jZbZUZxkGgdmyT+p4nCr5ujF7nLjp3mVtFydQLzuRmzcbytAXJUaFdFOQMW949+okzZD VI3g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772317154; x=1772921954; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-gg:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=JgUvB7vchkbEExheWs0hP4qVVuHRRrf4mV32RUQg+58=; b=uAVJFgIrvHUqoAnF5BE2sUixlAJPLkeDqY3rlckk0vB3qDZlyJ/CIdV6AdVXy68ZXw 6YLZaW7il/+UAXnQiSmCpD+nmrSn0OA2eCpMhRi/IRat76PVW1wR1b4sRdExmURXbe1J nPUv/IMwL6HKuSanjxynplLFteKZEz0Lgs1ON95PPkyWPaBJBgsMpa5PaW/xbltsXq/j JRVi6yvIm1UjS25X0RhQJAGmIh0A99OdrJAfQv3imeUJSZRCE9lZTJWq4TZ9i8Y8OjgT U00W/SqLUKFcVUSY4ZFosdl4ryqIB5aq+3xxDqDAoXuzH+WsoU0tkpI4dE0o8kos35SD rH/A== X-Forwarded-Encrypted: i=1; AJvYcCWmWMdDmjBwWS4ABaCi595tINsFzuQbYsROjwelHjCcFOy0BDhbkZcGKxwhmDdNuOsGDugX2qVSfjuYuzQ=@vger.kernel.org X-Gm-Message-State: AOJu0YxDXg4xZC13QHD/yHaYZOmt1fPi6aVYT4v+sT9tll8b+x2l1PSH VRznffU8grWdwJv+/V/FqVluSoo0KsmnpY4dPrWnnMVaSyfwfBnGtxmumWDl5GGAcp0= X-Gm-Gg: ATEYQzzSfPxUwqj19m9j+BWCTgt6BgdPiQVHflGknAfE7XaQiMB5e2esAcIscLtu8w7 O3xvv04F5rGIGFrwlEU7dwbKsSFhRS1mNcjSjUFiArA+mM3HVX+ch9MdvYLtLRZXZsLkQqmZHl+ x94CGXp5nHZsUVYxQ0O0ybqQM/0KSCh1sHBnL/F/5iKwRw+v8Dyhn46MvJU1FyzyxM0kJ7s+97H HzBVrv9uQz+8kxFZBU9Vj1fNhCYvphdHdE86OJhSUU9xGZ+39SHufwMOyi76rpurnDP6zADnU7s xCNXu7ws5iH91Ugz3hITGvbMZ/a2AoOPyk7G5RWgpdt+uzaYRDd+RWQpLlDuzRfzNRr6xmDHbjg IdnnV4mR3BWt7b3KEfrH+fUxEZ/RsFBcMxlY/dtbO1HIhRfR+5Qxk7TFEhvpSNQ1tTncuYbhfNy zUTu9F7U8P2YbTc638KEHdCSwiZnHpcEJEQg== X-Received: by 2002:a05:620a:2849:b0:8cb:4d64:e972 with SMTP id af79cd13be357-8cbc8d70f45mr790715885a.9.1772317153551; Sat, 28 Feb 2026 14:19:13 -0800 (PST) Received: from localhost ([140.174.219.137]) by smtp.gmail.com with ESMTPSA id af79cd13be357-8cbbf6f9905sm783702585a.28.2026.02.28.14.19.12 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 28 Feb 2026 14:19:12 -0800 (PST) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Sat, 28 Feb 2026 17:19:11 -0500 Message-Id: Cc: "Hui Zhu" Subject: Re: [PATCH bpf-next v3 0/3] Fix test_cgroup_iter_memcg issues found during back-porting From: "Emil Tsalapatis" To: "Emil Tsalapatis" , "Hui Zhu" , "Alexei Starovoitov" , "Daniel Borkmann" , "Andrii Nakryiko" , "Martin KaFai Lau" , "Eduard Zingerman" , "Song Liu" , "Yonghong Song" , "John Fastabend" , "KP Singh" , "Stanislav Fomichev" , "Hao Luo" , "Jiri Olsa" , "Shuah Khan" , "JP Kobryn" , "Roman Gushchin" , , , X-Mailer: aerc 0.20.1 References: In-Reply-To: On Sat Feb 28, 2026 at 4:26 PM EST, Emil Tsalapatis wrote: > On Sat Feb 28, 2026 at 2:11 AM EST, Hui Zhu wrote: >> From: Hui Zhu >> >> While back-porting "mm: bpf kfuncs to access memcg data", I >> encountered issues with test_cgroup_iter_memcg, specifically >> in test_kmem. >> >> These patches are my fixes for the problems I encountered. >> > > Nit: Could you be a bit more descriptive about the series' contents? > Cover letters normally focus on what what the patch does - for example > the issues being found during backporting is not pertinent to the patchse= t=20 > itself, because the issues are still present on bpf-next. > Also, responded on v3 instead of v4 sorry about that. Comments and tags still apply since the only delta is in the commit message. >> Changelog: >> v3: >> According to the comments of JP Kobryn, remove kmem subtest from >> cgroup_iter_memcg and fix assertion string in test_pgfault. >> v2: >> According to the comments of JP Kobryn, added bpf_core_enum_value() >> usage in the BPF program to handle cross-kernel enum value differences >> at load-time instead of compile-time. >> Dropped the mm/memcontrol.c patch. >> Modified test_kmem handling: instead of skipping when nokmem is set, >> verify that kmem value is zero as expected. >> According to the comments of bot, fixed assertion message: changed >> "bpf_mem_cgroup_page_state" to "bpf_mem_cgroup_vm_events" for PGFAULT >> check. >> >> Hui Zhu (3): >> selftests/bpf: Remove kmem subtest from cgroup_iter_memcg >> bpf: Use bpf_core_enum_value for stats in cgroup_iter_memcg >> selftests/bpf: Check bpf_mem_cgroup_page_state return value >> >> .../testing/selftests/bpf/cgroup_iter_memcg.h | 2 - >> .../bpf/prog_tests/cgroup_iter_memcg.c | 38 +++++-------------- >> .../selftests/bpf/progs/cgroup_iter_memcg.c | 18 ++++++--- >> 3 files changed, 22 insertions(+), 36 deletions(-)