Skip to contentSkip to conversation

package/qemu: disable Qemu user-land emulation for musl toolchains

PATCH New Romain Naour 2 participants 2 messages Nov 6, 2016

lore
Share & tools
lore

Romain Naour

author
#0 lore raw

PATCH v2 1/1 package/qemu: disable Qemu user-land emulation for musl toolchains

Like for cyclictest (rt-test)[1], linux-user/syscall.c use the definition of the "struct sigevent" from Glibc/uClibc which doesn't build with musl libc.

So, disable Qemu user-land emulation for musl toolchains.

Fixes:
http://autobuild.buildroot.net/results/b27/b277e668d9e22b624416bf46f6ccba802acd257e

[1] 43bffa59e7359610c8f00038ddd97795c0ca35e1

Signed-off-by Romain Naour <[email protected]>
Cc Francois Perrad <[email protected]>

---
v2: add a comment for BR2_TOOLCHAIN_USES_MUSL case (Arnout)

---
 package/qemu/Config.in | 5 +++++
 1 file changed, 5 insertions(+)
package/qemu/Config.in +5 -0
index ea6b946..8bae5f4 100644
@@ -69,11 +69,16 @@ comment "systems emulation needs a toolchain w/ dynamic library"
 
 config BR2_PACKAGE_QEMU_LINUX_USER
 	bool "Enable all Linux user-land emulation"
+	# Incompatible "struct sigevent" definition on musl
+	depends on !BR2_TOOLCHAIN_USES_MUSL
 	help
 	  Say 'y' to build all Linux user-land emulators that QEMU supports.
 
 # Note: bsd-user can not be build on Linux
 
+comment "Linux user-land emulation needs a glibc or uClibc toolchain"
+	depends on BR2_TOOLCHAIN_USES_MUSL
+
 endif # BR2_PACKAGE_QEMU_CUSTOM_TARGETS == ""
 
 config BR2_PACKAGE_QEMU_HAS_EMULS
Signature · 1 line
2.5.5
Applied
#1 lore raw

PATCH v2 1/1 package/qemu: disable Qemu user-land emulation for musl toolchains

Hello,

Quoted 11 lines · Romain Naour

Like for cyclictest (rt-test)[1], linux-user/syscall.c use the definition of the "struct sigevent" from Glibc/uClibc which doesn't build with musl libc.

So, disable Qemu user-land emulation for musl toolchains.

Fixes:
http://autobuild.buildroot.net/results/b27/b277e668d9e22b624416bf46f6ccba802acd257e

[1] 43bffa59e7359610c8f00038ddd97795c0ca35e1

Quoted 7 lines
Signed-off-by Romain Naour <[email protected]>
Cc Francois Perrad <[email protected]>

---
v2: add a comment for BR2_TOOLCHAIN_USES_MUSL case (Arnout)

---
 package/qemu/Config.in | 5 +++++
 1 file changed, 5 insertions(+)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com