* [dpdk-dev] [PATCH] ethdev: fix device capability to string translation
@ 2021-11-09 9:00 Dmitry Kozlyuk
2021-11-09 11:56 ` Ali Alnubani
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Kozlyuk @ 2021-11-09 9:00 UTC (permalink / raw)
To: dev
Cc: Ali Alnubani, Xueming Li, Thomas Monjalon, Ferruh Yigit,
Andrew Rybchenko
Add support for RTE_ETH_DEV_CAPA_FLOW_{RULE,SHARED_OBJECT}_KEEP
to rte_eth_dev_capability_name(), missed when adding the capabilities.
Fixes: 1d5a3d68c0f9 ("ethdev: add capability to keep flow rules on restart")
Fixes: 2c9cd45de7e6 ("ethdev: add capability to keep shared objects on restart")
Reported-by: Ali Alnubani <alialnu@nvidia.com>
Signed-off-by: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
Acked-by: Xueming Li <xuemingl@nvidia.com>
---
P.S. I don't know a good way to catch such errors in the future
except by a unit test, but adding a new suite for ethdev
seems worthy of its own patch (if needed at all).
lib/ethdev/rte_ethdev.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/lib/ethdev/rte_ethdev.c b/lib/ethdev/rte_ethdev.c
index 8e679e4003..a1d475a292 100644
--- a/lib/ethdev/rte_ethdev.c
+++ b/lib/ethdev/rte_ethdev.c
@@ -171,6 +171,8 @@ static const struct {
{RTE_ETH_DEV_CAPA_RUNTIME_RX_QUEUE_SETUP, "RUNTIME_RX_QUEUE_SETUP"},
{RTE_ETH_DEV_CAPA_RUNTIME_TX_QUEUE_SETUP, "RUNTIME_TX_QUEUE_SETUP"},
{RTE_ETH_DEV_CAPA_RXQ_SHARE, "RXQ_SHARE"},
+ {RTE_ETH_DEV_CAPA_FLOW_RULE_KEEP, "FLOW_RULE_KEEP"},
+ {RTE_ETH_DEV_CAPA_FLOW_SHARED_OBJECT_KEEP, "FLOW_SHARED_OBJECT_KEEP"},
};
/**
--
2.25.1
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-dev] [PATCH] ethdev: fix device capability to string translation
2021-11-09 9:00 [dpdk-dev] [PATCH] ethdev: fix device capability to string translation Dmitry Kozlyuk
@ 2021-11-09 11:56 ` Ali Alnubani
2021-11-09 18:43 ` Ferruh Yigit
0 siblings, 1 reply; 3+ messages in thread
From: Ali Alnubani @ 2021-11-09 11:56 UTC (permalink / raw)
To: Dmitry Kozlyuk, dev
Cc: Xueming(Steven) Li, NBU-Contact-Thomas Monjalon, Ferruh Yigit,
Andrew Rybchenko
> -----Original Message-----
> From: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
> Sent: Tuesday, November 9, 2021 11:01 AM
> To: dev@dpdk.org
> Cc: Ali Alnubani <alialnu@nvidia.com>; Xueming(Steven) Li
> <xuemingl@nvidia.com>; NBU-Contact-Thomas Monjalon
> <thomas@monjalon.net>; Ferruh Yigit <ferruh.yigit@intel.com>; Andrew
> Rybchenko <andrew.rybchenko@oktetlabs.ru>
> Subject: [PATCH] ethdev: fix device capability to string translation
>
> Add support for RTE_ETH_DEV_CAPA_FLOW_{RULE,SHARED_OBJECT}_KEEP
> to rte_eth_dev_capability_name(), missed when adding the capabilities.
>
> Fixes: 1d5a3d68c0f9 ("ethdev: add capability to keep flow rules on restart")
> Fixes: 2c9cd45de7e6 ("ethdev: add capability to keep shared objects on
> restart")
>
> Reported-by: Ali Alnubani <alialnu@nvidia.com>
> Signed-off-by: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
> Acked-by: Xueming Li <xuemingl@nvidia.com>
> ---
> P.S. I don't know a good way to catch such errors in the future
> except by a unit test, but adding a new suite for ethdev
> seems worthy of its own patch (if needed at all).
>
Tested-by: Ali Alnubani <alialnu@nvidia.com>
Thanks,
Ali
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-dev] [PATCH] ethdev: fix device capability to string translation
2021-11-09 11:56 ` Ali Alnubani
@ 2021-11-09 18:43 ` Ferruh Yigit
0 siblings, 0 replies; 3+ messages in thread
From: Ferruh Yigit @ 2021-11-09 18:43 UTC (permalink / raw)
To: Ali Alnubani, Dmitry Kozlyuk, dev
Cc: Xueming(Steven) Li, NBU-Contact-Thomas Monjalon, Andrew Rybchenko
On 11/9/2021 11:56 AM, Ali Alnubani wrote:
>> -----Original Message-----
>> From: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
>> Sent: Tuesday, November 9, 2021 11:01 AM
>> To: dev@dpdk.org
>> Cc: Ali Alnubani <alialnu@nvidia.com>; Xueming(Steven) Li
>> <xuemingl@nvidia.com>; NBU-Contact-Thomas Monjalon
>> <thomas@monjalon.net>; Ferruh Yigit <ferruh.yigit@intel.com>; Andrew
>> Rybchenko <andrew.rybchenko@oktetlabs.ru>
>> Subject: [PATCH] ethdev: fix device capability to string translation
>>
>> Add support for RTE_ETH_DEV_CAPA_FLOW_{RULE,SHARED_OBJECT}_KEEP
>> to rte_eth_dev_capability_name(), missed when adding the capabilities.
>>
>> Fixes: 1d5a3d68c0f9 ("ethdev: add capability to keep flow rules on restart")
>> Fixes: 2c9cd45de7e6 ("ethdev: add capability to keep shared objects on
>> restart")
>>
>> Reported-by: Ali Alnubani <alialnu@nvidia.com>
>> Signed-off-by: Dmitry Kozlyuk <dkozlyuk@nvidia.com>
>> Acked-by: Xueming Li <xuemingl@nvidia.com>
>
> Tested-by: Ali Alnubani <alialnu@nvidia.com>
>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Applied to dpdk-next-net/main, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-09 18:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-09 9:00 [dpdk-dev] [PATCH] ethdev: fix device capability to string translation Dmitry Kozlyuk
2021-11-09 11:56 ` Ali Alnubani
2021-11-09 18:43 ` 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).