Re: sparse - make __CHECK_ENDIAN__ default enabled?

From: Sam Ravnborg
Date: Wed Feb 20 2008 - 17:51:34 EST


On Wed, Feb 20, 2008 at 02:39:46PM -0800, Harvey Harrison wrote:
> On Wed, 2008-02-20 at 23:03 +0100, Sam Ravnborg wrote:
> > Hi Harvey.
> >
> > Can I ask you to look into the worst offenders so we
> > can make -D__CHECK_ENDIAN__ enabled per default
> > in the kernel.
> > Or maybe we should do it anyway?
>
> >From a quick test, the same places that spew sparse warnings, spew
> lots more sparse warnings, the places that are not so bad....are not
> so bad afterwards either.
>
> There's still so many of the trivial warnings that maybe it's not the
> right time for this yet. Ask me again when we get to the 2.6.26
> timeframe, we'll see how many I've gotten rid of by then.

OK - thanks for looking into it.

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