On Thu, Jun 22, 2006 at 12:32:19AM +0200, Andi Kleen wrote:On Thursday 22 June 2006 00:19, Rajesh Shah wrote:Yes, I meant it killed extended config space. Note that we areOn Wed, Jun 14, 2006 at 11:47:57PM -0700, Arjan van de Ven wrote:What do you mean with killed PCI Express? PCI Express shouldI just noticed today - this check killed PCI Express on 3 of the 4We need to improve this "mmconfig disabled" anyhow. Having the extendedit's unlikely to be many machines though.
config space unavailable on lots of machines is also far from a viable
solution :)
machines I normally use for testing.
work even without extended config space, except error handling.
about to send out code that enables PCI Express error handling,
so this will become more visible now.
You're saying that you have lots of machines where the mmconfigYes, I saw this in 3 out of 4 systems I checked. I'll go check
aperture is not fully reserved in e820?
some more systems too.
Is it partially reserved (not for all busses) or not at all?The MMCFG resources are not listed at all in the BIOS provided
memory map.
If someone does a patch to double check it against the ACPI name spaceOh I agree with you that booting is more important. My point with
I'm not opposed to let it overrule the e820 heuristic.
The point of this code is to pragmatically detect BIOS with obviously broken setups. It's not about standards lawyering.
the spec statement was that most BIOS developers may not even know
they are doing something "wrong" by not listing these resources in
the int15 E820 table, since the document they normally refer to
doesn't say so. I suspect there are many more systems out there
which do the same thing and will fail the check, but we never notice
since most users don't try to ever access the extended space today.