test/bonding: don't request CRC STRIP offload

Message ID 20180720155941.35021-1-ferruh.yigit@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series test/bonding: don't request CRC STRIP offload |

Checks

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

Commit Message

Ferruh Yigit July 20, 2018, 3:59 p.m. UTC
  bonding PMD doesn't announce CRC STRIP offload support, don't request
this offload in unit test otherwise unit test fails.

Fixes: f52f1a65e461 ("ethdev: force offloading API rules")
Cc: stable@dpdk.org

Reported-by: Meijuan Zhao <meijuanx.zhao@intel.com>
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
 test/test/test_link_bonding.c | 1 -
 1 file changed, 1 deletion(-)
  

Comments

Thomas Monjalon July 26, 2018, 7:46 p.m. UTC | #1
20/07/2018 17:59, Ferruh Yigit:
> bonding PMD doesn't announce CRC STRIP offload support, don't request
> this offload in unit test otherwise unit test fails.
> 
> Fixes: f52f1a65e461 ("ethdev: force offloading API rules")
> Cc: stable@dpdk.org
> 
> Reported-by: Meijuan Zhao <meijuanx.zhao@intel.com>
> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied, thanks
  

Patch

diff --git a/test/test/test_link_bonding.c b/test/test/test_link_bonding.c
index ff0576a54..0fe1d78eb 100644
--- a/test/test/test_link_bonding.c
+++ b/test/test/test_link_bonding.c
@@ -137,7 +137,6 @@  static struct rte_eth_conf default_pmd_conf = {
 		.mq_mode = ETH_MQ_RX_NONE,
 		.split_hdr_size = 0,
 		.max_rx_pkt_len = ETHER_MAX_LEN,
-		.offloads = DEV_RX_OFFLOAD_CRC_STRIP,
 	},
 	.txmode = {
 		.mq_mode = ETH_MQ_TX_NONE,