Commit e1dc1bfe authored by Masahiro Yamada's avatar Masahiro Yamada
Browse files

modpost: remove more symbol patterns from the section check whitelist 



These symbol patterns were whitelisted to allow them to reference to
functions with the old __devinit and __devexit annotations.

We stopped doing this a long time ago, for example, commit 6f039790
("Drivers: scsi: remove __dev* attributes.") remove those annotations
from the scsi drivers.

Keep *_ops, *_probe, and *_console, otherwise they will really cause
section mismatch warnings.

Signed-off-by: default avatarMasahiro Yamada <masahiroy@kernel.org>
parent 50cccec1

scripts/mod/modpost.c

+1 −7
Original line number Diff line number Diff line
@@ -997,13 +997,7 @@ static int secref_whitelist(const char *fromsec, const char *fromsym, 
	/* symbols in data sections that may refer to any init/exit sections */

	if (match(fromsec, PATTERNS(DATA_SECTIONS)) &&

	    match(tosec, PATTERNS(ALL_INIT_SECTIONS, ALL_EXIT_SECTIONS)) &&

	    match(fromsym, PATTERNS("*_template", // scsi uses *_template a lot

				    "*_timer", // arm uses ops structures named _timer a lot

				    "*_sht", // scsi also used *_sht to some extent

				    "*_ops",

				    "*_probe",

				    "*_probe_one",

				    "*_console")))

	    match(fromsym, PATTERNS("*_ops", "*_probe", "*_console")))

		return 0;



	/*