Thread (6 messages) 6 messages, 4 authors, 2021-02-24

Re: [dpdk-dev] [PATCH 21.05] app/testpmd: count outer IP checksum errors

From: Thomas Monjalon <hidden>
Date: 2021-02-23 20:25:35

Possibly related (same subject, not in this thread)

23/02/2021 19:39, Ajit Khaparde:
On Tue, Feb 23, 2021 at 10:36 AM Ferruh Yigit [off-list ref] wrote:
quoted
On 1/31/2021 11:53 AM, Wisam Monther wrote:
quoted
Hi,
quoted
-----Original Message-----
From: dev <redacted> On Behalf Of Lance Richardson
Sent: Saturday, January 30, 2021 12:36 AM
To: Wenzhuo Lu <redacted>; Xiaoyun Li
[off-list ref]; Bernard Iremonger
[off-list ref]
Cc: dev@dpdk.org; Ajit Kumar Khaparde <ajit.khaparde@broadcom.com>;
Kalesh Anakkur Purayil [off-list ref]
Subject: [dpdk-dev] [PATCH 21.05] app/testpmd: count outer IP checksum
errors

Count and display outer IP checksum errors in the checksum forwarder.

Example forwarder stats output:
   RX-packets: 158            RX-dropped: 0             RX-total: 158
   Bad-ipcsum: 48             Bad-l4csum: 48            Bad-outer-l4csum: 6
   Bad-outer-ipcsum: 40
   TX-packets: 0              TX-dropped: 0             TX-total: 0

Signed-off-by: Lance Richardson <redacted>
Reviewed-by: Ajit Kumar Khaparde <ajit.khaparde@broadcom.com>
Reviewed-by: Kalesh Anakkur Purayil <kalesh-
anakkur.purayil@broadcom.com>
Acked-by: Wisam Jaddo <redacted>
Reviewed-by: Ferruh Yigit <redacted>
Applied to dpdk-next-net/main, thanks.



'PKT_RX_EIP_CKSUM_BAD' is documented in mbuf header as:
"/** External IP header checksum error. */"

I think 'External' wording is confusing, as well as 'EIP' abbreviation, what do
you think to create another macro alias to existing one, and mark the old one as
deprecated?
+1
+1

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help