Skip to content

[BUG] Panic-ked at 'assertion failed: max_phys_addr < (1 << 48) / 512' #19

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

Closed
Stay1444 opened this issue Jul 7, 2023 · 5 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@Stay1444
Copy link

Stay1444 commented Jul 7, 2023

Describe the bug
After running cargo run --release i get panicket at 'assertion failed: max_phys_addr < (1 << 48) / 512' (build success, and running in QEMU)

To Reproduce
Steps to reproduce the behavior:

  1. Clone the repo
  2. run it in releasemode

Expected behavior
The system boots

Screenshots
image

Additional information

  • Architecture: x86_64
  • OS: Arch Linux Kernel 6.4.1
  • Build environment: linux
  • QEMU version: QEMU emulator version 8.0.2
  • rustc version: rustc 1.59.0-nightly (cfa4ac66c 2022-01-06)
  • Commit hash: 8daca65
@Stay1444 Stay1444 added the bug Something isn't working label Jul 7, 2023
@tomshoo
Copy link
Contributor

tomshoo commented Jul 7, 2023

This apparently is a bug in the bootloader crate, which was mentioned in issue #1138, which is apparently fixed in its pull request #234. I currently have a branch that bumps up everything to the latest, and apparently, everything seems to work just fine the system seems to boot, but I still am a tad hesitant to create a PR because bumping everything to the latest might just open another can of worms.

EDIT: I delete the branch mentioned by me because it contained way too many changes and it was getting hard to trace them.

@Stay1444
Copy link
Author

Stay1444 commented Jul 7, 2023

Amazing! Will check it out and see if i can contribute. Any recommendations?

@Stay1444 Stay1444 changed the title [BUG] Panicket at 'assertion failed: max_phys_addr < (1 << 48) / 512' [BUG] Panic-ked at 'assertion failed: max_phys_addr < (1 << 48) / 512' Jul 7, 2023
@tomshoo
Copy link
Contributor

tomshoo commented Jul 7, 2023

Any recommendations?

I just tested it on an older toolchain and seems like bumping the version for bootloader crate to 0.9.23 seems to be enough. You can try doing that and testing even further. I unfortunately cannot ATM since it is 12:00AM here and I need to get some sleep.

@tomshoo
Copy link
Contributor

tomshoo commented Jul 14, 2023

This issue has been fixed in PR #25.

@danielteberian
Copy link
Contributor

Thank you @tomshoo. I'm closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants