[SeaBIOS] [PATCH] Fix high memory zone initialization in CSM mode

José Martínez via SeaBIOS posted 1 patch 10 months, 3 weeks ago
Failed in applying to current master (apply log)
src/fw/csm.c | 4 ++--
src/malloc.c | 6 +++---
2 files changed, 5 insertions(+), 5 deletions(-)
[SeaBIOS] [PATCH] Fix high memory zone initialization in CSM mode
Posted by José Martínez via SeaBIOS 10 months, 3 weeks ago
malloc_high() cannot allocate any memory in CSM mode due to an empty
ZoneHigh. SeaBIOS cannot find any disk to boot from because device
initialization fails.

The bug was introduced in 1.16.1 (commit dc88f9b) when the meaning of
BUILD_MAX_HIGHTABLE changed but CSM code was not updated. This patch
reverts to the previous behavior by using BUILD_MIN_HIGHTABLE in CSM
methods.

Signed-off-by: José Martínez <xose@google.com>
---
 src/fw/csm.c | 4 ++--
 src/malloc.c | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/fw/csm.c b/src/fw/csm.c
index bc14a92..579debd 100644
--- a/src/fw/csm.c
+++ b/src/fw/csm.c
@@ -150,9 +150,9 @@ handle_csm_0002(struct bregs *regs)
     for (i=0; i < csm_compat_table.E820Length / sizeof(struct e820entry); i++)
         e820_add(p[i].start, p[i].size, p[i].type);

-    if (csm_init_table->HiPmmMemorySizeInBytes > BUILD_MAX_HIGHTABLE) {
+    if (csm_init_table->HiPmmMemorySizeInBytes > BUILD_MIN_HIGHTABLE) {
         u32 hi_pmm_end = csm_init_table->HiPmmMemory +
csm_init_table->HiPmmMemorySizeInBytes;
-        e820_add(hi_pmm_end - BUILD_MAX_HIGHTABLE,
BUILD_MAX_HIGHTABLE, E820_RESERVED);
+        e820_add(hi_pmm_end - BUILD_MIN_HIGHTABLE,
BUILD_MIN_HIGHTABLE, E820_RESERVED);
     }

     // For PCIBIOS 1ab10e
diff --git a/src/malloc.c b/src/malloc.c
index da84098..30b2c92 100644
--- a/src/malloc.c
+++ b/src/malloc.c
@@ -460,10 +460,10 @@ malloc_csm_preinit(u32 low_pmm, u32
low_pmm_size, u32 hi_pmm, u32 hi_pmm_size)
 {
     ASSERT32FLAT();

-    if (hi_pmm_size > BUILD_MAX_HIGHTABLE) {
+    if (hi_pmm_size > BUILD_MIN_HIGHTABLE) {
         u32 hi_pmm_end = hi_pmm + hi_pmm_size;
-        alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm_end - BUILD_MAX_HIGHTABLE);
-        alloc_add(&ZoneHigh, hi_pmm_end - BUILD_MAX_HIGHTABLE, hi_pmm_end);
+        alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm_end - BUILD_MIN_HIGHTABLE);
+        alloc_add(&ZoneHigh, hi_pmm_end - BUILD_MIN_HIGHTABLE, hi_pmm_end);
     } else {
         alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm + hi_pmm_size);
     }
--
2.40.1.606.ga4b1b128d6-goog
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org
[SeaBIOS] Re: [PATCH] Fix high memory zone initialization in CSM mode
Posted by Kevin O'Connor 9 months, 2 weeks ago
On Fri, May 12, 2023 at 02:14:47PM +0100, José Martínez via SeaBIOS wrote:
> malloc_high() cannot allocate any memory in CSM mode due to an empty
> ZoneHigh. SeaBIOS cannot find any disk to boot from because device
> initialization fails.
> 
> The bug was introduced in 1.16.1 (commit dc88f9b) when the meaning of
> BUILD_MAX_HIGHTABLE changed but CSM code was not updated. This patch
> reverts to the previous behavior by using BUILD_MIN_HIGHTABLE in CSM
> methods.

Thanks.  I committed this change.

-Kevin
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org
[SeaBIOS] Re: [PATCH] Fix high memory zone initialization in CSM mode
Posted by Kevin O'Connor 10 months ago
On Fri, May 12, 2023 at 02:14:47PM +0100, José Martínez via SeaBIOS wrote:
> malloc_high() cannot allocate any memory in CSM mode due to an empty
> ZoneHigh. SeaBIOS cannot find any disk to boot from because device
> initialization fails.
> 
> The bug was introduced in 1.16.1 (commit dc88f9b) when the meaning of
> BUILD_MAX_HIGHTABLE changed but CSM code was not updated. This patch
> reverts to the previous behavior by using BUILD_MIN_HIGHTABLE in CSM
> methods.

Thanks.  The change makes sense to me.  Gerd, do you have any comments
on this change?

-Kevin


> 
> Signed-off-by: José Martínez <xose@google.com>
> ---
>  src/fw/csm.c | 4 ++--
>  src/malloc.c | 6 +++---
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/fw/csm.c b/src/fw/csm.c
> index bc14a92..579debd 100644
> --- a/src/fw/csm.c
> +++ b/src/fw/csm.c
> @@ -150,9 +150,9 @@ handle_csm_0002(struct bregs *regs)
>      for (i=0; i < csm_compat_table.E820Length / sizeof(struct e820entry); i++)
>          e820_add(p[i].start, p[i].size, p[i].type);
> 
> -    if (csm_init_table->HiPmmMemorySizeInBytes > BUILD_MAX_HIGHTABLE) {
> +    if (csm_init_table->HiPmmMemorySizeInBytes > BUILD_MIN_HIGHTABLE) {
>          u32 hi_pmm_end = csm_init_table->HiPmmMemory +
> csm_init_table->HiPmmMemorySizeInBytes;
> -        e820_add(hi_pmm_end - BUILD_MAX_HIGHTABLE,
> BUILD_MAX_HIGHTABLE, E820_RESERVED);
> +        e820_add(hi_pmm_end - BUILD_MIN_HIGHTABLE,
> BUILD_MIN_HIGHTABLE, E820_RESERVED);
>      }
> 
>      // For PCIBIOS 1ab10e
> diff --git a/src/malloc.c b/src/malloc.c
> index da84098..30b2c92 100644
> --- a/src/malloc.c
> +++ b/src/malloc.c
> @@ -460,10 +460,10 @@ malloc_csm_preinit(u32 low_pmm, u32
> low_pmm_size, u32 hi_pmm, u32 hi_pmm_size)
>  {
>      ASSERT32FLAT();
> 
> -    if (hi_pmm_size > BUILD_MAX_HIGHTABLE) {
> +    if (hi_pmm_size > BUILD_MIN_HIGHTABLE) {
>          u32 hi_pmm_end = hi_pmm + hi_pmm_size;
> -        alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm_end - BUILD_MAX_HIGHTABLE);
> -        alloc_add(&ZoneHigh, hi_pmm_end - BUILD_MAX_HIGHTABLE, hi_pmm_end);
> +        alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm_end - BUILD_MIN_HIGHTABLE);
> +        alloc_add(&ZoneHigh, hi_pmm_end - BUILD_MIN_HIGHTABLE, hi_pmm_end);
>      } else {
>          alloc_add(&ZoneTmpHigh, hi_pmm, hi_pmm + hi_pmm_size);
>      }
> --
> 2.40.1.606.ga4b1b128d6-goog
> _______________________________________________
> SeaBIOS mailing list -- seabios@seabios.org
> To unsubscribe send an email to seabios-leave@seabios.org
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org
[SeaBIOS] Re: [PATCH] Fix high memory zone initialization in CSM mode
Posted by Gerd Hoffmann 10 months ago
On Tue, May 30, 2023 at 09:58:23PM -0400, Kevin O'Connor wrote:
> On Fri, May 12, 2023 at 02:14:47PM +0100, José Martínez via SeaBIOS wrote:
> > malloc_high() cannot allocate any memory in CSM mode due to an empty
> > ZoneHigh. SeaBIOS cannot find any disk to boot from because device
> > initialization fails.
> > 
> > The bug was introduced in 1.16.1 (commit dc88f9b) when the meaning of
> > BUILD_MAX_HIGHTABLE changed but CSM code was not updated. This patch
> > reverts to the previous behavior by using BUILD_MIN_HIGHTABLE in CSM
> > methods.
> 
> Thanks.  The change makes sense to me.  Gerd, do you have any comments
> on this change?

Looks good to me.

Possibly we should add support for larger allocations in CSM mode in a
followup patch.  Not sure this is needed though, when running in CSM
mode I'd suspect features needing large allocations will be handled by
the EFI firmware instead of seabios ...

take care,
  Gerd

_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-leave@seabios.org