Re: [PATCH] update MMConfig patches w/915 support

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Monday, January 8, 2007 12:45 pm, Olivier Galibert wrote:
> On Sun, Jan 07, 2007 at 11:44:16AM -0800, Jesse Barnes wrote:
> > For reference, here's the probe routine I tried for 965, probably
> > something dumb wrong with it that I'm not seeing atm.
>
> It shouldn't have mattered in your case, but base_address is limited
> to 32bits.  There is a 32 bits reserved zone after it so hope is not
> to be lost, but in any case the current code can't handle over-4G
> base addresses at that point.
>
> Does the bios or your '965 give a correct acpi mmconfig entry?

I should have captured the debug printk I put in while testing.  I think 
the base address specified in the register was 0xf0000000, with a size 
of 256M marked enabled.  Arjan points out that this likely conflicts 
with other BIOS mappings, which is probably why I saw my machine hang 
when I tried to use it.

As for ACPI, I assume you mean the MCFG table?  I haven't looked at it, 
but the stock kernel complains about a lack of the MCFG range in the 
e820 table and subsequently disables mmconfig.

But won't the bridge register value control what actually gets decoded?  
If so, it sounds like this BIOS is buggy wrt mmconfig mapping in 
general; good thing I'm not using any PCIe devices I guess...

Thanks,
Jesse