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 X-Spam-Level: X-Spam-Status: No, score=-8.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 151E0C433FE for ; Sun, 6 Dec 2020 10:03:00 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 7DD6C22D2A for ; Sun, 6 Dec 2020 10:02:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7DD6C22D2A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=bytedance.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id ADFA86B006C; Sun, 6 Dec 2020 05:02:58 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id A90606B006E; Sun, 6 Dec 2020 05:02:58 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9A7846B0070; Sun, 6 Dec 2020 05:02:58 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0033.hostedemail.com [216.40.44.33]) by kanga.kvack.org (Postfix) with ESMTP id 837A96B006C for ; Sun, 6 Dec 2020 05:02:58 -0500 (EST) Received: from smtpin19.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 47CE5180AD804 for ; Sun, 6 Dec 2020 10:02:58 +0000 (UTC) X-FDA: 77562418836.19.brain64_3212e3e273d4 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin19.hostedemail.com (Postfix) with ESMTP id 2A64B1AD1B4 for ; Sun, 6 Dec 2020 10:02:58 +0000 (UTC) X-HE-Tag: brain64_3212e3e273d4 X-Filterd-Recvd-Size: 6571 Received: from mail-pj1-f65.google.com (mail-pj1-f65.google.com [209.85.216.65]) by imf03.hostedemail.com (Postfix) with ESMTP for ; Sun, 6 Dec 2020 10:02:57 +0000 (UTC) Received: by mail-pj1-f65.google.com with SMTP id p21so5651445pjv.0 for ; Sun, 06 Dec 2020 02:02:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=yK27iUNadlI9MmI6B+s8BZZCxEUnviUifHj+P5KRXMg=; b=ilNSry1fFb2VSxzC6tm0xGYazjZD9Ar0wYnM5PmkQPzKeO6m/8kxm5rNlI7oyTPPbk i5xwLJoZlcE+oIS19DTByIbI8S3mWauEFjywj/RhH1PXOGTBSbM16m1Gl+AZFCYm0gyH bqu6gclEOOrruiFK87psVFo1l/07z01Iq65RLsp9vxd5+qGAHT97LUR7X2J4zBebULJg EbQ6P5WANSdYinm/+Jl5ZApt2ou6i8fmj/P4NnTZeBruaCmWfW2MUQXUTCXnA4EK+O5a yyTurylWJoFQxbjQg32Oh61ACS8XIfEJKlqAriWqjT9VSLCtNs7tOFl0UM+mD4P2rW87 b+1Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=yK27iUNadlI9MmI6B+s8BZZCxEUnviUifHj+P5KRXMg=; b=izRl6a00uDl6DuHIQPK0IqZ9xgAI47txwZazQIG4xZ9ezbLaQi0Olvj7f/ghIugi1W POW8eBr2Hzfw+0oOipjKMCV95orjPjzvEFKCH5BfNeJL1wx9mI1t4n8q71CLKLc5AuQk s48bFx6NjXGM7MPbnb0i7EZVeiq4d+NAb8TDT+mCDDYHYuwACZPu393fPWXHUIb+raAZ PHjM7Q7QZFPX71uirZqhgYzh4D5HWfGmjQQiY7QiZesC4ESQmF2PPOcAmkW17rQESeqX KYqhIyFQ3WLbI342JP2cvaK2MlU5wLpCxa/befdsSQoZSyldfY7pMjVMqSLP6lDyAHx+ mAaA== X-Gm-Message-State: AOAM533zgG1TM6fu7VAdpnV+fk1zvai/Az09s12ErRZnovBGFv7Gljln 0DHbKctTmX4C+QRgEzh/RVP6of+41ddH4NpvTleFxA== X-Google-Smtp-Source: ABdhPJzlTMynhkhES4yihb/sRbkNB9LxQF5JR1rRWkeIuqZTp8b5PPlWcwgHSCrkWCDSnWLuHFNfu1VIhoMtRrqcpDc= X-Received: by 2002:a17:902:ed0d:b029:da:c83b:5f40 with SMTP id b13-20020a170902ed0db02900dac83b5f40mr11300264pld.20.1607248976015; Sun, 06 Dec 2020 02:02:56 -0800 (PST) MIME-Version: 1.0 References: <20201206082318.11532-1-songmuchun@bytedance.com> In-Reply-To: <20201206082318.11532-1-songmuchun@bytedance.com> From: Muchun Song Date: Sun, 6 Dec 2020 18:02:19 +0800 Message-ID: Subject: Re: [PATCH v2 00/12] Convert all vmstat counters to pages or bytes To: Greg KH , rafael@kernel.org, Alexey Dobriyan , Andrew Morton , Johannes Weiner , Michal Hocko , Vladimir Davydov , Hugh Dickins , Will Deacon , Roman Gushchin , Mike Rapoport , Thomas Gleixner , esyr@redhat.com, peterx@redhat.com, krisman@collabora.com, Suren Baghdasaryan , avagin@openvz.org, Marco Elver , Randy Dunlap , Joonsoo Kim Cc: LKML , linux-fsdevel , Linux Memory Management List , Cgroups Content-Type: text/plain; charset="UTF-8" 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: I am very sorry that I have hit 'git send-email *' in a directory containing both v1 and v2 patchs. Please ignore this. I will resend this version. Very sorry for the noise. Thanks. On Sun, Dec 6, 2020 at 4:25 PM Muchun Song wrote: > > Hi, > > This patch series is aimed to convert all THP vmstat counters to pages > and some KiB vmstat counters to bytes. > > The unit of some vmstat counters are pages, some are bytes, some are > HPAGE_PMD_NR, and some are KiB. When we want to expose these vmstat > counters to the userspace, we have to know the unit of the vmstat counters > is which one. It makes the code complex. Because there are too many choices, > the probability of making a mistake will be greater. > > For example, the below is some bug fix: > - 7de2e9f195b9 ("mm: memcontrol: correct the NR_ANON_THPS counter of hierarchical memcg") > - not committed(it is the first commit in this series) ("mm: memcontrol: fix NR_ANON_THPS account") > > This patch series can make the code simple (161 insertions(+), 187 deletions(-)). > And make the unit of the vmstat counters are either pages or bytes. Fewer choices > means lower probability of making mistakes :). > > This was inspired by Johannes and Roman. Thanks to them. > > Changes in v1 -> v2: > - Change the series subject from "Convert all THP vmstat counters to pages" > to "Convert all vmstat counters to pages or bytes". > - Convert NR_KERNEL_SCS_KB account to bytes. > - Convert vmstat slab counters to bytes. > - Remove {global_}node_page_state_pages. > > Muchun Song (12): > mm: memcontrol: fix NR_ANON_THPS account > mm: memcontrol: convert NR_ANON_THPS account to pages > mm: memcontrol: convert NR_FILE_THPS account to pages > mm: memcontrol: convert NR_SHMEM_THPS account to pages > mm: memcontrol: convert NR_SHMEM_PMDMAPPED account to pages > mm: memcontrol: convert NR_FILE_PMDMAPPED account to pages > mm: memcontrol: convert kernel stack account to bytes > mm: memcontrol: convert NR_KERNEL_SCS_KB account to bytes > mm: memcontrol: convert vmstat slab counters to bytes > mm: memcontrol: scale stat_threshold for byted-sized vmstat > mm: memcontrol: make the slab calculation consistent > mm: memcontrol: remove {global_}node_page_state_pages > > drivers/base/node.c | 25 ++++----- > fs/proc/meminfo.c | 22 ++++---- > include/linux/mmzone.h | 21 +++----- > include/linux/vmstat.h | 21 ++------ > kernel/fork.c | 8 +-- > kernel/power/snapshot.c | 2 +- > kernel/scs.c | 4 +- > mm/filemap.c | 4 +- > mm/huge_memory.c | 9 ++-- > mm/khugepaged.c | 4 +- > mm/memcontrol.c | 131 ++++++++++++++++++++++++------------------------ > mm/oom_kill.c | 2 +- > mm/page_alloc.c | 17 +++---- > mm/rmap.c | 19 ++++--- > mm/shmem.c | 3 +- > mm/vmscan.c | 2 +- > mm/vmstat.c | 54 ++++++++------------ > 17 files changed, 161 insertions(+), 187 deletions(-) > > -- > 2.11.0 > -- Yours, Muchun