All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kaiyang2@cs.cmu.edu
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH 3/4] use memory.low local node protection for local node reclaim
Date: Sun, 22 Sep 2024 08:51:21 +0800	[thread overview]
Message-ID: <202409221032.DoTv9B0p-lkp@intel.com> (raw)
In-Reply-To: <20240920221202.1734227-4-kaiyang2@cs.cmu.edu>

Hi,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:

[auto build test WARNING on akpm-mm/mm-everything]
[also build test WARNING on linus/master next-20240920]
[cannot apply to tip/sched/core v6.11]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/kaiyang2-cs-cmu-edu/Add-get_cgroup_local_usage-for-estimating-the-top-tier-memory-usage/20240921-061404
base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-everything
patch link:    https://lore.kernel.org/r/20240920221202.1734227-4-kaiyang2%40cs.cmu.edu
patch subject: [RFC PATCH 3/4] use memory.low local node protection for local node reclaim
config: x86_64-rhel-8.3 (https://download.01.org/0day-ci/archive/20240922/202409221032.DoTv9B0p-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240922/202409221032.DoTv9B0p-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202409221032.DoTv9B0p-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> mm/memcontrol.c:4499: warning: Function parameter or struct member 'is_local' not described in 'mem_cgroup_calculate_protection'


vim +4499 mm/memcontrol.c

c077719be8e9e6 KAMEZAWA Hiroyuki   2009-01-07  4488  
241994ed8649f7 Johannes Weiner     2015-02-11  4489  /**
05395718b2fe48 Mel Gorman          2021-06-30  4490   * mem_cgroup_calculate_protection - check if memory consumption is in the normal range
34c81057927311 Sean Christopherson 2017-07-10  4491   * @root: the top ancestor of the sub-tree being checked
241994ed8649f7 Johannes Weiner     2015-02-11  4492   * @memcg: the memory cgroup to check
241994ed8649f7 Johannes Weiner     2015-02-11  4493   *
230671533d6463 Roman Gushchin      2018-06-07  4494   * WARNING: This function is not stateless! It can only be used as part
230671533d6463 Roman Gushchin      2018-06-07  4495   *          of a top-down tree iteration, not for isolated queries.
241994ed8649f7 Johannes Weiner     2015-02-11  4496   */
45c7f7e1ef17f0 Chris Down          2020-08-06  4497  void mem_cgroup_calculate_protection(struct mem_cgroup *root,
3ebe5883ec39d9 Kaiyang Zhao        2024-09-20  4498  				     struct mem_cgroup *memcg, int is_local)
241994ed8649f7 Johannes Weiner     2015-02-11 @4499  {
a8585ac6862198 Maarten Lankhorst   2024-07-03  4500  	bool recursive_protection =
a8585ac6862198 Maarten Lankhorst   2024-07-03  4501  		cgrp_dfl_root.flags & CGRP_ROOT_MEMORY_RECURSIVE_PROT;
230671533d6463 Roman Gushchin      2018-06-07  4502  
241994ed8649f7 Johannes Weiner     2015-02-11  4503  	if (mem_cgroup_disabled())
45c7f7e1ef17f0 Chris Down          2020-08-06  4504  		return;
241994ed8649f7 Johannes Weiner     2015-02-11  4505  
34c81057927311 Sean Christopherson 2017-07-10  4506  	if (!root)
34c81057927311 Sean Christopherson 2017-07-10  4507  		root = root_mem_cgroup;
22f7496f0b9012 Yafang Shao         2020-08-06  4508  
6f4c005a5f8b8f Kaiyang Zhao        2024-09-20  4509  	page_counter_calculate_protection(&root->memory, &memcg->memory,
3ebe5883ec39d9 Kaiyang Zhao        2024-09-20  4510  					recursive_protection, is_local);
241994ed8649f7 Johannes Weiner     2015-02-11  4511  }
241994ed8649f7 Johannes Weiner     2015-02-11  4512  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

  reply	other threads:[~2024-09-22  0:52 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-20 22:11 [RFC PATCH 0/4] memory tiering fairness by per-cgroup control of promotion and demotion kaiyang2
2024-09-20 22:11 ` [RFC PATCH 1/4] Add get_cgroup_local_usage for estimating the top-tier memory usage kaiyang2
2024-09-20 22:11 ` [RFC PATCH 2/4] calculate memory.low for the local node and track its usage kaiyang2
2024-09-21 23:18   ` kernel test robot
2024-09-22  8:39   ` kernel test robot
2024-10-15 22:05   ` Gregory Price
2024-09-20 22:11 ` [RFC PATCH 3/4] use memory.low local node protection for local node reclaim kaiyang2
2024-09-22  0:51   ` kernel test robot [this message]
2024-09-22 16:31   ` kernel test robot
2024-10-15 21:52   ` Gregory Price
2024-09-20 22:11 ` [RFC PATCH 4/4] reduce NUMA balancing scan size of cgroups over their local memory.low kaiyang2
2024-10-11 20:51 ` [RFC PATCH 0/4] memory tiering fairness by per-cgroup control of promotion and demotion Kaiyang Zhao
2024-11-08 19:01 ` kaiyang2

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202409221032.DoTv9B0p-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kaiyang2@cs.cmu.edu \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.