From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 412DDC43217 for ; Thu, 1 Dec 2022 19:24:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230120AbiLATYm (ORCPT ); Thu, 1 Dec 2022 14:24:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229775AbiLATYk (ORCPT ); Thu, 1 Dec 2022 14:24:40 -0500 Received: from out2.migadu.com (out2.migadu.com [188.165.223.204]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 70497BE6A7 for ; Thu, 1 Dec 2022 11:24:39 -0800 (PST) Date: Thu, 1 Dec 2022 11:24:19 -0800 X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Roman Gushchin To: Michal Hocko Cc: kernel test robot , Shakeel Butt , oe-lkp@lists.linux.dev, lkp@intel.com, Andrew Morton , kernel test robot , Soheil Hassas Yeganeh , Feng Tang , Muchun Song , Eric Dumazet , Johannes Weiner , Michal =?iso-8859-1?Q?Koutn=FD?= , linux-kernel@vger.kernel.org Subject: Re: [linus:master] [memcg] 1813e51eec: kernel-selftests.cgroup.test_kmem.test_kmem_memcg_deletion.fail Message-ID: References: <202212010958.c1053bd3-yujie.liu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Dec 01, 2022 at 11:16:34AM +0100, Michal Hocko wrote: > On Thu 01-12-22 16:05:44, kernel test robot wrote: > > Greeting, > > > > FYI, we noticed kernel-selftests.cgroup.test_kmem.test_kmem_memcg_deletion.fail due to commit (built with gcc-11): > > > > commit: 1813e51eece0ad6f4aacaeb738e7cced46feb470 ("memcg: increase MEMCG_CHARGE_BATCH to 64") > > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git master > > > > [test failed on linux-next/master 700e0cd3a5ce6a2cb90d9a2aab729b52f092a7d6] > > > > in testcase: kernel-selftests > > version: kernel-selftests-x86_64-2ed09c3b-1_20221128 > > with following parameters: > > > > group: cgroup > > > > test-description: The kernel contains a set of "self tests" under the tools/testing/selftests/ directory. These are intended to be small unit tests to exercise individual code paths in the kernel. > > test-url: https://www.kernel.org/doc/Documentation/kselftest.txt > > > > on test machine: 128 threads 2 sockets Intel(R) Xeon(R) Platinum 8358 CPU @ 2.60GHz (Ice Lake) with 128G memory > > > > caused below changes (please refer to attached dmesg/kmsg for entire log/backtrace): > > > > > > # memory.current = 40161280 > > # slab + anon + file + kernel_stack = 14478624 > > # slab = 13453184 > > # anon = 0 > > # file = 0 > > # kernel_stack = 0 > > # pagetables = 0 > > # percpu = 1025440 > > # sock = 0 > > # not ok 2 test_kmem_memcg_deletion <-- > > # ok 3 test_kmem_proc_kpagecgroup > > # ok 4 test_kmem_kernel_stacks > > # ok 5 test_kmem_dead_cgroups > > # ok 6 test_percpu_basic > > not ok 2 selftests: cgroup: test_kmem # exit=1 > > IIUC we need this > diff --git a/tools/testing/selftests/cgroup/test_kmem.c b/tools/testing/selftests/cgroup/test_kmem.c > index 22b31ebb3513..1d073e28254b 100644 > --- a/tools/testing/selftests/cgroup/test_kmem.c > +++ b/tools/testing/selftests/cgroup/test_kmem.c > @@ -24,7 +24,7 @@ > * the maximum discrepancy between charge and vmstat entries is number > * of cpus multiplied by 32 pages. > */ > -#define MAX_VMSTAT_ERROR (4096 * 32 * get_nprocs()) > +#define MAX_VMSTAT_ERROR (4096 * 64 * get_nprocs()) Yep. > > > static int alloc_dcache(const char *cgroup, void *arg) > > But honestly, I am rather dubious of tests like this one. Does it really > give us any useful testing coverage? As I remember, we've had some issues in the past when some memcg stats leftovers were not prpoerly propagated on the cgroup deletion, so that over time the numbers on the parent level beacame completely crazy. Thanks!