From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [PATCH] xen: don't use PCI BIOS service for configuration space accesses Date: Fri, 13 Apr 2012 16:25:13 +0100 Message-ID: <4F884559.5000800@citrix.com> References: <1334318915-9083-1-git-send-email-david.vrabel@citrix.com> <4F883848020000780007DCD2@nat28.tlf.novell.com> <4F88222F.7050703@citrix.com> <4F885120020000780007DD18@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4F885120020000780007DD18@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: Konrad Rzeszutek Wilk , xen-devel List-Id: xen-devel@lists.xenproject.org On 13/04/12 15:15, Jan Beulich wrote: >>>> On 13.04.12 at 14:55, David Vrabel wrote: >> On 13/04/12 13:29, Jan Beulich wrote: >>>>>> On 13.04.12 at 14:08, David Vrabel wrote: >>>> From: David Vrabel >>>> >>>> The accessing PCI configuration space with the PCI BIOS service does >>>> not work in PV guests. >>>> >>>> This fixes boot on systems without MMCONFIG or where the BIOS hasn't >>>> marked the MMCONFIG region as reserved in the e820 map. >>> >>> ... and where "direct" access doesn't work either? Are there really >>> machines where Xen works on but this doesn't work? (Or, in case >>> this is disabled in your config, is it really useful to have >>> CONFIG_PCI_DIRECT disabled?) >> >> If you have CONFIG_PCI_GOANY (the default) BIOS is preferred over >> direct. So this change makes it skip BIOS and fall back to direct. > > How is that? When I look at pci_arch_init(), I see pci_direct_probe() > being called first. Hmm. Direct /is/ preferred over BIOS. But the BIOS is initialized even if direct is eventually used and it's the init of the BIOS that dies. I'll fixup the description to make this clearer. David