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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4AB38C77B60 for ; Fri, 28 Apr 2023 09:44:16 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id A4D7E6B0071; Fri, 28 Apr 2023 05:44:15 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 9FE846B0072; Fri, 28 Apr 2023 05:44:15 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8ECE26B0074; Fri, 28 Apr 2023 05:44:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 819A76B0071 for ; Fri, 28 Apr 2023 05:44:15 -0400 (EDT) Received: from smtpin14.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 513A21C6C40 for ; Fri, 28 Apr 2023 09:44:15 +0000 (UTC) X-FDA: 80730314070.14.3FEA219 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf15.hostedemail.com (Postfix) with ESMTP id 734CCA0011 for ; Fri, 28 Apr 2023 09:44:13 +0000 (UTC) Authentication-Results: imf15.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b="AGa/xCEj"; spf=pass (imf15.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1682675053; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=fiJTpK/XRgtTp6ZTdBZkBeDGODxz+KV44UNCy4wvN4k=; b=Xlf2HCpm6z+JUL6LiYQhLh2whwSlzVrD5XzygZMvIuQVfKSHJsIDrEwe02UPWmTjVDC+iJ QKBiw0foXSFC80dTU40dWT572EJqnEshYzOv5VO2LR3fbb5adbPj7z5HsMqzWT9E/Wevcs ZoYvQfzkV6qmHjswPdzEm77reHdEMqA= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1682675053; a=rsa-sha256; cv=none; b=aZ9K88xza36lkLuhq3j8McZdgrxwhLk2RIIu1OetpQrUYETvschyyZX+dIptARDL57Eabn NJH8zzxRjk8WeEBsNED0SyMWea6rjg1q+383OBHhgPh+1OdP3uvW1bPKOv7/9U+9NT3HOL c954EsWqDvpAz5FCeNJk1OrB7vsOMv4= ARC-Authentication-Results: i=1; imf15.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b="AGa/xCEj"; spf=pass (imf15.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id BA7B521EE2; Fri, 28 Apr 2023 09:44:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1682675051; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=fiJTpK/XRgtTp6ZTdBZkBeDGODxz+KV44UNCy4wvN4k=; b=AGa/xCEjEb3qK0MkSqbET4JkZvo2UGTc8z/NuldzPCjYt42vE3dxHHEbB6d5sy76QyDlKB JZvN9n3qMA7C3W9yiQNLWfKs3iEzV/H7hFRrViMiISFVM6/YwbFIvopZPQ3WQxIfW9qSHl k/+k/CJqK0lRhf7+++JDZcpd7axXQv0= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id AC0771390E; Fri, 28 Apr 2023 09:44:11 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id tI34KWuVS2TmHwAAMHmgww (envelope-from ); Fri, 28 Apr 2023 09:44:11 +0000 Date: Fri, 28 Apr 2023 11:44:11 +0200 From: Michal Hocko To: Yosry Ahmed Cc: Johannes Weiner , Roman Gushchin , Shakeel Butt , Andrew Morton , Muchun Song , Sergey Senozhatsky , Steven Rostedt , Petr Mladek , Chris Li , cgroups@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] memcg: dump memory.stat during cgroup OOM for v1 Message-ID: References: <20230426133919.1342942-1-yosryahmed@google.com> <20230426133919.1342942-3-yosryahmed@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspam-User: X-Rspamd-Queue-Id: 734CCA0011 X-Rspamd-Server: rspam09 X-Stat-Signature: ucwraayjp6zygrz8rkbu44gemq4owdhp X-HE-Tag: 1682675053-808430 X-HE-Meta: U2FsdGVkX1/+GdcIndRR3byQYcxHApnn2rfIDX3AKB6rYX8NfWuqszEm5LNGtnXYlEwCoPI1Z18EqSTjHfP2oHSBi2eonb5w34EqiSDIyhJRUup7dYXRLEPELb+F/Ug939qO5CVrv/J2yp7jRZgI1faYf0R2WPpKd6Gr/pDrDyhBOu0Q7lYSM0Q12MirKRfwnHqt8o+0s1eh7Nr7/rqslw6mwuoEUsX3U9IGWUhirkOjy6E8Dj5GfVMbF6zLtqRcgXigauOAhjBgFjATMHGZ0ZoiPIfD8V+Jvl/uYhIqQybqzCt82o4LRneKOxsB3n3rGOqbqRlYgJPBsi2QWVxJZ4VHk4Jwq2eFNW77k4cxmp4mqeVDHpNRtKUiGWV4Y189khckD3j40XDT8YpugndnatYCgtoq2ZZVyFwnzNCj9DlALE+54WC7YmMb9KAsNjqqLSUm6BeEegD2o1DLspcAeCE7AbvZ9QHMYCzyMoensitY5qj6/b+XfjIA7qcr68zNYIKdz8GCadK86zUa33T0jt/xibSewTTBt44FC6SXma7VXK1smnaOg1e9La+MIa42aKAs5inUS7uI9kF3nGwjJrILJJnRpTPzRiHkmIJ+DpnPasSv7uFDfhZVMkVGIodrrux28KOyZsgAHzSvdTAJ4UKRpfZH108k2adzzjqhYYVPqRyyP9IvZ90/alAWGI7K/bt5ZFPWHelVBgKh0fTimvp3rvA86bghFxxnIdpRyHRd3xrvLDmdxmjtoOrr9pa2u5jOoMBKaQ0GNc27PthmmmD0/InmxdtqYqMi93DrIZfeTYFsaR6T7pr2D+v2luegSkNR+CQ+n1xNMsS7fuzzDqROC78bWHjLQ98bscNem6Qh4djhdvB68E6tOp1V2TRisAa2iRBk57zHX3Zvs2lGpmaZMoifR85dQk3gp1EkVRvdgQMXfx7elpSr6Sa6YZOvTEyUwaSA2ZgP6LMrUDq 9xk/UwkX PsmkKEvyWaz2mnuChHQX2+B4LC89VgVKyzw5bg7kMqaZRfQlNrO9oOS5M6R2MFMlVFFQ/WPCkA/7lpSDLp3A8uVW64lqZxlrGztOU4BYptilor/VlFOGSziF0Db9wnDwRHX+Yt8AuL5ysB8rUt9pj2LzeIg== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu 27-04-23 15:12:46, Yosry Ahmed wrote: [...] > However, I still think this change is valuable. Like you mentioned, > the OOM log is not set in stone, but we shouldn't just change it for > no reason. In this case, for cgroup v1 users, the OOM log changed for > no reason beyond a side effect of another patch. Upon upgrading our > kernel we noticed the behavior change. This patch restores the old > behavior without any cost really, and it makes the code a tiny bit > more consistent. Fair enough. Just make sure you go into more details about why this is causing problems/inconveniences. I am slightly worried this might cause problems to other people who would like to have the same report for both v1 and v2 so we should at least have some solid argumetns to revert rather than "it used has changed and we liked it more that way". I personally do not care all that much. It kinda sucks to dump counters that are not tracked or fully tracked in v1 because that can mislead people and that would be a bigger problem from my POV. -- Michal Hocko SUSE Labs