mm: hugetlb.c:5449:62: error: expected ';' before '{' token for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL)

mm: hugetlb.c:5449:62: error: expected ';' before '{' token for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL)

[Naresh Kamboju]

Linux next build failed on arm beagleboard x15.

Build error:
/mm/hugetlb.c: In function 'hugetlb_cma_reserve':
/mm/hugetlb.c:5449:3: error: implicit declaration of function
'for_each_mem_pfn_range'; did you mean 'for_each_mem_range'?
[-Werror=implicit-function-declaration]
   for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL) {
   ^~~~~~~~~~~~~~~~~~~~~~
   for_each_mem_range
/mm/hugetlb.c:5449:62: error: expected ';' before '{' token
   for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL) {

config:
http://snapshots.linaro.org/openembedded/lkft/lkft/sumo/am57xx-evm/lkft/linux-next/718/config
                                                              ^
full build url:
https://ci.linaro.org/view/lkft/job/openembedded-lkft-linux-next/DISTRO=lkft,MACHINE=am57xx-evm,label=docker-lkft/727/consoleText

-- 
Linaro LKFT
https://lkft.linaro.org

Re: mm: hugetlb.c:5449:62: error: expected ';' before '{' token for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL)

[Roman Gushchin]

On Wed, Mar 18, 2020 at 08:03:09PM +0530, Naresh Kamboju wrote:
> Linux next build failed on arm beagleboard x15.
> 
> Build error:
> /mm/hugetlb.c: In function 'hugetlb_cma_reserve':
> /mm/hugetlb.c:5449:3: error: implicit declaration of function
> 'for_each_mem_pfn_range'; did you mean 'for_each_mem_range'?
> [-Werror=implicit-function-declaration]
>    for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL) {
>    ^~~~~~~~~~~~~~~~~~~~~~
>    for_each_mem_range
> /mm/hugetlb.c:5449:62: error: expected ';' before '{' token
>    for_each_mem_pfn_range(i, nid, &start_pfn, &end_pfn, NULL) {
> 
> config:
> https://urldefense.proofpoint.com/v2/url?u=http-3A__snapshots.linaro.org_openembedded_lkft_lkft_sumo_am57xx-2Devm_lkft_linux-2Dnext_718_config&d=DwIBaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=jJYgtDM7QT-W-Fz_d29HYQ&m=36B2rBvL7m63G1ffiQY8uKTRD5-t1fCOMaiubLFeCFI&s=wvwNdQn_ZM2uS8WDrAnK3MNOjgdZH17_m0W6nU6B6jE&e= 
>                                                               ^
> full build url:
> https://urldefense.proofpoint.com/v2/url?u=https-3A__ci.linaro.org_view_lkft_job_openembedded-2Dlkft-2Dlinux-2Dnext_DISTRO-3Dlkft-2CMACHINE-3Dam57xx-2Devm-2Clabel-3Ddocker-2Dlkft_727_consoleText&d=DwIBaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=jJYgtDM7QT-W-Fz_d29HYQ&m=36B2rBvL7m63G1ffiQY8uKTRD5-t1fCOMaiubLFeCFI&s=ziWfEC6V94C4hMva9LMHYjbVQiDIL0PqFolj52WtAps&e= 
> 
> -- 
> Linaro LKFT
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lkft.linaro.org&d=DwIBaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=jJYgtDM7QT-W-Fz_d29HYQ&m=36B2rBvL7m63G1ffiQY8uKTRD5-t1fCOMaiubLFeCFI&s=KkC9c04y9ZU97zV5xpwJ4tHeoFRdC45JRuzeGbqb_Gs&e= 

Hello Naresh!

Thank you for the report!

We've been discussing the issue for a couple of days, and I've proposed a fix.
I've just resent it to linux-mm@ (you're in cc), so hopefully it will be merged today.

Thanks!