Re: [PATCH net] bnxt: fix head underflow on XDP head-grow

[Michael Chan]

On Tue, Jun 9, 2026 at 7:51 AM Joe Damato <joe@xxxxxxx> wrote:
>
> On Mon, Jun 08, 2026 at 09:22:07PM -0700, Michael Chan wrote:
> > On Mon, Jun 8, 2026 at 1:31 PM Joe Damato <joe@xxxxxxx> wrote:
> > >
> > > To fix this, we need to do a bit of math to recover the offset if this
> > > is a page fragment since it is not passed into rx_skb_func
> > > (bnxt_rx_multi_page_skb, in this case).
> >
> > I wonder if we should add an offset field to struct bnxt_sw_rx_bd to
> > simplify things for page mode.  Struct bnxt_sw_rx_agg_bd has the
> > offset field for a similar purpose.  Thanks.
>
> I don't mind doing that, but I wonder if that's better material for net-next?
>
> In other words, we get the minimal fix in (this patch?) and then do the
> cleanup and struct tweaking as a follow-up in net-next?
>
> I could definitely be wrong; I had just assumed patches for net were supposed
> to be as minimal as possible.

I think some minor refactoring/struct changes to make the fix
cleaner/simpler should be fine for -net.  I think the total number of
line changes should be about the same either way.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature