Re: [PATCH] checkpatch: Add a test for const with __read_mostly uses

From: Joe Perches
Date: Wed Apr 08 2015 - 20:33:47 EST


On Thu, 2015-04-09 at 02:28 +0200, Andi Kleen wrote:
> On Wed, Apr 08, 2015 at 05:14:49PM -0700, Joe Perches wrote:
> > const objects shouldn't be __read_mostly. They are read-only.
> >
> > Marking these objects as __read_mostly causes section conflicts
> > with LTO linking.
> >
> > So add a test to try to avoid this issue.
>
> I suspect excluding *
> will miss quite a few cases, but there's no way around it.

That excludes only "const <foo> *" not "const foo * const"
so I believe it shouldn't exclude anything that's not
pointer to const.


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