DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
@ 2017-10-17  2:46 Xueming Li
  2017-10-17  7:56 ` Nélio Laranjeiro
  2017-10-21  1:05 ` Ferruh Yigit
  0 siblings, 2 replies; 5+ messages in thread
From: Xueming Li @ 2017-10-17  2:46 UTC (permalink / raw)
  To: Nelio Laranjeiro, FerruhYigitferruh.yigit; +Cc: Xueming Li, dev

Initialize variable to avoid ICC compiler warning:
http://www.dpdk.org/ml/archives/dev/2017-October/077971.html

Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file...")

Signed-off-by: Xueming Li <xuemingl@mellanox.com>
---
 drivers/net/mlx5/mlx5_txq.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c
index 9deaa7e..93f1388 100644
--- a/drivers/net/mlx5/mlx5_txq.c
+++ b/drivers/net/mlx5/mlx5_txq.c
@@ -246,6 +246,7 @@
 	int already_mapped;
 	size_t page_size = sysconf(_SC_PAGESIZE);
 
+	memset(pages, 0, priv->txqs_n * sizeof(uintptr_t));
 	/*
 	 * As rdma-core, UARs are mapped in size of OS page size.
 	 * Use aligned address to avoid duplicate mmap.
-- 
1.8.3.1

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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
  2017-10-17  2:46 [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning Xueming Li
@ 2017-10-17  7:56 ` Nélio Laranjeiro
  2017-10-23 21:54   ` Ferruh Yigit
  2017-10-21  1:05 ` Ferruh Yigit
  1 sibling, 1 reply; 5+ messages in thread
From: Nélio Laranjeiro @ 2017-10-17  7:56 UTC (permalink / raw)
  To: Xueming Li; +Cc: FerruhYigitferruh.yigit, dev

On Tue, Oct 17, 2017 at 10:46:43AM +0800, Xueming Li wrote:
> Initialize variable to avoid ICC compiler warning:
> http://www.dpdk.org/ml/archives/dev/2017-October/077971.html
> 
> Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file...")
> 
> Signed-off-by: Xueming Li <xuemingl@mellanox.com>
> ---
>  drivers/net/mlx5/mlx5_txq.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c
> index 9deaa7e..93f1388 100644
> --- a/drivers/net/mlx5/mlx5_txq.c
> +++ b/drivers/net/mlx5/mlx5_txq.c
> @@ -246,6 +246,7 @@
>  	int already_mapped;
>  	size_t page_size = sysconf(_SC_PAGESIZE);
>  
> +	memset(pages, 0, priv->txqs_n * sizeof(uintptr_t));
>  	/*
>  	 * As rdma-core, UARs are mapped in size of OS page size.
>  	 * Use aligned address to avoid duplicate mmap.
> -- 
> 1.8.3.1
 

Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>

-- 
Nélio Laranjeiro
6WIND

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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
  2017-10-17  2:46 [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning Xueming Li
  2017-10-17  7:56 ` Nélio Laranjeiro
@ 2017-10-21  1:05 ` Ferruh Yigit
  1 sibling, 0 replies; 5+ messages in thread
From: Ferruh Yigit @ 2017-10-21  1:05 UTC (permalink / raw)
  To: Xueming Li, Nelio Laranjeiro, FerruhYigitferruh.yigit; +Cc: dev

On 10/16/2017 7:46 PM, Xueming Li wrote:
> Initialize variable to avoid ICC compiler warning:
> http://www.dpdk.org/ml/archives/dev/2017-October/077971.html

Hi Xueming, Nelio,

I disabled that warning for ICC in Makefile [1], as mentioned in above mail.

If you prefer to fix it instead of disable, can you please update the patch,
and remove that workaround in this patch?

Thanks,
ferruh


[1]
http://dpdk.org/browse/dpdk/tree/drivers/net/mlx5/Makefile#n93
 # Disable false positive warning
 ifeq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
 CFLAGS_mlx5_txq.o += -wd3656
 endif



> 
> Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file...")
> 
> Signed-off-by: Xueming Li <xuemingl@mellanox.com>
> ---
>  drivers/net/mlx5/mlx5_txq.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/mlx5/mlx5_txq.c b/drivers/net/mlx5/mlx5_txq.c
> index 9deaa7e..93f1388 100644
> --- a/drivers/net/mlx5/mlx5_txq.c
> +++ b/drivers/net/mlx5/mlx5_txq.c
> @@ -246,6 +246,7 @@
>  	int already_mapped;
>  	size_t page_size = sysconf(_SC_PAGESIZE);
>  
> +	memset(pages, 0, priv->txqs_n * sizeof(uintptr_t));
>  	/*
>  	 * As rdma-core, UARs are mapped in size of OS page size.
>  	 * Use aligned address to avoid duplicate mmap.
> 

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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
  2017-10-17  7:56 ` Nélio Laranjeiro
@ 2017-10-23 21:54   ` Ferruh Yigit
  2017-10-24 12:28     ` Xueming(Steven) Li
  0 siblings, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2017-10-23 21:54 UTC (permalink / raw)
  To: Nélio Laranjeiro, Xueming Li; +Cc: Ferruh Yigit, dev

On 10/17/2017 12:56 AM, Nélio Laranjeiro wrote:
> On Tue, Oct 17, 2017 at 10:46:43AM +0800, Xueming Li wrote:
>> Initialize variable to avoid ICC compiler warning:
>> http://www.dpdk.org/ml/archives/dev/2017-October/077971.html
>>
>> Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a file...")
>>
>> Signed-off-by: Xueming Li <xuemingl@mellanox.com>

> Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>

Applied to dpdk-next-net/master, thanks.


(icc workaround to disable warning [1] removed as well with patch, please check)

[1]
http://dpdk.org/ml/archives/dev/2017-October/079836.html

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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
  2017-10-23 21:54   ` Ferruh Yigit
@ 2017-10-24 12:28     ` Xueming(Steven) Li
  0 siblings, 0 replies; 5+ messages in thread
From: Xueming(Steven) Li @ 2017-10-24 12:28 UTC (permalink / raw)
  To: Ferruh Yigit, Nélio Laranjeiro; +Cc: dev

Thanks very much, sorry that I forgot to reply...

> -----Original Message-----
> From: Ferruh Yigit [mailto:ferruh.yigit@intel.com]
> Sent: Tuesday, October 24, 2017 5:54 AM
> To: Nélio Laranjeiro <nelio.laranjeiro@6wind.com>; Xueming(Steven) Li
> <xuemingl@mellanox.com>
> Cc: Ferruh Yigit <ferruh.yigit@intel.com>; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning
> 
> On 10/17/2017 12:56 AM, Nélio Laranjeiro wrote:
> > On Tue, Oct 17, 2017 at 10:46:43AM +0800, Xueming Li wrote:
> >> Initialize variable to avoid ICC compiler warning:
> >>
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.dpd
> k.org%2Fml%2Farchives%2Fdev%2F2017-
> October%2F077971.html&data=02%7C01%7Cxuemingl%40mellanox.com%7Cf2aa5d8fb9
> be404fc57b08d51a60a28b%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C63644
> 3924711609271&sdata=z%2Fiv8vs4MAbdnHr%2Bmq7ZMcGUfNa%2F1ItUhCq95sI11iU%3D&
> reserved=0
> >>
> >> Fixes: f8b9a3bad467 ("net/mlx5: install a socket to exchange a
> file...")
> >>
> >> Signed-off-by: Xueming Li <xuemingl@mellanox.com>
> 
> > Acked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>
> 
> Applied to dpdk-next-net/master, thanks.
> 
> 
> (icc workaround to disable warning [1] removed as well with patch, please
> check)
> 
> [1]
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.or
> g%2Fml%2Farchives%2Fdev%2F2017-
> October%2F079836.html&data=02%7C01%7Cxuemingl%40mellanox.com%7Cf2aa5d8fb9
> be404fc57b08d51a60a28b%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C63644
> 3924711609271&sdata=RaHRnvt0Xaiu8qb34MfY4sGXOt1auJQhyXgtU7AWgrw%3D&reserv
> ed=0

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

end of thread, other threads:[~2017-10-24 12:28 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-10-17  2:46 [dpdk-dev] [PATCH] net/mlx5: fix ICC compiler warning Xueming Li
2017-10-17  7:56 ` Nélio Laranjeiro
2017-10-23 21:54   ` Ferruh Yigit
2017-10-24 12:28     ` Xueming(Steven) Li
2017-10-21  1:05 ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).