Skip to content
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

The "Error: Locate graphic output protocol fail: (0xE) Not Found." always be showed if platform doesn't provide EFI_GRAPHICS_OUTPUT_PROTOCOL #440

Closed
joeyli opened this issue Dec 15, 2021 · 3 comments

Comments

@joeyli
Copy link
Contributor

joeyli commented Dec 15, 2021

On some platform, like GCP or OVMF which does NOT provide EFI_GRAPHICS_OUTPUT_PROTOCOL when no display device (or display device be disabled). It causes that the "Error: Locate graphic output protocol fail: (0xE) Not Found." always be showed on console when we enroll MOK through MokManager.

@joeyli
Copy link
Contributor Author

joeyli commented Dec 15, 2021

I have sent a pull request here: #441

The patch removed the "Error: Locate graphic output protocol fail: (0xE) Not Found." message box.

@dennis-tseng99
Copy link
Contributor

If gop(Graphic Output Protocol) is not implemented in your platform, then a small window box will prompt and be waiting for a keystroke pressed by user, such that no more action can be implicitly continued thereafter. For example, VM cannot be accessed . There are 2 solutions:
[sol-1]
Following Joey's suggestion, we can just remove the warning window box generated by console_error() function.
efi_status = gBS->LocateProtocol(&gop_guid, NULL, (void **)&gop);
if (EFI_ERROR(efi_status)) {
console_error(L"Locate graphic output protocol fail", efi_status);
return;
}
[sol-2]
Let timer to control warning window box generated by console_countdown() function.
efi_status = gBS->LocateProtocol(&gop_guid, NULL, (void **)&gop);
if (EFI_ERROR(efi_status)) {
console_countdown(title,L"cannot find a graphic output protocol",timeout);
return;
}
Any comments are welcome :-)

@joeyli
Copy link
Contributor Author

joeyli commented Nov 23, 2022

Close this issue because patch be merged to shim mainline.

@joeyli joeyli closed this as completed Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants