Commit a79afa78 authored by Alexander Lobakin's avatar Alexander Lobakin Committed by Jakub Kicinski
Browse files

net: use the new dev_page_is_reusable() instead of private versions 



Now we can remove a bunch of identical functions from the drivers and
make them use common dev_page_is_reusable(). All {,un}likely() checks
are omitted since it's already present in this helper.
Also update some comments near the call sites.

Suggested-by: default avatarDavid Rientjes <rientjes@google.com>
Suggested-by: default avatarJakub Kicinski <kuba@kernel.org>
Cc: John Hubbard <jhubbard@nvidia.com>
Signed-off-by: default avatarAlexander Lobakin <alobakin@pm.me>
Reviewed-by: default avatarJesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent bc38f30f

drivers/net/ethernet/hisilicon/hns3/hns3_enet.c

+6 −11
Original line number Diff line number Diff line
@@ -2800,12 +2800,6 @@ static void hns3_nic_alloc_rx_buffers(struct hns3_enet_ring *ring, 
	writel(i, ring->tqp->io_base + HNS3_RING_RX_RING_HEAD_REG);

}



static bool hns3_page_is_reusable(struct page *page)

{

	return page_to_nid(page) == numa_mem_id() &&

		!page_is_pfmemalloc(page);

}



static bool hns3_can_reuse_page(struct hns3_desc_cb *cb)

