[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 2/6] ieee1275: drop len -= 1 quirk in heap_init
From: |
Stefan Berger |
Subject: |
[PATCH v3 2/6] ieee1275: drop len -= 1 quirk in heap_init |
Date: |
Mon, 30 Jan 2023 14:13:00 -0500 |
From: Daniel Axtens <dja@axtens.net>
This was apparently 'required by some firmware': commit dc9468500919
("2007-02-12 Hollis Blanchard <hollis@penguinppc.org>").
It's not clear what firmware that was, and what platform from 14 years ago
which exhibited the bug then is still both in use and buggy now.
It doesn't cause issues on qemu (mac99 or pseries) or under PFW for Power8.
I don't have access to old Mac hardware, but if anyone feels especially
strongly we can put it under some feature flag. I really want to disable
it under pseries because it will mess with region merging.
Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
---
grub-core/kern/ieee1275/init.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/grub-core/kern/ieee1275/init.c b/grub-core/kern/ieee1275/init.c
index 0bc571e3e..bb234b268 100644
--- a/grub-core/kern/ieee1275/init.c
+++ b/grub-core/kern/ieee1275/init.c
@@ -166,7 +166,6 @@ heap_init (grub_uint64_t addr, grub_uint64_t len,
grub_memory_type_t type,
addr = 0x180000;
}
}
- len -= 1; /* Required for some firmware. */
/* Never exceed HEAP_MAX_SIZE */
if (*total + len > HEAP_MAX_SIZE)
--
2.25.1
- [PATCH v3 0/6] Dynamic allocation of memory regions and IBM vTPM v2, Stefan Berger, 2023/01/30
- [PATCH v3 2/6] ieee1275: drop len -= 1 quirk in heap_init,
Stefan Berger <=
- [PATCH v3 1/6] ieee1275: request memory with ibm, client-architecture-support, Stefan Berger, 2023/01/30
- [PATCH v3 6/6] ibmvtpm: Add support for trusted boot using a vTPM 2.0, Stefan Berger, 2023/01/30
- [PATCH v3 4/6] ieee1275: implement vec5 for cas negotiation, Stefan Berger, 2023/01/30
- [PATCH v3 3/6] ieee1275: support runtime memory claiming, Stefan Berger, 2023/01/30
- [PATCH v3 5/6] Add memtool module with memory allocation stress-test, Stefan Berger, 2023/01/30