Re: [RFC net-next 14/15] net/mlx5e: Add Rx data path offload

Cosmin Ratiu
2024-05-30T09:04:55Z

On Wed, 2024-05-29 at 11:38 -0700, Jakub Kicinski wrote:
> On Sun, 12 May 2024 21:54:38 -0400 Willem de Bruijn wrote:
> > > + /* TBD: report errors as SW counters to ethtool, any further handling ? */
> > > + switch (MLX5_NISP_METADATA_SYNDROM(nisp_meta_data)) {
> > > + case MLX5E_NISP_OFFLOAD_RX_SYNDROME_DECRYPTED:
> > > + if (psp_rcv(skb))
> > > + netdev_warn_once(netdev, "PSP handling failed");
> > > + skb->decrypted = 1;
> >
> > Do not set skb->decrypted if psp_rcv failed? But drop the packet and
> > account the drop, likely.
>
> nVidia folks does this seem reasonable?

This seems reasonable. It's also what the comment above the switch
suggests should be done.
psp_rcv unrefs the skb on errors (doesn't even return error in all
cases) and I think it's no longer safe to touch it, unless there's
another ref held somewhere.

I've tried implementing this tweak in the shared repo we have, but it
seems it doesn't have the versions of this patch that you sent.

Cosmin.