Re: [PATCH net] net/stmmac: Set Rx queue page_pool to NULL when freeing DMA resources
From: Jakub Raczynski
Date: Thu Jul 02 2026 - 06:38:44 EST
On Thu, Jul 02, 2026 at 11:24:41AM +0200, Maxime Chevallier wrote:
> >
> > kfree(rx_q->buf_pool);
> > - if (rx_q->page_pool)
> > + if (rx_q->page_pool) {
> > page_pool_destroy(rx_q->page_pool);
> > + rx_q->page_pool = NULL;
> > + }
>
> page_pool_destroy can be passed NULL, as you're fixing this I think you might as well
> drop the if(rx_q->page_pool) at the same time :) I agree with the change otherwise.
>
> with that changed,
>
> Reviewed-by: Maxime Chevallier <maxime.chevallier@xxxxxxxxxxx>
>
> Maxime
>
Agreed, will send V2 soon.
BR
Jakub Raczynski