Commit 5a4910fb authored by Sam Ravnborg's avatar Sam Ravnborg

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: default avatarSam Ravnborg <sam@ravnborg.org>
parent a61b2dfd
...@@ -616,6 +616,15 @@ static int strrcmp(const char *s, const char *sub) ...@@ -616,6 +616,15 @@ static int strrcmp(const char *s, const char *sub)
* fromsec = .text * fromsec = .text
* atsym = kernel_init * atsym = kernel_init
* Some symbols belong to init section but still it is ok to reference * 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, static int secref_whitelist(const char *modname, const char *tosec,
const char *fromsec, const char *atsym, const char *fromsec, const char *atsym,
...@@ -687,6 +696,12 @@ static int secref_whitelist(const char *modname, const char *tosec, ...@@ -687,6 +696,12 @@ static int secref_whitelist(const char *modname, const char *tosec,
(strcmp(fromsec, ".text") == 0) && (strcmp(fromsec, ".text") == 0) &&
(strcmp(refsymname, "kernel_init") == 0)) (strcmp(refsymname, "kernel_init") == 0))
return 1; 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; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment