[PATCH net-next v2] nfp: using napi_build_skb() to replace build_skb()

[PATCH net-next v2] nfp: using napi_build_skb() to replace build_skb()

[Louis Peens]

From: Fei Qin <fei.qin@corigine.com>

The napi_build_skb() can reuse the skb in skb cache per CPU or
can allocate skbs in bulk, which helps improve the performance.

Signed-off-by: Fei Qin <fei.qin@corigine.com>
Signed-off-by: Louis Peens <louis.peens@corigine.com>
---
v1->v2:
Dropped the changes for the *_ctrl_* paths, as they were not within
napi context. Thanks Wojciech for pointing this out.

 drivers/net/ethernet/netronome/nfp/nfd3/dp.c | 2 +-
 drivers/net/ethernet/netronome/nfp/nfdk/dp.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/netronome/nfp/nfd3/dp.c b/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
index 0cc026b0aefd..17381bfc15d7 100644
--- a/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
+++ b/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
@@ -1070,7 +1070,7 @@ static int nfp_nfd3_rx(struct nfp_net_rx_ring *rx_ring, int budget)
 				nfp_repr_inc_rx_stats(netdev, pkt_len);
 		}
 
-		skb = build_skb(rxbuf->frag, true_bufsz);
+		skb = napi_build_skb(rxbuf->frag, true_bufsz);
 		if (unlikely(!skb)) {
 			nfp_nfd3_rx_drop(dp, r_vec, rx_ring, rxbuf, NULL);
 			continue;
diff --git a/drivers/net/ethernet/netronome/nfp/nfdk/dp.c b/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
index 33b6d74adb4b..8d78c6faefa8 100644
--- a/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
+++ b/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
@@ -1189,7 +1189,7 @@ static int nfp_nfdk_rx(struct nfp_net_rx_ring *rx_ring, int budget)
 				nfp_repr_inc_rx_stats(netdev, pkt_len);
 		}
 
-		skb = build_skb(rxbuf->frag, true_bufsz);
+		skb = napi_build_skb(rxbuf->frag, true_bufsz);
 		if (unlikely(!skb)) {
 			nfp_nfdk_rx_drop(dp, r_vec, rx_ring, rxbuf, NULL);
 			continue;
-- 
2.34.1

Re: [PATCH net-next v2] nfp: using napi_build_skb() to replace build_skb()

[Wojciech Drewek]

On 26.10.2023 10:00, Louis Peens wrote:
> From: Fei Qin <fei.qin@corigine.com>
> 
> The napi_build_skb() can reuse the skb in skb cache per CPU or
> can allocate skbs in bulk, which helps improve the performance.
> 
> Signed-off-by: Fei Qin <fei.qin@corigine.com>
> Signed-off-by: Louis Peens <louis.peens@corigine.com>
> ---
> v1->v2:
> Dropped the changes for the *_ctrl_* paths, as they were not within
> napi context. Thanks Wojciech for pointing this out.

No problem :)
Reviewed-by: Wojciech Drewek <wojciech.drewek@intel.com>

> 
>  drivers/net/ethernet/netronome/nfp/nfd3/dp.c | 2 +-
>  drivers/net/ethernet/netronome/nfp/nfdk/dp.c | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ethernet/netronome/nfp/nfd3/dp.c b/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
> index 0cc026b0aefd..17381bfc15d7 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfd3/dp.c
> @@ -1070,7 +1070,7 @@ static int nfp_nfd3_rx(struct nfp_net_rx_ring *rx_ring, int budget)
>  				nfp_repr_inc_rx_stats(netdev, pkt_len);
>  		}
>  
> -		skb = build_skb(rxbuf->frag, true_bufsz);
> +		skb = napi_build_skb(rxbuf->frag, true_bufsz);
>  		if (unlikely(!skb)) {
>  			nfp_nfd3_rx_drop(dp, r_vec, rx_ring, rxbuf, NULL);
>  			continue;
> diff --git a/drivers/net/ethernet/netronome/nfp/nfdk/dp.c b/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
> index 33b6d74adb4b..8d78c6faefa8 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfdk/dp.c
> @@ -1189,7 +1189,7 @@ static int nfp_nfdk_rx(struct nfp_net_rx_ring *rx_ring, int budget)
>  				nfp_repr_inc_rx_stats(netdev, pkt_len);
>  		}
>  
> -		skb = build_skb(rxbuf->frag, true_bufsz);
> +		skb = napi_build_skb(rxbuf->frag, true_bufsz);
>  		if (unlikely(!skb)) {
>  			nfp_nfdk_rx_drop(dp, r_vec, rx_ring, rxbuf, NULL);
>  			continue;

Re: [PATCH net-next v2] nfp: using napi_build_skb() to replace build_skb()

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Thu, 26 Oct 2023 10:00:58 +0200 you wrote:
> From: Fei Qin <fei.qin@corigine.com>
> 
> The napi_build_skb() can reuse the skb in skb cache per CPU or
> can allocate skbs in bulk, which helps improve the performance.
> 
> Signed-off-by: Fei Qin <fei.qin@corigine.com>
> Signed-off-by: Louis Peens <louis.peens@corigine.com>
> 
> [...]

Here is the summary with links:
  - [net-next,v2] nfp: using napi_build_skb() to replace build_skb()
    https://git.kernel.org/netdev/net-next/c/1a86a77a2328

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html