All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 0/2] add more dependencies on atomic operations
@ 2014-09-08  6:32 Alexey Brodkin
  2014-09-08  6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin
  2014-09-08  6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin
  0 siblings, 2 replies; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-08  6:32 UTC (permalink / raw)
  To: buildroot

More packages were discovered to use atomic operatoins.
With this series dependencies are applied to libraries that use atomic
operations and packages that explicitly select mentioned libraries.

Alexey Brodkin (2):
  libtirpc: add dependency on atomic operations
  zeromq: add dependency on atomic operations

 package/argus/Config.in           | 1 +
 package/conntrack-tools/Config.in | 1 +
 package/cppzmq/Config.in          | 1 +
 package/czmq/Config.in            | 1 +
 package/filemq/Config.in          | 1 +
 package/libtirpc/Config.in        | 1 +
 package/lmbench/Config.in         | 1 +
 package/mongrel2/Config.in        | 1 +
 package/nfs-utils/Config.in       | 1 +
 package/php-zmq/Config.in         | 1 +
 package/python-pyzmq/Config.in    | 1 +
 package/quota/Config.in           | 1 +
 package/rpcbind/Config.in         | 1 +
 package/zeromq/Config.in          | 1 +
 package/zmqpp/Config.in           | 1 +
 package/zyre/Config.in            | 1 +
 16 files changed, 16 insertions(+)

-- 
1.9.3

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-08  6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin
@ 2014-09-08  6:32 ` Alexey Brodkin
  2014-09-08  7:23   ` Thomas Petazzoni
  2014-09-08  6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin
  1 sibling, 1 reply; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-08  6:32 UTC (permalink / raw)
  To: buildroot

Fixes:
http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3
http://autobuild.buildroot.net/results/994a84d192848b90a70e28e5ebaa769701bcaec9

Following packages select "libtirpc", so adding dependency as well:
 * argus
 * conntrack-tools
 * lmbench
 * nfs-utils
 * quota
 * rpcbind

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/argus/Config.in           | 1 +
 package/conntrack-tools/Config.in | 1 +
 package/libtirpc/Config.in        | 1 +
 package/lmbench/Config.in         | 1 +
 package/nfs-utils/Config.in       | 1 +
 package/quota/Config.in           | 1 +
 package/rpcbind/Config.in         | 1 +
 7 files changed, 7 insertions(+)

diff --git a/package/argus/Config.in b/package/argus/Config.in
index 6d47734..2fab07a 100644
--- a/package/argus/Config.in
+++ b/package/argus/Config.in
@@ -1,6 +1,7 @@
 config BR2_PACKAGE_ARGUS
 	bool "argus"
 	depends on BR2_INET_IPV6
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_MMU # fork()
 	select BR2_PACKAGE_LIBPCAP
diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in
index ec19d8e..36b12b7 100644
--- a/package/conntrack-tools/Config.in
+++ b/package/conntrack-tools/Config.in
@@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS
 	depends on BR2_INET_IPV6
 	depends on BR2_LARGEFILE
 	depends on BR2_USE_MMU # fork()
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc
 	select BR2_PACKAGE_LIBNETFILTER_CONNTRACK
 	select BR2_PACKAGE_LIBNETFILTER_CTHELPER
diff --git a/package/libtirpc/Config.in b/package/libtirpc/Config.in
index a8e6dd3..a5fe175 100644
--- a/package/libtirpc/Config.in
+++ b/package/libtirpc/Config.in
@@ -7,6 +7,7 @@ config BR2_PACKAGE_LIBTIRPC
 	# used instead of libtirpc.
 	depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_ARCH_HAS_ATOMICS
 	help
 	  Libtirpc is a port of Sun's Transport-Independent RPC
 	  library to Linux.
diff --git a/package/lmbench/Config.in b/package/lmbench/Config.in
index c4ea522..5b88d62 100644
--- a/package/lmbench/Config.in
+++ b/package/lmbench/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_LMBENCH
 	bool "lmbench"
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc
 	select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC
 	depends on BR2_USE_MMU # fork()
diff --git a/package/nfs-utils/Config.in b/package/nfs-utils/Config.in
index dff5150..3168333 100644
--- a/package/nfs-utils/Config.in
+++ b/package/nfs-utils/Config.in
@@ -5,6 +5,7 @@ comment "nfs-utils needs a toolchain w/ largefile, threads"
 
 config BR2_PACKAGE_NFS_UTILS
 	bool "nfs-utils"
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc
 	select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC
 	depends on BR2_LARGEFILE
