Page MenuHomeFreeBSD

loader: lsefi: Print more information
ClosedPublic

Authored by manu on Nov 15 2021, 12:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 24, 12:26 PM
Unknown Object (File)
Fri, Nov 15, 2:51 AM
Unknown Object (File)
Fri, Nov 15, 12:30 AM
Unknown Object (File)
Sun, Nov 3, 1:09 PM
Unknown Object (File)
Sun, Nov 3, 1:08 PM
Unknown Object (File)
Sun, Nov 3, 1:08 PM
Unknown Object (File)
Sun, Nov 3, 12:53 PM
Unknown Object (File)
Oct 14 2024, 3:10 PM
Subscribers

Details

Summary

Printing the EFI_HANDLE pointer isn't very useful.
If the handle have a IMAGE_DEVICE_PATH or a DEVICE_PATH protocol print it.
This makes it easier to see which devices are present and what protocol they
expose.

Sponsored by: Beckhoff Automation GmbH & Co. KG

Test Plan

Tested on u-boot based and edk2 based firmware

Diff Detail

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

Event Timeline

manu requested review of this revision.Nov 15 2021, 12:58 PM

It's better than what we have as is... Question about whether or not we have multiple paths that would be useful or not (I can't recall that deatil). If so, please adjust to print both (I still approve), if not just indicate in comments (and I also still approve)

stand/efi/loader/main.c
1417

Can you have both an image path and a devpath for the handle? And would you want to print them both?

This revision is now accepted and ready to land.Nov 15 2021, 3:41 PM
stand/efi/loader/main.c
1417

You can't if I understand correctly the UEFI driver model.

stand/efi/loader/main.c
1417

OK then. forget I said anything... I wasn't quite sure... feel free to commit when you think the review is complete.

This revision was automatically updated to reflect the committed changes.