Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Randy Dunlap]

On 5/3/19 5:54 AM, kbuild test robot wrote:
> Hi Ming,
> 
> First bad commit (maybe != root cause):
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   ea9866793d1e925b4d320eaea409263b2a568f38
> commit: 6dc4f100c175dd0511ae8674786e7c9006cdfbfa block: allow bio_for_each_segment_all() to iterate over multi-page bvec
> date:   3 months ago
> config: ia64-bigsur_defconfig (attached as .config)
> compiler: ia64-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
>         git checkout 6dc4f100c175dd0511ae8674786e7c9006cdfbfa
>         # save the attached .config to linux build tree
>         GCC_VERSION=8.1.0 make.cross ARCH=ia64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All errors (new ones prefixed by >>):
> 
>    ERROR: "paddr_to_nid" [sound/core/snd-pcm.ko] undefined!
>    ERROR: "paddr_to_nid" [net/sunrpc/sunrpc.ko] undefined!
>    ERROR: "paddr_to_nid" [fs/cifs/cifs.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/video/fbdev/core/fb.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/usb/mon/usbmon.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/usb/core/usbcore.ko] undefined!
>>> ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/md/dm-mod.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/md/dm-crypt.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/md/dm-bufio.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/ide/ide-core.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/ide/ide-cd_mod.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/gpu/drm/drm.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/char/agp/agpgart.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/block/nbd.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/block/loop.ko] undefined!
>    ERROR: "paddr_to_nid" [drivers/block/brd.ko] undefined!
>    ERROR: "paddr_to_nid" [crypto/ccm.ko] undefined!
> 

---
Exporting paddr_to_nid() in arch/ia64/mm/numa.c fixes all of these build errors.
Is there a problem with doing that?


-- 
~Randy

RE: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Luck, Tony]

From: Randy Dunlap [mailto:rdunlap@infradead.org] 
>>    ERROR: "paddr_to_nid" [drivers/block/brd.ko] undefined!
>>    ERROR: "paddr_to_nid" [crypto/ccm.ko] undefined!
>> 
>
> ---
> Exporting paddr_to_nid() in arch/ia64/mm/numa.c fixes all of these build errors.
> Is there a problem with doing that?

I don't see a problem with exporting it.

-Tony

RE: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Luck, Tony]

>> Exporting paddr_to_nid() in arch/ia64/mm/numa.c fixes all of these build errors.
>> Is there a problem with doing that?
>
> I don't see a problem with exporting it.

But I also don't see these build errors.  I'm using the same HEAD commit. I think the
same .config (derived from arch/ia64/configs/bigsur_defconfig.

Big difference is I'm doing a natinve build with a much older compiler (4.6.4)

-Tony

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Randy Dunlap]

On 5/3/19 1:25 PM, Luck, Tony wrote:
>>> Exporting paddr_to_nid() in arch/ia64/mm/numa.c fixes all of these build errors.
>>> Is there a problem with doing that?
>>
>> I don't see a problem with exporting it.
> 
> But I also don't see these build errors.  I'm using the same HEAD commit. I think the
> same .config (derived from arch/ia64/configs/bigsur_defconfig.
> 
> Big difference is I'm doing a natinve build with a much older compiler (4.6.4)
> 
> -Tony
> 

I was able to reproduce the build errors.
using gcc 8.1.0 cross-compile.

-- 
~Randy

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[John Paul Adrian Glaubitz]

Hi Tony!

On 5/3/19 10:25 PM, Luck, Tony wrote:
> Big difference is I'm doing a natinve build with a much older compiler (4.6.4)

Just as a heads-up: There are updated installation images available
for Debian unstable for ia64 [1]. Those install a fresh system using GRUB
as a bootloader and come with the latest versions of the toolchain.

Please keep in mind that the kernel is currently unbootable on some machines
due to a miscompiled kernel due to a bug in gcc-8 [2]. However, there is already
a fix for this particular gcc bug and it has also been backported to gcc-8 as
of yesterday, so we're just waiting for Debian's gcc-8 package to be updated
so we can rebuild the kernel packages and the installation image.

The network mirror and remote directory which are being prompted for during
installation are:

- Mirror: http://ftp.ports.debian.org
- Directory: debian-ports

Thanks,
Adrian

> [1] https://cdimage.debian.org/cdimage/ports/grub-test/
> [2] https://gcc.gnu.org/bugzilla/show_bug.cgi?id†984

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaubitz@debian.org
`. `'   Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Christoph Hellwig]

On Sat, May 04, 2019 at 07:47:08AM +0200, John Paul Adrian Glaubitz wrote:
> Hi Tony!
> 
> On 5/3/19 10:25 PM, Luck, Tony wrote:
> > Big difference is I'm doing a natinve build with a much older compiler (4.6.4)
> 
> Just as a heads-up: There are updated installation images available
> for Debian unstable for ia64 [1]. Those install a fresh system using GRUB
> as a bootloader and come with the latest versions of the toolchain.

Do we also have an x86 to ia64 cross compiler package for Debian?
Those packages really help me with changes to other architectures, and
the lack of them for ia64 makes compile-testing ia64 code rather painful.

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[John Paul Adrian Glaubitz]

On 5/4/19 8:27 AM, Christoph Hellwig wrote:
>> Just as a heads-up: There are updated installation images available
>> for Debian unstable for ia64 [1]. Those install a fresh system using GRUB
>> as a bootloader and come with the latest versions of the toolchain.
> 
> Do we also have an x86 to ia64 cross compiler package for Debian?
> Those packages really help me with changes to other architectures, and
> the lack of them for ia64 makes compile-testing ia64 code rather painful.

Hmm, I just realized that building the cross-compiler for ia64 is disabled
at the moment because we were having issues with libunwind back then. But
normally, on Debian and Ubuntu, you can just install the compiler with:

# apt install gcc-8-$ARCH-linux-gnu

And that should also be the case for ia64, hence I will look into fixing this
as it should have been long time ago. Thanks for the reminder.

Adrian

-- 
 .''`.  John Paul Adrian Glaubitz
: :' :  Debian Developer - glaubitz@debian.org
`. `'   Freie Universitaet Berlin - glaubitz@physik.fu-berlin.de
  `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913

Re: ERROR: "paddr_to_nid" [drivers/md/raid1.ko] undefined!

[Christoph Hellwig]

On Sat, May 04, 2019 at 08:33:53AM +0200, John Paul Adrian Glaubitz wrote:
> On 5/4/19 8:27 AM, Christoph Hellwig wrote:
> >> Just as a heads-up: There are updated installation images available
> >> for Debian unstable for ia64 [1]. Those install a fresh system using GRUB
> >> as a bootloader and come with the latest versions of the toolchain.
> > 
> > Do we also have an x86 to ia64 cross compiler package for Debian?
> > Those packages really help me with changes to other architectures, and
> > the lack of them for ia64 makes compile-testing ia64 code rather painful.
> 
> Hmm, I just realized that building the cross-compiler for ia64 is disabled
> at the moment because we were having issues with libunwind back then. But
> normally, on Debian and Ubuntu, you can just install the compiler with:
> 
> # apt install gcc-8-$ARCH-linux-gnu
> 
> And that should also be the case for ia64, hence I will look into fixing this
> as it should have been long time ago. Thanks for the reminder.

Yes, and I've done that for all other architectures available, it is
just ia64 that has been missing.