Firmware/opensbi
8
0
Fork 0
forked from Mirrors/opensbi
Code Pull Requests Activity

platform: introduce DT-based configurable heap size

Browse Source 
The default heap size will work for most platforms, but for some
special platforms, the heap is too small to hold all the information
or is too big so that it take too much ram. Introduce configurable
heap should solve this problem and make all generic platforms happy.

Add DT-based heap-size for the generic platform.

Signed-off-by: Inochi Amaoto <inochiama@gmail.com>
Reviewed-by: Xiang W <wxjstz@126.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
This commit is contained in:
Inochi Amaoto
2024-11-11 10:27:34 +08:00
committed by Anup Patel
parent dcb68dd17d
commit 5d4e3fb2ee
2 changed files with 29 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
docs/opensbi_config.md
View File

@@ -23,6 +23,9 @@ The DT properties of a domain configuration DT node are as follows:
set of harts is permitted to perform a cold boot. Otherwise, all
harts are allowed to cold boot.
* **heap-size** (Optional) - When present, the specified value is used
as the size of the heap in bytes.
* **system-suspend-test** (Optional) - When present, enable a system
suspend test implementation which simply waits five seconds and issues a WFI.
@@ -36,6 +39,7 @@ The OpenSBI Configuration Node will be deleted at the end of cold boot
opensbi-config {
compatible = "opensbi,config";
cold-boot-harts = <&cpu1 &cpu2 &cpu3 &cpu4>;
heap-size = <0x400000>;
system-suspend-test;
};
};

24
platform/generic/platform.c
View File

@@ -69,6 +69,28 @@ static u32 fw_platform_calculate_heap_size(u32 hart_count)
return BIT_ALIGN(heap_size, HEAP_BASE_ALIGN);
}
static u32 fw_platform_get_heap_size(const void *fdt, u32 hart_count)
{
int chosen_offset, config_offset, len;
const fdt32_t *val;
/* Get the heap size from device tree */
chosen_offset = fdt_path_offset(fdt, "/chosen");
if (chosen_offset < 0)
goto default_config;
config_offset = fdt_node_offset_by_compatible(fdt, chosen_offset, "opensbi,config");
if (config_offset < 0)
goto default_config;
val = (fdt32_t *)fdt_getprop(fdt, config_offset, "heap-size", &len);
if (len > 0 && val)
return BIT_ALIGN(fdt32_to_cpu(*val), HEAP_BASE_ALIGN);
default_config:
return fw_platform_calculate_heap_size(hart_count);
}
extern struct sbi_platform platform;
static bool platform_has_mlevel_imsic = false;
static u32 generic_hart_index2id[SBI_HARTMASK_MAX_BITS] = { 0 };
@@ -185,7 +207,7 @@ unsigned long fw_platform_init(unsigned long arg0, unsigned long arg1,
}
platform.hart_count = hart_count;
platform.heap_size = fw_platform_calculate_heap_size(hart_count);
platform.heap_size = fw_platform_get_heap_size(fdt, hart_count);
platform_has_mlevel_imsic = fdt_check_imsic_mlevel(fdt);
fw_platform_coldboot_harts_init(fdt);