Re: [PATCH] eth: mlx4: Fix IS_ERR() vs NULL check bug in mlx4_en_create_rx_ring

From: Jakub Kicinski
Date: Tue Aug 05 2025 - 19:09:53 EST


On Tue, 5 Aug 2025 06:50:57 +0400 Miaoqian Lin wrote:
> ring->pp = page_pool_create(&pp);
> - if (!ring->pp)
> + if (IS_ERR(ring->pp))
> goto err_ring;

Thanks for fixing! Looks we previously depended on err being initialized
to -ENOMEM, but since we have an errno now, I think it'd be better to
use it:

if (IS_ERR(ring->pp)) {
err = PTR_ERR(ring->pp);
goto err_ring;
}