From: Andrew Morton <akpm@linux-foundation.org>
To: Andreas Gruenbacher <agruen@suse.de>
Cc: linux-kernel@vger.kernel.org, Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [genksyms patch 3/4] Track symbol checksum changes
Date: Tue, 2 Sep 2008 22:02:31 -0700 [thread overview]
Message-ID: <20080902220231.0197237e.akpm@linux-foundation.org> (raw)
In-Reply-To: <20080721025913.063165574@suse.de>
On Mon, 21 Jul 2008 04:28:26 +0200 Andreas Gruenbacher <agruen@suse.de> wrote:
> Sometimes it is preferable to avoid changes of exported symbol checksums (to
> avoid breaking externally provided modules). When a checksum change occurs, it
> can be hard to figure out what caused this change: underlying types may have
> changed, or additional type information may simply have become available at the
> point where a symbol is exported.
>
> Add a new --reference option to genksyms which allows it to report why
> checksums change, based on the type information dumps it creates with
> the --dump-types flag. Genksyms will read in such a dump from a previous run,
> and report which symbols have changed (and why).
>
> The behavior can be controlled for an entire build as follows: If
> KBUILD_SYMTYPES is set, genksyms uses --dump-types to produce *.symtypes dump
> files. If any *.symref files exist, those will be used as the reference to
> check against. If KBUILD_PRESERVE is set, checksum changes will fail the
> build.
This breaks `make M=...'
With i386 allmodconfig,
akpm2:/usr/src/25> make M=drivers/rtc
WARNING: Symbol version dump /usr/src/devel/Module.symvers
is missing; modules will have no dependencies and modversions.
LD drivers/rtc/built-in.o
CC [M] drivers/rtc/rtc-lib.o
scripts/genksyms/genksyms: invalid option -- r
Usage:
genksyms [-adDTwqhV] > /path/to/.tmp_obj.ver
-a, --arch Select architecture
-d, --debug Increment the debug level (repeatable)
-D, --dump Dump expanded symbol defs (for debugging only)
-T, --dump-types file Dump expanded types into file (for debugging only)
-w, --warnings Enable warnings
-q, --quiet Disable warnings (default)
-h, --help Print this message
-V, --version Print the release version
make[1]: *** [drivers/rtc/rtc-lib.o] Error 1
make: *** [_module_drivers/rtc] Error 2
next prev parent reply other threads:[~2008-09-03 5:02 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-21 2:28 [genksyms patch 0/4] Checksum tracking and tweaking Andreas Gruenbacher
2008-07-21 2:28 ` [genksyms patch 1/4] parser: fix the __attribute__ rule Andreas Gruenbacher
2008-07-30 20:43 ` Sam Ravnborg
2008-07-30 22:03 ` Andreas Gruenbacher
2008-07-31 21:17 ` Sam Ravnborg
2008-07-21 2:28 ` [genksyms patch 2/4] Include extern information in dumps Andreas Gruenbacher
2008-07-31 21:18 ` Sam Ravnborg
2008-07-21 2:28 ` [genksyms patch 3/4] Track symbol checksum changes Andreas Gruenbacher
2008-09-03 5:02 ` Andrew Morton [this message]
2008-09-03 8:13 ` Andreas Gruenbacher
2008-07-21 2:28 ` [genksyms patch 4/4] Allow to ignore " Andreas Gruenbacher
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080902220231.0197237e.akpm@linux-foundation.org \
--to=akpm@linux-foundation.org \
--cc=agruen@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=sam@ravnborg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox