Skip to content

Commit

Permalink
malloc: use variable for ZoneHigh size
Browse files Browse the repository at this point in the history
Use the variable highram_size instead of the BUILD_MAX_HIGHTABLE #define
for the ZoneHigh size. Initialize the new variable with the old #define,
so behavior does not change.

This allows to easily adjust the ZoneHigh size at runtime in a followup
patch.

Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
  • Loading branch information
kraxel committed Apr 27, 2022
1 parent 0177400 commit 3b91e8e
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions src/malloc.c
Original file line number Diff line number Diff line change
Expand Up @@ -422,7 +422,8 @@ malloc_preinit(void)
e820_add(BUILD_BIOS_ADDR, BUILD_BIOS_SIZE, E820_RESERVED);

// Populate temp high ram
u32 highram = 0;
u32 highram_start = 0;
u32 highram_size = BUILD_MAX_HIGHTABLE;
int i;
for (i=e820_count-1; i>=0; i--) {
struct e820entry *en = &e820_list[i];
Expand All @@ -432,10 +433,10 @@ malloc_preinit(void)
if (en->type != E820_RAM || end > 0xffffffff)
continue;
u32 s = en->start, e = end;
if (!highram) {
u32 newe = ALIGN_DOWN(e - BUILD_MAX_HIGHTABLE, MALLOC_MIN_ALIGN);
if (!highram_start) {
u32 newe = ALIGN_DOWN(e - highram_size, MALLOC_MIN_ALIGN);
if (newe <= e && newe >= s) {
highram = newe;
highram_start = newe;
e = newe;
}
}
Expand All @@ -444,9 +445,9 @@ malloc_preinit(void)

// Populate regions
alloc_add(&ZoneTmpLow, BUILD_STACK_ADDR, BUILD_EBDA_MINIMUM);
if (highram) {
alloc_add(&ZoneHigh, highram, highram + BUILD_MAX_HIGHTABLE);
e820_add(highram, BUILD_MAX_HIGHTABLE, E820_RESERVED);
if (highram_start) {
alloc_add(&ZoneHigh, highram_start, highram_start + highram_size);
e820_add(highram_start, highram_size, E820_RESERVED);
}
}

Expand Down

0 comments on commit 3b91e8e

Please sign in to comment.