All of lore.kernel.org
 help / color / mirror / Atom feed
* Problems with arm64 compat vdso
@ 2019-09-20 14:27 Will Deacon
  2019-09-20 16:33 ` Vincenzo Frascino
                   ` (3 more replies)
  0 siblings, 4 replies; 127+ messages in thread
From: Will Deacon @ 2019-09-20 14:27 UTC (permalink / raw)
  To: vincenzo.frascino
  Cc: linux-kernel, linux-kernel, tglx, linux, ard.biesheuvel, catalin.marinas

Hi Vincenzo,

I've been running into a few issues with the COMPAT vDSO. Please could
you have a look?

If I do the following:

$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- defconfig
[...]
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- menuconfig
[set CONFIG_CROSS_COMPILE_COMPAT_VDSO="arm-linux-gnueabihf-"]
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu-

Then I see the following warning:

arch/arm64/Makefile:62: CROSS_COMPILE_COMPAT not defined or empty, the compat vDSO will not be built

even though the compat vDSO *has* been built:

$ file arch/arm64/kernel/vdso32/vdso.so
arch/arm64/kernel/vdso32/vdso.so: ELF 32-bit LSB pie executable, ARM, EABI5 version 1 (SYSV), dynamically linked, BuildID[sha1]=c67f6c786f2d2d6f86c71f708595594aa25247f6, stripped

However, I also get some warnings because arm64 headers are being included
in the compat vDSO build:

In file included from ./arch/arm64/include/asm/thread_info.h:17:0,
                 from ./include/linux/thread_info.h:38,
                 from ./arch/arm64/include/asm/preempt.h:5,
                 from ./include/linux/preempt.h:78,
                 from ./include/linux/spinlock.h:51,
                 from ./include/linux/seqlock.h:36,
                 from ./include/linux/time.h:6,
                 from /usr/local/google/home/willdeacon/work/linux/lib/vdso/gettimeofday.c:7,
                 from <command-line>:0:
./arch/arm64/include/asm/memory.h: In function ‘__tag_set’:
./arch/arm64/include/asm/memory.h:233:15: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  u64 __addr = (u64)addr & ~__tag_shifted(0xff);
               ^
In file included from ./arch/arm64/include/asm/pgtable-hwdef.h:8:0,
                 from ./arch/arm64/include/asm/processor.h:34,
                 from ./arch/arm64/include/asm/elf.h:118,
                 from ./include/linux/elf.h:5,
                 from ./include/linux/elfnote.h:62,
                 from arch/arm64/kernel/vdso32/note.c:11:
./arch/arm64/include/asm/memory.h: In function ‘__tag_set’:
./arch/arm64/include/asm/memory.h:233:15: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  u64 __addr = (u64)addr & ~__tag_shifted(0xff);
               ^
Worse, if your compat binutils isn't up-to-date, you'll actually run into
a build failure:

/tmp/ccFCrjUg.s:80: Error: invalid barrier type -- `dmb ishld'
/tmp/ccFCrjUg.s:124: Error: invalid barrier type -- `dmb ishld'

There also appears to be a problem getting the toolchain prefix from Kconfig.
If, for example, I do:

$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- defconfig
[...]
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- menuconfig
[set CONFIG_CROSS_COMPILE_COMPAT_VDSO="vincenzo"]
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu-
arch/arm64/Makefile:64: *** vincenzogcc not found, check CROSS_COMPILE_COMPAT.  Stop.
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- menuconfig
[set CONFIG_CROSS_COMPILE_COMPAT_VDSO="arm-linux-gnueabihf-"]
$ make ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu-
arch/arm64/Makefile:64: *** vincenzogcc not found, check CROSS_COMPILE_COMPAT.  Stop.
$ grep CONFIG_CROSS_COMPILE_COMPAT_VDSO .config
CONFIG_CROSS_COMPILE_COMPAT_VDSO="arm-linux-gnueabihf-"

which is irritating, because it seems to force a 'mrproper' if you don't
get the prefix right first time.

Cheers,

Will

^ permalink raw reply	[flat|nested] 127+ messages in thread

end of thread, other threads:[~2019-10-01 17:14 UTC | newest]

Thread overview: 127+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-20 14:27 Problems with arm64 compat vdso Will Deacon
2019-09-20 16:33 ` Vincenzo Frascino
2019-09-23 16:41   ` Ard Biesheuvel
2019-09-24 12:11     ` Vincenzo Frascino
2019-09-24 18:06       ` Nick Desaulniers
2019-09-24 20:42         ` Vincenzo Frascino
2019-09-25 15:58     ` Ard Biesheuvel
2019-09-26  6:03 ` [PATCH 0/4] arm64: vdso32: Address various issues Vincenzo Frascino
2019-09-26  6:03   ` Vincenzo Frascino
2019-09-26  6:03   ` [PATCH 1/4] arm64: vdso32: Introduce COMPAT_CC_IS_GCC Vincenzo Frascino
2019-09-26  6:03     ` Vincenzo Frascino
2019-09-26  8:06     ` Catalin Marinas
2019-09-26  8:06       ` Catalin Marinas
2019-09-26 10:56       ` Vincenzo Frascino
2019-09-26 10:56         ` Vincenzo Frascino
2019-09-26 11:02         ` Vincenzo Frascino
2019-09-26 11:02           ` Vincenzo Frascino
2019-09-26 15:23           ` Catalin Marinas
2019-09-26 15:23             ` Catalin Marinas
2019-09-26 12:22         ` Vincenzo Frascino
2019-09-26 12:22           ` Vincenzo Frascino
2019-09-26 15:17           ` Catalin Marinas
2019-09-26 15:17             ` Catalin Marinas
2019-09-26 20:17             ` Vincenzo Frascino
2019-09-26 20:17               ` Vincenzo Frascino
2019-09-26  6:03   ` [PATCH 2/4] arm64: vdso32: Detect binutils support for dmb ishld Vincenzo Frascino
2019-09-26  6:03     ` Vincenzo Frascino
2019-09-26  8:30     ` Catalin Marinas
2019-09-26  8:30       ` Catalin Marinas
2019-09-26 10:59       ` Vincenzo Frascino
2019-09-26 10:59         ` Vincenzo Frascino
2019-09-26  6:03   ` [PATCH 3/4] arm64: vdso32: Fix compilation warning Vincenzo Frascino
2019-09-26  6:03     ` Vincenzo Frascino
2019-09-26  8:32     ` Catalin Marinas
2019-09-26  8:32       ` Catalin Marinas
2019-09-26 11:03       ` Vincenzo Frascino
2019-09-26 11:03         ` Vincenzo Frascino
2019-09-26  6:03   ` [PATCH 4/4] arm64: Remove gettimeofday.S Vincenzo Frascino
2019-09-26  6:03     ` Vincenzo Frascino
2019-09-26  8:34     ` Catalin Marinas
2019-09-26  8:34       ` Catalin Marinas
2019-09-26 13:38 ` [PATCH v2 0/4] arm64: vdso32: Address various issues Vincenzo Frascino
2019-09-26 13:38   ` Vincenzo Frascino
2019-09-26 13:38   ` [PATCH v2 1/4] arm64: vdso32: Introduce COMPAT_CC_IS_GCC Vincenzo Frascino
2019-09-26 13:38     ` Vincenzo Frascino
2019-09-26 15:13     ` Catalin Marinas
2019-09-26 15:13       ` Catalin Marinas
2019-09-26 13:38   ` [PATCH v2 2/4] arm64: vdso32: Detect binutils support for dmb ishld Vincenzo Frascino
2019-09-26 13:38     ` Vincenzo Frascino
2019-09-26 14:26     ` Catalin Marinas
2019-09-26 14:26       ` Catalin Marinas
2019-09-26 20:19       ` Vincenzo Frascino
2019-09-26 20:19         ` Vincenzo Frascino
2019-09-26 15:59     ` Catalin Marinas
2019-09-26 15:59       ` Catalin Marinas
2019-09-26 21:17       ` Vincenzo Frascino
2019-09-26 21:17         ` Vincenzo Frascino
2019-09-26 16:23     ` Nick Desaulniers
2019-09-26 16:23       ` Nick Desaulniers
2019-09-26 13:38   ` [PATCH v2 3/4] arm64: vdso32: Fix compilation warning Vincenzo Frascino
2019-09-26 13:38     ` Vincenzo Frascino
2019-09-26 15:22     ` Catalin Marinas
2019-09-26 15:22       ` Catalin Marinas
2019-09-26 13:38   ` [PATCH v2 4/4] arm64: Remove gettimeofday.S Vincenzo Frascino
2019-09-26 13:38     ` Vincenzo Frascino
2019-09-26 15:31   ` [PATCH v2 0/4] arm64: vdso32: Address various issues Catalin Marinas
2019-09-26 15:31     ` Catalin Marinas
2019-09-26 20:29     ` Vincenzo Frascino
2019-09-26 20:29       ` Vincenzo Frascino
2019-09-26 21:43 ` [PATCH v3 0/5]arm64: " Vincenzo Frascino
2019-09-26 21:43   ` Vincenzo Frascino
2019-09-26 21:43   ` [PATCH v3 1/5] arm64: vdso32: Introduce COMPAT_CC_IS_GCC Vincenzo Frascino
2019-09-26 21:43     ` Vincenzo Frascino
2019-09-27  9:24     ` Catalin Marinas
2019-09-27  9:24       ` Catalin Marinas
2019-10-01 13:14     ` Will Deacon
2019-10-01 13:14       ` Will Deacon
2019-10-01 13:27       ` Catalin Marinas
2019-10-01 13:27         ` Catalin Marinas
2019-10-01 14:20         ` Vincenzo Frascino
2019-10-01 14:20           ` Vincenzo Frascino
2019-10-01 14:20           ` Will Deacon
2019-10-01 14:20             ` Will Deacon
2019-10-01 14:37             ` Vincenzo Frascino
2019-10-01 14:37               ` Vincenzo Frascino
2019-10-01 14:43               ` Will Deacon
2019-10-01 14:43                 ` Will Deacon
2019-10-01 14:59                 ` Vincenzo Frascino
2019-10-01 14:59                   ` Vincenzo Frascino
2019-10-01 15:30                 ` Catalin Marinas
2019-10-01 15:30                   ` Catalin Marinas
2019-10-01 16:46                   ` Will Deacon
2019-10-01 16:46                     ` Will Deacon
2019-10-01 16:59                     ` Nick Desaulniers
2019-10-01 16:59                       ` Nick Desaulniers
2019-10-01 17:07                       ` Will Deacon
2019-10-01 17:07                         ` Will Deacon
2019-10-01 17:12                         ` Nick Desaulniers
2019-10-01 17:12                           ` Nick Desaulniers
2019-10-01 17:14                           ` Will Deacon
2019-10-01 17:14                             ` Will Deacon
2019-09-26 21:43   ` [PATCH v3 2/5] arm64: vdso32: Detect binutils support for dmb ishld Vincenzo Frascino
2019-09-26 21:43     ` Vincenzo Frascino
2019-09-27  9:25     ` Catalin Marinas
2019-09-27  9:25       ` Catalin Marinas
2019-10-01 13:26     ` Will Deacon
2019-10-01 13:26       ` Will Deacon
2019-10-01 14:23       ` Vincenzo Frascino
2019-10-01 14:23         ` Vincenzo Frascino
2019-09-26 21:43   ` [PATCH v3 3/5] arm64: vdso32: Fix compilation warning Vincenzo Frascino
2019-09-26 21:43     ` Vincenzo Frascino
2019-09-27  9:26     ` Catalin Marinas
2019-09-27  9:26       ` Catalin Marinas
2019-10-01 13:21     ` Will Deacon
2019-10-01 13:21       ` Will Deacon
2019-10-01 14:41       ` Vincenzo Frascino
2019-10-01 14:41         ` Vincenzo Frascino
2019-09-26 21:43   ` [PATCH v3 4/5] arm64: Remove gettimeofday.S Vincenzo Frascino
2019-09-26 21:43     ` Vincenzo Frascino
2019-09-26 21:43   ` [PATCH v3 5/5] arm64: vdso32: Remove jump label config option in Makefile Vincenzo Frascino
2019-09-26 21:43     ` Vincenzo Frascino
2019-09-27  9:27     ` Catalin Marinas
2019-09-27  9:27       ` Catalin Marinas
2019-09-27 10:46   ` [PATCH v3 0/5]arm64: vdso32: Address various issues Catalin Marinas
2019-09-27 10:46     ` Catalin Marinas
2019-10-01 13:27   ` Will Deacon
2019-10-01 13:27     ` Will Deacon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.