[dpdk-dev] [PATCH] mk: fix link with gcc

2014-06-10 Thread Thomas Monjalon
2014-06-02 09:40, Olivier MATZ: > Hi Thomas, > > On 05/29/2014 08:48 AM, Thomas Monjalon wrote: > > You're right, title should be: > > mk: fix 32-bit link with gcc > > > > And I should add some details in the commit log: > > > > I didn't see any error with -z muldefs but it isn't documented in

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-06-02 Thread Olivier MATZ
Hi Thomas, On 05/29/2014 08:48 AM, Thomas Monjalon wrote: > You're right, title should be: > mk: fix 32-bit link with gcc > > And I should add some details in the commit log: > > I didn't see any error with -z muldefs but it isn't documented in gcc manual. > So it's safer to explicitly pass

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-29 Thread Thomas Monjalon
Hi Olivier, 2014-05-28 13:47, Olivier MATZ: > On 05/27/2014 02:55 PM, Thomas Monjalon wrote: > > Some linker options were not prefixed by -Wl, when using gcc: > > -z muldefs > > -melf_i386 (32-bit config) > > > > Using macro linkerprefix is fixing it. > > > > Signed-off-by: Thomas Monjal

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-29 Thread Thomas Monjalon
Hi Neil, 2014-05-28 10:17, Neil Horman: > On Tue, May 27, 2014 at 02:55:16PM +0200, Thomas Monjalon wrote: > > Some linker options were not prefixed by -Wl, when using gcc: > > -z muldefs > > -melf_i386 (32-bit config) > > > > Using macro linkerprefix is fixing it. > > > > Signed-off-by:

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-29 Thread Neil Horman
On Thu, May 29, 2014 at 08:24:56AM +0200, Thomas Monjalon wrote: > Hi Neil, > > 2014-05-28 10:17, Neil Horman: > > On Tue, May 27, 2014 at 02:55:16PM +0200, Thomas Monjalon wrote: > > > Some linker options were not prefixed by -Wl, when using gcc: > > > -z muldefs > > > -melf_i386 (32-bit conf

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-28 Thread Olivier MATZ
Hi Thomas, On 05/27/2014 02:55 PM, Thomas Monjalon wrote: > Some linker options were not prefixed by -Wl, when using gcc: > -z muldefs > -melf_i386 (32-bit config) > > Using macro linkerprefix is fixing it. > > Signed-off-by: Thomas Monjalon The patch looks correct, but from the comm

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-28 Thread Neil Horman
On Tue, May 27, 2014 at 02:55:16PM +0200, Thomas Monjalon wrote: > Some linker options were not prefixed by -Wl, when using gcc: > -z muldefs > -melf_i386 (32-bit config) > > Using macro linkerprefix is fixing it. > > Signed-off-by: Thomas Monjalon > --- > mk/rte.lib.mk | 6 -- >

[dpdk-dev] [PATCH] mk: fix link with gcc

2014-05-27 Thread Thomas Monjalon
Some linker options were not prefixed by -Wl, when using gcc: -z muldefs -melf_i386 (32-bit config) Using macro linkerprefix is fixing it. Signed-off-by: Thomas Monjalon --- mk/rte.lib.mk | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/mk/rte.lib.mk b/m