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

[tracking] shim 15.4 critical regressions #165

Closed
julian-klode opened this issue Apr 20, 2021 · 9 comments
Closed

[tracking] shim 15.4 critical regressions #165

julian-klode opened this issue Apr 20, 2021 · 9 comments
Labels
meta Not a review request, but an issue or notice wrt the signing process

Comments

@julian-klode
Copy link
Collaborator

julian-klode commented Apr 20, 2021

Posting this here for awareness of submitters regarding fatal issues that can break boot on machines and other errors of unknown extend.

Critical issues:

Important issues:

Unknown extent:

You might want to hold off submitting shims until there is a 15.5 release, or cherry-pick patches.

@julian-klode julian-klode changed the title [tracking] shim 15.4 regressions [tracking] shim 15.4 critical regressions Apr 20, 2021
@julian-klode julian-klode pinned this issue Apr 20, 2021
@steve-mcintyre steve-mcintyre unpinned this issue Apr 20, 2021
@julian-klode julian-klode pinned this issue Apr 20, 2021
@Doncuppjr
Copy link

So we need 822d07ad4f07ef66fe447a130e1027c88d02a394 if we want Mac support?

@steve-mcintyre
Copy link
Collaborator

aarch64 builds are really awkward - they will break using binutils versions before 2.35, and even after that they may give unpredictable results. We really need proper toolchain support.

@Doncuppjr
Copy link

I guess the better question is, do they want to make a 15.5?

@steve-mcintyre
Copy link
Collaborator

822d07ad4f07ef66fe447a130e1027c88d02a394 is for really old Macs that won't do SB anyway. But without that patch you'll fail to boot the machine at all

@Doncuppjr
Copy link

And I guess we are waiting on 8b59591775a0412863aab9596ab87bdd493a9c1e to be merged or QA'ed?

@cmurf
Copy link

cmurf commented May 9, 2021

Fedora's shim-15.4-4 has fixes for some of the items listed in this issue. But there are two issues we're seeing with a subset of Lenovo and Dell systems that don't yet have upstream issues filed (and I don't understand the problem well enough to file them).
https://bugzilla.redhat.com/show_bug.cgi?id=1955416
https://bugzilla.redhat.com/show_bug.cgi?id=1954245

@realnickel
Copy link

As this issue is pinned I'd like to draw attention of new submitters to an old comment, calling for cross review between submitters themselves:
#120 (comment)
I think that might speed up review process for everyone by eliminating most common inconsistencies before authorized review-board members pay attention to submission.

@pnardini
Copy link

This was a good reminder @realnickel, thanks. If submitters are hesitant to review because they "don't know where to start" they can begin with some of the reviewer guidelines captured here: https://github.com/rhboot/shim/wiki/reviewer-guidelines.

@julian-klode
Copy link
Collaborator Author

shim 15.5 is out, closing this.

@julian-klode julian-klode unpinned this issue Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Not a review request, but an issue or notice wrt the signing process
Projects
None yet
Development

No branches or pull requests

6 participants