All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC v1] memcg: add memcg lru for page reclaiming
Date: Tue, 22 Oct 2019 10:17:34 +0800	[thread overview]
Message-ID: <201910221000.RGstQIqk%lkp@intel.com> (raw)
In-Reply-To: <20191021115654.14740-1-hdanton@sina.com>

[-- Attachment #1: Type: text/plain, Size: 3202 bytes --]

Hi Hillf,

[FYI, it's a private test report for your RFC patch.]
[auto build test ERROR on linus/master]
[cannot apply to v5.4-rc4 next-20191021]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Hillf-Danton/memcg-add-memcg-lru-for-page-reclaiming/20191022-082625
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 7d194c2100ad2a6dded545887d02754948ca5241
config: x86_64-lkp (attached as .config)
compiler: gcc-7 (Debian 7.4.0-14) 7.4.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   mm/memcontrol.c: In function 'reclaim_high':
   mm/memcontrol.c:2385:8: error: 'start' undeclared (first use in this function); did you mean 'stat'?
       if (start != memcg) {
           ^~~~~
           stat
   mm/memcontrol.c:2385:8: note: each undeclared identifier is reported only once for each function it appears in
>> mm/memcontrol.c:2386:5: error: implicit declaration of function 'memcg_add_lru'; did you mean 'numa_add_cpu'? [-Werror=implicit-function-declaration]
        memcg_add_lru(memcg);
        ^~~~~~~~~~~~~
        numa_add_cpu
   mm/memcontrol.c: In function 'mem_cgroup_soft_limit_reclaim':
   mm/memcontrol.c:3202:30: error: implicit declaration of function 'memcg_pick_lru'; did you mean 'lock_page_lru'? [-Werror=implicit-function-declaration]
      struct mem_cgroup *memcg = memcg_pick_lru();
                                 ^~~~~~~~~~~~~~
                                 lock_page_lru
   mm/memcontrol.c:3202:30: warning: initialization makes pointer from integer without a cast [-Wint-conversion]
   mm/memcontrol.c: In function 'mem_cgroup_alloc':
   mm/memcontrol.c:5119:26: error: 'struct mem_cgroup' has no member named 'lru_node'; did you mean 'scan_nodes'?
      INIT_LIST_HEAD(&memcg->lru_node);
                             ^~~~~~~~
                             scan_nodes
   cc1: some warnings being treated as errors

vim +2386 mm/memcontrol.c

  2372	
  2373	static void reclaim_high(struct mem_cgroup *memcg,
  2374				 unsigned int nr_pages,
  2375				 gfp_t gfp_mask)
  2376	{
  2377	#ifdef CONFIG_MEMCG_LRU
  2378		struct mem_cgroup *start = memcg;
  2379	#endif
  2380		do {
  2381			if (page_counter_read(&memcg->memory) <= memcg->high)
  2382				continue;
  2383			memcg_memory_event(memcg, MEMCG_HIGH);
  2384			if (IS_ENABLED(CONFIG_MEMCG_LRU))
> 2385				if (start != memcg) {
> 2386					memcg_add_lru(memcg);
  2387					return;
  2388				}
  2389			try_to_free_mem_cgroup_pages(memcg, nr_pages, gfp_mask, true);
  2390		} while ((memcg = parent_mem_cgroup(memcg)));
  2391	}
  2392	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 28617 bytes --]

  parent reply	other threads:[~2019-10-22  2:17 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-21 11:56 [RFC v1] memcg: add memcg lru for page reclaiming Hillf Danton
2019-10-21 12:14 ` Michal Hocko
2019-10-22  2:17 ` kbuild test robot [this message]
2019-10-22  3:37 ` kbuild test robot
2019-10-22 13:30 ` Hillf Danton
2019-10-22 13:58   ` Michal Hocko
2019-10-23  4:44   ` Hillf Danton
2019-10-23  8:08     ` Michal Hocko

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=201910221000.RGstQIqk%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /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.