[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20090225214542.GB5757@movementarian.org>
Date: Wed, 25 Feb 2009 16:45:42 -0500
From: wli@...ementarian.org
To: linux-kernel@...r.kernel.org
Cc: Mark Langsdorf <mark.langsdorf@....com>,
Ingo Molnar <mingo@...e.hu>,
Thomas Gleixner <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>,
Rusty Russell <rusty@...tcorp.com.au>,
Eric Lammerts <eric@...merts.org>,
Andrew Morton <akpm@...ux-foundation.org>
Subject: [patch] mm: k8_northbridges[] build fix
k8_northbridges[] isn't a defined symbol unless arch/x86/kernel/k8.c
is compiled, which is conditional on CONFIG_K8_NB. This patch resolves
the link error by making the sysfs access to the k8 northbridge
conditional on CONFIG_K8_NB in like fashion to the k8_northbridges[]
array itself. The issue was encountered on a 32-bit build with the
processor family specified as CONFIG_MCORE2=y.
The link error was introduced in
x86-conform-l3-cache-index-disable-to-linux-standards.patch
Signed-off-by: William Irwin <wli@...ementarian.org>
Index: mmotm-2.6.29-rc6/arch/x86/kernel/cpu/intel_cacheinfo.c
===================================================================
--- mmotm-2.6.29-rc6.orig/arch/x86/kernel/cpu/intel_cacheinfo.c
+++ mmotm-2.6.29-rc6/arch/x86/kernel/cpu/intel_cacheinfo.c
@@ -613,7 +613,7 @@ static int __cpuinit detect_cache_attrib
return retval;
}
-#ifdef CONFIG_SYSFS
+#if defined(CONFIG_SYSFS) && defined(CONFIG_K8_NB)
#include <linux/kobject.h>
#include <linux/sysfs.h>
@@ -986,4 +986,4 @@ static int __cpuinit cache_sysfs_init(vo
device_initcall(cache_sysfs_init);
-#endif
+#endif /* defined(CONFIG_SYSFS) && defined(CONFIG_K8_NB) */
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists