On Tue, Feb 23, 2021 at 10:36 AM Ferruh Yigit <[email protected]> wrote:
>
> On 1/31/2021 11:53 AM, Wisam Monther wrote:
> > Hi,
> >
> >> -----Original Message-----
> >> From: dev <[email protected]> On Behalf Of Lance Richardson
> >> Sent: Saturday, January 30, 2021 12:36 AM
> >> To: Wenzhuo Lu <[email protected]>; Xiaoyun Li
> >> <[email protected]>; Bernard Iremonger
> >> <[email protected]>
> >> Cc: [email protected]; Ajit Kumar Khaparde <[email protected]>;
> >> Kalesh Anakkur Purayil <[email protected]>
> >> 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 <[email protected]>
> >> Reviewed-by: Ajit Kumar Khaparde <[email protected]>
> >> Reviewed-by: Kalesh Anakkur Purayil <kalesh-
> >> [email protected]>
> >
> > Acked-by: Wisam Jaddo <[email protected]>
> >
>
> Reviewed-by: Ferruh Yigit <[email protected]>
> 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

>

Reply via email to