Date: Wed, 20 Feb 2008 23:03:26 +0100
From: Sam Ravnborg <sam@ravnborg.org>
To: Harvey Harrison <harvey.harrison@gmail.com>
Cc: LKML <linux-kernel@vger.kernel.org>
Subject: sparse - make __CHECK_ENDIAN__ default enabled?
Message-ID: <20080220220326.GA22328@uranus.ravnborg.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.4.2.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 848
Lines: 27

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?

I made a quick test-run with a x86 64 bit defconfig.
My first thought was that this was just really bad
because the amount of warnings roughly doubled.

But then inspecting it a little closer I could see
that 8 files had an increase of > 100 additional
warnings when we enabled __CHECK_ENDIAN__ and
that smells like easy targets to bring down the noise.

I did not dare do it for an allyesconfig run - I 
am not that brave.

Comments?

	Sam
--
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/