page_alloc.c:(.init.text+0xa7c): undefined reference to `vmalloc_huge'

page_alloc.c:(.init.text+0xa7c): undefined reference to `vmalloc_huge'

[Naresh Kamboju]

Linux mainline arm architecture tinyconfig and allnoconfig builds failed.
These builds have config mmu not set.

# CONFIG_MMU is not set


Regressions found on arm:

   - arm-clang-14-tinyconfig
   - arm-clang-nightly-tinyconfig
   - arm-gcc-11-tinyconfig
   - arm-clang-13-tinyconfig
   - arm-gcc-11-allnoconfig
   - arm-clang-14-allnoconfig
   - arm-clang-nightly-allnoconfig
   - arm-gcc-8-allnoconfig
   - arm-clang-11-allnoconfig
   - arm-clang-11-tinyconfig
   - arm-gcc-9-allnoconfig
   - arm-gcc-9-tinyconfig
   - arm-gcc-10-allnoconfig
   - arm-gcc-8-tinyconfig
   - arm-gcc-10-tinyconfig
   - arm-clang-12-allnoconfig
   - arm-clang-12-tinyconfig
   - arm-clang-13-allnoconfig


Build error:
------------
arm-linux-gnueabihf-ld: mm/page_alloc.o: in function `alloc_large_system_hash':
page_alloc.c:(.init.text+0xa7c): undefined reference to `vmalloc_huge'
make[1]: *** [Makefile:1158: vmlinux] Error 1


# To install tuxmake on your system globally:
# sudo pip3 install -U tuxmake

tuxmake --runtime podman --target-arch arm --toolchain gcc-11
--kconfig tinyconfig


Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>


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

[1] https://builds.tuxbuild.com/28GNGapkjXsbIbRSnHoGILqmaFp/

Re: page_alloc.c:(.init.text+0xa7c): undefined reference to `vmalloc_huge'

[Linus Torvalds]

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

On Mon, Apr 25, 2022 at 1:10 AM Naresh Kamboju
<naresh.kamboju@linaro.org> wrote:
>
> # CONFIG_MMU is not set

I just sent the stupid and straightforward patch in

  https://lore.kernel.org/all/CAHk-=wi5DYKbFE4j-jC2HGsKVuf1RpZbEiYt4tSXuxGKiN9oJg@mail.gmail.com/

but then I thought maybe the alias patch is as simple as the attached.

So does either (or both) of these patches work?

                  Linus

[-- Attachment #2: patch.diff --]
[-- Type: text/x-patch, Size: 407 bytes --]

 mm/nommu.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/mm/nommu.c b/mm/nommu.c
index 55a9e48a7a02..9d7afc2d959e 100644
--- a/mm/nommu.c
+++ b/mm/nommu.c
@@ -226,6 +226,8 @@ void *vmalloc(unsigned long size)
 }
 EXPORT_SYMBOL(vmalloc);
 
+void *vmalloc_huge(unsigned long size, gfp_t gfp_mask) __weak __alias(__vmalloc);
+
 /*
  *	vzalloc - allocate virtually contiguous memory with zero fill
  *

Re: page_alloc.c:(.init.text+0xa7c): undefined reference to `vmalloc_huge'

[Naresh Kamboju]

Hi Linus,

On Mon, 25 Apr 2022 at 13:58, Linus Torvalds
<torvalds@linux-foundation.org> wrote:
>
> On Mon, Apr 25, 2022 at 1:10 AM Naresh Kamboju
> <naresh.kamboju@linaro.org> wrote:
> >
> > # CONFIG_MMU is not set
>
> I just sent the stupid and straightforward patch in
>
>   https://lore.kernel.org/all/CAHk-=wi5DYKbFE4j-jC2HGsKVuf1RpZbEiYt4tSXuxGKiN9oJg@mail.gmail.com/
>
> but then I thought maybe the alias patch is as simple as the attached.
>
> So does either (or both) of these patches work?

Yes.
I have tested these two patches and both the way the build passes.

Tested-by: Linux Kernel Functional Testing <lkft@linaro.org>

- Naresh
>
>                   Linus