Messages in this thread | | | Date | Tue, 24 Aug 2021 16:52:08 -0700 | From | Jakub Kicinski <> | Subject | Re: [PATCH v3 net-next] net: bridge: change return type of br_handle_ingress_vlan_tunnel |
| |
On Mon, 23 Aug 2021 13:25:20 +0300 Nikolay Aleksandrov wrote: > On 23/08/2021 13:21, Kangmin Park wrote: > > br_handle_ingress_vlan_tunnel() is only referenced in > > br_handle_frame(). If br_handle_ingress_vlan_tunnel() is called and > > return non-zero value, goto drop in br_handle_frame(). > > > > But, br_handle_ingress_vlan_tunnel() always return 0. So, the > > routines that check the return value and goto drop has no meaning. > > > > Therefore, change return type of br_handle_ingress_vlan_tunnel() to > > void and remove if statement of br_handle_frame(). > > > > Signed-off-by: Kangmin Park <l4stpr0gr4m@gmail.com> > > Looks good to me, > Acked-by: Nikolay Aleksandrov <nikolay@nvidia.com>
Applied, thanks!
| |