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 63C2BC3DA61 for ; Wed, 24 Jul 2024 23:54:24 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D22576B008C; Wed, 24 Jul 2024 19:54:23 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id CD1E26B0093; Wed, 24 Jul 2024 19:54:23 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id B99B16B0095; Wed, 24 Jul 2024 19:54:23 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 9AFBA6B008C for ; Wed, 24 Jul 2024 19:54:23 -0400 (EDT) Received: from smtpin14.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 1C13A1C130C for ; Wed, 24 Jul 2024 23:54:23 +0000 (UTC) X-FDA: 82376302806.14.23959B9 Received: from out-178.mta0.migadu.com (out-178.mta0.migadu.com [91.218.175.178]) by imf01.hostedemail.com (Postfix) with ESMTP id C10F640007 for ; Wed, 24 Jul 2024 23:54:19 +0000 (UTC) Authentication-Results: imf01.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=JG+hdaJe; spf=pass (imf01.hostedemail.com: domain of roman.gushchin@linux.dev designates 91.218.175.178 as permitted sender) smtp.mailfrom=roman.gushchin@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1721865214; 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=yk2XYWVfb0ajgjBqpBjfB0JQbWXRZtaSTamq2SyxgJA=; b=vKTRU/BaqCS3dGhtrdy00SKRnZM6ehk6xDmKsUrVN1gHScIvvV9Hy1qQ/ih6g1iSRUk1p7 hKuRly1VptUx7sdgyVFk+sLxOYp4ao7B8/SuL+OkK10Vsrjw6EOavsNmpp00QR4hyaVXka cyxQqzO30Z2Na7dl/qreJ5FZXjLIx0Y= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1721865214; a=rsa-sha256; cv=none; b=mhrivd4azQaGRYcLRAMxhgqAIgZJipm7VeD6VXNrrR6XpJ17wjMQ5okgC8v7Yu6PNVTgLc I1WnMMnfVGImIe7v2lfNlBDnSy1xPcB1UaKFIML5OE2i8UG1eZBTagxRAr7xmS9eagnL8R NQnlMtGLJRaWg2043180JmQ9E8ntOpM= ARC-Authentication-Results: i=1; imf01.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=JG+hdaJe; spf=pass (imf01.hostedemail.com: domain of roman.gushchin@linux.dev designates 91.218.175.178 as permitted sender) smtp.mailfrom=roman.gushchin@linux.dev; dmarc=pass (policy=none) header.from=linux.dev Date: Wed, 24 Jul 2024 23:54:11 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1721865256; h=from:from:reply-to:subject:subject: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=yk2XYWVfb0ajgjBqpBjfB0JQbWXRZtaSTamq2SyxgJA=; b=JG+hdaJepFrLRf9MvG7gveKe6Fko9GfH+pTQ5JnzanNmI6F2eIBwUCBzr/kee5wMiYS/uy 9Y4V52GMdNnTeCqovstywhptovwqORs3ZYGX6+H5WxE/k4tOlMHsCThjBKH+G3S4sDS9zb jX1f/qnbtDByGQvtFpDSexaYJ9HpWBU= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Roman Gushchin To: Shakeel Butt Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Johannes Weiner , Michal Hocko , Muchun Song Subject: Re: [PATCH v2 5/5] mm: memcg: convert enum res_type to mem_counter_type Message-ID: References: <20240724202103.1210065-1-roman.gushchin@linux.dev> <20240724202103.1210065-6-roman.gushchin@linux.dev> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Migadu-Flow: FLOW_OUT X-Rspamd-Queue-Id: C10F640007 X-Stat-Signature: s586ce9o817mr5ayf5ye7qp8qo3sq7hy X-Rspamd-Server: rspam09 X-Rspam-User: X-HE-Tag: 1721865259-843548 X-HE-Meta: U2FsdGVkX1+8tcj7J6XpeIU6e8pPOElmjZOjx1GTobzrgRRFpFArxiQJ7t5oV2V+3RZn8Z2AyiyhbTkBhz1FZ0pdWvohcN13GWHOImlOVF/0jnCcT/GxOytijf443H2iD/SlkDW1M55ooO2PsQhDnamckGIf+JbQkbYP/PAy+TUbk+uUlT9EGBlzbwPiws8gFvt02v80WJ/++gJrVRUYTunwMEV2nUzNl9+5eXphBuib89KArIsUZaeEodePNRna+GNChaJjb39N7doPafCgfgpYe4IVOWFwVtb2noaamE+z1rbV0/XFysZnGpnmmwOzB7XP0MsBm4wjhPGOgSIvRmHJQAZkI/VeyYs4bUr4XTOM8s8jkAjI7H4Ejf1QqT0NsruCAmXgasSq0wt5L/h/ss4NizpPENlalVdTPJBD1Ahbw8+vAG+0Erd7aMrkQuWxigkCMKx/lsxIcejrn769ViXIv/8TaRweTm033FJKb4c3kGIAWUeUYTAL1n2RCAam6Cm1b7TIBlHqpkW1IcHP7pNTbBbnV+7NCZwm9Yityhkomp+lMs2oaCRZ4fpy8KmGumTkmaNz9xXrZaGk3HFQJV3lappjgVDS/htuH2xned0jaT/4hZYp48o1JLC6SsrEA9kriRDHIijuPQygSXdJVm97J0rVWMPZ78kWMsv1Rni/8nEKPCvHDsZj9ui5yFnJTH6PfHnYJeRHLREXYGfpHQHeURkKmRz0Hn5j2nwwKpXNpY/2Ss0pUnng4J7CfDaJqKenlhK4UmyKZkHOABFCnpTuuXjf4ENzDeQdnJ9ANqCeOwyYbdev6GUHWp50Ihb3Gzew8OB3rWSC1KALlzCfQQIczgRnASWPJkvE8wmC+lhYwCT7y6oCRKUZokQqMcRGwgc57/q9hGrfCyoIHcwFQBjWou5RHoSiHEuVk5TzqVRJ4cF5OQmafKRk/gnjjSFj4VUIrsBEzbBEIoi3PNT ErzMRt2M Nkv162X5gjTxtzKWGwl1I4a+2GYv39s+MFsa9SaluGaWsAgCPzPJXX5JPPsoZuExlGtH4wrt0+MHzdqUpVXWxmq7x+52NkQuOnf3oy5yMud6Ww26pgL2gsi44vw== 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: List-Subscribe: List-Unsubscribe: On Wed, Jul 24, 2024 at 04:48:47PM -0700, Shakeel Butt wrote: > On Wed, Jul 24, 2024 at 08:21:03PM GMT, Roman Gushchin wrote: > > The res_type enum is listing all types of memory tracked by memory > > cgroups: generic memory, swap, kernel memory, tcp etc; and it's > > currently used only for dealing with corresponding sysfs files. > > > > To prepare for tracking of various types of memory by a single > > page_counter structure, a similar enumeration is needed. Instead > > of introducing a completely new enumeration, let's re-purpose > > the existing one: rename it into mem_counter_type, change items > > names to be more meaningful and move to page_counter.h. The latter > > is needed to have the total number of different memory types > > available. > > > > This change doesn't bring any functional difference, it's a pure > > refactoring. > > > > Signed-off-by: Roman Gushchin > > Acked-by: Shakeel Butt Thank you for reviewing the series!