Should we not just revert 9e9f46c44e487af0a82eb61b624553e2f7118f5b? The thing says: "At this point, it seems to solve more problems than it causes, so let's try using it by default. It's an easy revert if it ends up causing trouble." and it clearly does _not_ solve more problems than it causes, and the whole message in that commit implies we should revert it. I'm happy to apply various patches to fix it up, but regardless, I thinkwe should revert that commit as bogus. We can try making it the default again next round, when maybe it will be true that it doesn't cause issues. What did it even ever help with? Linus On Wed, 24 Jun 2009, Yinghai Lu wrote: > > for AMD system, when only one PCI root, just set PCI_NO_ROOT_CRS for it > > Signed-off-by: Yinghai Lu <yinghai@xxxxxxxxxx> > > --- > arch/x86/pci/amd_bus.c | 4 ++++ > 1 file changed, 4 insertions(+) > > Index: linux-2.6/arch/x86/pci/amd_bus.c > =================================================================== > --- linux-2.6.orig/arch/x86/pci/amd_bus.c > +++ linux-2.6/arch/x86/pci/amd_bus.c > @@ -561,6 +561,10 @@ static int __init early_fill_mp_bus_info > } > } > > + /* don't use _CRS if we only have one root */ > + if (pci_root_num <= 1) > + pci_probe |= PCI_NO_ROOT_CRS; > + > return 0; > } > > -- 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
