Page MenuHomeFreeBSD
Differential D53656

stand: Rename EFI_ERROR_CODE to DECODE_ERROR to avoid Edk2 Clash
ClosedPublic
Actions

Authored by imp on Nov 10 2025, 2:38 AM.
Tags
None
Referenced Files
F169642202: D53656.id166181.diff
Wed, Sep 2, 3:51 AM
F169605152: D53656.id166551.diff
Wed, Sep 2, 1:19 AM
F169495921: D53656.id166102.diff
Tue, Sep 1, 5:17 PM
F169487556: D53656.id.diff
Tue, Sep 1, 4:38 PM
Unknown Object (File)
Fri, Aug 28, 6:00 AM
Unknown Object (File)
Wed, Aug 26, 7:07 AM
Unknown Object (File)
Wed, Aug 26, 7:03 AM
Unknown Object (File)
Sun, Aug 23, 9:55 PM
View All Files
Subscribers
bcran
dab

Details

Reviewers
manu
kevans
tsoome
Commits
rG87795c6e95b0: stand: Rename EFI_ERROR_CODE to DECODE_ERROR to avoid Edk2 Clash
Summary

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68491
Build 65374: arc lint + arc unit

Event Timeline

imp created this revision.Nov 10 2025, 2:38 AM
Herald added a reviewer: manu. · View Herald TranscriptNov 10 2025, 2:38 AM
Herald added subscribers: dab, bcran. · View Herald Transcript
imp requested review of this revision.Nov 10 2025, 2:38 AM
Harbormaster completed remote builds in B68491: Diff 166102.Nov 10 2025, 2:38 AM
imp added a child revision: D53651: efi: Move edk2 compat defines to sys/efi-edk2.h.Nov 10 2025, 2:39 AM
imp added reviewers: kevans, tsoome.
tsoome accepted this revision.Nov 10 2025, 4:11 PM

I used DECODE_ERROR() in illumos loader in pair with ENCODE_ERROR(). (https://github.com/illumos/illumos-gate/commit/f334afcfaebea1b7dc3430015651d8d748fa8a3e).

This revision is now accepted and ready to land.Nov 10 2025, 4:11 PM
imp retitled this revision from stand: Rename EFI_ERROR_CODE to EFI_ERRCODE to avoid Edk2 Clash to stand: Rename EFI_ERROR_CODE to DECODE_ERROR to avoid Edk2 Clash.Nov 10 2025, 11:14 PM
imp updated this revision to Diff 166181.Nov 10 2025, 11:14 PM

Use DECODE_ERROR as the name, as suggested. It's maybe less likely to conflict?

This revision now requires review to proceed.Nov 10 2025, 11:14 PM
Harbormaster completed remote builds in B68524: Diff 166181.Nov 10 2025, 11:14 PM
tsoome accepted this revision.Nov 11 2025, 1:19 PM
This revision is now accepted and ready to land.Nov 11 2025, 1:19 PM
Closed by commit rG87795c6e95b0: stand: Rename EFI_ERROR_CODE to DECODE_ERROR to avoid Edk2 Clash (authored by imp). · Explain WhyNov 16 2025, 6:29 PM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG87795c6e95b0: stand: Rename EFI_ERROR_CODE to DECODE_ERROR to avoid Edk2 Clash.

Revision Contents
Changeset List

PathSize
stand/
efi/
boot1/
8 lines
4 lines
2 lines
2 lines
gptboot/
4 lines
include/
2 lines
libefi/
2 lines
2 lines
2 lines
10 lines
4 lines
8 lines
loader/
arch/
amd64/
4 lines
8 lines
6 lines
4 lines
6 lines
2 lines

Diff 166102

View Options

stand/efi/boot1/boot1.c

Loading...
View Options

stand/efi/boot1/proto.c

Loading...
View Options

stand/efi/boot1/ufs_module.c

Loading...
View Options

stand/efi/boot1/zfs_module.c

Loading...
View Options

stand/efi/gptboot/proto.c

Loading...
View Options

stand/efi/include/efierr.h

Loading...
View Options

stand/efi/libefi/devpath.c

Loading...
View Options

stand/efi/libefi/efi_driver_utils.c

Loading...
View Options

stand/efi/libefi/eficom.c

Loading...
View Options

stand/efi/libefi/efinet.c

Loading...
View Options

stand/efi/libefi/efipart.c

Loading...
View Options

stand/efi/libefi/env.c

Loading...
View Options

stand/efi/loader/arch/amd64/multiboot2.c

Loading...
View Options

stand/efi/loader/arch/amd64/trap.c

Loading...
View Options

stand/efi/loader/bootinfo.c

Loading...
View Options

stand/efi/loader/copy.c

Loading...
View Options

stand/efi/loader/framebuffer.c

Loading...
View Options

stand/efi/loader/main.c

Loading...