Nico Huber has uploaded this change for review.
device: Introduce RESOURCE_ALLOCATION_TOP_DOWN
Add option to resource allocators v4 and v4.5 that restores the top-down
allocation approach at the domain level.
Change-Id: Iaf463d3e6b37d52e46761d8e210034fded58a8a4
Signed-off-by: Nico Huber <nico.h@gmx.de>
---
M src/device/Kconfig
M src/device/resource_allocator_v4.5.c
M src/device/resource_allocator_v4.c
M src/include/memrange.h
M src/lib/memrange.c
5 files changed, 37 insertions(+), 21 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/57/41957/1
diff --git a/src/device/Kconfig b/src/device/Kconfig
index ee6fb21..35433b0 100644
--- a/src/device/Kconfig
+++ b/src/device/Kconfig
@@ -821,4 +821,8 @@
endchoice
+config RESOURCE_ALLOCATION_TOP_DOWN
+ bool "Allocate resources from top down"
+ depends on RESOURCE_ALLOCATOR_V4 || RESOURCE_ALLOCATOR_V4_5
+
endmenu
diff --git a/src/device/resource_allocator_v4.5.c b/src/device/resource_allocator_v4.5.c
index 6b9e712..4a03a23 100644
--- a/src/device/resource_allocator_v4.5.c
+++ b/src/device/resource_allocator_v4.5.c
@@ -403,7 +403,8 @@
if (!res->size)
continue;
- if (!memranges_steal(&ranges, res->limit, res->size, res->align, type, &base)) {
+ if (!memranges_steal(&ranges, res->limit, res->size, res->align, type, &base,
+ CONFIG(RESOURCE_ALLOCATION_TOP_DOWN))) {
printk(BIOS_ERR, "ERROR: Resource didn't fit!!!\n");
print_failed_res(dev, res);
continue;
diff --git a/src/device/resource_allocator_v4.c b/src/device/resource_allocator_v4.c
index 3a11032..4e967f2 100644
--- a/src/device/resource_allocator_v4.c
+++ b/src/device/resource_allocator_v4.c
@@ -365,6 +365,9 @@
static void allocate_child_resources(struct bus *bus, struct memranges *ranges,
unsigned long type_mask, unsigned long type_match)
{
+ const bool allocate_top_down =
+ bus->dev->path.type == DEVICE_PATH_DOMAIN &&
+ CONFIG(RESOURCE_ALLOCATION_TOP_DOWN);
struct resource *resource = NULL;
const struct device *dev;
@@ -373,8 +376,9 @@
if (!resource->size)
continue;
- if (memranges_steal(ranges, resource->limit, resource->size, resource->align,
- type_match, &resource->base) == false) {
+ if (memranges_steal(ranges, resource->limit, resource->size,
+ resource->align, type_match, &resource->base,
+ allocate_top_down)) == false) {
printk(BIOS_ERR, " ERROR: Resource didn't fit!!! ");
printk(BIOS_DEBUG, " %s %02lx * size: 0x%llx limit: %llx %s\n",
dev_path(dev), resource->index,
diff --git a/src/include/memrange.h b/src/include/memrange.h
index 2b8c00e..d50a9c8 100644
--- a/src/include/memrange.h
+++ b/src/include/memrange.h
@@ -161,15 +161,17 @@
const struct range_entry *r);
/* Steals memory from the available list in given ranges as per the constraints:
- * limit = Upper bound for the memory range to steal (Inclusive).
- * size = Requested size for the stolen memory.
- * align = Required alignment(log 2) for the starting address of the stolen memory.
- * tag = Use a range that matches the given tag.
+ * limit = Upper bound for the memory range to steal (Inclusive).
+ * size = Requested size for the stolen memory.
+ * align = Required alignment(log 2) for the starting address of the stolen memory.
+ * tag = Use a range that matches the given tag.
+ * from_top = Steal the highest possible range.
*
* If the constraints can be satisfied, this function creates a hole in the memrange,
* writes the base address of that hole to stolen_base and returns true. Otherwise it returns
* false. */
bool memranges_steal(struct memranges *ranges, resource_t limit, resource_t size,
- unsigned char align, unsigned long tag, resource_t *stolen_base);
+ unsigned char align, unsigned long tag, resource_t *stolen_base,
+ bool from_top);
#endif /* MEMRANGE_H_ */
diff --git a/src/lib/memrange.c b/src/lib/memrange.c
index 39f502c..2f6d55c 100644
--- a/src/lib/memrange.c
+++ b/src/lib/memrange.c
@@ -378,11 +378,11 @@
/* Find a range entry that satisfies the given constraints to fit a hole that matches the
* required alignment, is big enough, does not exceed the limit and has a matching tag. */
-static const struct range_entry *memranges_find_entry(struct memranges *ranges,
- resource_t limit, resource_t size,
- unsigned char align, unsigned long tag)
+static const struct range_entry *
+memranges_find_entry(struct memranges *ranges, resource_t limit, resource_t size,
+ unsigned char align, unsigned long tag, bool last)
{
- const struct range_entry *r;
+ const struct range_entry *r, *last_entry = NULL;
resource_t base, end;
if (size == 0)
@@ -407,25 +407,30 @@
if (end > limit)
break;
- return r;
+ if (!last)
+ return r;
+
+ last_entry = r;
}
- return NULL;
+ return last_entry;
}
bool memranges_steal(struct memranges *ranges, resource_t limit, resource_t size,
- unsigned char align, unsigned long tag, resource_t *stolen_base)
+ unsigned char align, unsigned long tag, resource_t *stolen_base,
+ bool from_top)
{
- resource_t base;
- const struct range_entry *r = memranges_find_entry(ranges, limit, size, align, tag);
+ const struct range_entry *r;
+ r = memranges_find_entry(ranges, limit, size, align, tag, from_top);
if (r == NULL)
return false;
- base = ALIGN_UP(r->begin, POWER_OF_2(align));
-
- memranges_create_hole(ranges, base, size);
- *stolen_base = base;
+ if (from_top)
+ *stolen_base = ALIGN_DOWN(range_entry_end(r) - size, POWER_OF_2(align));
+ else
+ *stolen_base = ALIGN_UP(r->begin, POWER_OF_2(align));
+ memranges_create_hole(ranges, *stolen_base, size);
return true;
}
To view, visit change 41957. To unsubscribe, or for help writing mail filters, visit settings.