Skip to content

Commit e1dc1bf

Browse files
committed
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 6f03979 ("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: Masahiro Yamada <[email protected]>
1 parent 50cccec commit e1dc1bf

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

scripts/mod/modpost.c

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -997,13 +997,7 @@ static int secref_whitelist(const char *fromsec, const char *fromsym,
997997
/* symbols in data sections that may refer to any init/exit sections */
998998
if (match(fromsec, PATTERNS(DATA_SECTIONS)) &&
999999
match(tosec, PATTERNS(ALL_INIT_SECTIONS, ALL_EXIT_SECTIONS)) &&
1000-
match(fromsym, PATTERNS("*_template", // scsi uses *_template a lot
1001-
"*_timer", // arm uses ops structures named _timer a lot
1002-
"*_sht", // scsi also used *_sht to some extent
1003-
"*_ops",
1004-
"*_probe",
1005-
"*_probe_one",
1006-
"*_console")))
1000+
match(fromsym, PATTERNS("*_ops", "*_probe", "*_console")))
10071001
return 0;
10081002

10091003
/*

0 commit comments

Comments
 (0)