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

RunningMode::RunningInstallerRecovery is unreliable #2248

Closed
VisualEhrmanntraut opened this issue Mar 18, 2023 · 11 comments · Fixed by acidanthera/Lilu#88
Closed

RunningMode::RunningInstallerRecovery is unreliable #2248

VisualEhrmanntraut opened this issue Mar 18, 2023 · 11 comments · Fixed by acidanthera/Lilu#88

Comments

@VisualEhrmanntraut
Copy link

VisualEhrmanntraut commented Mar 18, 2023

RunningMode::RunningInstallerRecovery is only set at 2nd update stage (first reboot). Recovery partition and next update stages seem to have it unset.
macOS Version: 11.7.4 (Big Sur)
Lilu version: 1.6.4

@vit9696
Copy link
Contributor

vit9696 commented May 7, 2023

Do you have an idea how can we detect these stages? E.g. some boot arguments or device tree specifics?

@VisualEhrmanntraut
Copy link
Author

VisualEhrmanntraut commented May 7, 2023

I don't know about anything else, but the boot args remain the same as the ones in the config.plist. I tested it by printing the contents of the value returned by PE_boot_args

@VisualEhrmanntraut
Copy link
Author

(This test was conducted on the Big Sur recovery dmg, aka com.apple.recovery.boot/BaseSystem.dmg)

@vit9696
Copy link
Contributor

vit9696 commented May 7, 2023

Sorry, if I was unclear, I mean the boot arguments one can use to detect the recovery. Perhaps you would be able to provide the IORegistry dump (IOService and IODeviceTree planes) in the recovery?

@VisualEhrmanntraut
Copy link
Author

I understood perfectly fine. My response was there's no new boot args appended while booting the macOS 11 Recovery DMG; they remain the same as the ones in the config.plist according to my debug print with the contents of PE_boot_args(). I'll try to fetch the IORegistry dump while in recovery for further inspection

@VisualEhrmanntraut
Copy link
Author

Here you go @vit9696
MacBook Pro.ioreg.zip

vit9696 added a commit to acidanthera/Lilu that referenced this issue May 7, 2023
@vit9696
Copy link
Contributor

vit9696 commented May 7, 2023

Please test.

@VisualEhrmanntraut
Copy link
Author

Works fine, @vit9696! Don't forget to fix the Analysis issues, lol

@VisualEhrmanntraut
Copy link
Author

Note: Only normal boot and the recovery update was tested. I'm going to perform an update now to make sure it works here as well

vit9696 added a commit to acidanthera/Lilu that referenced this issue May 7, 2023
@VisualEhrmanntraut
Copy link
Author

Unfortunately, due to unexpected issues I'm unable to perform an update at this time. I will need someone else to test this

@vit9696
Copy link
Contributor

vit9696 commented May 7, 2023

I fixed the analysers and will merge the patch. I suggest you to raise a new issue if the update fails.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants