aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Vrabel <david.vrabel@citrix.com>2012-05-04 09:29:46 -0400
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>2012-05-07 12:16:21 -0400
commit76a8df7b49168509df02461f83fab117a4a86e08 (patch)
tree64da15c892104735332bc1fe2783579aac7f3793
parentb7e5ffe5d83fa40d702976d77452004abbe35791 (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.c7
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);