CVS commit: [netbsd-10] src/sys/arch/i386/stand/lib

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Tue Sep  3 15:23:41 UTC 2024

Modified Files:
        src/sys/arch/i386/stand/lib [netbsd-10]: biosmemx.S bootinfo_memmap.c
            getextmemx.c

Log Message:
Pull up following revision(s) (requested by andvar in ticket #815):

        sys/arch/i386/stand/lib/bootinfo_memmap.c: revision 1.7
        sys/arch/i386/stand/lib/bootinfo_memmap.c: revision 1.8
        sys/arch/i386/stand/lib/getextmemx.c: revision 1.11
        sys/arch/i386/stand/lib/getextmemx.c: revision 1.12
        sys/arch/i386/stand/lib/biosmemx.S: revision 1.12
        sys/arch/i386/stand/lib/biosmemx.S: revision 1.13

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.
PR install/49470

i386/stand/lib: Nix trailing whitespace and tidy comments a bit.
No functional change intended.
PR port-amd64/49470: NetBSD 7 BETA reboots after bootloader (HP 2133)


To generate a diff of this commit:
cvs rdiff -u -r1.11 -r1.11.8.1 src/sys/arch/i386/stand/lib/biosmemx.S
cvs rdiff -u -r1.6 -r1.6.26.1 src/sys/arch/i386/stand/lib/bootinfo_memmap.c
cvs rdiff -u -r1.10 -r1.10.84.1 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.