Commit 7639a6e0 authored by Shannon Nelson's avatar Shannon Nelson Committed by Jakub Kicinski

ionic: rename ionic_xdp_rx_put_bufs

We aren't "putting" buf, we're just unlinking them from our tracking in
order to let the XDP_TX and XDP_REDIRECT tx clean paths take care of the
pages when they are done with them.  This rename clears up the intent.
Signed-off-by: default avatarShannon Nelson <shannon.nelson@amd.com>
Signed-off-by: default avatarBrett Creeley <brett.creeley@amd.com>
Link: https://patch.msgid.link/20240906232623.39651-3-brett.creeley@amd.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 4a0ec348
...@@ -481,7 +481,7 @@ int ionic_xdp_xmit(struct net_device *netdev, int n, ...@@ -481,7 +481,7 @@ int ionic_xdp_xmit(struct net_device *netdev, int n,
return nxmit; return nxmit;
} }
static void ionic_xdp_rx_put_bufs(struct ionic_queue *q, static void ionic_xdp_rx_unlink_bufs(struct ionic_queue *q,
struct ionic_buf_info *buf_info, struct ionic_buf_info *buf_info,
int nbufs) int nbufs)
{ {
...@@ -600,7 +600,7 @@ static bool ionic_run_xdp(struct ionic_rx_stats *stats, ...@@ -600,7 +600,7 @@ static bool ionic_run_xdp(struct ionic_rx_stats *stats,
netdev_dbg(netdev, "tx ionic_xdp_post_frame err %d\n", err); netdev_dbg(netdev, "tx ionic_xdp_post_frame err %d\n", err);
goto out_xdp_abort; goto out_xdp_abort;
} }
ionic_xdp_rx_put_bufs(rxq, buf_info, nbufs); ionic_xdp_rx_unlink_bufs(rxq, buf_info, nbufs);
stats->xdp_tx++; stats->xdp_tx++;
/* the Tx completion will free the buffers */ /* the Tx completion will free the buffers */
...@@ -612,7 +612,7 @@ static bool ionic_run_xdp(struct ionic_rx_stats *stats, ...@@ -612,7 +612,7 @@ static bool ionic_run_xdp(struct ionic_rx_stats *stats,
netdev_dbg(netdev, "xdp_do_redirect err %d\n", err); netdev_dbg(netdev, "xdp_do_redirect err %d\n", err);
goto out_xdp_abort; goto out_xdp_abort;
} }
ionic_xdp_rx_put_bufs(rxq, buf_info, nbufs); ionic_xdp_rx_unlink_bufs(rxq, buf_info, nbufs);
rxq->xdp_flush = true; rxq->xdp_flush = true;
stats->xdp_redirect++; stats->xdp_redirect++;
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment