Opened 2 years ago
Closed 2 years ago
#17857 closed bug (fixed)
"General Protection Error" during boot
Reported by: | widespace_ | Owned by: | nobody |
---|---|---|---|
Priority: | normal | Milestone: | R1/beta4 |
Component: | System/Kernel | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | x86-64 |
Description
On hrev56326 trying to boot results in the kernel panicking with a "General Protection Error" without "turning on" any of the boot indicators. It does not appear to generate a syslog.
Attachments (1)
Change History (11)
by , 2 years ago
Attachment: | 20220804_171051.jpg added |
---|
comment:1 by , 2 years ago
I just checked and hrev56325 boots fine so the problematic change is between these two revisions.
comment:2 by , 2 years ago
Thanks. How much RAM on this computer? https://github.com/haiku/haiku/blob/e83979afa4cf62fd7980a863da80be11644feabe/src/system/kernel/fs/EntryCache.cpp#L87 looks like it checks against 1MB instead of 1GB.
comment:4 by , 2 years ago
shaka reported something similar via IRC: https://matrix.org/_matrix/media/r0/download/matrix.org/cptKGazpyXmnOZZLmeHyiItT/ima_9f0a5f2.jpeg
korli, how does that check 1MB? 1024^3
is the number of bytes in 1GB. Anyway 32GB of RAM is obviously more than enough for this. I'm not sure how we wound up with a crash here, in the worst case we'd get a NULL dereference but this isn't even that.
comment:5 by , 2 years ago
Seems more likely that the refactoring changes somehow caused this, really, not the last commit to have larger cache sizes.
comment:8 by , 2 years ago
korli, how does that check 1MB?
Right, read too much strange code. The coding style didn't help either, "1024 * 1024 * 1024" would be more readable...
comment:9 by , 2 years ago
Replying to waddlesplash:
Should be fixed in hrev56330, please test.
It appears to be fixed, hrev56332 boots fine.
comment:10 by , 2 years ago
Milestone: | Unscheduled → R1/beta4 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Image of KDL output