Re: [BUG] mtd redboot (also gcc 4.1 warning fix)

[Steven Rostedt <rostedt@goodmis.org>]

On Wed, 10 May 2006, Adrian Bunk wrote:

> On Tue, May 09, 2006 at 07:56:04PM -0700, Daniel Walker wrote:
>
> > unsigned long may not always be 32 bits, right ? This patch fixes the
> > warning, but not the bug .
> >...
>
> IOW, all your patch does is to hide a bug?
>
> Not exactly an improvement...
>

Perfect example of what Andrew Morten stated in his keynote at LinuxTag.
He mentioned patches that fixed warnings but not the problem that they
warn about.  He stated that these are very dangerous, because, not only is
the problem not fixed, but now no one knows of the problem.

Daniel,

We appreciate the clean up of the kernel.  Just try to focus on the
obvious stuff, and maybe only flag those that you're not sure is still a
problem.  Email the author of the code, or kick his butt to get him to
move and fix it.

Unintialized variables and bad compares may not be as trivial as they
appear.  Look closely at them to see if there isn't really a problem that
lies underneath.

Thanks,

-- Steve

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/