patches for DPDK stable branches
 help / color / Atom feed
* [dpdk-stable] [PATCH] net/bnx2x: fix the meson compile issue
@ 2019-10-29 11:09 Gavin Hu
  2019-10-30 11:23 ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Gavin Hu @ 2019-10-29 11:09 UTC (permalink / raw)
  To: dev; +Cc: nd, ajit.khaparde, somnath.kotur, stable

The definition is conflicting with that of the compiler, as show the
following error: ../drivers/net/bnx2x/bnx2x_osal.h:27:17: error:
conflicting types for ‘uint64_t’ #define __le64  uint64_t ^~~~ In file
included from /usr/include/stdint.h:37, from
/usr/lib/gcc/aarch64-linux-gnu/8/include/stdint.h:9, from
../lib/librte_eal/common/include/arch/arm/rte_byteorder.h:16, from
../drivers/net/bnx2x/bnx2x.h:17, from ../drivers/net/bnx2x/bnx2x_rxtx.c:8:
/usr/include/aarch64-linux-gnu/bits/stdint-uintn.h:27:20: note: previous
declaration of ‘uint64_t’ was here
typedef __uint64_t uint64_t;

The fix is to remove the duplicate definition.

Fixes: 38dff79ba736 ("net/bnx2x: update HSI")
Cc: stable@dpdk.org

Signed-off-by: Gavin Hu <gavin.hu@arm.com>
---
 drivers/net/bnx2x/bnx2x_osal.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/net/bnx2x/bnx2x_osal.h b/drivers/net/bnx2x/bnx2x_osal.h
index 7cd2932..aad4e0b 100644
--- a/drivers/net/bnx2x/bnx2x_osal.h
+++ b/drivers/net/bnx2x/bnx2x_osal.h
@@ -22,8 +22,4 @@
 #undef __LITTLE_ENDIAN
 #endif
 
-#define __le16		uint16_t
-#define __le32		uint32_t
-#define __le64		uint64_t
-
 #endif /* BNX2X_OSAL_H */
-- 
2.7.4


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

* Re: [dpdk-stable] [dpdk-dev] [PATCH] net/bnx2x: fix the meson compile issue
  2019-10-29 11:09 [dpdk-stable] [PATCH] net/bnx2x: fix the meson compile issue Gavin Hu
@ 2019-10-30 11:23 ` " Ferruh Yigit
  2019-10-30 18:35   ` [dpdk-stable] [EXT] " Rasesh Mody
  0 siblings, 1 reply; 3+ messages in thread
From: Ferruh Yigit @ 2019-10-30 11:23 UTC (permalink / raw)
  To: Gavin Hu, dev, Rasesh Mody, Shahed Shaikh
  Cc: nd, ajit.khaparde, somnath.kotur, stable, Jerin Jacob Kollanukkaran

On 10/29/2019 11:09 AM, Gavin Hu wrote:
> The definition is conflicting with that of the compiler, as show the
> following error: ../drivers/net/bnx2x/bnx2x_osal.h:27:17: error:
> conflicting types for ‘uint64_t’ #define __le64  uint64_t ^~~~ In file
> included from /usr/include/stdint.h:37, from
> /usr/lib/gcc/aarch64-linux-gnu/8/include/stdint.h:9, from
> ../lib/librte_eal/common/include/arch/arm/rte_byteorder.h:16, from
> ../drivers/net/bnx2x/bnx2x.h:17, from ../drivers/net/bnx2x/bnx2x_rxtx.c:8:
> /usr/include/aarch64-linux-gnu/bits/stdint-uintn.h:27:20: note: previous
> declaration of ‘uint64_t’ was here
> typedef __uint64_t uint64_t;
> 
> The fix is to remove the duplicate definition.
> 
> Fixes: 38dff79ba736 ("net/bnx2x: update HSI")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Gavin Hu <gavin.hu@arm.com>

Hi Rasesh, Shahed,

The driver is causing the build error for the arm, I remember same issue was
valid for x86 and fixed later but seems not fixed for arm.

Can you please check for the build error, if we can't find a solution, may have
to merge this patch.

Thanks,
ferruh


> ---
>  drivers/net/bnx2x/bnx2x_osal.h | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/net/bnx2x/bnx2x_osal.h b/drivers/net/bnx2x/bnx2x_osal.h
> index 7cd2932..aad4e0b 100644
> --- a/drivers/net/bnx2x/bnx2x_osal.h
> +++ b/drivers/net/bnx2x/bnx2x_osal.h
> @@ -22,8 +22,4 @@
>  #undef __LITTLE_ENDIAN
>  #endif
>  
> -#define __le16		uint16_t
> -#define __le32		uint32_t
> -#define __le64		uint64_t
> -
>  #endif /* BNX2X_OSAL_H */
> 


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

* Re: [dpdk-stable] [EXT] Re: [dpdk-dev] [PATCH] net/bnx2x: fix the meson compile issue
  2019-10-30 11:23 ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
@ 2019-10-30 18:35   ` " Rasesh Mody
  0 siblings, 0 replies; 3+ messages in thread
From: Rasesh Mody @ 2019-10-30 18:35 UTC (permalink / raw)
  To: Ferruh Yigit, Gavin Hu, dev, Shahed Shaikh
  Cc: nd, ajit.khaparde, somnath.kotur, stable, Jerin Jacob Kollanukkaran

>From: Ferruh Yigit <ferruh.yigit@intel.com>
>Sent: Wednesday, October 30, 2019 4:23 AM
>
>On 10/29/2019 11:09 AM, Gavin Hu wrote:
>> The definition is conflicting with that of the compiler, as show the
>> following error: ../drivers/net/bnx2x/bnx2x_osal.h:27:17: error:
>> conflicting types for ‘uint64_t’ #define __le64  uint64_t ^~~~ In file
>> included from /usr/include/stdint.h:37, from
>> /usr/lib/gcc/aarch64-linux-gnu/8/include/stdint.h:9, from
>> ../lib/librte_eal/common/include/arch/arm/rte_byteorder.h:16, from
>> ../drivers/net/bnx2x/bnx2x.h:17, from ../drivers/net/bnx2x/bnx2x_rxtx.c:8:
>> /usr/include/aarch64-linux-gnu/bits/stdint-uintn.h:27:20: note:
>> previous declaration of ‘uint64_t’ was here typedef __uint64_t
>> uint64_t;
>>
>> The fix is to remove the duplicate definition.
>>
>> Fixes: 38dff79ba736 ("net/bnx2x: update HSI")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Gavin Hu <gavin.hu@arm.com>
>
>Hi Rasesh, Shahed,
>
>The driver is causing the build error for the arm, I remember same issue was
>valid for x86 and fixed later but seems not fixed for arm.
>
>Can you please check for the build error, if we can't find a solution, may have
>to merge this patch.
>

Hi Ferruh,

We have a solution to fix the bnx2x build error, it's part of the bug 355. I plan to send it out once tested on failing setup.

Thanks!
-Rasesh

>Thanks,
>ferruh
>
>
>> ---
>>  drivers/net/bnx2x/bnx2x_osal.h | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/drivers/net/bnx2x/bnx2x_osal.h
>> b/drivers/net/bnx2x/bnx2x_osal.h index 7cd2932..aad4e0b 100644
>> --- a/drivers/net/bnx2x/bnx2x_osal.h
>> +++ b/drivers/net/bnx2x/bnx2x_osal.h
>> @@ -22,8 +22,4 @@
>>  #undef __LITTLE_ENDIAN
>>  #endif
>>
>> -#define __le16		uint16_t
>> -#define __le32		uint32_t
>> -#define __le64		uint64_t
>> -
>>  #endif /* BNX2X_OSAL_H */
>>



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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-29 11:09 [dpdk-stable] [PATCH] net/bnx2x: fix the meson compile issue Gavin Hu
2019-10-30 11:23 ` [dpdk-stable] [dpdk-dev] " Ferruh Yigit
2019-10-30 18:35   ` [dpdk-stable] [EXT] " Rasesh Mody

patches for DPDK stable branches

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/stable/0 stable/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 stable stable/ http://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


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