[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20080615153145.27982962@infradead.org>
Date: Sun, 15 Jun 2008 15:31:45 -0700
From: Arjan van de Ven <arjan@...radead.org>
To: David Miller <davem@...emloft.net>
Cc: mingo@...e.hu, torvalds@...ux-foundation.org,
akpm@...ux-foundation.org, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, rjw@...k.pl
Subject: Re: [GIT]: Networking
On Sat, 14 Jun 2008 16:24:30 -0700 (PDT)
David Miller <davem@...emloft.net> wrote:
> From: Arjan van de Ven <arjan@...radead.org>
> Date: Sat, 14 Jun 2008 02:39:16 -0700
>
> > the interesting thing is that there is a clear bias on which drivers
> > are tripping this (r8169 is top when I counted yesterday, with
> > sis900 second and then a long tail of nothing) that I think the
> > WARN_ON() is useful in addition to the always-there printk. Eg it
> > does help in seeing which driver is most likely to trigger this.
> > (Andrew also thought this would trigger a *LOT*, so far it's only a
> > rather modest amount, but it's waiting for Fedora or others to ship
> > a kernel with this in to be sure)
>
> I agree that it's useful and should stay.
>
> One thing I noticed is that you can't tell which driver is
> to blame just from the warning and backtrace. Somehow getting
> a driver name in that warning message would be useful and help
> diagnose problems as well as make it easier for you to compute
> those statistics.
There's a patch in -mm to add a WARN_ON variant that takes printk like
arguments... once that goes to mainline that'd be the thing to use
here...
--
If you want to reach me at my work email, use arjan@...ux.intel.com
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists