On Tue, Jun 24, 2025 at 7:41 PM Jiawen Wu jiawenwu@trustnetic.com wrote:
'rx_ring->size' means the count of ring descriptors multiplied by the size of one descriptor. When increasing the count of ring descriptors, it may exceed the limit of pool size.
[ 864.209610] page_pool_create_percpu() gave up with errno -7 [ 864.209613] txgbe 0000:11:00.0: Page pool creation failed: -7
Fix to set the pool_size to the count of ring descriptors.
Fixes: 850b971110b2 ("net: libwx: Allocate Rx and Tx resources") Cc: stable@vger.kernel.org Signed-off-by: Jiawen Wu jiawenwu@trustnetic.com
Looks correct to me. pool_size is meant to be the number of entries in the ptr_ring indeed, not a size in bytes. wx_setup_rx_resources does set up rx_ring->size to be the size in bytes based on the rx_ring->count.
Reviewed-by: Mina Almasry almasrymina@google.com