NACK: [PATCH 1/4] net: mana: Use napi_build_skb in RX path
Roxana Nicolescu
roxana.nicolescu at canonical.com
Wed Apr 19 08:21:35 UTC 2023
On 18/04/2023 17:56, Tim Gardner wrote:
> From: Haiyang Zhang <haiyangz at microsoft.com>
>
> BugLink: https://bugs.launchpad.net/bugs/2016898
>
> Use napi_build_skb() instead of build_skb() to take advantage of the
> NAPI percpu caches to obtain skbuff_head.
>
> Signed-off-by: Haiyang Zhang <haiyangz at microsoft.com>
> Reviewed-by: Jesse Brandeburg <jesse.brandeburg at intel.com>
> Signed-off-by: David S. Miller <davem at davemloft.net>
> (cherry picked from commit ce518bc3e9ca342309995c9270c3ec4892963695 linux-next)
> Signed-off-by: Tim Gardner <tim.gardner at canonical.com>
> ---
> drivers/net/ethernet/microsoft/mana/mana_en.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c b/drivers/net/ethernet/microsoft/mana/mana_en.c
> index 55bf40e5ee71..a1b7905ed2f7 100644
> --- a/drivers/net/ethernet/microsoft/mana/mana_en.c
> +++ b/drivers/net/ethernet/microsoft/mana/mana_en.c
> @@ -1188,7 +1188,7 @@ static void mana_post_pkt_rxq(struct mana_rxq *rxq)
> static struct sk_buff *mana_build_skb(void *buf_va, uint pkt_len,
> struct xdp_buff *xdp)
> {
> - struct sk_buff *skb = build_skb(buf_va, PAGE_SIZE);
> + struct sk_buff *skb = napi_build_skb(buf_va, PAGE_SIZE);
>
> if (!skb)
> return NULL;
>
This does not apply to jammy because commit "net: mana: Add XDP support"
is missing, thus `mana_build_skb` func is not defined.
Roxana
More information about the kernel-team
mailing list