[v2] net/ixgbe: update PMD Tx offload mask

Message ID 20181025145840.38440-1-zhirun.yan@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Qi Zhang
Headers
Series [v2] net/ixgbe: update PMD Tx offload mask |

Checks

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

Commit Message

Yan, Zhirun Oct. 25, 2018, 2:58 p.m. UTC
  Tx offload mask is updated in following commit: commit 1037ed842c37
("mbuf: fix Tx offload mask"). Currently, the new added offload
flags are not supported in PMD and application will fail to call
PMD transmit prepare function.

This patch updates IXGBE_TX_OFFFLOAD_MASK.

Fixes: 1037ed842c37 ("mbuf: fix Tx offload mask")

Signed-off-by: Zhirun Yan <zhirun.yan@intel.com>
---
 drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++++
 1 file changed, 4 insertions(+)
  

Comments

Xiaolong Ye Oct. 25, 2018, 7:45 a.m. UTC | #1
On 10/25, Zhirun Yan wrote:
>Tx offload mask is updated in following commit: commit 1037ed842c37
>("mbuf: fix Tx offload mask"). Currently, the new added offload
>flags are not supported in PMD and application will fail to call
>PMD transmit prepare function.
>
>This patch updates IXGBE_TX_OFFFLOAD_MASK.
>
>Fixes: 1037ed842c37 ("mbuf: fix Tx offload mask")
>
>Signed-off-by: Zhirun Yan <zhirun.yan@intel.com>
>---
> drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
>index e6f61c900..e462a03d8 100644
>--- a/drivers/net/ixgbe/ixgbe_rxtx.c
>+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
>@@ -58,6 +58,10 @@
> #endif
> /* Bit Mask to indicate what bits required for building TX context */
> #define IXGBE_TX_OFFLOAD_MASK (			 \
>+		PKT_TX_OUTER_IPV6 |		 \
>+		PKT_TX_OUTER_IPV4 |		 \
>+		PKT_TX_IPV6 |			 \
>+		PKT_TX_IPV4 |			 \
> 		PKT_TX_VLAN_PKT |		 \
> 		PKT_TX_IP_CKSUM |		 \
> 		PKT_TX_L4_MASK |		 \

Reviewed-by: Ye Xiaolong <xiaolong.ye@intel.com>

Thanks,
Xiaolong
  
Qi Zhang Oct. 25, 2018, 3:23 p.m. UTC | #2
> -----Original Message-----
> From: Ye, Xiaolong
> Sent: Thursday, October 25, 2018 2:45 AM
> To: Yan, Zhirun <zhirun.yan@intel.com>
> Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zhang@intel.com>; Xing, Beilei
> <beilei.xing@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v2] net/ixgbe: update PMD Tx offload mask
> 
> On 10/25, Zhirun Yan wrote:
> >Tx offload mask is updated in following commit: commit 1037ed842c37
> >("mbuf: fix Tx offload mask"). Currently, the new added offload flags
> >are not supported in PMD and application will fail to call PMD transmit
> >prepare function.
> >
> >This patch updates IXGBE_TX_OFFFLOAD_MASK.
> >
> >Fixes: 1037ed842c37 ("mbuf: fix Tx offload mask")
> >
> >Signed-off-by: Zhirun Yan <zhirun.yan@intel.com>
> >---
> > drivers/net/ixgbe/ixgbe_rxtx.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> >diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c
> >b/drivers/net/ixgbe/ixgbe_rxtx.c index e6f61c900..e462a03d8 100644
> >--- a/drivers/net/ixgbe/ixgbe_rxtx.c
> >+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
> >@@ -58,6 +58,10 @@
> > #endif
> > /* Bit Mask to indicate what bits required for building TX context */
> > #define IXGBE_TX_OFFLOAD_MASK (			 \
> >+		PKT_TX_OUTER_IPV6 |		 \
> >+		PKT_TX_OUTER_IPV4 |		 \
> >+		PKT_TX_IPV6 |			 \
> >+		PKT_TX_IPV4 |			 \
> > 		PKT_TX_VLAN_PKT |		 \
> > 		PKT_TX_IP_CKSUM |		 \
> > 		PKT_TX_L4_MASK |		 \
> 
> Reviewed-by: Ye Xiaolong <xiaolong.ye@intel.com>

Applied to dpdk-next-net-intel.

Thanks
Qi

> 
> Thanks,
> Xiaolong
  

Patch

diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c
index e6f61c900..e462a03d8 100644
--- a/drivers/net/ixgbe/ixgbe_rxtx.c
+++ b/drivers/net/ixgbe/ixgbe_rxtx.c
@@ -58,6 +58,10 @@ 
 #endif
 /* Bit Mask to indicate what bits required for building TX context */
 #define IXGBE_TX_OFFLOAD_MASK (			 \
+		PKT_TX_OUTER_IPV6 |		 \
+		PKT_TX_OUTER_IPV4 |		 \
+		PKT_TX_IPV6 |			 \
+		PKT_TX_IPV4 |			 \
 		PKT_TX_VLAN_PKT |		 \
 		PKT_TX_IP_CKSUM |		 \
 		PKT_TX_L4_MASK |		 \