> -----Original Message-----
> From: Thomas Monjalon <[email protected]>
> Sent: Tuesday, September 1, 2020 1:39 PM
> To: Nipun Gupta <[email protected]>; Stephen Hemminger
> <[email protected]>; [email protected]
> Cc: [email protected]; [email protected]; [email protected];
> Hemant Agrawal <[email protected]>; Rohit Raj <[email protected]>;
> [email protected]
> Subject: Re: [dpdk-dev] [PATCH] ethdev: add rx offload to drop error packets
>
> 31/08/2020 19:00, Stephen Hemminger:
> > On Mon, 31 Aug 2020 13:23:33 +0530
> > Nipun Gupta <[email protected]> wrote:
> >
> > > This change adds a RX offload capability where hardware can drop the
> > > packets in case there is an error in the packet such as L3 checksum
> > > error or L4 checksum.
> > >
> > > Signed-off-by: Nipun Gupta <[email protected]>
> > > Signed-off-by: Rohit Raj <[email protected]>
> > > ---
> > > +#define DEV_RX_OFFLOAD_ERR_PKT_DROP 0x00100000
>
> Please add RTE_ prefix, even if older macros don't have it.
> We could (in a separate effort) alias old ones with RTE_ prefixed names.
Agree, will update and send the change along with testpmd and dpaa2 driver
change.
>
> > Could/should this be an rte_flow action as well?
>
> I feel rte_flow API is not appropriate here.
> Ori, any opinion?
>
