Re: [PATCH] bonding: simplify code and get rid of warning

2008-02-20 Thread Stephen Hemminger
On Wed, 20 Feb 2008 08:57:56 -0800 Jay Vosburgh <[EMAIL PROTECTED]> wrote: > Stephen Hemminger <[EMAIL PROTECTED]> wrote: > > >Get rid of warning and simplify code that looks up vlan tag. > >No need to get tag, then copy it. Also no need for a local status > >variable. > > Granted, the cur

Re: [PATCH] bonding: simplify code and get rid of warning

2008-02-20 Thread Jay Vosburgh
Stephen Hemminger <[EMAIL PROTECTED]> wrote: >Get rid of warning and simplify code that looks up vlan tag. >No need to get tag, then copy it. Also no need for a local status >variable. Granted, the current code is suboptimal, but I don't see any warnings compiling this (gcc 4.1.2). What

[PATCH] bonding: simplify code and get rid of warning

2008-02-18 Thread Stephen Hemminger
Get rid of warning and simplify code that looks up vlan tag. No need to get tag, then copy it. Also no need for a local status variable. Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --- Patch against current 2.6.25 version. --- a/drivers/net/bonding/bond_alb.c2008-02-18 20:58:53.0