Skip to content

Documentation#134

Merged
phil-opp merged 1 commit intorust-osdev:masterfrom
Qubasa:documentation
Dec 16, 2020
Merged

Documentation#134
phil-opp merged 1 commit intorust-osdev:masterfrom
Qubasa:documentation

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

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

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.

4 participants