{

	return (page_count(cb->priv) - cb->pagecnt_bias) == 1;
@@ -2823,10 +2817,11 @@ static void hns3_nic_reuse_page(struct sk_buff *skb, int i, 
	skb_add_rx_frag(skb, i, desc_cb->priv, desc_cb->page_offset + pull_len,

			size - pull_len, truesize);



	/* Avoid re-using remote pages, or the stack is still using the page

	 * when page_offset rollback to zero, flag default unreuse

	/* Avoid re-using remote and pfmemalloc pages, or the stack is still

	 * using the page when page_offset rollback to zero, flag default

	 * unreuse

	 */

	if (unlikely(!hns3_page_is_reusable(desc_cb->priv)) ||

	if (!dev_page_is_reusable(desc_cb->priv) ||

	    (!desc_cb->page_offset && !hns3_can_reuse_page(desc_cb))) {

		__page_frag_cache_drain(desc_cb->priv, desc_cb->pagecnt_bias);

		return;
@@ -3083,8 +3078,8 @@ static int hns3_alloc_skb(struct hns3_enet_ring *ring, unsigned int length, 
	if (length <= HNS3_RX_HEAD_SIZE) {

		memcpy(__skb_put(skb, length), va, ALIGN(length, sizeof(long)));



		/* We can reuse buffer as-is, just make sure it is local */

		if (likely(hns3_page_is_reusable(desc_cb->priv)))

		/* We can reuse buffer as-is, just make sure it is reusable */

		if (dev_page_is_reusable(desc_cb->priv))

			desc_cb->reuse_flag = 1;

		else /* This page cannot be reused so discard it */

			__page_frag_cache_drain(desc_cb->priv,

drivers/net/ethernet/intel/fm10k/fm10k_main.c

+4 −9
Original line number Diff line number Diff line
@@ -194,17 +194,12 @@ static void fm10k_reuse_rx_page(struct fm10k_ring *rx_ring, 
					 DMA_FROM_DEVICE);

}



static inline bool fm10k_page_is_reserved(struct page *page)

{

	return (page_to_nid(page) != numa_mem_id()) || page_is_pfmemalloc(page);

}



static bool fm10k_can_reuse_rx_page(struct fm10k_rx_buffer *rx_buffer,

				    struct page *page,

				    unsigned int __maybe_unused truesize)

{

	/* avoid re-using remote pages */

	if (unlikely(fm10k_page_is_reserved(page)))

	/* avoid re-using remote and pfmemalloc pages */

	if (!dev_page_is_reusable(page))

		return false;



#if (PAGE_SIZE < 8192)
@@ -265,8 +260,8 @@ static bool fm10k_add_rx_frag(struct fm10k_rx_buffer *rx_buffer, 
	if (likely(size <= FM10K_RX_HDR_LEN)) {

		memcpy(__skb_put(skb, size), va, ALIGN(size, sizeof(long)));



		/* page is not reserved, we can reuse buffer as-is */

		if (likely(!fm10k_page_is_reserved(page)))

		/* page is reusable, we can reuse buffer as-is */

		if (dev_page_is_reusable(page))

			return true;



		/* this page cannot be reused so discard it */

drivers/net/ethernet/intel/i40e/i40e_txrx.c

+1 −14
Original line number Diff line number Diff line
@@ -1843,19 +1843,6 @@ static bool i40e_cleanup_headers(struct i40e_ring *rx_ring, struct sk_buff *skb, 
	return false;

}



/**

 * i40e_page_is_reusable - check if any reuse is possible

 * @page: page struct to check

 *

 * A page is not reusable if it was allocated under low memory

 * conditions, or it's not in the same NUMA node as this CPU.

 */

static inline bool i40e_page_is_reusable(struct page *page)

{

	return (page_to_nid(page) == numa_mem_id()) &&

		!page_is_pfmemalloc(page);

}



/**

 * i40e_can_reuse_rx_page - Determine if this page can be reused by

 * the adapter for another receive
@@ -1891,7 +1878,7 @@ static bool i40e_can_reuse_rx_page(struct i40e_rx_buffer *rx_buffer, 
	struct page *page = rx_buffer->page;



	/* Is any reuse possible? */

	if (unlikely(!i40e_page_is_reusable(page)))

	if (!dev_page_is_reusable(page))

		return false;



#if (PAGE_SIZE < 8192)

drivers/net/ethernet/intel/iavf/iavf_txrx.c

+1 −14
Original line number Diff line number Diff line
@@ -1141,19 +1141,6 @@ static void iavf_reuse_rx_page(struct iavf_ring *rx_ring, 
	new_buff->pagecnt_bias	= old_buff->pagecnt_bias;

}



/**

 * iavf_page_is_reusable - check if any reuse is possible

 * @page: page struct to check

 *

 * A page is not reusable if it was allocated under low memory

 * conditions, or it's not in the same NUMA node as this CPU.

 */

static inline bool iavf_page_is_reusable(struct page *page)

{

	return (page_to_nid(page) == numa_mem_id()) &&

		!page_is_pfmemalloc(page);

}



/**

 * iavf_can_reuse_rx_page - Determine if this page can be reused by

 * the adapter for another receive
@@ -1187,7 +1174,7 @@ static bool iavf_can_reuse_rx_page(struct iavf_rx_buffer *rx_buffer) 
	struct page *page = rx_buffer->page;



	/* Is any reuse possible? */

	if (unlikely(!iavf_page_is_reusable(page)))

	if (!dev_page_is_reusable(page))

		return false;



#if (PAGE_SIZE < 8192)

drivers/net/ethernet/intel/ice/ice_txrx.c

+2 −11
Original line number Diff line number Diff line
@@ -728,15 +728,6 @@ bool ice_alloc_rx_bufs(struct ice_ring *rx_ring, u16 cleaned_count) 
	return !!cleaned_count;

}



/**

 * ice_page_is_reserved - check if reuse is possible

 * @page: page struct to check

 */

static bool ice_page_is_reserved(struct page *page)

{

	return (page_to_nid(page) != numa_mem_id()) || page_is_pfmemalloc(page);

}



/**

 * ice_rx_buf_adjust_pg_offset - Prepare Rx buffer for reuse

 * @rx_buf: Rx buffer to adjust
@@ -775,8 +766,8 @@ ice_can_reuse_rx_page(struct ice_rx_buf *rx_buf, int rx_buf_pgcnt) 
	unsigned int pagecnt_bias = rx_buf->pagecnt_bias;

	struct page *page = rx_buf->page;



	/* avoid re-using remote pages */

	if (unlikely(ice_page_is_reserved(page)))

	/* avoid re-using remote and pfmemalloc pages */

	if (!dev_page_is_reusable(page))

		return false;



#if (PAGE_SIZE < 8192)