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