On Fri, Aug 29, 2008 at 9:41 PM, Yinghai Lu <yhlu.kernel@xxxxxxxxx> wrote: > On Fri, Aug 29, 2008 at 8:24 PM, Linus Torvalds > <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote: >> >> >> On Fri, 29 Aug 2008, Yinghai Lu wrote: >>> >>> we need to use insert_resource_split_to_fit instead... >>> >>> otherwise __request_region will not be happy. >> >> Are you really really sure? >> >> Try just removing the IORESOURCE_BUSY. As mentioned, if we expect the PCI >> BAR's to work with the e820 resources, then BUSY really is simply not >> right any more. Not that I think it should matter either.. >> >> The ones that are added _early_ should be IORESOURCE_BUSY (ie the ones >> that cover RAM), but the others we now expect to nest with PCI BARs. > > not all. some are MMCONF, some are for GART, and some for fixed lapic, > and fixed ioapic, and fixed HPET. we may not need put reserve entries from e820 into resource tree. and only insert those sticky resources (with _BUSY) before pci_assign_unassign and _request_region etc. YH -- To unsubscribe from this list: send the line "unsubscribe kernel-testers" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html