I wrote: > But the code in the init path is ridiculously, embarrassingly complicated ... On Fri, Jun 15, 2012 at 1:15 AM, Yinghai Lu <yinghai@xxxxxxxxxx> wrote: > here cache cfg too early. should do that after > > pci_mmcfg_reject_broken(). > > otherwise will use mcfg even try to reject that before. ... > that really change the logic. > > looks like it will break mrst/sfi path. > > the scan from pci_legacy_init() for mrst/sfi will not have ext_pci_ops > set for bus 0. I rest my case. The current init path is unmaintainable. I'll wait for a v8 (or later) with fixes for the build issues Fengguang found and Yinghai's ack. It will make things somewhat easier for me if you start with my topic/jiang-mmconfig-v7 branch. Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html