diff --git a/package/quota/Config.in b/package/quota/Config.in
index 69f03af..cceb2ab 100644
--- a/package/quota/Config.in
+++ b/package/quota/Config.in
@@ -4,6 +4,7 @@ config BR2_PACKAGE_QUOTA
 	depends on BR2_USE_WCHAR
 	depends on BR2_USE_MMU # fork()
 	depends on !(BR2_microblazeel || BR2_microblazebe) # util-linux
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc
 	select BR2_PACKAGE_UTIL_LINUX
 	select BR2_PACKAGE_UTIL_LINUX_BINARIES
diff --git a/package/rpcbind/Config.in b/package/rpcbind/Config.in
index 2a79bdc..bc658d6 100644
--- a/package/rpcbind/Config.in
+++ b/package/rpcbind/Config.in
@@ -2,6 +2,7 @@ config BR2_PACKAGE_RPCBIND
 	bool "rpcbind"
 	# We really need libtirpc and can't work with the native RPC
 	# implementation of toolchains.
+	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libtirpc
 	select BR2_PACKAGE_LIBTIRPC
 	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
-- 
1.9.3

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

* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations
  2014-09-08  6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin
  2014-09-08  6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin
@ 2014-09-08  6:32 ` Alexey Brodkin
  2014-09-08  7:23   ` Thomas Petazzoni
  1 sibling, 1 reply; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-08  6:32 UTC (permalink / raw)
  To: buildroot

As it is seen from http://autobuild.buildroot.org/results/3bd/3bda860a269dd9a4e38533cc9350892c68102618
"zeromq" uses "___sync_add_and_fetch_2" & "___sync_fetch_and_add_2" in "libpgm"
so make it dependent on atomic operations.

Following packages select "zeromq", so adding dependency as well:
 * cppzmq
 * czmq
 * filemq
 * mongrel2
 * php-zmq
 * python-pyzmq
 * zmqpp
 * zyre

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/cppzmq/Config.in       | 1 +
 package/czmq/Config.in         | 1 +
 package/filemq/Config.in       | 1 +
 package/mongrel2/Config.in     | 1 +
 package/php-zmq/Config.in      | 1 +
 package/python-pyzmq/Config.in | 1 +
 package/zeromq/Config.in       | 1 +
 package/zmqpp/Config.in        | 1 +
 package/zyre/Config.in         | 1 +
 9 files changed, 9 insertions(+)

diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in
index e17963e..9f32c51 100644
--- a/package/cppzmq/Config.in
+++ b/package/cppzmq/Config.in
@@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	select BR2_PACKAGE_ZEROMQ
 	help
 	  C++ binding for zeromq (ZeroMQ, 0MQ, zmq).
diff --git a/package/czmq/Config.in b/package/czmq/Config.in
index fbdedd1..b005a0e 100644
--- a/package/czmq/Config.in
+++ b/package/czmq/Config.in
@@ -7,6 +7,7 @@ config BR2_PACKAGE_CZMQ
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	help
 	  High-level C Binding for 0MQ
 
diff --git a/package/filemq/Config.in b/package/filemq/Config.in
index 1604846..0fea057 100644
--- a/package/filemq/Config.in
+++ b/package/filemq/Config.in
@@ -8,6 +8,7 @@ config BR2_PACKAGE_FILEMQ
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	depends on BR2_USE_MMU # czmq
 	help
 	  FileMQ is a publish-subscribe file service based on 0MQ.
diff --git a/package/mongrel2/Config.in b/package/mongrel2/Config.in
index 680e0ce..21e0706 100644
--- a/package/mongrel2/Config.in
+++ b/package/mongrel2/Config.in
@@ -14,6 +14,7 @@ config BR2_PACKAGE_MONGREL2
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
 	depends on BR2_LARGEFILE # zeromq -> util-linux
 	depends on BR2_USE_WCHAR # zeromq -> util-linux
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	# {get,make,swap}context functions present in
 	# {e,}glibc and in uClibc's master branch. Source has arm workaround
 	depends on BR2_UCLIBC_VERSION_SNAPSHOT || \
diff --git a/package/php-zmq/Config.in b/package/php-zmq/Config.in
index f32b5a3..ff34082 100644
--- a/package/php-zmq/Config.in
+++ b/package/php-zmq/Config.in
@@ -10,6 +10,7 @@ config BR2_PACKAGE_PHP_ZMQ
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	select BR2_PACKAGE_ZEROMQ
 	help
 	  ZeroMQ messaging bindings for PHP
diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in
index 0f868d0..b2a60c2 100644
--- a/package/python-pyzmq/Config.in
+++ b/package/python-pyzmq/Config.in
@@ -5,6 +5,7 @@ config BR2_PACKAGE_PYTHON_PYZMQ
 	depends on BR2_USE_WCHAR # zeromq
 	depends on BR2_INSTALL_LIBSTDCPP # zeromq
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	select BR2_PACKAGE_ZEROMQ
 	help
 	  This package contains the python language binding for zeromq.
diff --git a/package/zeromq/Config.in b/package/zeromq/Config.in
index 778a503..4178014 100644
--- a/package/zeromq/Config.in
+++ b/package/zeromq/Config.in
@@ -9,6 +9,7 @@ config BR2_PACKAGE_ZEROMQ
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS
+	depends on BR2_ARCH_HAS_ATOMICS
 	select BR2_PACKAGE_UTIL_LINUX
 	select BR2_PACKAGE_UTIL_LINUX_LIBUUID
 	help
diff --git a/package/zmqpp/Config.in b/package/zmqpp/Config.in
index 2ccaf7e..7740679 100644
--- a/package/zmqpp/Config.in
+++ b/package/zmqpp/Config.in
@@ -8,6 +8,7 @@ config BR2_PACKAGE_ZMQPP
 	depends on BR2_LARGEFILE # util-linux
 	depends on BR2_USE_WCHAR # util-linux
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	select BR2_PACKAGE_ZEROMQ
 	help
 	  C++ binding for zeromq (ZeroMQ, 0MQ, zmq).
diff --git a/package/zyre/Config.in b/package/zyre/Config.in
index 86fff35..2c32068 100644
--- a/package/zyre/Config.in
+++ b/package/zyre/Config.in
@@ -5,6 +5,7 @@ config BR2_PACKAGE_ZYRE
 	depends on BR2_LARGEFILE # zeromq
 	depends on BR2_USE_WCHAR # zeromq
 	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
+	depends on BR2_ARCH_HAS_ATOMICS # zeromq
 	depends on BR2_USE_MMU # czmq
 	# uClibc toolchains provided by ADI don't have AI_ADDRCONFIG
 	# support, needed by czmq
-- 
1.9.3

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-08  6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin
@ 2014-09-08  7:23   ` Thomas Petazzoni
  2014-09-08  9:23     ` Alexey Brodkin
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2014-09-08  7:23 UTC (permalink / raw)
  To: buildroot

Dear Alexey Brodkin,

On Mon,  8 Sep 2014 10:32:43 +0400, Alexey Brodkin wrote:
> Fixes:
> http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3
> http://autobuild.buildroot.net/results/994a84d192848b90a70e28e5ebaa769701bcaec9
> 
> Following packages select "libtirpc", so adding dependency as well:
>  * argus
>  * conntrack-tools
>  * lmbench
>  * nfs-utils
>  * quota
>  * rpcbind
> 
> Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>

That's not that simple because you forget to update the comments: they
should not be displayed if the architecture doesn't have the atomic
intrinsics. And this is where things start to be really, really weird.


> diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in
> index ec19d8e..36b12b7 100644
> --- a/package/conntrack-tools/Config.in
> +++ b/package/conntrack-tools/Config.in
> @@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS
>  	depends on BR2_INET_IPV6
>  	depends on BR2_LARGEFILE
>  	depends on BR2_USE_MMU # fork()
> +	depends on BR2_ARCH_HAS_ATOMICS # libtirpc

That's not correct: if the toolchain has native RPC support, we do not
need libtirpc, so there is no need to depend on BR2_ARCH_HAS_ATOMICS in
this case.

I already tried to solve this thing, but it's much more complicated
than just adding a bunch of "depends on".

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

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

* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations
  2014-09-08  6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin
@ 2014-09-08  7:23   ` Thomas Petazzoni
  2014-09-08  9:27     ` Alexey Brodkin
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2014-09-08  7:23 UTC (permalink / raw)
  To: buildroot

Dear Alexey Brodkin,

On Mon,  8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote:

> diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in
> index e17963e..9f32c51 100644
> --- a/package/cppzmq/Config.in
> +++ b/package/cppzmq/Config.in
> @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ
>  	depends on BR2_LARGEFILE # util-linux
>  	depends on BR2_USE_WCHAR # util-linux
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
> +	depends on BR2_ARCH_HAS_ATOMICS # zeromq
>  	select BR2_PACKAGE_ZEROMQ
>  	help
>  	  C++ binding for zeromq (ZeroMQ, 0MQ, zmq).

