Re: Enable mmconf access to PCI ECS for all AMD fam10h systems

[Robert Richter]

Thanks Yinghai and Arjan,

On 23.05.08 11:18:25, Arjan van de Ven wrote:
> On Fri, 23 May 2008 10:57:22 -0700
> "Yinghai Lu" <yhlu.kernel@xxxxxxxxx> wrote:
> > Someone express the concern about touching MSR. Arjan or Andi?
> > 
> > for those of system that doesn't have MCFG table there, they could use
> > pci=check_enable_amd_mmconf to enable it forcely.
> 
> using MMIO if the bios doesn't allow it via MCFG is a huge mistake and
> shouldn't be done.

I see your concerns here, though it could possibly be save for access
to processor config space only. Andi also stated before the BIOS
should do the work.

> Using the extended port IO version is actually perfectly reasonable to
> me and I would encourage that to be in the kernel (and be used in favor
> of MCFG if the port IO method is there)

Glad to hear that IO access code is also accepted to access ECS, at
least for segment 0. I already submitted patches for this some times
ago and will send an update. This would ensure access to ECS of
processor config space if MMCONFIG is disabled. Also, this could be a
solution for 32bit code.

-Robert

-- 
Advanced Micro Devices, Inc.
Operating System Research Center
email: robert.richter@xxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/