Re: [RFC PATCH] kbuild: silence section mismatch warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On May 11, 2007, at 4:08 PM, Sam Ravnborg wrote:

----- Forwarded message from Sam Ravnborg <[email protected]> -----

Forgot lkml in first mail...

	Sam

Subject: [RFC PATCH] kbuild: silence section mismatch warnings
From: Sam Ravnborg <[email protected]>
Date: Fri, 11 May 2007 23:03:46 +0200
User-Agent: Mutt/1.4.2.1i
To: Chris Wedgwood <[email protected]>, Andrew Morton <[email protected]>,
	"David S. Miller" <[email protected]>,
	Russell King <[email protected]>,
	Satyam Sharma <[email protected]>
Cc: [email protected]

Following patch allow us in specific places to silence section mismatch warnings. There is a few legitime places that modpost does not yet recognize where
reference from .text to .init.text (likewise for data) are legitime.
This allow us to spot the few places and annotate them so we do not
get false warnings that in the end will let real warnings pass.

The annotation is simple to grep for so revieing all uses in a few
months time are trivial. It is assumed that a few places will
use this to shut up the warning as replacement for the real fix.
But these cases are esay to spot and to fix up.

Its unclear if you expect that some things will be tagged __init_refok/__initdata_refok forever or if we'll find some way to fix/change the code so the things tagged no longer need it.

With this and the following two patches I have a section mismatch free
build.
The plan is that a section mismatch soon will graduate from a warning to an error. This will hurt a few drives but then they have an incentive to fix it up.

Comments welcome.

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux