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

Adding MBR partitioning and using it to create a boot image #298

Merged
merged 1 commit into from
May 7, 2021

Conversation

DrDeano
Copy link
Member

@DrDeano DrDeano commented Apr 8, 2021

Sorry for this being a bit long. I was hoping to split this up into two PR, one for the MBR stuff and one for building a boot image.
But what this adds:

  • Simple MBR partition scheme implementation
  • Refactored makefs to include making a MBR partition table.
  • Refactored some common filesystem operations
  • Removed echfs and parted dependencies (used MBR and FAT32 in house code)
    Pls don't look at the build file. It really needs a big refactor, it is a mess, but is working ;) This should be fixed in another PR, contributor friendly :)

@DrDeano DrDeano added enhancement New feature or request arch: none Affects common code across architectures building Relates to building the kernel labels Apr 8, 2021
@DrDeano DrDeano added this to the v0.4 milestone Apr 8, 2021
@DrDeano DrDeano requested a review from SamTebbs33 April 8, 2021 20:25
src/kernel/filesystem/filesystem_common.zig Outdated Show resolved Hide resolved
src/kernel/filesystem/filesystem_common.zig Outdated Show resolved Hide resolved
src/kernel/time.zig Show resolved Hide resolved
@DrDeano DrDeano force-pushed the feature/mbr-partitioning branch from b1e690b to cd2db39 Compare April 21, 2021 16:38
@DrDeano DrDeano requested a review from SamTebbs33 April 22, 2021 19:02
@DrDeano DrDeano force-pushed the feature/mbr-partitioning branch from cd2db39 to a067b5d Compare May 7, 2021 19:09
This adds the a MBR partition implementation. 
Refactored some file system code to have a common FS helper functions
This also enables the using the MBR and FAT32 to create the boot image removing the echfs dependency.
Moved file system boot sector to common x86 arch.
As this boot sector is written in x86, where would need to be an ARM version, so would be arch dependent.
Added MAP file path
Fixed running the x86_64 port as this wasn't making the boot image, just the elf
@DrDeano DrDeano force-pushed the feature/mbr-partitioning branch from d9fc95e to b3ac0aa Compare May 7, 2021 19:33
@DrDeano DrDeano merged commit ebdea66 into feature/x86_64-port May 7, 2021
@DrDeano DrDeano deleted the feature/mbr-partitioning branch May 7, 2021 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch: none Affects common code across architectures building Relates to building the kernel enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants