Re: small change

[Ben Greear <greearb@candelatech.com>]

Vishwas Manral wrote:
> Hi Ben,
> 
> I am going through the 2.4 code. We can make small changes in br_input.c
> to make the code slightly better and slightly more optimized. Maybe when
> you are changing something else you can club it along.

Please send this to the netdev mailing list:

netdev@oss.sgi.com

I don't maintain the bridging code but there are people on
that list who do.

They would also prefer a unified diff instead of the cut-n-paste
like you sent.

Thanks,
Ben


> 
> Thanks,
> Vishwas
> 
> ========================================================================
> =======
> Change from ->
> 
>  87         if (dst != NULL && dst->is_local) {
>  88                 if (!passedup)
>  89                         br_pass_frame_up(br, skb);
>  90                 else
>  91                         kfree_skb(skb);
>  92                 br_fdb_put(dst);
>  93                 goto out;
>  94         }
>  95 
>  96         if (dst != NULL) {
>  97                 br_forward(dst->dst, skb);
>  98                 br_fdb_put(dst);
>  99                 goto out;
> 100         }
> 
> To ->
> 
>  87         if (dst != NULL) {
>                 if(dst->is_local) {
>  88                 if (!passedup)
>  89                         br_pass_frame_up(br, skb);
>  90                 else
>  91                         kfree_skb(skb);
>                  } else {
>  97                 br_forward(dst->dst, skb);
>                  }
>  92              br_fdb_put(dst);
>  93              goto out;
>  94         }
> 
> 


-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com