Bug 134744 - RHEL4: modules: put srcversion checksum in each modinfo section
Summary: RHEL4: modules: put srcversion checksum in each modinfo section
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: kernel   
(Show other bugs)
Version: 4.0
Hardware: All
OS: Linux
Target Milestone: ---
: ---
Assignee: Dave Jones
QA Contact:
Keywords: FutureFeature
Depends On:
Blocks: 135876
TreeView+ depends on / blocked
Reported: 2004-10-05 22:15 UTC by Matt Domsch
Modified: 2015-01-04 22:10 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2004-11-08 21:20:01 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
linux-2.6.9-rc3-srcversion.patch (10.28 KB, patch)
2004-10-05 22:16 UTC, Matt Domsch
no flags Details | Diff

Description Matt Domsch 2004-10-05 22:15:58 UTC
Date: Tue, 5 Oct 2004 17:02:11 -0500
From: Matt Domsch <Matt_Domsch@dell.com>
To: akpm@osdl.org, Rusty Russell <rusty@rustcorp.com.au>
Cc: linux-kernel@vger.kernel.org
Subject: [PATCH 2.6.9-rc3] modules: put srcversion checksum in each 

Separate the module source and header checksum into a separate 
modinfo field 
With CONFIG_MODULE_SRCVERSION_ALL=y, put srcversion into every module,
not just those with MODULE_VERSION("something").

Patch by Rusty Russell, trivial merging and testing by Matt Domsch

Signed-off-by: Matt Domsch <Matt_Domsch@dell.com>

 include/linux/module.h   |    6 +--
 init/Kconfig             |   12 +++++++
 scripts/Makefile.modpost |    1
 scripts/mod/modpost.c    |   65 +++++++++++++++++++++++++++++++++++--
 scripts/mod/modpost.h    |   10 +++--
 scripts/mod/sumversion.c |   80 ++++++++-----------------------------
 6 files changed, 96 insertions, 78 deletions

DKMS could use this information to tell if a driver source has 
changed or not, 
regardless of if the driver author put a MODULE_VERSION() tag in it.

w/o patch, modinfo doesn't show a 'srcversion' field
w patch and CONFIG_MODULE_SRCVERSION_ALL=y, modinfo shows 
a 'srcversion' field 
for all modules.

Comment 1 Matt Domsch 2004-10-05 22:16:45 UTC
Created attachment 104813 [details]

Patch sent to akpm today.

Comment 3 Matt Domsch 2004-10-21 14:33:51 UTC
This patch was just committed to Linus's 2.6 bitkeeper repo.

Comment 8 Tim Burke 2004-11-08 21:20:01 UTC

Our intention is include the MODULE_VERSION in all of the main network
and storage drivers in RHEL4 RC.  In conjunction, we are implementing
the RPM based dependencies which Jeremy described to you. We can't
really make use of the checksum from a versioning standpoint. Unless
we went through the unmaintainable trauma of having a database of
every shipped driver & respective version. 

For this reason, the incremental value of adding in this change isn't
compelling to put in this late in the game.  In a later update, when
we rebase, it would then be picked up. 

Comment 9 Matt Domsch 2004-11-08 21:27:15 UTC
OK, but you probably won't rebase for an update, so it becomes a "you 
get it for free in RHEL5" thing.

Note You need to log in before you can comment on or make changes to this bug.