From: Ram Pai <linuxram@us.ibm.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH] crc generation fix for EXPORT_SYMBOL_GPL
Date: Mon, 06 Feb 2006 22:49:01 -0800 [thread overview]
Message-ID: <1139294941.4723.50.camel@localhost> (raw)
In-Reply-To: <1139234973.3131.83.camel@laptopd505.fenrus.org>
On Mon, 2006-02-06 at 15:09 +0100, Arjan van de Ven wrote:
> > Eventually we want to generate a tool that can report API changes across
> > kernel releases and put it in some friendly(docbook) format.
>
> the CRC's are only very lightly related to API though (or even ABI) so I
> suspect this isn't too useful a thing to persue in the first place
actually the CRC's capture a pretty good picture of the changes to API
as well as ABI. The crc is run on the prototype of the exported symbol,
recursively expanding each and every datastructure involved in the
prototype.
Hence it mostly captures the ABI signature of the exported symbols. The
only part it misses is, it does not capture the GPL'ness of the exported
symbol. And that was what I was trying to fix, because changing the
export nature of the symbol changes the ABI. With the fix one would
be able to exactly detect API/ABI changes to exported symbols.
Do you think its a bad idea still? Its a good indication for out-of-tree
modules that the ABI/API of some exported symbols they depend on, has
changed.
>(using CRC I mean, documenting real API changes I can see being useful)
Sure. Nice to learn that this work will be of value. Also
looking for ideas on what information would be useful to report and in
what format.
Thanks,
RP
>
prev parent reply other threads:[~2006-02-07 6:50 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-02 4:15 [RFC PATCH] crc generation fix for EXPORT_SYMBOL_GPL Ram Pai
2006-02-04 20:31 ` Arjan van de Ven
2006-02-06 5:24 ` Ram Pai
2006-02-06 13:42 ` Arjan van de Ven
2006-02-06 14:09 ` Arjan van de Ven
2006-02-07 6:49 ` Ram Pai [this message]
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=1139294941.4723.50.camel@localhost \
--to=linuxram@us.ibm.com \
--cc=arjan@infradead.org \
--cc=linux-kernel@vger.kernel.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