Re: [PATCH] Bridge allows the userspace to configure the bridge parameters with values not defined in the IEEE 802.1D std

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Sun, Mar 27, 2011 at 12:01 AM, Sasikanth V <sasikanth.v19@xxxxxxxxx> wrote:


On Sat, Mar 26, 2011 at 9:25 PM, Stephen Hemminger <shemminger@xxxxxxxxxxxxxxxxxxxx> wrote:
On Sat, 26 Mar 2011 20:19:57 +0530
Sasikanth V <sasikanth.v19@xxxxxxxxx> wrote:

>
> Signed-off-by: Sasikanth V <sasikanth.v19@xxxxxxxxx>
> ---

I will clean this up. There are things like introducing
global function names (set_forward_delay, etc) that need
to be fixed.

   Thanks for looking at the patch.  I made 5 set functions (set_forward_delay, set_max_age,
   set_hello_time, set_priority and set_ageing_time) in br_sysfs_br.c global to use in br_ioctl.c to
   avoid code duplication (but it looks it is not a good idea).

   Now  i thought something like making those 5 functions static inline and moving it to br_private.h.
   If this is fine , I will resend the patch
 
  

  Since I am new to patch states and workflow. Can you guys please let me, what  I have to do when the
  patch state is in Awaiting Stream.

  Thanks
 
_______________________________________________
Bridge mailing list
Bridge@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/bridge

[Index of Archives]     [Netdev]     [AoE Tools]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux