diff options
author | David Vrabel <david.vrabel@citrix.com> | 2012-05-04 09:29:46 -0400 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2012-05-07 12:16:21 -0400 |
commit | 76a8df7b49168509df02461f83fab117a4a86e08 (patch) | |
tree | 64da15c892104735332bc1fe2783579aac7f3793 | |
parent | b7e5ffe5d83fa40d702976d77452004abbe35791 (diff) |
xen/pci: don't use PCI BIOS service for configuration space accesses
The accessing PCI configuration space with the PCI BIOS32 service does
not work in PV guests.
On systems without MMCONFIG or where the BIOS hasn't marked the
MMCONFIG region as reserved in the e820 map, the BIOS service is
probed (even though direct access is preferred) and this hangs.
CC: stable@kernel.org
Acked-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
[v1: Fixed compile error when CONFIG_PCI is not set]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
-rw-r--r-- | arch/x86/xen/enlighten.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/arch/x86/xen/enlighten.c b/arch/x86/xen/enlighten.c index 4f437dedbdf5..95dccce8e979 100644 --- a/arch/x86/xen/enlighten.c +++ b/arch/x86/xen/enlighten.c | |||
@@ -63,6 +63,7 @@ | |||
63 | #include <asm/stackprotector.h> | 63 | #include <asm/stackprotector.h> |
64 | #include <asm/hypervisor.h> | 64 | #include <asm/hypervisor.h> |
65 | #include <asm/mwait.h> | 65 | #include <asm/mwait.h> |
66 | #include <asm/pci_x86.h> | ||
66 | 67 | ||
67 | #ifdef CONFIG_ACPI | 68 | #ifdef CONFIG_ACPI |
68 | #include <linux/acpi.h> | 69 | #include <linux/acpi.h> |
@@ -1398,8 +1399,10 @@ asmlinkage void __init xen_start_kernel(void) | |||
1398 | /* Make sure ACS will be enabled */ | 1399 | /* Make sure ACS will be enabled */ |
1399 | pci_request_acs(); | 1400 | pci_request_acs(); |
1400 | } | 1401 | } |
1401 | 1402 | #ifdef CONFIG_PCI | |
1402 | 1403 | /* PCI BIOS service won't work from a PV guest. */ | |
1404 | pci_probe &= ~PCI_PROBE_BIOS; | ||
1405 | #endif | ||
1403 | xen_raw_console_write("about to get started...\n"); | 1406 | xen_raw_console_write("about to get started...\n"); |
1404 | 1407 | ||
1405 | xen_setup_runstate_info(0); | 1408 | xen_setup_runstate_info(0); |