On Sat, Aug 11, 2018 at 07:00:52PM +0800, kbuild test robot wrote:
> tree: git://git.cmpxchg.org/linux-mmotm.git master
> head: b1da01df1aa700864692a49a7007fc96cc1da7d9
> commit: e3dcfdaca81e86f21335a0b6d39162ad574c8574 [208/394] mm/page_alloc: Introduce free_area_init_core_hotplug
> config: x86_64-fedora-25 (attached as .config)
> compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
> reproduce:
> git checkout e3dcfdaca81e86f21335a0b6d39162ad574c8574
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> Note: the mmotm/master HEAD b1da01df1aa700864692a49a7007fc96cc1da7d9 builds fine.
> It only hurts bisectibility.
>
> All errors (new ones prefixed by >>):
>
> >> mm/page_alloc.c:6245:26: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'zone_init_internals'
> static void __paginginit zone_init_internals(struct zone *zone, enum zone_type idx, int nid,
> ^~~~~~~~~~~~~~~~~~~
> mm/page_alloc.c: In function 'free_area_init_core_hotplug':
> >> mm/page_alloc.c:6272:3: error: implicit declaration of function 'zone_init_internals'; did you mean 'pgdat_init_internals'? [-Werror=implicit-function-declaration]
> zone_init_internals(&pgdat->node_zones[z], z, nid, 0);
It looks like this linux-mmotm's tree is not testing the right version of the patch.
In V6 [1], zone_init_internals is declared as __meminit because the __paginginit macro was just dropped.
I just checked out the last version of the linux-mmotm tree, and it contains the right thing there:
<--
static void __meminit zone_init_internals(struct zone *zone, enum zone_type idx, int nid,
unsigned long remaining_pages)
{
zone->managed_pages = remaining_pages;
zone_set_nid(zone, nid);
zone->name = zone_names[idx];
zone->zone_pgdat = NODE_DATA(nid);
-->
So this error looks invalid.
[1] https://patchwork.kernel.org/patch/10552231/
--
Oscar Salvador
SUSE L3
