Messages in this thread | | | Date | Wed, 28 Jan 2009 14:41:14 -0800 | From | Arjan van de Ven <> | Subject | Re: [PATCH 5/6] module: make modversion_info contain a pointer, not an array. |
| |
On Thu, 29 Jan 2009 08:59:40 +1030 Rusty Russell <rusty@rustcorp.com.au> wrote:
> On Thursday 29 January 2009 01:22:31 Arjan van de Ven wrote: > > On Thu, 29 Jan 2009 00:05:52 +1030 > > Rusty Russell <rusty@rustcorp.com.au> wrote: > > > > > > > > With allmodconfig (minus non-building modules) on 32-bit x86: > > > Total size of modules before: 60009790 bytes > > > Total size of modules after: 55927866 bytes > > > > > > Saving 7% of module size for CONFIG_MODVERSIONS=y; and these > > > sections are kept resident as well. > > > > > > > that reminds me.. can we just simplify MODVERSIONS to be a md5sum > > (or sha1 whatver) of the .config file in the VERMAGIC ? > > it's a lot more reliable in detecting incompatibilities, and a lot > > less space consumed. > > Unfortunately people really seem to want the finer granularity that > MODVERSIONS (sometimes) provides :( I've tried killing it off > several times, and always failed.
but we could just stick the result in VERMAGIC right? rather than tacking it to every symbol.
how you calculate the global checksum is almost a separate debate.
-- Arjan van de Ven Intel Open Source Technology Centre For development, discussion and tips for power savings, visit http://www.lesswatts.org
| |