DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/mlx5: fix compiling without definition
@ 2019-11-10 16:37 Bing Zhao
  2019-11-11  7:00 ` Slava Ovsiienko
  0 siblings, 1 reply; 3+ messages in thread
From: Bing Zhao @ 2019-11-10 16:37 UTC (permalink / raw)
  To: viacheslavo, rasland; +Cc: orika, dev

When compiling the driver without macro for direct rules, the flow
table reference count should also be in the flow table resource
structure.

Fixes: c7455199284a ("net/mlx5: reorganize flow tables with hash list")

Signed-off-by: Bing Zhao <bingz@mellanox.com>
---
 drivers/net/mlx5/mlx5.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h
index 3296f10..feac5a3 100644
--- a/drivers/net/mlx5/mlx5.h
+++ b/drivers/net/mlx5/mlx5.h
@@ -587,9 +587,7 @@ struct mlx5_ibv_shared_port {
 /* Table structure. */
 struct mlx5_flow_tbl_resource {
 	void *obj; /**< Pointer to DR table object. */
-#ifdef HAVE_MLX5DV_DR
 	rte_atomic32_t refcnt; /**< Reference counter. */
-#endif
 };
 
 #define MLX5_MAX_TABLES UINT16_MAX
-- 
1.8.3.1


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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix compiling without definition
  2019-11-10 16:37 [dpdk-dev] [PATCH] net/mlx5: fix compiling without definition Bing Zhao
@ 2019-11-11  7:00 ` Slava Ovsiienko
  2019-11-11 13:10   ` Ferruh Yigit
  0 siblings, 1 reply; 3+ messages in thread
From: Slava Ovsiienko @ 2019-11-11  7:00 UTC (permalink / raw)
  To: Bing Zhao, Raslan Darawsheh; +Cc: Ori Kam, dev

> -----Original Message-----
> From: Bing Zhao <bingz@mellanox.com>
> Sent: Sunday, November 10, 2019 18:37
> To: Slava Ovsiienko <viacheslavo@mellanox.com>; Raslan Darawsheh
> <rasland@mellanox.com>
> Cc: Ori Kam <orika@mellanox.com>; dev@dpdk.org
> Subject: [PATCH] net/mlx5: fix compiling without definition
> 
> When compiling the driver without macro for direct rules, the flow table
> reference count should also be in the flow table resource structure.
> 
> Fixes: c7455199284a ("net/mlx5: reorganize flow tables with hash list")
> 
> Signed-off-by: Bing Zhao <bingz@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>

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

* Re: [dpdk-dev] [PATCH] net/mlx5: fix compiling without definition
  2019-11-11  7:00 ` Slava Ovsiienko
@ 2019-11-11 13:10   ` Ferruh Yigit
  0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2019-11-11 13:10 UTC (permalink / raw)
  To: Slava Ovsiienko, Bing Zhao, Raslan Darawsheh; +Cc: Ori Kam, dev

On 11/11/2019 7:00 AM, Slava Ovsiienko wrote:
>> -----Original Message-----
>> From: Bing Zhao <bingz@mellanox.com>
>> Sent: Sunday, November 10, 2019 18:37
>> To: Slava Ovsiienko <viacheslavo@mellanox.com>; Raslan Darawsheh
>> <rasland@mellanox.com>
>> Cc: Ori Kam <orika@mellanox.com>; dev@dpdk.org
>> Subject: [PATCH] net/mlx5: fix compiling without definition
>>
>> When compiling the driver without macro for direct rules, the flow table
>> reference count should also be in the flow table resource structure.
>>
>> Fixes: c7455199284a ("net/mlx5: reorganize flow tables with hash list")
>>
>> Signed-off-by: Bing Zhao <bingz@mellanox.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>
> 

Squashed into relevant commit in next-net, thanks.

Please confirm the change in latest next-net.

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

end of thread, other threads:[~2019-11-11 13:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-10 16:37 [dpdk-dev] [PATCH] net/mlx5: fix compiling without definition Bing Zhao
2019-11-11  7:00 ` Slava Ovsiienko
2019-11-11 13:10   ` 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).