Same thing here: you should change the comment to *not* display it if
the architecture doesn't have atomic intrinsics.

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

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-08  7:23   ` Thomas Petazzoni
@ 2014-09-08  9:23     ` Alexey Brodkin
  2014-09-08  9:27       ` Thomas Petazzoni
  0 siblings, 1 reply; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-08  9:23 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote:
> Dear Alexey Brodkin,
> > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
> 
> That's not that simple because you forget to update the comments: they
> should not be displayed if the architecture doesn't have the atomic
> intrinsics. And this is where things start to be really, really weird.

> > diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in
> > index ec19d8e..36b12b7 100644
> > --- a/package/conntrack-tools/Config.in
> > +++ b/package/conntrack-tools/Config.in
> > @@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS
> >  	depends on BR2_INET_IPV6
> >  	depends on BR2_LARGEFILE
> >  	depends on BR2_USE_MMU # fork()
> > +	depends on BR2_ARCH_HAS_ATOMICS # libtirpc
> 
> That's not correct: if the toolchain has native RPC support, we do not
> need libtirpc, so there is no need to depend on BR2_ARCH_HAS_ATOMICS in
> this case.
> 
> I already tried to solve this thing, but it's much more complicated
> than just adding a bunch of "depends on".

So I assume there's no sense for me to try fixing comments, right?
I mean if this won't work either or introduce new problems as you meant.

Then what could be our next steps to accommodate possibility of
existence/absence of atomic ops?

This is especially important for ARC where these ops are optional.
The simplest solution could be to enable BR2_ARCH_HAS_ATOMICS for ARC by
default, but this will just attempt to escape from real problem.

And as I saw from other autobuild results other architectures are
affected like Blackfin for example.

If this is that complicated we may try to discuss it in person on
Buildroot summit in few weeks.

-Alexey

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-08  9:23     ` Alexey Brodkin
@ 2014-09-08  9:27       ` Thomas Petazzoni
  2014-09-12 13:12         ` Alexey Brodkin
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2014-09-08  9:27 UTC (permalink / raw)
  To: buildroot

Dear Alexey Brodkin,

On Mon, 8 Sep 2014 09:23:02 +0000, Alexey Brodkin wrote:

> > I already tried to solve this thing, but it's much more complicated
> > than just adding a bunch of "depends on".
> 
> So I assume there's no sense for me to try fixing comments, right?
> I mean if this won't work either or introduce new problems as you meant.

There is definitely some sense in fixing this stuff. I'm not sure yet
on how to do this. I had some discussion with Thomas DS a week ago or
so on IRC about this.

> This is especially important for ARC where these ops are optional.
> The simplest solution could be to enable BR2_ARCH_HAS_ATOMICS for ARC by
> default, but this will just attempt to escape from real problem.
> 
> And as I saw from other autobuild results other architectures are
> affected like Blackfin for example.

