summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSam Ravnborg <sam@ravnborg.org>2007-02-27 09:14:58 +0100
committerSam Ravnborg <sam@ravnborg.org>2007-05-02 20:58:08 +0200
commit5a4910fbbeef14cc91daa41086449a1a4acebc96 (patch)
treed81c13661f886b960712cdfd11f97a95a81d5922
parenta61b2dfd1823506dbf1f9b046e0b09237ec1b985 (diff)
downloadop-kernel-dev-5a4910fbbeef14cc91daa41086449a1a4acebc96.zip
op-kernel-dev-5a4910fbbeef14cc91daa41086449a1a4acebc96.tar.gz
kbuild: whitelist logo references from .text to .init.data
drivers/video/logo has references from .text to .init.data but function is only used during early init. So reference is OK and we do not want to warn about them => whitelist the reference. Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
-rw-r--r--scripts/mod/modpost.c15
1 files changed, 15 insertions, 0 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index 281abb7..5f2ecd5 100644
--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -616,6 +616,15 @@ static int strrcmp(const char *s, const char *sub)
* fromsec = .text
* atsym = kernel_init
* Some symbols belong to init section but still it is ok to reference
+ *
+ * Pattern 7:
+ * Logos used in drivers/video/logo reside in __initdata but the
+ * funtion that references them are EXPORT_SYMBOL() so cannot be
+ * marker __init. So we whitelist them here.
+ * The pattern is:
+ * tosec = .init.data
+ * fromsec = .text*
+ * refsymname = logo_
**/
static int secref_whitelist(const char *modname, const char *tosec,
const char *fromsec, const char *atsym,
@@ -687,6 +696,12 @@ static int secref_whitelist(const char *modname, const char *tosec,
(strcmp(fromsec, ".text") == 0) &&
(strcmp(refsymname, "kernel_init") == 0))
return 1;
+
+ /* Check for pattern 7 */
+ if ((strcmp(tosec, ".init.data") == 0) &&
+ (strncmp(fromsec, ".text", strlen(".text")) == 0) &&
+ (strncmp(refsymname, "logo_", strlen("logo_")) == 0))
+ return 1;
return 0;
}
OpenPOWER on IntegriCloud