diff options
author | Borislav Petkov <bp@amd64.org> | 2010-03-12 09:43:03 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2010-03-13 02:36:16 -0500 |
commit | 0e152cd7c16832bd5cadee0c2e41d9959bc9b6f9 (patch) | |
tree | c84df0d86adf5347268939be88426394cd28e18e /arch/x86/Kconfig | |
parent | 8447b360a3897bdfb0677107564d1dd9ab6e63be (diff) |
x86, k8 nb: Fix boot crash: enable k8_northbridges unconditionally on AMD systems
de957628ce7c84764ff41331111036b3ae5bad0f changed setting of the
x86_init.iommu.iommu_init function ptr only when GART IOMMU is
found.
One side effect of it is that num_k8_northbridges
is not initialized anymore if not explicitly
called. This resulted in uninitialized pointers in
<arch/x86/kernel/cpu/intel_cacheinfo.c:amd_calc_l3_indices()>,
for example, which uses the num_k8_northbridges thing through
node_to_k8_nb_misc().
Fix that through an initcall that runs right after the PCI
subsystem and does all the scanning. Then, remove initialization
in gart_iommu_init() which is a rootfs_initcall and we're
running before that.
What is more, since num_k8_northbridges is being used in other
places beside GART IOMMU, include it whenever we add AMD CPU
support. The previous dependency chain in kconfig contained
K8_NB depends on AGP_AMD64|GART_IOMMU
which was clearly incorrect. The more natural way in terms of
hardware dependency should be
AGP_AMD64|GART_IOMMU depends on K8_NB depends on CPU_SUP_AMD &&
PCI. Make it so Number One!
Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Cc: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: Joerg Roedel <joerg.roedel@amd.com>
LKML-Reference: <20100312144303.GA29262@aftab>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Tested-by: Joerg Roedel <joerg.roedel@amd.com>
Diffstat (limited to 'arch/x86/Kconfig')
-rw-r--r-- | arch/x86/Kconfig | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig index eb4092568f9e..ddb52b8d38a7 100644 --- a/arch/x86/Kconfig +++ b/arch/x86/Kconfig | |||
@@ -627,7 +627,7 @@ config GART_IOMMU | |||
627 | bool "GART IOMMU support" if EMBEDDED | 627 | bool "GART IOMMU support" if EMBEDDED |
628 | default y | 628 | default y |
629 | select SWIOTLB | 629 | select SWIOTLB |
630 | depends on X86_64 && PCI | 630 | depends on X86_64 && PCI && K8_NB |
631 | ---help--- | 631 | ---help--- |
632 | Support for full DMA access of devices with 32bit memory access only | 632 | Support for full DMA access of devices with 32bit memory access only |
633 | on systems with more than 3GB. This is usually needed for USB, | 633 | on systems with more than 3GB. This is usually needed for USB, |
@@ -2026,7 +2026,7 @@ endif # X86_32 | |||
2026 | 2026 | ||
2027 | config K8_NB | 2027 | config K8_NB |
2028 | def_bool y | 2028 | def_bool y |
2029 | depends on AGP_AMD64 || (X86_64 && (GART_IOMMU || (PCI && NUMA))) | 2029 | depends on CPU_SUP_AMD && PCI |
2030 | 2030 | ||
2031 | source "drivers/pcmcia/Kconfig" | 2031 | source "drivers/pcmcia/Kconfig" |
2032 | 2032 | ||