]> git.baikalelectronics.ru Git - kernel.git/commit
bonding: reduce noise during init
authorAndy Gospodarek <andy@greyhouse.net>
Wed, 27 Jul 2011 10:09:26 +0000 (10:09 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 28 Jul 2011 05:39:30 +0000 (22:39 -0700)
commit2fa16b0bc01dd1fbac3001b1cea463e07404195a
treeb84df061504072115fbc7d455730304c3129b5c0
parent1a035b32ae3e957d08a9785efb17d47cf20d39b7
bonding: reduce noise during init

On Tue, Jul 26, 2011 at 05:40:27PM -0700, Joe Perches wrote:
> On Tue, 2011-07-26 at 17:37 -0700, Jay Vosburgh wrote:
> > Joe Perches <joe@perches.com> wrote:
> > >I'd prefer you don't separate the format string
> > >into multiple pieces.
> > Why not?  To me, it looks easier to read split into sections
> > that don't wrap lines.
>
> Harder to grep for a dmesg and the
> defect rate of these split formats is
> typically higher than single strings
> because of bad spacing between string
> segments.
>

I noticed that you took some time back in late 2009 to 'consolidate' the
split format-strings present in the bonding driver at the time and I've
decided I'm fine to leave them the way they are.  The main point of my
patch was to change the output and I would like to get that included.
Here is my updated patch...

Subject: [PATCH net-next-2.6 v2] bonding: reduce noise during init

Many are using sysfs to configure bonding rather than module options, so
there is no need for bonding to throw this warning in normal cases.

Keep the message around when debugging is enabled as it might be useful
for someone desperate enough to enable debugging, but eliminate it
otherwise.

Signed-off-by: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_main.c