loader.efi: add error printouts for chain command
Needs ReviewPublic
Actions

Authored by tsoome on May 1 2025, 7:37 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jun 10, 3:46 AM

	Unknown Object (File)
	Mon, Jun 2, 4:42 AM

	Unknown Object (File)
	Wed, May 28, 1:45 PM

	Unknown Object (File)
	Wed, May 28, 4:41 AM

	Unknown Object (File)
	Mon, May 19, 12:12 PM

	Unknown Object (File)
	May 6 2025, 3:55 AM

	Unknown Object (File)
	May 5 2025, 11:48 AM

	Unknown Object (File)
	May 5 2025, 6:54 AM

Subscribers

Details

Reviewers

manu

Summary

Current loader.efi chain command can fail with mysterious
"LoadImage failed" or "StartImage failed". Since the error codes
may have multiple causes, print out the UEFI error codes.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63815
Build 60699: arc lint + arc unit

Event Timeline

tsoome created this revision.May 1 2025, 7:37 AM

Herald added a reviewer: manu. · View Herald TranscriptMay 1 2025, 7:37 AM

Herald added subscribers: dab, bcran, imp. · View Herald Transcript

tsoome requested review of this revision.May 1 2025, 7:37 AM

Harbormaster completed remote builds in B63815: Diff 154606.May 1 2025, 7:37 AM

imp added inline comments.May 1 2025, 5:35 PM

stand/efi/loader/main.c
1907	Shouldn't we be also setting command_errmsg? I'm confused why you removed that...
1924	Either remove (void) entirely, or remove the space between it and the function call.

Revision Contents
Changeset List

Path

Size

stand/

efi/

loader/

main.c

11 lines

Diff 154606

View Options

loader.efi: add error printouts for chain commandNeeds ReviewPublicActions