[v2] ethdev: fix missing names in Tx offload name array

Message ID 1536740882-60426-1-git-send-email-dekelp@mellanox.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series [v2] ethdev: fix missing names in Tx offload name array |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Dekel Peled Sept. 12, 2018, 8:28 a.m. UTC
  Patch 5355f443 added two definitions of DEV_TX_OFFLOAD_xxx.
If new Tx offload capabilities are defined, they also must be mentioned
in rte_tx_offload_names in rte_ethdev.c file.

This patch adds the required lines in array rte_tx_offload_names.

Fixes: 5355f4439e2e ("ethdev: introduce generic IP/UDP tunnel checksum and TSO")

Cc: xuemingl@mellanox.com

Signed-off-by: Dekel Peled <dekelp@mellanox.com>
---
v2:
* Discard change of comments location in file rte_ethdev.h.
---

 lib/librte_ethdev/rte_ethdev.c | 2 ++
 1 file changed, 2 insertions(+)
  

Comments

Andrew Rybchenko Sept. 12, 2018, 8:33 a.m. UTC | #1
On 09/12/2018 11:28 AM, Dekel Peled wrote:
> Patch 5355f443 added two definitions of DEV_TX_OFFLOAD_xxx.
> If new Tx offload capabilities are defined, they also must be mentioned
> in rte_tx_offload_names in rte_ethdev.c file.
>
> This patch adds the required lines in array rte_tx_offload_names.
>
> Fixes: 5355f4439e2e ("ethdev: introduce generic IP/UDP tunnel checksum and TSO")
>
> Cc: xuemingl@mellanox.com
>
> Signed-off-by: Dekel Peled <dekelp@mellanox.com>

Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>
  
Ferruh Yigit Sept. 12, 2018, 5:20 p.m. UTC | #2
On 9/12/2018 9:33 AM, Andrew Rybchenko wrote:
> On 09/12/2018 11:28 AM, Dekel Peled wrote:
>> Patch 5355f443 added two definitions of DEV_TX_OFFLOAD_xxx.
>> If new Tx offload capabilities are defined, they also must be mentioned
>> in rte_tx_offload_names in rte_ethdev.c file.
>>
>> This patch adds the required lines in array rte_tx_offload_names.
>>
>> Fixes: 5355f4439e2e ("ethdev: introduce generic IP/UDP tunnel checksum and TSO")
>>
>> Cc: xuemingl@mellanox.com
>>
>> Signed-off-by: Dekel Peled <dekelp@mellanox.com>
> 
> Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>

     Cc: stable@dpdk.org

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

Patch

diff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c
index 3f8de93..5004b9f 100644
--- a/lib/librte_ethdev/rte_ethdev.c
+++ b/lib/librte_ethdev/rte_ethdev.c
@@ -156,6 +156,8 @@  struct rte_eth_xstats_name_off {
 	RTE_TX_OFFLOAD_BIT2STR(MULTI_SEGS),
 	RTE_TX_OFFLOAD_BIT2STR(MBUF_FAST_FREE),
 	RTE_TX_OFFLOAD_BIT2STR(SECURITY),
+	RTE_TX_OFFLOAD_BIT2STR(UDP_TNL_TSO),
+	RTE_TX_OFFLOAD_BIT2STR(IP_TNL_TSO),
 };
 
 #undef RTE_TX_OFFLOAD_BIT2STR