Re: [patch] Make MMCONFIG space (extended PCI config space) adriver opt-in issue

From: Arjan van de Ven
Date: Thu Dec 27 2007 - 09:12:28 EST


On Thu, 27 Dec 2007 06:46:22 -0500
Jeff Garzik <jeff@xxxxxxxxxx> wrote:

> Linus Torvalds wrote:
> > Well, the *current* behaviour as far as setup is concerned is
> > unacceptable. But yes, longer term, we should be able to just have
> > quirk entries for saying "enable mmconfig because I know it's
> > safe", except we should not enable them until after the core PCI
> > probing has completed.
>
>
> IMO that should be an arch decision, buried somewhere in arch/x86.
>
> If other arches implement extended config space sanely -- and
> possibly via some arch-specific means that is /not/ mmconfig -- then
> they should be able to make an arch decision that extended PCI config
> space accesses Just Work(tm).
>
> For such arches, pci_enable_ext_cfg_space(pdev) would be a no-op,
> always returning success.
>

both the first and the second patch already have this behavior.


--
If you want to reach me at my work email, use arjan@xxxxxxxxxxxxxxx
For development, discussion and tips for power savings,
visit http://www.lesswatts.org
--
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/