diff mbox

ARC: disable packages that require atomic builtins

Message ID 1404742914-21583-1-git-send-email-abrodkin@synopsys.com
State Rejected
Headers show

Commit Message

Alexey Brodkin July 7, 2014, 2:21 p.m. UTC
In particular libtirpc is reported to be broken here: http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3

"rpcbind" in its turn selects "libtirpc", so it's required to disable it as well

As it is seen from http://autobuild.buildroot.org/results/3bd/3bda860a269dd9a4e38533cc9350892c68102618
"zeromq" also uses "___sync_add_and_fetch_2" & "___sync_fetch_and_add_2" in
"libpgm" so need to disable it.

And finally "zyre" selects "zeromq" so we disable it for ARC too.

signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>

Cc: Anton Kolesov <akolesov@synopsys.com>
Cc: Gustavo Zacarias <gustavo@zacarias.com.ar>
Cc: Peter Korsgaard <peter@korsgaard.com>
CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/libtirpc/Config.in | 2 ++
 package/rpcbind/Config.in  | 2 ++
 package/zeromq/Config.in   | 1 +
 package/zyre/Config.in     | 2 ++
 4 files changed, 7 insertions(+)

Comments

Arnout Vandecappelle July 7, 2014, 3:52 p.m. UTC | #1
On 07/07/14 16:21, Alexey Brodkin wrote:
> In particular libtirpc is reported to be broken here: http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3
> 
> "rpcbind" in its turn selects "libtirpc", so it's required to disable it as well

 What about the packages that select libtirpc if !BR2_TOOLCHAIN_HAS_NATIVE_RPC?
argus, conntrack-tools, lmbench, nfs-utils, quota

> 
> As it is seen from http://autobuild.buildroot.org/results/3bd/3bda860a269dd9a4e38533cc9350892c68102618
> "zeromq" also uses "___sync_add_and_fetch_2" & "___sync_fetch_and_add_2" in
> "libpgm" so need to disable it.
> 
> And finally "zyre" selects "zeromq" so we disable it for ARC too.

 What about the other packages that use zeromq? cppzmq, czmq, filemq, mongrel2,
php-zmq, python-pyzmq, zmqpp?

 Because this is becoming a lot of changes, it's probably better to split into
two patches:

tirpc: disable on ARC

zeromq: disable on ARC


> 
> signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>

 Signed-off-by should start with capital S (preferably use "git commit -s").


 Regards,
 Arnout

[snip]
Alexey Brodkin July 9, 2014, 5:19 a.m. UTC | #2
Hi Arnout,


On Mon, 2014-07-07 at 17:52 +0200, Arnout Vandecappelle wrote:
>  What about the other packages that use zeromq? cppzmq, czmq, filemq, mongrel2,
> php-zmq, python-pyzmq, zmqpp?

I see that dependencies list is getting longer and longer.

In coming days we're about to release a new version of tools for ARC and
as a part of this upcoming release we'll enable atomic operations by
default for ARC. This should immediately fix all those previously broken
packages.

So I'd prefer to wait for mentioned update and if it works as we expect
there will be no need in fixing more packages... moreover we'll need to
re-enable packages we already disabled like "jack2", "msgpack" and some
others.

Regards,
Alexey
Peter Korsgaard July 9, 2014, 7:12 a.m. UTC | #3
>>>>> "Alexey" == Alexey Brodkin <Alexey.Brodkin@synopsys.com> writes:

Hi,

 > I see that dependencies list is getting longer and longer.

 > In coming days we're about to release a new version of tools for ARC and
 > as a part of this upcoming release we'll enable atomic operations by
 > default for ARC. This should immediately fix all those previously broken
 > packages.

 > So I'd prefer to wait for mentioned update and if it works as we expect
 > there will be no need in fixing more packages... moreover we'll need to
 > re-enable packages we already disabled like "jack2", "msgpack" and some
 > others.

Ok, that sounds great!
diff mbox

Patch

diff --git a/package/libtirpc/Config.in b/package/libtirpc/Config.in
index a8e6dd3..4fb3eeb 100644
--- a/package/libtirpc/Config.in
+++ b/package/libtirpc/Config.in
@@ -6,6 +6,7 @@  config BR2_PACKAGE_LIBTIRPC
 	# dependencies of libtirpc for which native RPC support can be
 	# used instead of libtirpc.
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
+	depends on !BR2_arc # atomic builtins
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	help
 	  Libtirpc is a port of Sun's Transport-Independent RPC
@@ -16,3 +17,4 @@  config BR2_PACKAGE_LIBTIRPC
 comment "libtirpc needs a toolchain w/ threads"
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_arc
diff --git a/package/rpcbind/Config.in b/package/rpcbind/Config.in
index 2a79bdc..136553a 100644
--- a/package/rpcbind/Config.in
+++ b/package/rpcbind/Config.in
@@ -7,6 +7,7 @@  config BR2_PACKAGE_RPCBIND
 	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
 	# support, needed for libtirpc
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
+	depends on !BR2_arc # atomic builtins @libtirpc
 	# fork()
 	depends on BR2_USE_MMU
 	help
@@ -16,4 +17,5 @@  config BR2_PACKAGE_RPCBIND
 comment "rpcbind needs a toolchain w/ threads"
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
+	depends on !BR2_arc
 	depends on BR2_USE_MMU
diff --git a/package/zeromq/Config.in b/package/zeromq/Config.in
index 778a503..5382c98 100644
--- a/package/zeromq/Config.in
+++ b/package/zeromq/Config.in
@@ -34,6 +34,7 @@  config BR2_PACKAGE_ZEROMQ_PGM
 	depends on !BR2_avr32 # openpgm
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2013R1 # openpgm
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 # openpgm
+	depends on !BR2_arc # atomic builtins
 	select BR2_PACKAGE_OPENPGM
 	help
 	  Add support for Pragmatic General Multicast protocol (RFC 3208)
diff --git a/package/zyre/Config.in b/package/zyre/Config.in
index 86fff35..d955b0e 100644
--- a/package/zyre/Config.in
+++ b/package/zyre/Config.in
@@ -9,6 +9,7 @@  config BR2_PACKAGE_ZYRE
 	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
 	# support, needed by czmq
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
+	depends on !BR2_arc # atomic builtins @zeromq
 	select BR2_PACKAGE_CZMQ
 	select BR2_PACKAGE_ZEROMQ
 	help
@@ -19,6 +20,7 @@  config BR2_PACKAGE_ZYRE
 
 comment "zyre needs a toolchain w/ C++, IPv6, largefile, wchar, threads"
 	depends on BR2_USE_MMU
+	depends on !BR2_arc
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
 	depends on !(BR2_INSTALL_LIBSTDCPP && BR2_INET_IPV6 && BR2_LARGEFILE \
 		&& BR2_USE_WCHAR && BR2_TOOLCHAIN_HAS_THREADS)