Subject: Re: [PATCH net-next v4 06/12] net: vxlan: make vxlan_snoop() return drop reasons

From: Simon Horman <[email-masked]>
Date: Wed, 2 Oct 2024 14:35:43 +0100

On Tue, Oct 01, 2024 at 03:32:19PM +0800, Menglong Dong wrote:
> Change the return type of vxlan_snoop() from bool to enum
> skb_drop_reason. In this commit, two drop reasons are introduced:
> 
>   SKB_DROP_REASON_VXLAN_INVALID_SMAC
>   SKB_DROP_REASON_VXLAN_ENTRY_EXISTS
> 
> Signed-off-by: Menglong Dong <[email-masked]>
> ---
> v4:
> - rename SKB_DROP_REASON_VXLAN_INVALID_SMAC to
>   SKB_DROP_REASON_MAC_INVALID_SOURCE

super-nit: SKB_DROP_REASON_VXLAN_INVALID_SMAC was renamed in the code below
           but not the patch description above

In any case, this looks good to me.

Reviewed-by: Simon Horman <[email-masked]>