[v2,04/13] net/vmxnet3: replace abort() with rte_panic()

Message ID 20230818174537.290222-5-stephen@networkplumber.org (mailing list archive)
State Rejected, archived
Delegated to: Thomas Monjalon
Headers
Series Replace us of term abort |

Checks

Context Check Description
ci/checkpatch warning coding style issues

Commit Message

Stephen Hemminger Aug. 18, 2023, 5:45 p.m. UTC
  Do the same thing but be better about wording.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/vmxnet3/vmxnet3_rxtx.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
  

Patch

diff --git a/drivers/net/vmxnet3/vmxnet3_rxtx.c b/drivers/net/vmxnet3/vmxnet3_rxtx.c
index 380f41f98be2..695a1877185f 100644
--- a/drivers/net/vmxnet3/vmxnet3_rxtx.c
+++ b/drivers/net/vmxnet3/vmxnet3_rxtx.c
@@ -565,9 +565,9 @@  vmxnet3_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts,
 						dgram_cksum);
 				break;
 			default:
-				PMD_TX_LOG(WARNING, "requested cksum offload not supported %#llx",
-					   txm->ol_flags & RTE_MBUF_F_TX_L4_MASK);
-				abort();
+				/* should probably count as tx error and drop instead */
+				rte_panic("unsupported cksum offload: %"PRIx64"\n",
+					  txm->ol_flags);
 			}
 			deferred++;
 		} else {