Skip to content

Commit

Permalink
gve: change to use page_pool_put_full_page when recycling pages
Browse files Browse the repository at this point in the history
The driver currently uses page_pool_put_page() to recycle
page pool pages. Since gve uses split pages, if the fragment
being recycled is not the last fragment in the page, there
is no dma sync operation. When the last fragment is recycled,
dma sync is performed by page pool infra according to the
value passed as dma_sync_size which right now is set to the
size of fragment.

But the correct thing to do is to dma sync the entire page when
the last fragment is recycled. Hence change to using
page_pool_put_full_page().

Link: https://lore.kernel.org/netdev/89d7ce83-cc1d-4791-87b5-6f7af29a031d@huawei.com/

Suggested-by: Yunsheng Lin <linyunsheng@huawei.com>
Reviewed-by: Praveen Kaligineedi <pkaligineedi@google.com>
Reviewed-by: Willem de Bruijn <willemb@google.com>
Signed-off-by: Harshitha Ramamurthy <hramamurthy@google.com>
Reviewed-by: Yunsheng Lin <linyunsheng@huawei.com>
Fixes: ebdfae0 ("gve: adopt page pool for DQ RDA mode")
Link: https://patch.msgid.link/20241023221141.3008011-1-pkaligineedi@google.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
  • Loading branch information
hramamurthy12 authored and kuba-moo committed Oct 31, 2024
1 parent e110225 commit 4ddf7cc
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions drivers/net/ethernet/google/gve/gve_buffer_mgmt_dqo.c
Original file line number Diff line number Diff line change
Expand Up @@ -210,8 +210,7 @@ void gve_free_to_page_pool(struct gve_rx_ring *rx,
if (!page)
return;

page_pool_put_page(page->pp, page, buf_state->page_info.buf_size,
allow_direct);
page_pool_put_full_page(page->pp, page, allow_direct);
buf_state->page_info.page = NULL;
}

Expand Down

0 comments on commit 4ddf7cc

Please sign in to comment.