Bug 63981 - rhconfig.h __module__bigmem test missing underscore
Summary: rhconfig.h __module__bigmem test missing underscore
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: kernel
Version: 7.3
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Arjan van de Ven
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2002-04-23 01:22 UTC by Matt Domsch
Modified: 2007-04-18 16:42 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2002-04-23 01:22:45 UTC
Embargoed:


Attachments (Terms of Use)

Description Matt Domsch 2002-04-23 01:22:40 UTC
Description of Problem:
linux/include/linux/rhconfig.h test for setting of __module__bigmem is missing 
an underscore.  (it has __module_bigmem, not __module__bigmem).  This prevents 
_ver_str() from being properly defined for bigmem kernels (it doesn't get the 
smp decoration as it should).

This causes binary or shim'd closed source modules to not get the proper symbol 
names for linking.  I'm going to open this as a Sev. 2 issue as it causes 
problems, but I believe can be worked around...

Version-Release number of selected component (if applicable):
2.4.18-3

Additional Information:
Patch to fix this.

--- linux-2.4.18-3/include/linux/rhconfig.h.orig        Mon Apr 22 20:12:47 2002
+++ linux-2.4.18-3/include/linux/rhconfig.h     Mon Apr 22 20:12:57 2002
@@ -218,7 +218,7 @@
 # endif
 #endif

-#if defined(__module__smp) || defined(__module__BOOTsmp) || defined(__module__e
nterprise) || defined(__module_bigmem)
+#if defined(__module__smp) || defined(__module__BOOTsmp) || defined(__module__e
nterprise) || defined(__module__bigmem)
 #define _ver_str(x) smp_ ## x
 #else
 #define _ver_str(x) x

Comment 1 Matt Domsch 2002-07-24 14:26:40 UTC
Fixed in 2.4.18-4 and above.


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