Yes, sure, we need to fix the problem. It's just not as simple as
adding more "depends on", unfortunately :-(

> If this is that complicated we may try to discuss it in person on
> Buildroot summit in few weeks.

Ah, you're planning on coming to the Buildroot meeting? If so, could
you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or
confirm me that you are coming so that I can add your name to the list
of participants?

Thanks!

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

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

* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations
  2014-09-08  7:23   ` Thomas Petazzoni
@ 2014-09-08  9:27     ` Alexey Brodkin
  2014-09-08  9:33       ` Thomas Petazzoni
  0 siblings, 1 reply; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-08  9:27 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote:
> Dear Alexey Brodkin,
> 
> On Mon,  8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote:
> 
> > diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in
> > index e17963e..9f32c51 100644
> > --- a/package/cppzmq/Config.in
> > +++ b/package/cppzmq/Config.in
> > @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ
> >  	depends on BR2_LARGEFILE # util-linux
> >  	depends on BR2_USE_WCHAR # util-linux
> >  	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
> > +	depends on BR2_ARCH_HAS_ATOMICS # zeromq
> >  	select BR2_PACKAGE_ZEROMQ
> >  	help
> >  	  C++ binding for zeromq (ZeroMQ, 0MQ, zmq).
> 
> Same thing here: you should change the comment to *not* display it if
> the architecture doesn't have atomic intrinsics.

I assume comments from the previous patch apply here as well, right?
Especially since atomic ops are only used in "libpgm", i.e. if only
BR2_PACKAGE_ZEROMQ_PGM is enabled.

-Alexey

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

* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations
  2014-09-08  9:27     ` Alexey Brodkin
@ 2014-09-08  9:33       ` Thomas Petazzoni
  0 siblings, 0 replies; 12+ messages in thread
From: Thomas Petazzoni @ 2014-09-08  9:33 UTC (permalink / raw)
  To: buildroot

Dear Alexey Brodkin,

On Mon, 8 Sep 2014 09:27:21 +0000, Alexey Brodkin wrote:
> Hi Thomas,
> 
> On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote:
> > Dear Alexey Brodkin,
> > 
> > On Mon,  8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote:
> > 
> > > diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in
> > > index e17963e..9f32c51 100644
> > > --- a/package/cppzmq/Config.in
> > > +++ b/package/cppzmq/Config.in
> > > @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ
> > >  	depends on BR2_LARGEFILE # util-linux
> > >  	depends on BR2_USE_WCHAR # util-linux
> > >  	depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq
> > > +	depends on BR2_ARCH_HAS_ATOMICS # zeromq
> > >  	select BR2_PACKAGE_ZEROMQ
> > >  	help
> > >  	  C++ binding for zeromq (ZeroMQ, 0MQ, zmq).
> > 
> > Same thing here: you should change the comment to *not* display it if
> > the architecture doesn't have atomic intrinsics.
> 
> I assume comments from the previous patch apply here as well, right?
> Especially since atomic ops are only used in "libpgm", i.e. if only
> BR2_PACKAGE_ZEROMQ_PGM is enabled.

If atomic operations are only used when BR2_PACKAGE_ZEROMQ_PGM is
enabled, why do we need to propagate all those dependencies then? If a
package selects BR2_PACKAGE_ZEROMQ and not BR2_PACKAGE_ZEROMQ_PGM,
there's no reason to propagate the dependency.

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

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-08  9:27       ` Thomas Petazzoni
@ 2014-09-12 13:12         ` Alexey Brodkin
  2014-09-12 13:25           ` Thomas Petazzoni
  0 siblings, 1 reply; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-12 13:12 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Mon, 2014-09-08 at 11:27 +0200, Thomas Petazzoni wrote:
> Dear Alexey Brodkin,
> 
> On Mon, 8 Sep 2014 09:23:02 +0000, Alexey Brodkin wrote:

> Ah, you're planning on coming to the Buildroot meeting? If so, could
> you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or
> confirm me that you are coming so that I can add your name to the list
> of participants?

Do you mind to add another atomic ops discussion as another item to
"Topics to discuss" during the event?

-Alexey

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-12 13:12         ` Alexey Brodkin
@ 2014-09-12 13:25           ` Thomas Petazzoni
  2014-09-12 13:31             ` Alexey Brodkin
  0 siblings, 1 reply; 12+ messages in thread
From: Thomas Petazzoni @ 2014-09-12 13:25 UTC (permalink / raw)
  To: buildroot

Dear Alexey Brodkin,

On Fri, 12 Sep 2014 13:12:03 +0000, Alexey Brodkin wrote:

> > Ah, you're planning on coming to the Buildroot meeting? If so, could
> > you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or
> > confirm me that you are coming so that I can add your name to the list
> > of participants?
> 
> Do you mind to add another atomic ops discussion as another item to
> "Topics to discuss" during the event?

I've added it. But to be honest, I hope to be able to make a proposal
to solve the problem before the meeting.

Thanks!

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

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

* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations
  2014-09-12 13:25           ` Thomas Petazzoni
@ 2014-09-12 13:31             ` Alexey Brodkin
  0 siblings, 0 replies; 12+ messages in thread
From: Alexey Brodkin @ 2014-09-12 13:31 UTC (permalink / raw)
  To: buildroot

Hi Thomas,

On Fri, 2014-09-12 at 15:25 +0200, Thomas Petazzoni wrote:
> Dear Alexey Brodkin,
> 
> On Fri, 12 Sep 2014 13:12:03 +0000, Alexey Brodkin wrote:
> 
> > > Ah, you're planning on coming to the Buildroot meeting? If so, could
> > > you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or
> > > confirm me that you are coming so that I can add your name to the list
> > > of participants?
> > 
> > Do you mind to add another atomic ops discussion as another item to
> > "Topics to discuss" during the event?
> 
> I've added it. But to be honest, I hope to be able to make a proposal
> to solve the problem before the meeting.

Good to know it! Please let me know if I may somehow help you with it.

-Alexey

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

end of thread, other threads:[~2014-09-12 13:31 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-09-08  6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin
2014-09-08  6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin
2014-09-08  7:23   ` Thomas Petazzoni
2014-09-08  9:23     ` Alexey Brodkin
2014-09-08  9:27       ` Thomas Petazzoni
2014-09-12 13:12         ` Alexey Brodkin
2014-09-12 13:25           ` Thomas Petazzoni
2014-09-12 13:31             ` Alexey Brodkin
2014-09-08  6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin
2014-09-08  7:23   ` Thomas Petazzoni
2014-09-08  9:27     ` Alexey Brodkin
2014-09-08  9:33       ` 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.