* [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair
@ 2022-09-27 14:18 Shang XiaoJing
2022-09-28 1:05 ` shangxiaojing
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Shang XiaoJing @ 2022-09-27 14:18 UTC (permalink / raw)
To: simon.horman, kuba, davem, edumazet, pabeni, ast, daniel, hawk,
john.fastabend, niklas.soderlund, oss-drivers, netdev
Cc: shangxiaojing
Use skb_put_data() instead of skb_put() and memcpy(), which is clear.
Signed-off-by: Shang XiaoJing <shangxiaojing@huawei.com>
---
changes in v2:
- no change
---
drivers/net/ethernet/netronome/nfp/nfd3/xsk.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
index 65e243168765..5d9db8c2a5b4 100644
--- a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
+++ b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
@@ -84,7 +84,7 @@ static void nfp_nfd3_xsk_rx_skb(struct nfp_net_rx_ring *rx_ring,
nfp_net_xsk_rx_drop(r_vec, xrxbuf);
return;
}
- memcpy(skb_put(skb, pkt_len), xrxbuf->xdp->data, pkt_len);
+ skb_put_data(skb, xrxbuf->xdp->data, pkt_len);
skb->mark = meta->mark;
skb_set_hash(skb, meta->hash, meta->hash_type);
--
2.17.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair
2022-09-27 14:18 [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair Shang XiaoJing
@ 2022-09-28 1:05 ` shangxiaojing
2022-09-28 8:20 ` Niklas Söderlund
2022-09-29 9:00 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: shangxiaojing @ 2022-09-28 1:05 UTC (permalink / raw)
To: simon.horman, kuba, davem, edumazet, pabeni, ast, daniel, hawk,
john.fastabend, niklas.soderlund, oss-drivers, netdev
On 2022/9/27 22:18, Shang XiaoJing wrote:
> Use skb_put_data() instead of skb_put() and memcpy(), which is clear.
>
> Signed-off-by: Shang XiaoJing <shangxiaojing@huawei.com>
> ---
> changes in v2:
> - no change
> ---
> drivers/net/ethernet/netronome/nfp/nfd3/xsk.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> index 65e243168765..5d9db8c2a5b4 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> @@ -84,7 +84,7 @@ static void nfp_nfd3_xsk_rx_skb(struct nfp_net_rx_ring *rx_ring,
> nfp_net_xsk_rx_drop(r_vec, xrxbuf);
> return;
> }
> - memcpy(skb_put(skb, pkt_len), xrxbuf->xdp->data, pkt_len);
> + skb_put_data(skb, xrxbuf->xdp->data, pkt_len);
>
> skb->mark = meta->mark;
> skb_set_hash(skb, meta->hash, meta->hash_type);
forgot to add the reviewed tag, sorry.
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Thanks,
Shang XiaoJing
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair
2022-09-27 14:18 [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair Shang XiaoJing
2022-09-28 1:05 ` shangxiaojing
@ 2022-09-28 8:20 ` Niklas Söderlund
2022-09-29 9:00 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: Niklas Söderlund @ 2022-09-28 8:20 UTC (permalink / raw)
To: Shang XiaoJing
Cc: simon.horman, kuba, davem, edumazet, pabeni, ast, daniel, hawk,
john.fastabend, oss-drivers, netdev
Hi Shang,
Thanks for your work.
On 2022-09-27 22:18:35 +0800, Shang XiaoJing wrote:
> Use skb_put_data() instead of skb_put() and memcpy(), which is clear.
>
> Signed-off-by: Shang XiaoJing <shangxiaojing@huawei.com>
Reviewed-by: Niklas Söderlund <niklas.soderlund@corigine.com>
> ---
> changes in v2:
> - no change
> ---
> drivers/net/ethernet/netronome/nfp/nfd3/xsk.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> index 65e243168765..5d9db8c2a5b4 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfd3/xsk.c
> @@ -84,7 +84,7 @@ static void nfp_nfd3_xsk_rx_skb(struct nfp_net_rx_ring *rx_ring,
> nfp_net_xsk_rx_drop(r_vec, xrxbuf);
> return;
> }
> - memcpy(skb_put(skb, pkt_len), xrxbuf->xdp->data, pkt_len);
> + skb_put_data(skb, xrxbuf->xdp->data, pkt_len);
>
> skb->mark = meta->mark;
> skb_set_hash(skb, meta->hash, meta->hash_type);
> --
> 2.17.1
>
--
Kind Regards,
Niklas Söderlund
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair
2022-09-27 14:18 [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair Shang XiaoJing
2022-09-28 1:05 ` shangxiaojing
2022-09-28 8:20 ` Niklas Söderlund
@ 2022-09-29 9:00 ` patchwork-bot+netdevbpf
2 siblings, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-09-29 9:00 UTC (permalink / raw)
To: Shang XiaoJing
Cc: simon.horman, kuba, davem, edumazet, pabeni, ast, daniel, hawk,
john.fastabend, niklas.soderlund, oss-drivers, netdev
Hello:
This patch was applied to netdev/net-next.git (master)
by Paolo Abeni <pabeni@redhat.com>:
On Tue, 27 Sep 2022 22:18:35 +0800 you wrote:
> Use skb_put_data() instead of skb_put() and memcpy(), which is clear.
>
> Signed-off-by: Shang XiaoJing <shangxiaojing@huawei.com>
> ---
> changes in v2:
> - no change
>
> [...]
Here is the summary with links:
- [-next,v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair
https://git.kernel.org/netdev/net-next/c/d49e265b66d9
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-09-29 9:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-27 14:18 [PATCH -next v2] nfp: Use skb_put_data() instead of skb_put/memcpy pair Shang XiaoJing
2022-09-28 1:05 ` shangxiaojing
2022-09-28 8:20 ` Niklas Söderlund
2022-09-29 9:00 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.