Re: [PATCH 2/3] bootloader: Handle 64-bit bootinfo
Taylor R Campbell <[email protected]>
| Newsgroups | gmane.os.netbsd.devel.kernel |
|---|---|
| Message-ID | <[email protected]> |
> Date: Sun, 2 Mar 2025 15:18:08 +0300 > From: Vladimir Serbinenko <[email protected]> > > On my darp10-b the bootloader is loaded around 32GiB mark and so all > BI_ADD with a static address end up way over 4GiB and are truncated > when passed to the kernel. Instead keep the whole pointer until we move > all bootinfos to the heap Thanks, I filed PR port-amd64/NNNNN: `x86 efiboot: bootinfo pointers truncated when mapped >4GB' <https://gnats.NetBSD.org/NNNNN> to track this. I wrote a note about the issue jakllsch@ raised with non-efi boot in the `Fix' section before I saw what you said about uintptr_t being 32-bit for non-EFI boot. Feel free to follow up on the PR -- just send mail with subject `Re: port-amd64/NNNNN: x86 efiboot: bootinfo pointers truncated when mapped >4GB' to [email protected] cc netbsd-bugs@NetBSD>org -- with clarification, and yes, we should have a CTASSERT in the non-EFIBOOT case.