CVS commit: src/sys/arch/i386/stand/lib

["Andrius Varanavicius" <andvar%netbsd.org@localhost>]

Module Name:    src
Committed By:   andvar
Date:           Sat Aug 17 08:38:31 UTC 2024

Modified Files:
        src/sys/arch/i386/stand/lib: biosmemx.S bootinfo_memmap.c getextmemx.c

Log Message:
Allocate buf[6] instead of buf[5] to work around buggy firmware, which may
write mementry to 24 bytes instead 20 requested with ACPI 3.0 enabled,
causing corrupted stack and wrong marks[] values in common_load_kernel().
This in turn was leading to erroneous caluclations and memory addresses.
This eventually may result in null pointer dereference in the kernel and sudden
reboot for VIA based systems.

Commit should fix boot issues for VIA C7-M based HP 2133 Mini-Note,
mentioned in the original bug report. For AMD based HP Compaq 6005 Pro
discussed in a separate thread, the issue may be different.

Tested on my VIA C7-D based Biostar Viotech 3100+ motherboard.

needs-pullups to 9, 10.

PR install/49470


To generate a diff of this commit:
cvs rdiff -u -r1.11 -r1.12 src/sys/arch/i386/stand/lib/biosmemx.S
cvs rdiff -u -r1.6 -r1.7 src/sys/arch/i386/stand/lib/bootinfo_memmap.c
cvs rdiff -u -r1.10 -r1.11 src/sys/arch/i386/stand/lib/getextmemx.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.