On Tue, 5 Feb 2008 17:08:37 +1100 Rusty Russell <[EMAIL PROTECTED]> wrote:
> On Tuesday 05 February 2008 14:53:18 Andrew Morton wrote: > > On Tue, 5 Feb 2008 14:43:31 +1100 Rusty Russell <[EMAIL PROTECTED]> > wrote: > > > On Tuesday 05 February 2008 02:42:15 Alexey Dobriyan wrote: > > > > One head-scratching session could be noticeably shorter with this > > > > patch... > > > > > > > > Signed-off-by: Alexey Dobriyan <[EMAIL PROTECTED]> > > > > > > If we want to prevent > 0 returns, let's just BUG_ON(). > > > > That risks killing previously-working setups. WARN_ON is sufficient. > > I disagree. WARN_ON is useful for developers, but they can handle BUG_ON, > too. For developers, BUG_ON has zero benefit relative to WARN_ON. For non-developers, BUG_ON has large disadvantages relative to WARN_ON. It's a no-brainer. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/