DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system
@ 2014-10-22 16:36 Sergio Gonzalez Monroy
  2014-10-28 16:30 ` De Lara Guarch, Pablo
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Sergio Gonzalez Monroy @ 2014-10-22 16:36 UTC (permalink / raw)
  To: dev

Incompatible libraries error when building shared libraries for 32bits on
a 64bits system.
Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.

Signed-off-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>
---
 mk/rte.lib.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mk/rte.lib.mk b/mk/rte.lib.mk
index f458258..d83e808 100644
--- a/mk/rte.lib.mk
+++ b/mk/rte.lib.mk
@@ -61,7 +61,7 @@ exe2cmd = $(strip $(call dotfile,$(patsubst %,%.cmd,$(1))))
 
 ifeq ($(LINK_USING_CC),1)
 # Override the definition of LD here, since we're linking with CC
-LD := $(CC)
+LD := $(CC) $(CPU_CFLAGS)
 LD_MULDEFS := $(call linkerprefix,-z$(comma)muldefs)
 CPU_LDFLAGS := $(call linkerprefix,$(CPU_LDFLAGS))
 endif
-- 
1.9.3

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

* Re: [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system
  2014-10-22 16:36 [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system Sergio Gonzalez Monroy
@ 2014-10-28 16:30 ` De Lara Guarch, Pablo
  2014-11-28 15:48 ` Thomas Monjalon
  2014-12-08 14:52 ` [dpdk-dev] " Neil Horman
  2 siblings, 0 replies; 6+ messages in thread
From: De Lara Guarch, Pablo @ 2014-10-28 16:30 UTC (permalink / raw)
  To: Gonzalez Monroy, Sergio, dev



> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Sergio Gonzalez
> Monroy
> Sent: Wednesday, October 22, 2014 5:36 PM
> To: dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system
> 
> Incompatible libraries error when building shared libraries for 32bits on
> a 64bits system.
> Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.
> 
> Signed-off-by: Sergio Gonzalez Monroy
> <sergio.gonzalez.monroy@intel.com>

Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>

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

* Re: [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system
  2014-10-22 16:36 [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system Sergio Gonzalez Monroy
  2014-10-28 16:30 ` De Lara Guarch, Pablo
@ 2014-11-28 15:48 ` Thomas Monjalon
  2014-12-01  9:51   ` Gonzalez Monroy, Sergio
  2014-12-08 14:52 ` [dpdk-dev] " Neil Horman
  2 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2014-11-28 15:48 UTC (permalink / raw)
  To: Sergio Gonzalez Monroy; +Cc: dev

Hi Sergio,

2014-10-22 17:36, Sergio Gonzalez Monroy:
> Incompatible libraries error when building shared libraries for 32bits on
> a 64bits system.
> Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.

This issue looks really strange. If that's the only way to fix it,
it would be better to have a comment in the makefile and a detailed
explanation in the commit log.

Thanks
-- 
Thomas

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

* Re: [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system
  2014-11-28 15:48 ` Thomas Monjalon
@ 2014-12-01  9:51   ` Gonzalez Monroy, Sergio
  0 siblings, 0 replies; 6+ messages in thread
From: Gonzalez Monroy, Sergio @ 2014-12-01  9:51 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: dev

> From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com]
> Sent: Friday, November 28, 2014 3:49 PM
> 
> Hi Sergio,
> 
> 2014-10-22 17:36, Sergio Gonzalez Monroy:
> > Incompatible libraries error when building shared libraries for 32bits
> > on a 64bits system.
> > Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.
> 
> This issue looks really strange. If that's the only way to fix it, it would be
> better to have a comment in the makefile and a detailed explanation in the
> commit log.
> 
Hi Thomas,

There may be a better way to deal with this, please feel free to suggest alternate method.
We do specify -m32 or -m64 when building the DPDK.
The issue is that we were not specifying -m32 when linking, therefore the wrong libraries were being picked.

At the time CPU_CFLAGS was only being set with -m32/-m64, reason why I used such variable.
Would it be better to create another var and pass it down?

Thanks,
Sergio

> Thanks
> --
> Thomas

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

* Re: [dpdk-dev] mk: fix build 32bits shared libs on 64bits system
  2014-10-22 16:36 [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system Sergio Gonzalez Monroy
  2014-10-28 16:30 ` De Lara Guarch, Pablo
  2014-11-28 15:48 ` Thomas Monjalon
@ 2014-12-08 14:52 ` Neil Horman
  2014-12-11  0:49   ` Thomas Monjalon
  2 siblings, 1 reply; 6+ messages in thread
From: Neil Horman @ 2014-12-08 14:52 UTC (permalink / raw)
  To: Sergio Gonzalez Monroy; +Cc: dev

On Wed, Oct 22, 2014 at 05:36:22PM +0100, Sergio Gonzalez Monroy wrote:
> Incompatible libraries error when building shared libraries for 32bits on
> a 64bits system.
> Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.
> 
> Signed-off-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>
> Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> 
> ---
> mk/rte.lib.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mk/rte.lib.mk b/mk/rte.lib.mk
> index f458258..d83e808 100644
> --- a/mk/rte.lib.mk
> +++ b/mk/rte.lib.mk
> @@ -61,7 +61,7 @@ exe2cmd = $(strip $(call dotfile,$(patsubst %,%.cmd,$(1))))
>  
>  ifeq ($(LINK_USING_CC),1)
>  # Override the definition of LD here, since we're linking with CC
> -LD := $(CC)
> +LD := $(CC) $(CPU_CFLAGS)
>  LD_MULDEFS := $(call linkerprefix,-z$(comma)muldefs)
>  CPU_LDFLAGS := $(call linkerprefix,$(CPU_LDFLAGS))
>  endif

Acked-by: Neil Horman <nhorman@tuxdriver.com>

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

* Re: [dpdk-dev] mk: fix build 32bits shared libs on 64bits system
  2014-12-08 14:52 ` [dpdk-dev] " Neil Horman
@ 2014-12-11  0:49   ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2014-12-11  0:49 UTC (permalink / raw)
  To: Sergio Gonzalez Monroy; +Cc: dev

> > Incompatible libraries error when building shared libraries for 32bits on
> > a 64bits system.
> > Fix issue by passing CPU_CFLAGS to CC when LINK_USING_CC is enabled.
> > 
> > Signed-off-by: Sergio Gonzalez Monroy <sergio.gonzalez.monroy@intel.com>
> > Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> 
> Acked-by: Neil Horman <nhorman@tuxdriver.com>

Applied

Thanks
-- 
Thomas

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

end of thread, other threads:[~2014-12-11  0:49 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-22 16:36 [dpdk-dev] [PATCH] mk: fix build 32bits shared libs on 64bits system Sergio Gonzalez Monroy
2014-10-28 16:30 ` De Lara Guarch, Pablo
2014-11-28 15:48 ` Thomas Monjalon
2014-12-01  9:51   ` Gonzalez Monroy, Sergio
2014-12-08 14:52 ` [dpdk-dev] " Neil Horman
2014-12-11  0:49   ` Thomas Monjalon

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git