grub-ieee1275 should not change boot-device for every upgrade
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
grub2 (Ubuntu) |
Fix Released
|
Critical
|
Mathieu Trudel-Lapierre | ||
Trusty |
Fix Released
|
Critical
|
Unassigned | ||
grub2-signed (Ubuntu) |
Fix Released
|
Critical
|
Unassigned | ||
Trusty |
Fix Released
|
Critical
|
Unassigned |
Bug Description
[Impact]
This affects all ppc64el users whenever they upgrade grub.
[Test case]
1) run 'nvram --print-
2) upgrade grub-ieee1275
2) run 'nvram --print-
The value should not change between runs, it should also not be touched at all (even if it's set to the same value).
This can be further verified by modifying boot-device to an arbitrary value before running the grub upgrade.
[Regression potential]
This has minimal potential for regression. It removes a code path which might otherwise break booting after upgrading grub on the affected systems. One possibility for regression is that if code paths are subsequently added to register openfirmware devices, this should be guarded with an 'if (update_nvram)' test to make sure it only ever happens when the --no-nvram flag is *NOT* passed to grub-install.
---
grub-ieee1275 should call grub-install with --no-nvram in postinst, so as not to modify boot-device in OpenFirmware everytime the grub package is updated.
description: | updated |
Changed in grub2 (Ubuntu): | |
status: | New → In Progress |
importance: | Undecided → Critical |
assignee: | nobody → Mathieu Trudel-Lapierre (mathieu-tl) |
Changed in grub2 (Ubuntu Trusty): | |
importance: | Undecided → Critical |
status: | New → In Progress |
Changed in grub2-signed (Ubuntu Trusty): | |
status: | New → In Progress |
Changed in grub2-signed (Ubuntu Trusty): | |
importance: | Undecided → Critical |
Changed in grub2-signed (Ubuntu): | |
importance: | Undecided → Critical |
Changed in grub2-signed (Ubuntu): | |
status: | New → Invalid |
Hello Mathieu, or anyone else affected,
Accepted grub2 into trusty-proposed. The package will build now and be available at https:/ /launchpad. net/ubuntu/ +source/ grub2/2. 02~beta2- 9ubuntu1. 8 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification- failed. In either case, details of your testing will help us make a better decision.
Further information regarding the verification process can be found at https:/ /wiki.ubuntu. com/QATeam/ PerformingSRUVe rification . Thank you in advance!