diff mbox

[dpdk-dev] net/ixgbe: fix CRC strip enabling

Message ID 1518068715-76903-1-git-send-email-xiaoyun.li@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Helin Zhang
Headers

Checks

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

Commit Message

Li, Xiaoyun Feb. 8, 2018, 5:45 a.m. UTC 
  New testpmd will get CRC strip offload from rx_offload_capa. Ixgbe card
has the capability. This patch adds CRC strip flag into rx_offload_capa
in ixgbe and ixgbevf.

Fixes: 8b9bd0efe0b6 ("app/testpmd: disable Rx VLAN offloads by default")
Cc: stable@dpdk.org

Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
---
 drivers/net/ixgbe/ixgbe_ethdev.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Wenzhuo Lu Feb. 8, 2018, 6:07 a.m. UTC | #1 
Hi,

> -----Original Message-----
> From: Li, Xiaoyun
> Sent: Thursday, February 8, 2018 1:45 PM
> To: Lu, Wenzhuo <wenzhuo.lu@intel.com>
> Cc: dev@dpdk.org; Zhang, Helin <helin.zhang@intel.com>; Li, Xiaoyun
> <xiaoyun.li@intel.com>; stable@dpdk.org
> Subject: [PATCH] net/ixgbe: fix CRC strip enabling
> 
> New testpmd will get CRC strip offload from rx_offload_capa. Ixgbe card has
> the capability. This patch adds CRC strip flag into rx_offload_capa in ixgbe
> and ixgbevf.
> 
> Fixes: 8b9bd0efe0b6 ("app/testpmd: disable Rx VLAN offloads by default")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Zhang, Helin Feb. 8, 2018, 6:19 a.m. UTC | #2 
> -----Original Message-----
> From: Lu, Wenzhuo
> Sent: Thursday, February 8, 2018 2:08 PM
> To: Li, Xiaoyun
> Cc: dev@dpdk.org; Zhang, Helin; stable@dpdk.org
> Subject: RE: [PATCH] net/ixgbe: fix CRC strip enabling
> 
> Hi,
> 
> > -----Original Message-----
> > From: Li, Xiaoyun
> > Sent: Thursday, February 8, 2018 1:45 PM
> > To: Lu, Wenzhuo <wenzhuo.lu@intel.com>
> > Cc: dev@dpdk.org; Zhang, Helin <helin.zhang@intel.com>; Li, Xiaoyun
> > <xiaoyun.li@intel.com>; stable@dpdk.org
> > Subject: [PATCH] net/ixgbe: fix CRC strip enabling
> >
> > New testpmd will get CRC strip offload from rx_offload_capa. Ixgbe
> > card has the capability. This patch adds CRC strip flag into
> > rx_offload_capa in ixgbe and ixgbevf.
> >
> > Fixes: 8b9bd0efe0b6 ("app/testpmd: disable Rx VLAN offloads by
> > default")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Xiaoyun Li <xiaoyun.li@intel.com>
> Acked-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
Applied to dpdk-next-dpdk-intel, with commit log changes. Thanks!

/Helin
diff mbox

Patch

diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
index c423cf2..4483258 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.c
+++ b/drivers/net/ixgbe/ixgbe_ethdev.c
@@ -3651,7 +3651,8 @@  ixgbe_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)
 		DEV_RX_OFFLOAD_VLAN_STRIP |
 		DEV_RX_OFFLOAD_IPV4_CKSUM |
 		DEV_RX_OFFLOAD_UDP_CKSUM  |
-		DEV_RX_OFFLOAD_TCP_CKSUM;
+		DEV_RX_OFFLOAD_TCP_CKSUM  |
+		DEV_RX_OFFLOAD_CRC_STRIP;
 
 	/*
 	 * RSC is only supported by 82599 and x540 PF devices in a non-SR-IOV
@@ -3798,7 +3799,8 @@  ixgbevf_dev_info_get(struct rte_eth_dev *dev,
 	dev_info->rx_offload_capa = DEV_RX_OFFLOAD_VLAN_STRIP |
 				DEV_RX_OFFLOAD_IPV4_CKSUM |
 				DEV_RX_OFFLOAD_UDP_CKSUM  |
-				DEV_RX_OFFLOAD_TCP_CKSUM;
+				DEV_RX_OFFLOAD_TCP_CKSUM  |
+				DEV_RX_OFFLOAD_CRC_STRIP;
 	dev_info->tx_offload_capa = DEV_TX_OFFLOAD_VLAN_INSERT |
 				DEV_TX_OFFLOAD_IPV4_CKSUM  |
 				DEV_TX_OFFLOAD_UDP_CKSUM   |