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

Documentation #134

Merged
merged 1 commit into from
Dec 16, 2020
Merged

Documentation #134

merged 1 commit into from
Dec 16, 2020

Conversation

Qubasa
Copy link
Contributor

@Qubasa Qubasa commented Dec 10, 2020

I documented the build process and added sse to the feature list and a little debug help section

@@ -87,13 +157,28 @@ dd if=target/x86_64-bootloader/release/bootloader.bin of=/dev/sdX && sync

Where sdX is the device name of your USB stick. **Be careful** to choose the correct device name, because everything on that device is overwritten.

## Debugging
Set a breakpoint at address `0x7c00`. Disassemble instructions with gdb:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This address will differ between versions of bootloader and possibly between assembler versions.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the address BIOS jumps to to start code execution, we'll always have something there

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, ok.

@ethindp
Copy link
Contributor

ethindp commented Dec 13, 2020

I disagree; when the bootloader gains UEFI support 0x7C00 will no longer be a constant address, I believe.

@Qubasa
Copy link
Contributor Author

Qubasa commented Dec 13, 2020

I disagree; when the bootloader gains UEFI support 0x7C00 will no longer be a constant address, I believe.

I mean, the documentation documents the current behavior if this changes the documentation should be changed accordingly

Copy link
Member

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks a lot!

I agree that we can leave the 0x7c00 debug address in for now, we will have to update the documentation in many places for the new UEFI bootloader anyway.

@phil-opp phil-opp merged commit 530b26f into rust-osdev:master Dec 16, 2020
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

Successfully merging this pull request may close these issues.

5 participants