Ahahhh! My bad. I think this was actually discussed back when the whole
reserved-memory-handling thing was being worked on, but it got ignored
because none of the developers actually ever had any machines with this
issue, and the whole (and only) reason for the change was some odd sound
card initialization issue if I recall correctly.
The problem (I think) is that the e820 resource handling doesn't insert
the resources as some kind of magic PCI resource, but it inserts them as a
"I am a driver, and I actively _use_ this resource".
Which means that it all ends up being very busy, and then when a PCI
driver says "I now want to use this", you get EBUSY.
Does this patch work for you?
A resource doesn't have to be busy for the resource allocator to try to
avoid it, so the only thing that BUSY bit does is to not allow people who
_do_ want to use the resources they know about from using them.
So we should mark the resources busy only if we _really_ use them (like
the kernel *RAM* resources).
Oh, and this time the patch even compiles. It includes the previous
change, obviously.
Btw, this shows another (unrelated) issue: the BUSY bit (along with
various other resource flags) doesn't show up in /proc/iomem, so these
kinds of issues end up being debugged totally "blind". Not good.
Linus
---
arch/x86/kernel/e820.c | 8 +++++---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/arch/x86/kernel/e820.c b/arch/x86/kernel/e820.c
index ce97bf3..7aafeb5 100644
--- a/arch/x86/kernel/e820.c
+++ b/arch/x86/kernel/e820.c
@@ -1290,15 +1290,17 @@ void __init e820_reserve_resources(void)
res->start = e820.map[i].addr;
res->end = end;
- res->flags = IORESOURCE_MEM | IORESOURCE_BUSY;
+ res->flags = IORESOURCE_MEM;
/*
* don't register the region that could be conflicted with
* pci device BAR resource and insert them later in
* pcibios_resource_survey()
*/
- if (e820.map[i].type != E820_RESERVED || res->start < (1ULL<<20))
+ if (e820.map[i].type != E820_RESERVED || res->start < (1ULL<<20)) {
+ res->flags |= IORESOURCE_BUSY;
insert_resource(&iomem_resource, res);
+ }
res++;
}
@@ -1318,7 +1320,7 @@ void __init e820_reserve_resources_late(void)
res = e820_res;
for (i = 0; i < e820.nr_map; i++) {
if (!res->parent && res->end)
- reserve_region_with_split(&iomem_resource, res->start, res->end, res->name);
+ insert_resource_expand_to_fit(&iomem_resource, res);
res++;
}
}
--
