Re: [mmotm:master 208/394] mm/page_alloc.c:6245:26: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'zone_init_internals'

[Oscar Salvador <osalvador@techadventures.net>]

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