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=-6.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no 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 2F6BCC433E7 for ; Mon, 19 Oct 2020 07:38:07 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id A79C022200 for ; Mon, 19 Oct 2020 07:38:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A79C022200 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id C41886B005D; Mon, 19 Oct 2020 03:38:05 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id BF10D6B0062; Mon, 19 Oct 2020 03:38:05 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id AB8F96B006E; Mon, 19 Oct 2020 03:38:05 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0099.hostedemail.com [216.40.44.99]) by kanga.kvack.org (Postfix) with ESMTP id 7DF2D6B005D for ; Mon, 19 Oct 2020 03:38:05 -0400 (EDT) Received: from smtpin18.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 25E42180AD811 for ; Mon, 19 Oct 2020 07:38:05 +0000 (UTC) X-FDA: 77387871330.18.cent82_4e0a41627235 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin18.hostedemail.com (Postfix) with ESMTP id F18761015A7B3 for ; Mon, 19 Oct 2020 07:38:04 +0000 (UTC) X-HE-Tag: cent82_4e0a41627235 X-Filterd-Recvd-Size: 4707 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by imf44.hostedemail.com (Postfix) with ESMTP for ; Mon, 19 Oct 2020 07:38:03 +0000 (UTC) IronPort-SDR: cVU4u6taTy7BF76nsTdVIdilEjUZTTNAsVg25CmYpvmJcCuafIV5xmyJv4s8H77ek+LkbRr6VF BIfF2majJdww== X-IronPort-AV: E=McAfee;i="6000,8403,9778"; a="166204681" X-IronPort-AV: E=Sophos;i="5.77,394,1596524400"; d="scan'208";a="166204681" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Oct 2020 00:37:58 -0700 IronPort-SDR: v8n1GIO3V1Krbh0UsCrd3j9O/YLzI+PkbPAIwaMn10BgwQfpOzLrEVWroHYqT9DOXKJnq/gbGb up0eraJF1i/g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,394,1596524400"; d="scan'208";a="347328811" Received: from yhuang-dev.sh.intel.com (HELO yhuang-dev) ([10.239.159.65]) by fmsmga004.fm.intel.com with ESMTP; 19 Oct 2020 00:37:55 -0700 From: "Huang\, Ying" To: Dave Hansen Cc: , , , , , Subject: Re: [RFC][PATCH 6/9] mm/vmscan: add page demotion counter References: <20201007161736.ACC6E387@viggo.jf.intel.com> <20201007161747.FE7288F0@viggo.jf.intel.com> Date: Mon, 19 Oct 2020 15:37:55 +0800 In-Reply-To: <20201007161747.FE7288F0@viggo.jf.intel.com> (Dave Hansen's message of "Wed, 7 Oct 2020 09:17:47 -0700") Message-ID: <87v9f6hdik.fsf@yhuang-dev.intel.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=ascii 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: Dave Hansen writes: > From: Yang Shi > > Account the number of demoted pages into reclaim_state->nr_demoted. It appears that you don't add "nr_demoted" into struct reclaim_state. > Add pgdemote_kswapd and pgdemote_direct VM counters showed in > /proc/vmstat. > > [ daveh: > - __count_vm_events() a bit, and made them look at the THP > size directly rather than getting data from migrate_pages() It appears that we get the data from migrate_pages() now. > ] > > Signed-off-by: Yang Shi > Signed-off-by: Dave Hansen > Cc: David Rientjes > Cc: Huang Ying > Cc: Dan Williams > Cc: David Hildenbrand > --- > > b/include/linux/vm_event_item.h | 2 ++ > b/mm/vmscan.c | 6 ++++++ > b/mm/vmstat.c | 2 ++ > 3 files changed, 10 insertions(+) > > diff -puN include/linux/vm_event_item.h~mm-vmscan-add-page-demotion-counter include/linux/vm_event_item.h > --- a/include/linux/vm_event_item.h~mm-vmscan-add-page-demotion-counter 2020-10-07 09:15:32.171642439 -0700 > +++ b/include/linux/vm_event_item.h 2020-10-07 09:15:32.179642439 -0700 > @@ -33,6 +33,8 @@ enum vm_event_item { PGPGIN, PGPGOUT, PS > PGREUSE, > PGSTEAL_KSWAPD, > PGSTEAL_DIRECT, > + PGDEMOTE_KSWAPD, > + PGDEMOTE_DIRECT, > PGSCAN_KSWAPD, > PGSCAN_DIRECT, > PGSCAN_DIRECT_THROTTLE, > diff -puN mm/vmscan.c~mm-vmscan-add-page-demotion-counter mm/vmscan.c > --- a/mm/vmscan.c~mm-vmscan-add-page-demotion-counter 2020-10-07 09:15:32.173642439 -0700 > +++ b/mm/vmscan.c 2020-10-07 09:15:32.180642439 -0700 > @@ -147,6 +147,7 @@ struct scan_control { > unsigned int immediate; > unsigned int file_taken; > unsigned int taken; > + unsigned int demoted; It appears that this newly added field isn't used in the patch. > } nr; > > /* for recording the reclaimed slab by now */ > @@ -1134,6 +1135,11 @@ static unsigned int demote_page_list(str > target_nid, MIGRATE_ASYNC, MR_DEMOTION, > &nr_succeeded); > > + if (current_is_kswapd()) > + __count_vm_events(PGDEMOTE_KSWAPD, nr_succeeded); > + else > + __count_vm_events(PGDEMOTE_DIRECT, nr_succeeded); > + > return nr_succeeded; > } > > diff -puN mm/vmstat.c~mm-vmscan-add-page-demotion-counter mm/vmstat.c > --- a/mm/vmstat.c~mm-vmscan-add-page-demotion-counter 2020-10-07 09:15:32.175642439 -0700 > +++ b/mm/vmstat.c 2020-10-07 09:15:32.181642439 -0700 > @@ -1244,6 +1244,8 @@ const char * const vmstat_text[] = { > "pgreuse", > "pgsteal_kswapd", > "pgsteal_direct", > + "pgdemote_kswapd", > + "pgdemote_direct", > "pgscan_kswapd", > "pgscan_direct", > "pgscan_direct_throttle", > _ Best Regards, Huang, Ying