From: kbuild test robot <lkp@intel.com>
To: Pingfan Liu <kernelfans@gmail.com>
Cc: Michal Hocko <mhocko@suse.com>, "H. Peter Anvin" <hpa@zytor.com>,
Paul Mackerras <paulus@samba.org>, Ingo Molnar <mingo@redhat.com>,
x86@kernel.org, linux-kernel@vger.kernel.org,
Pingfan Liu <kernelfans@gmail.com>,
linux-mm@kvack.org, Mike Rapoport <rppt@linux.vnet.ibm.com>,
Borislav Petkov <bp@alien8.de>,
kbuild-all@01.org, Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Bjorn Helgaas <bhelgaas@google.com>,
David Rientjes <rientjes@google.com>,
Andrew Morton <akpm@linux-foundation.org>,
linuxppc-dev@lists.ozlabs.org,
Thomas Gleixner <tglx@linutronix.de>,
Vlastimil Babka <vbabka@suse.cz>
Subject: Re: [PATCHv2 1/3] mm/numa: change the topo of build_zonelist_xx()
Date: Fri, 21 Dec 2018 15:56:39 +0800 [thread overview]
Message-ID: <201812211505.W391FoVW%fengguang.wu@intel.com> (raw)
In-Reply-To: <1545299439-31370-2-git-send-email-kernelfans@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 4272 bytes --]
Hi Pingfan,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[also build test ERROR on v4.20-rc7 next-20181220]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Pingfan-Liu/mm-bugfix-for-NULL-reference-in-mm-on-all-archs/20181221-152625
config: riscv-tinyconfig (attached as .config)
compiler: riscv64-linux-gcc (GCC) 8.1.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=8.1.0 make.cross ARCH=riscv
All errors (new ones prefixed by >>):
mm/page_alloc.c: In function 'build_zonelists':
>> mm/page_alloc.c:5288:12: error: 'local_node' redeclared as different kind of symbol
int node, local_node;
^~~~~~~~~~
mm/page_alloc.c:5286:66: note: previous definition of 'local_node' was here
static void build_zonelists(struct zonelist *node_zonelists, int local_node)
~~~~^~~~~~~~~~
vim +/local_node +5288 mm/page_alloc.c
^1da177e4 Linus Torvalds 2005-04-16 5285
e6ee0d8bd Pingfan Liu 2018-12-20 5286 static void build_zonelists(struct zonelist *node_zonelists, int local_node)
^1da177e4 Linus Torvalds 2005-04-16 5287 {
19655d348 Christoph Lameter 2006-09-25 @5288 int node, local_node;
9d3be21bf Michal Hocko 2017-09-06 5289 struct zoneref *zonerefs;
9d3be21bf Michal Hocko 2017-09-06 5290 int nr_zones;
^1da177e4 Linus Torvalds 2005-04-16 5291
e6ee0d8bd Pingfan Liu 2018-12-20 5292 zonerefs = node_zonelists[ZONELIST_FALLBACK]._zonerefs;
e6ee0d8bd Pingfan Liu 2018-12-20 5293 nr_zones = build_zonerefs_node(local_node, zonerefs);
9d3be21bf Michal Hocko 2017-09-06 5294 zonerefs += nr_zones;
^1da177e4 Linus Torvalds 2005-04-16 5295
^1da177e4 Linus Torvalds 2005-04-16 5296 /*
^1da177e4 Linus Torvalds 2005-04-16 5297 * Now we build the zonelist so that it contains the zones
^1da177e4 Linus Torvalds 2005-04-16 5298 * of all the other nodes.
^1da177e4 Linus Torvalds 2005-04-16 5299 * We don't want to pressure a particular node, so when
^1da177e4 Linus Torvalds 2005-04-16 5300 * building the zones for node N, we make sure that the
^1da177e4 Linus Torvalds 2005-04-16 5301 * zones coming right after the local ones are those from
^1da177e4 Linus Torvalds 2005-04-16 5302 * node N+1 (modulo N)
^1da177e4 Linus Torvalds 2005-04-16 5303 */
^1da177e4 Linus Torvalds 2005-04-16 5304 for (node = local_node + 1; node < MAX_NUMNODES; node++) {
^1da177e4 Linus Torvalds 2005-04-16 5305 if (!node_online(node))
^1da177e4 Linus Torvalds 2005-04-16 5306 continue;
e6ee0d8bd Pingfan Liu 2018-12-20 5307 nr_zones = build_zonerefs_node(node, zonerefs);
9d3be21bf Michal Hocko 2017-09-06 5308 zonerefs += nr_zones;
^1da177e4 Linus Torvalds 2005-04-16 5309 }
^1da177e4 Linus Torvalds 2005-04-16 5310 for (node = 0; node < local_node; node++) {
^1da177e4 Linus Torvalds 2005-04-16 5311 if (!node_online(node))
^1da177e4 Linus Torvalds 2005-04-16 5312 continue;
e6ee0d8bd Pingfan Liu 2018-12-20 5313 nr_zones = build_zonerefs_node(node, zonerefs);
9d3be21bf Michal Hocko 2017-09-06 5314 zonerefs += nr_zones;
^1da177e4 Linus Torvalds 2005-04-16 5315 }
^1da177e4 Linus Torvalds 2005-04-16 5316
9d3be21bf Michal Hocko 2017-09-06 5317 zonerefs->zone = NULL;
9d3be21bf Michal Hocko 2017-09-06 5318 zonerefs->zone_idx = 0;
^1da177e4 Linus Torvalds 2005-04-16 5319 }
^1da177e4 Linus Torvalds 2005-04-16 5320
:::::: The code at line 5288 was first introduced by commit
:::::: 19655d3487001d7df0e10e9cbfc27c758b77c2b5 [PATCH] linearly index zone->node_zonelists[]
:::::: TO: Christoph Lameter <clameter@sgi.com>
:::::: CC: Linus Torvalds <torvalds@g5.osdl.org>
---
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: 4416 bytes --]
next prev parent reply other threads:[~2018-12-21 7:58 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-20 9:50 [PATCHv2 0/3] mm: bugfix for NULL reference in mm on all archs Pingfan Liu
2018-12-20 9:50 ` [PATCHv2 1/3] mm/numa: change the topo of build_zonelist_xx() Pingfan Liu
2018-12-21 7:56 ` kbuild test robot [this message]
2018-12-20 9:50 ` [PATCHv2 2/3] mm/numa: build zonelist when alloc for device on offline node Pingfan Liu
2018-12-20 11:35 ` Michal Hocko
2018-12-20 12:26 ` Pingfan Liu
2018-12-20 12:44 ` Michal Hocko
2018-12-21 6:17 ` Pingfan Liu
2018-12-20 9:50 ` [PATCHv2 3/3] powerpc/numa: make all possible node be instanced against NULL reference in node_zonelist() Pingfan Liu
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=201812211505.W391FoVW%fengguang.wu@intel.com \
--to=lkp@intel.com \
--cc=Jonathan.Cameron@huawei.com \
--cc=akpm@linux-foundation.org \
--cc=bhelgaas@google.com \
--cc=bp@alien8.de \
--cc=hpa@zytor.com \
--cc=kbuild-all@01.org \
--cc=kernelfans@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mhocko@suse.com \
--cc=mingo@redhat.com \
--cc=paulus@samba.org \
--cc=rientjes@google.com \
--cc=rppt@linux.vnet.ibm.com \
--cc=tglx@linutronix.de \
--cc=vbabka@suse.cz \
--cc=x86@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).