Re: [PATCH -mm 7/9] netconsole: Support multiple logging targets

2007-07-07 Thread Satyam Sharma
Hi, On Sun, 8 Jul 2007, KII Keiichi wrote: > Hi Satyam, > > The following comments aren't essential. > > > if (!(event == NETDEV_UP || event == NETDEV_DOWN || > > event == NETDEV_CHANGEADDR || event == NETDEV_CHANGENAME)) > > - goto done; > > + goto done; > > T

Re: [PATCH -mm 7/9] netconsole: Support multiple logging targets

2007-07-07 Thread KII Keiichi
Hi Satyam, The following comments aren't essential. > if (!(event == NETDEV_UP || event == NETDEV_DOWN || > event == NETDEV_CHANGEADDR || event == NETDEV_CHANGENAME)) > - goto done; > + goto done; The above diff lines are extra. > + spin_lock_irqsav

[PATCH -mm 7/9] netconsole: Support multiple logging targets

2007-07-04 Thread Satyam Sharma
From: Satyam Sharma <[EMAIL PROTECTED]> [7/9] netconsole: Support multiple logging targets This patch introduces support for multiple targets: Let's keep this out of CONFIG_NETCONSOLE_DYNAMIC as well -- this is useful even in the default case and (including the infrastructure introduced in previ