Messages in this thread | | | Subject | Re: [PATCH v3 net-next] net: bridge: change return type of br_handle_ingress_vlan_tunnel | From | Nikolay Aleksandrov <> | Date | Mon, 23 Aug 2021 13:25:20 +0300 |
| |
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> > --- > v3: > - remove unnecessary return statement > > v2: > - cleanup instead of modifying ingress function > - change prototype of ingress function > - cleanup br_handle_frame function > - change commit message accordingly > > net/bridge/br_input.c | 7 ++----- > net/bridge/br_private_tunnel.h | 6 +++--- > net/bridge/br_vlan_tunnel.c | 14 ++++++-------- > 3 files changed, 11 insertions(+), 16 deletions(-) >
Looks good to me, Acked-by: Nikolay Aleksandrov <nikolay@nvidia.com>
| |