All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option
@ 2016-08-01 16:30 Romain Naour
  2016-08-01 16:30 ` [Buildroot] [PATCH 2/3] package/xenomai: disable for cortex-M cpus Romain Naour
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Romain Naour @ 2016-08-01 16:30 UTC (permalink / raw)
  To: buildroot

Introduce BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS to hold the
architectures than can build and use Xenomai on the target.

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
 package/xenomai/Config.in | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/package/xenomai/Config.in b/package/xenomai/Config.in
index d4c8a89..a4a2795 100644
--- a/package/xenomai/Config.in
+++ b/package/xenomai/Config.in
@@ -1,13 +1,17 @@
-comment "xenomai needs a toolchain w/ threads"
-	depends on !BR2_TOOLCHAIN_HAS_THREADS
+config BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
+	bool
+	default y
 	depends on BR2_i386 || BR2_x86_64 || BR2_arm || \
 		   BR2_bfin || BR2_powerpc || BR2_sh4
 
+comment "xenomai needs a toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
+
 config BR2_PACKAGE_XENOMAI
 	bool "Xenomai Userspace"
+	depends on BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_THREADS
-	depends on BR2_i386 || BR2_x86_64 || BR2_arm || \
-		BR2_bfin || BR2_powerpc || BR2_sh4
 	help
 	  Real-Time Framework for Linux
 	  http://www.xenomai.org
-- 
2.5.5

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

* [Buildroot] [PATCH 2/3] package/xenomai: disable for cortex-M cpus
  2016-08-01 16:30 [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Romain Naour
@ 2016-08-01 16:30 ` Romain Naour
  2016-08-01 16:30 ` [Buildroot] [PATCH 3/3] package/xenomai: disable for musl toolchains Romain Naour
  2016-08-01 20:09 ` [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Thomas Petazzoni
  2 siblings, 0 replies; 4+ messages in thread
From: Romain Naour @ 2016-08-01 16:30 UTC (permalink / raw)
  To: buildroot

Following include/asm-arm/features.h, Xenomai doesn't support cortex-M
cpus.

error "Could not find current ARM architecture"

Fixes:
http://autobuild.buildroot.net/results/06a/06a52b2c2c23df595f58feb6b95437fc65a86d31

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
 package/xenomai/Config.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/xenomai/Config.in b/package/xenomai/Config.in
index a4a2795..3de7bde 100644
--- a/package/xenomai/Config.in
+++ b/package/xenomai/Config.in
@@ -1,8 +1,8 @@
 config BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
 	bool
 	default y
-	depends on BR2_i386 || BR2_x86_64 || BR2_arm || \
-		   BR2_bfin || BR2_powerpc || BR2_sh4
+	depends on BR2_i386 || BR2_x86_64 || (BR2_arm && !BR2_ARM_CPU_ARMV7M) || \
+		BR2_bfin || BR2_powerpc || BR2_sh4
 
 comment "xenomai needs a toolchain w/ threads"
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
-- 
2.5.5

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

* [Buildroot] [PATCH 3/3] package/xenomai: disable for musl toolchains
  2016-08-01 16:30 [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Romain Naour
  2016-08-01 16:30 ` [Buildroot] [PATCH 2/3] package/xenomai: disable for cortex-M cpus Romain Naour
@ 2016-08-01 16:30 ` Romain Naour
  2016-08-01 20:09 ` [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Thomas Petazzoni
  2 siblings, 0 replies; 4+ messages in thread
From: Romain Naour @ 2016-08-01 16:30 UTC (permalink / raw)
  To: buildroot

After fixing the <bits/local_lim.h> issue by using <limits.h>, the
build still fail with udefined __WORDSIZE.

Adding <sys/user.h> to define __WORDSIZE reveal that xenomai use
<error.h> which is not provided by musl.

The discussion on the musl mailing list [1] about glibc error reporting
functions concluded to not add this support in the C library.

For now, disable Xenomai user space support for musl toolchains since
it require several patches to build correctly and needs to remove each
glibc error reporting functions.

Users interested in musl support for Xenomai can work with upstream to
fix these issues and revert this patch.

[1] http://www.openwall.com/lists/musl/2014/06/29/8

Fixes:
http://autobuild.buildroot.net/results/e36/e36a21c1df33bdd3fbc61d516a3e8f7c5f7c41af

Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
 package/xenomai/Config.in | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/package/xenomai/Config.in b/package/xenomai/Config.in
index 3de7bde..32c5124 100644
--- a/package/xenomai/Config.in
+++ b/package/xenomai/Config.in
@@ -4,14 +4,16 @@ config BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
 	depends on BR2_i386 || BR2_x86_64 || (BR2_arm && !BR2_ARM_CPU_ARMV7M) || \
 		BR2_bfin || BR2_powerpc || BR2_sh4
 
-comment "xenomai needs a toolchain w/ threads"
-	depends on !BR2_TOOLCHAIN_HAS_THREADS
+comment "xenomai needs an glibc or uClibc toolchain w/ threads"
+	depends on !BR2_TOOLCHAIN_HAS_THREADS || BR2_TOOLCHAIN_USES_MUSL
 	depends on BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
 
 config BR2_PACKAGE_XENOMAI
 	bool "Xenomai Userspace"
 	depends on BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	# uses <error.h>, __WORDSIZE and bits/local_lim.h
+	depends on !BR2_TOOLCHAIN_USES_MUSL
 	help
 	  Real-Time Framework for Linux
 	  http://www.xenomai.org
-- 
2.5.5

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

* [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option
  2016-08-01 16:30 [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Romain Naour
  2016-08-01 16:30 ` [Buildroot] [PATCH 2/3] package/xenomai: disable for cortex-M cpus Romain Naour
  2016-08-01 16:30 ` [Buildroot] [PATCH 3/3] package/xenomai: disable for musl toolchains Romain Naour
@ 2016-08-01 20:09 ` Thomas Petazzoni
  2 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2016-08-01 20:09 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon,  1 Aug 2016 18:30:19 +0200, Romain Naour wrote:
> Introduce BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS to hold the
> architectures than can build and use Xenomai on the target.
> 
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
> ---
>  package/xenomai/Config.in | 12 ++++++++----
>  1 file changed, 8 insertions(+), 4 deletions(-)

Series applied, thanks!

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

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

end of thread, other threads:[~2016-08-01 20:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-01 16:30 [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Romain Naour
2016-08-01 16:30 ` [Buildroot] [PATCH 2/3] package/xenomai: disable for cortex-M cpus Romain Naour
2016-08-01 16:30 ` [Buildroot] [PATCH 3/3] package/xenomai: disable for musl toolchains Romain Naour
2016-08-01 20:09 ` [Buildroot] [PATCH 1/3] package/xenomai: introduce a BR2_PACKAGE_XENOMAI_ARCH_SUPPORTS option Thomas Petazzoni

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.