Re: [PATCH net-next 08/10] bnxt_en: Reallocate Rx completion ring for TPH support

[Michal Swiatkowski <michal.swiatkowski@linux.intel.com>]

On Tue, Jan 14, 2025 at 01:42:42PM -0800, Michael Chan wrote:
> On Mon, Jan 13, 2025 at 12:38 AM Michal Swiatkowski
> <michal.swiatkowski@linux.intel.com> wrote:
> >
> > On Sun, Jan 12, 2025 at 10:39:25PM -0800, Michael Chan wrote:
> > > From: Somnath Kotur <somnath.kotur@broadcom.com>
> > > @@ -15669,11 +15689,13 @@ static int bnxt_queue_stop(struct net_device *dev, void *qmem, int idx)
> > >       cancel_work_sync(&rxr->bnapi->cp_ring.dim.work);
> > >       bnxt_hwrm_rx_ring_free(bp, rxr, false);
> > >       bnxt_hwrm_rx_agg_ring_free(bp, rxr, false);
> > > -     rxr->rx_next_cons = 0;
> > Unrelated?
> 
> This line is unneeded.  It gets overwritten by the clone during
> queue_start().  We remove it since we are making related changes in
> this function.  I'll add a comment about this in the Changelog.
> Thanks.

Sure, thanks.

> 
> >
> > >       page_pool_disable_direct_recycling(rxr->page_pool);
> > >       if (bnxt_separate_head_pool())
> > >               page_pool_disable_direct_recycling(rxr->head_pool);
> > >
> > > +     bnxt_hwrm_cp_ring_free(bp, rxr->rx_cpr);
> > > +     bnxt_clear_one_cp_ring(bp, rxr->rx_cpr);
> > > +
> > >       memcpy(qmem, rxr, sizeof(*rxr));
> > >       bnxt_init_rx_ring_struct(bp, qmem);
> > >
> >
> > Rest looks fine:
> > Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>