Hi，can you tell me what's the worst effect if i apply the following patch?

In order to boot a BIG vm (with 4T mem, 255 vCPUs, 60 virtio-scsi disk...), i have to increase the

BUILD_MAX_HIGHTABLE to 512KB.

But, then i found i can not boot a specific VM anymore (oracle linux 6.7 64bits with kernel 3.8.13, 2G mem).

It seems be related with the unused high ram given VM back by SeaBios (maybe also related to the memory align).

I've tested if setting giveback memory 64KB alignment other than 4KB, SeaBios with 512KB BUILD_MAX_HIGHTABLE

can boot the specific VM. But , i don't know the reason...

As far as i know, if seabios does not give back the unused ZoneHigh memory seems will not trigger any problems

but just waste only a little memory ,right?

diff --git a/src/config.h b/src/config.h
index baca029..b3536f1 100644
index baca029..b3536f1 100644
--- a/src/config.h
+++ b/src/config.h
@@ -17,7 +17,7 @@
// Maximum number of map entries in the e820 map
#define BUILD_MAX_E820 32
// Space to reserve in high-memory for tables
-#define BUILD_MAX_HIGHTABLE (256*1024)
+#define BUILD_MAX_HIGHTABLE (512*1024)

diff --git a/src/malloc.c b/src/malloc.c
index 3733855..6302525 100644
--- a/src/malloc.c
+++ b/src/malloc.c
@@ -549,13 +549,7 @@ malloc_prepboot(void)
     dprintf(1, "Space available for UMB: %x-%x, %x-%x\n"
             , RomEnd, base, info->range_start, info->range_end);

-    // Give back unused high ram.
-    info = alloc_find_lowest(&ZoneHigh);
-    if (info) {
-        u32 giveback = ALIGN_DOWN(info->range_end-info->range_start, PAGE_SIZE);
-        e820_add(info->range_start, giveback, E820_RAM);
-        dprintf(1, "Returned %d bytes of ZoneHigh\n", giveback);
-    }

     calcRamSize();