-
Notifications
You must be signed in to change notification settings - Fork 246
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
Fix for 'error:unrecognized number' when booting ISO on PPC #1978
Conversation
Issue is caused by incorrect, according to PAPR specification, bootinfo entity parsing code in grub2. In PAPR spec "B.4.1.6.1 Bootinfo Entities" partition and path separator (,) is optional and if not present bootinfo may be treated as directory and filename components unless it begins with number: &device;:[&partition;][,]&directory;&filename; If (,) is missing and directory begins with a hexadecimal number GRUB2 treats this as partition number immediately followed by a directory/filename. Note that with (,) present in bootpath issue isn't in the case as GRUB2 returns empty string for GRUB_PARSE_PARTITIONS. This effectively meaning that GRUB2 code requires (,) as mandatory. This is a GRUB2 bug, but until it is fixed, we can make the message disappear by specifying a comma in bootinfo.txt. Signed-off-by: Renaud Métrich <rmetrich@redhat.com>
@schabrolles |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The explanation looks fine to me to accept the PR
@schabrolles |
@schabrolles It's as simple as making an ISO and booting it :-) |
Hi team, I'm currently testing it ... Sorry for the delay, as you say I'm a bit overloaded and I've completely forgotten to test it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rmetrich @schabrolles |
Relax-and-Recover (ReaR) Pull Request Template
Please fill in the following items before submitting a new pull request:
Pull Request Details:
Type: Bug Fix
Impact: Low
Reference to related issue (URL): No issue created
How was this pull request tested? Not tested, I don't have the hardware for now, please may someone with hardware test.
Brief description of the changes in this pull request: