Messages in this thread Patch in this message | | | Date | Fri, 18 Jan 2008 20:03:02 +0100 | From | Sam Ravnborg <> | Subject | [PATCH for -rc] kbuild: fix false section mismatch warning referring to .init.text.2 |
| |
Hi Linus.
Please pull following two-liner fix for modpost. It silence an annoying section mismatch warning.
Pull from:
ssh://master.kernel.org/pub/scm/linux/kernel/git/sam/fix-kbuild.git
Sam
From 57ff41164d27cb577620a6e8085dd67cd8a87649 Mon Sep 17 00:00:00 2001 From: Sam Ravnborg <sam@ravnborg.org> Date: Fri, 18 Jan 2008 19:54:51 +0100 Subject: [PATCH] kbuild: fix false section mismatch warning referring to .init.text.2 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit
The following section mismatch warning are false: WARNING: vmlinux.o(.text.head+0x247): Section mismatch: reference to .init.text.1:start_kernel (between 'is386' and 'check_x87')
The problem was that modpost did not see a section named ".init.text.1" as an init section and thus it warned.
This patch fixes this so we no loneg see this warning. Thanks to report from: Toralf Förster
Signed-off-by: Sam Ravnborg <sam@ravnborg.org> Cc: Toralf Förster <toralf.foerster@gmx.de> Cc: WANG Cong <xiyou.wangcong@gmail.com> --- scripts/mod/modpost.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c index 93ac52a..b5f1cff 100644 --- a/scripts/mod/modpost.c +++ b/scripts/mod/modpost.c @@ -736,8 +736,8 @@ static int secref_whitelist(const char *modname, const char *tosec, /* Check for pattern 3 */ if ((strcmp(fromsec, ".text.head") == 0) && - ((strcmp(tosec, ".init.data") == 0) || - (strcmp(tosec, ".init.text") == 0))) + ((strncmp(tosec, ".init.data", strlen(".init.data")) == 0) || + (strncmp(tosec, ".init.text", strlen(".init.text")) == 0))) return 1; /* Check for pattern 4 */ -- 1.5.4.rc3.14.g44397 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |