[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070320203529.GA13349@uranus.ravnborg.org>
Date: Tue, 20 Mar 2007 21:35:29 +0100
From: Sam Ravnborg <sam@...nborg.org>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: Miles Lane <miles.lane@...il.com>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: 2.6.21-rc4-mm1 + 3 hot-fixes -- WARNING: could not find versions for .tmp_versions/built-in.mod
On Tue, Mar 20, 2007 at 08:11:38AM -0800, Andrew Morton wrote:
> On Tue, 20 Mar 2007 07:35:06 -0700 "Miles Lane" <miles.lane@...il.com> wrote:
>
> > It looks like the number of section mismatches is much reduced, which
> > is great.
>
> We fixed something? Maybe we just broke the checker ;)
>
> > But, I don't remember seeing "WARNING: could not find
> > versions for ..." warnings before. Is this an artifact of the
> > init/.missing_syscalls.h problem I encountered earlier?
>
> Sam says these are harmless and he's working on making them go away.
>
> > MODPOST vmlinux
> > WARNING: could not find versions for .tmp_versions/head.mod
> > WARNING: could not find versions for .tmp_versions/init_task.mod
They were caused by modpost now being called with several .o files
that are not modules. The fix was just to get rid of the
warning.
I have committed the following to kbuild.git.
Sam
commit 84b48a0275b4474587cefecb455372e8df6dabda
Author: Sam Ravnborg <sam@...nborg.org>
Date: Tue Mar 20 21:30:23 2007 +0100
kbuild: do not emit src version warning for non-modules
modpost is now called with .o files that are not modules.
So do not warn if there is no corresponding .mod
file listing .o files (in .tmp_versions/).
Signed-off-by: Sam Ravnborg <sam@...nborg.org>
diff --git a/scripts/mod/sumversion.c b/scripts/mod/sumversion.c
index 8a28756..6873d5a 100644
--- a/scripts/mod/sumversion.c
+++ b/scripts/mod/sumversion.c
@@ -397,10 +397,9 @@ void get_src_version(const char *modname, char sum[], unsigned sumlen)
(int) strlen(basename) - 2, basename);
file = grab_file(filelist, &len);
- if (!file) {
- warn("could not find versions for %s\n", filelist);
+ if (!file)
+ /* not a module or .mod file missing - ignore */
return;
- }
sources = strchr(file, '\n');
if (!sources) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists