Page MenuHomeFreeBSD

stand/multiboot: adjust the protocol between loader and kernel
ClosedPublic

Authored by royger on Jan 29 2021, 11:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 20, 3:34 AM
Unknown Object (File)
Oct 2 2024, 10:57 AM
Unknown Object (File)
Oct 2 2024, 1:05 AM
Unknown Object (File)
Oct 1 2024, 1:30 PM
Unknown Object (File)
Sep 29 2024, 11:21 PM
Unknown Object (File)
Sep 29 2024, 11:09 PM
Unknown Object (File)
Sep 27 2024, 10:11 AM
Unknown Object (File)
Sep 24 2024, 5:21 PM
Subscribers

Details

Summary

There's a currently ad-hoc protocol to hand off the FreeBSD kernel
payload between the loader and the kernel itself when Xen is in the
middle of the picture. Such protocol wasn't very resilient to changes
to the loader itself, because it relied on moving metadata around to
package it using a certain layout. This has proven to be fragile, so
replace it with a more robust version.

The new protocol requires using a xen_header structure that will be
used to pass data between the FreeBSD loader and the FreeBSD kernel
when booting in dom0 mode. At the moment the only data conveyed is the
offset of the start of the module metadata relative to the stat of the
module itself.

This is a slightly disruptive change since it also requires a change
to the kernel which is contained in this patch. In order to update
with this change the kernel must be updated before updating the
loader, as described in the handbook. Note this is only required when
booting a FreeBSD/Xen dom0. This change doesn't affect the normal
FreeBSD boot protocol.

This fixes booting FreeBSD/Xen in dom0 mode after
3630506b9daec9167a89bc4525638ea45a00769e.

Sponsored by: Citrix Systems R&D
MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

stand/i386/libi386/multiboot.c
298

strdup() can fail and return NULL.

303

we probably should use archsw.arch_copyin() but then again, this is i386 tree anyhow...

This revision is now accepted and ready to land.Jan 29 2021, 12:41 PM