summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSam Ravnborg <sam@mars.ravnborg.org>2006-02-22 21:24:50 +0100
committerSam Ravnborg <sam@mars.ravnborg.org>2006-02-22 21:24:50 +0100
commit6e10133fa4b2366e8ef18bc2ce34afe727b1c4ba (patch)
tree10204f957a5a42c2e9496eb3cdd074eb7132481c
parentfededcd2af6219de69b252b7d3ea4b4ec2f33c7a (diff)
downloadop-kernel-dev-6e10133fa4b2366e8ef18bc2ce34afe727b1c4ba.zip
op-kernel-dev-6e10133fa4b2366e8ef18bc2ce34afe727b1c4ba.tar.gz
kbuild: do not warn when unwind sections references .init/.exit sections
Andrew Morton reported a number of false positives for ia64 - like these: WARNING: drivers/acpi/button.o - Section mismatch: reference to .init.text: from .IA_64.unwind.init.text after '' (at offset 0x0) WARNING: drivers/acpi/button.o - Section mismatch: reference to .exit.text: from .IA_64.unwind.exit.text after '' (at offset 0x0) WARNING: drivers/acpi/processor.o - Section mismatch: reference to .init.text: from .IA_64.unwind after '' (at offset 0x1e8) They are all false positives - or at least the .c code looks OK. It is not known why sometimes a section name is appended and sometimes not. Fix is to accept references from all sections that includes "unwind." in the name. Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
-rw-r--r--scripts/mod/modpost.c18
1 files changed, 17 insertions, 1 deletions
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
index 5b076ef..7f25354 100644
--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -671,13 +671,21 @@ static int init_section_ref_ok(const char *name)
".debug",
NULL
};
-
+ /* part of section name */
+ const char *namelist3 [] = {
+ ".unwind", /* sample: IA_64.unwind.init.text */
+ NULL
+ };
+
for (s = namelist1; *s; s++)
if (strcmp(*s, name) == 0)
return 1;
for (s = namelist2; *s; s++)
if (strncmp(*s, name, strlen(*s)) == 0)
return 1;
+ for (s = namelist3; *s; s++)
+ if (strstr(*s, name) != NULL)
+ return 1;
return 0;
}
@@ -727,6 +735,11 @@ static int exit_section_ref_ok(const char *name)
".debug",
NULL
};
+ /* part of section name */
+ const char *namelist3 [] = {
+ ".unwind", /* Sample: IA_64.unwind.exit.text */
+ NULL
+ };
for (s = namelist1; *s; s++)
if (strcmp(*s, name) == 0)
@@ -734,6 +747,9 @@ static int exit_section_ref_ok(const char *name)
for (s = namelist2; *s; s++)
if (strncmp(*s, name, strlen(*s)) == 0)
return 1;
+ for (s = namelist3; *s; s++)
+ if (strstr(*s, name) != NULL)
+ return 1;
return 0;
}
OpenPOWER on IntegriCloud