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

Disrecommend RW_LEGACY #218

Open
ninelore opened this issue Dec 31, 2024 · 19 comments
Open

Disrecommend RW_LEGACY #218

ninelore opened this issue Dec 31, 2024 · 19 comments

Comments

@ninelore
Copy link
Member

ninelore commented Dec 31, 2024

  • RW_LEGACY should only be used for dual booting with ChromeOS, otherwise there is a risk of lockout on GSC reboot.
  • Recommend submarine instead
@sadlerm4
Copy link
Contributor

sadlerm4 commented Jan 5, 2025

Perhaps we can add documentation on enabling the GBB flag so that it won't matter even if enable_dev_altfw is reset.

@ninelore
Copy link
Member Author

ninelore commented Jan 5, 2025

Submarine should be the primary recommendation because iirc RW_LEGACY causes bugs in Linux that arent easily solved. @ellyq or @MrChromebox might be able to elaborate further or correct me.

@WeirdTreeThing
Copy link
Member

And what does submarine do differently to fix these issues?

@ninelore
Copy link
Member Author

ninelore commented Jan 5, 2025

And what does submarine do differently to fix these issues?

Made this issue because of #217 (comment). I don't want to rule out that I misremember that part. Pinged these people for clarification for that reason.

@WeirdTreeThing
Copy link
Member

By that logic, we shouldn't recommend submarine because if you wipe ChromeOS and dev_boot_signed_only gets cleared then you are locked out.

@WeirdTreeThing
Copy link
Member

In both cases you can use recomod to fix the nvram flags being reset. I'm also not sure how they would reset unless you intentionally did it.

@ninelore
Copy link
Member Author

ninelore commented Jan 5, 2025

I'm also not sure how they would reset unless you intentionally did it.

dead battery / GSC reboot?

@WeirdTreeThing
Copy link
Member

Google fixed the dead battery thing ages ago. And I've never heard of GSC reboot causing nvram to be wiped. Do you have anything to show these will cause nvram to be lost?

@ninelore
Copy link
Member Author

ninelore commented Jan 5, 2025

Mind specifying "ages"? I don't think MrChromebox would write that without reason.

@MrChromebox
Copy link
Collaborator

I certainly remember post-Haswell/Baytrail devices having the dev_boot_legacy/altfw crossystem flags cleared and users ending up in a situation where they couldn't boot

@WeirdTreeThing
Copy link
Member

We already don't recommend RW_LEGACY on EOL devices, so as long as this issue doesn't affect anything still supported I don't think it should matter. And as I said above, submarine would have the exact same issue.

@MrChromebox
Copy link
Collaborator

I honestly don't care either way. I'd be happy to drop RWL support entirely

@ninelore
Copy link
Member Author

ninelore commented Jan 5, 2025

Is that due to maintenance effort of the RWL images? That'd be one issue submarine could solve. But dropping RWL support is a question whether we want to screw Dual-boot users.

@MrChromebox
Copy link
Collaborator

combination of:

  • getting edk2 to boot as RWL without ability to debug via serial
  • no NVRAM
  • issues with i2c devices in stock google firmware
  • and other misc issues

@WeirdTreeThing
Copy link
Member

Technically you could build a custom rom with serial enabled and depthcharge, but that's more effort than it's worth.

@ninelore
Copy link
Member Author

ninelore commented Jan 6, 2025

a question whether we want to screw Dual-boot users.

I think its your call @MrChromebox

@MrChromebox
Copy link
Collaborator

I think as long as it exists, we just document the limitations, and let the users go from there

@ninelore
Copy link
Member Author

ninelore commented Jan 8, 2025

We could declare it deprecated, that future breakages and/or issues wont be addresses anymore

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

4 participants