-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Advanced logger for Non DXE phases #801
Conversation
Signed-off-by: kuqin12 <42554914+kuqin12@users.noreply.github.com>
Signed-off-by: kuqin12 <42554914+kuqin12@users.noreply.github.com>
…ano_platforms into adv_logger_non_dxe
…ano_platforms into adv_logger_non_dxe
@kuqin12, what is the current state of this PR? |
Should be ready to go. Tested on both Q35 and SBSA. |
Regarding #522. It states:
For future reference, I think the issue should mention that for QemuSbsaPkg, |
Ah, yes. I was wondering whether I should actually create a new one for that. But that will be a deeper conversation of how we want to pursue TFA modifications. I will at least add a comment there for better reference. |
Description
This change introduces non DXE advanced logger, which should enable us to verify the advanced logger related changes better. Resolves #522.
flow, or firmware?
validation improvement, ...
in build or boot behavior?
a function in a new library class in a pre-existing module, ...
outside direct code modifications (and comments)?
on an a separate Web page, ...
How This Was Tested
This change was tested on both QEMU Q35 and SBSA packages and verified bootable to UEFI shell.
Integration Instructions
N/A