rust-osdev · Freax13 · Aug 25, 2024 · Aug 21, 2024 · Aug 25, 2024
diff --git a/bios/boot_sector/README.md b/bios/boot_sector/README.md
@@ -4,13 +4,13 @@ This executable needs to fit into the 512-byte boot sector, so we need to use al
 
 ## Build Commands
 
-1. `cargo build --release -Zbuild-std=core --target x86-16bit.json -Zbuild-std-features=compiler-builtins-mem`
-2. `objcopy -I elf32-i386 -O binary target/x86-16bit/release/first_stage target/disk_image.bin
+1. `cargo build --profile=stage-1 -Zbuild-std=core --target ../../i386-code16-boot-sector.json -Zbuild-std-features=compiler-builtins-mem`
+2. `objcopy -I elf32-i386 -O binary ../../target/i386-code16-boot-sector/stage-1/bootloader-x86_64-bios-boot-sector ../../target/disk_image.img`
 
 To run in QEMU:
 
-- `qemu-system-x86_64 -drive format=raw,file=target/disk_image.bin`
+- `qemu-system-x86_64 -drive format=raw,file=../../target/disk_image.img`
 
 To print the contents of the ELF file, e.g. for trying to bring the size down:
 
-- `objdump -xsdS -M i8086,intel target/x86-16bit/release/first_stage`
+- `objdump -xsdS -M i8086,intel ../../target/i386-code16-boot-sector/stage-1/bootloader-x86_64-bios-boot-sector`
diff --git a/bios/boot_sector/src/boot.s b/bios/boot_sector/src/boot.s
@@ -36,13 +36,18 @@ check_int13h_extensions:
     mov bx, 0x55aa
     # dl contains drive number
     int 0x13
-    jc fail
+    jnc .int13_pass
+    call fail
+.int13_pass:
     pop ax      # pop error code again
 
 rust:
     # push arguments
     push dx     # disk number
     call first_stage
+    # Fail code if first stage returns
+    push 'x'
+    call fail
 
 spin:
     hlt

diff --git a/bios/boot_sector/src/dap.rs b/bios/boot_sector/src/dap.rs
@@ -38,11 +38,13 @@ impl DiskAddressPacket {
         let self_addr = self as *const Self as u16;
         unsafe {
             asm!(
-                "push 0x7a", // error code `z`, passed to `fail` on error
+                "push 'z'", // error code `z`, passed to `fail` on error
                 "mov {1:x}, si", // backup the `si` register, whose contents are required by LLVM
                 "mov si, {0:x}",
                 "int 0x13",
-                "jc fail",
+                "jnc 2f", // carry is set on fail
+                "call fail",
+                "2:",
                 "pop si", // remove error code again
                 "mov si, {1:x}", // restore the `si` register to its prior state
                 in(reg) self_addr,

diff --git a/bios/boot_sector/src/mbr.rs b/bios/boot_sector/src/mbr.rs
@@ -16,14 +16,14 @@ pub(crate) fn get_partition(partitions_raw: &[u8], index: usize) -> PartitionTab
             .get(8..)
             .and_then(|s| s.get(..4))
             .and_then(|s| s.try_into().ok())
-            .unwrap_or_fail(b'e'),
+            .unwrap_or_fail(b'f'),
     );
     let len = u32::from_le_bytes(
         buffer
             .get(12..)
             .and_then(|s| s.get(..4))
             .and_then(|s| s.try_into().ok())
-            .unwrap_or_fail(b'f'),
+            .unwrap_or_fail(b'g'),
     );
     PartitionTableEntry::new(bootable, partition_type, lba, len)
 }

diff --git a/bios/stage-2/src/dap.rs b/bios/stage-2/src/dap.rs
@@ -38,11 +38,13 @@ impl DiskAddressPacket {
     pub unsafe fn perform_load(&self, disk_number: u16) {
         let self_addr = self as *const Self as u16;
         asm!(
-            "push 0x7a", // error code `z`, passed to `fail` on error
+            "push 'z'", // error code `z`, passed to `fail` on error
             "mov {1:x}, si",
             "mov si, {0:x}",
             "int 0x13",
-            "jc fail",
+            "jnc 2f", // carry is set on fail
+            "call fail",
+            "2:",
             "pop si", // remove error code again
             "mov si, {1:x}",
             in(reg) self_addr,

diff --git a/build.rs b/build.rs
    );
 }

 #[cfg(not(docsrs_dummy_build))]
 #[cfg(feature = "uefi")]
 async fn build_uefi_bootloader(out_dir: &Path) -> PathBuf {
    let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".into());

 // dummy implementation because docsrs builds have no network access.
 // This will put an empty file in out_dir and return its path.
 #[cfg(docsrs_dummy_build)]
 #[cfg(feature = "uefi")]
 async fn build_uefi_bootloader(out_dir: &Path) -> PathBuf {
    use std::fs::File;
    path
 }

 #[cfg(not(docsrs_dummy_build))]
 #[cfg(feature = "bios")]
 async fn build_bios_boot_sector(out_dir: &Path) -> PathBuf {
    let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".into());

 // dummy implementation because docsrs builds have no network access.
 // This will put an empty file in out_dir and return its path.
 #[cfg(docsrs_dummy_build)]
 #[cfg(feature = "bios")]
 async fn build_bios_boot_sector(out_dir: &Path) -> PathBuf {
    use std::fs::File;
    path
 }

 #[cfg(not(docsrs_dummy_build))]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_2(out_dir: &Path) -> PathBuf {
    let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".into());

 // dummy implementation because docsrs builds have no network access.
 // This will put an empty file in out_dir and return its path.
 #[cfg(docsrs_dummy_build)]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_2(out_dir: &Path) -> PathBuf {
    use std::fs::File;
    path
 }

 #[cfg(not(docsrs_dummy_build))]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_3(out_dir: &Path) -> PathBuf {
    let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".into());

 // dummy implementation because docsrs builds have no network access.
 // This will put an empty file in out_dir and return its path.
 #[cfg(docsrs_dummy_build)]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_3(out_dir: &Path) -> PathBuf {
    use std::fs::File;
    path
 }

 #[cfg(not(docsrs_dummy_build))]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_4(out_dir: &Path) -> PathBuf {
    let cargo = std::env::var("CARGO").unwrap_or_else(|_| "cargo".into());

 // dummy implementation because docsrs builds have no network access.
 // This will put an empty file in out_dir and return its path.
 #[cfg(docsrs_dummy_build)]
 #[cfg(feature = "bios")]
 async fn build_bios_stage_4(out_dir: &Path) -> PathBuf {
    use std::fs::File;
	);
	}

	#[cfg(not(docsrs_dummy_build))]
Check warning on line 62 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 62 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "uefi")]
	async fn build_uefi_bootloader(out_dir: &Path) -> PathBuf {
	let cargo = std::env::var("CARGO").unwrap_or_else(\|_\| "cargo".into());

	// dummy implementation because docsrs builds have no network access.
	// This will put an empty file in out_dir and return its path.
	#[cfg(docsrs_dummy_build)]
Check warning on line 101 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 101 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "uefi")]
	async fn build_uefi_bootloader(out_dir: &Path) -> PathBuf {
	use std::fs::File;
	path
	}

	#[cfg(not(docsrs_dummy_build))]
Check warning on line 119 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 119 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_boot_sector(out_dir: &Path) -> PathBuf {
	let cargo = std::env::var("CARGO").unwrap_or_else(\|_\| "cargo".into());

	// dummy implementation because docsrs builds have no network access.
	// This will put an empty file in out_dir and return its path.
	#[cfg(docsrs_dummy_build)]
Check warning on line 165 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 165 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_boot_sector(out_dir: &Path) -> PathBuf {
	use std::fs::File;
	path
	}

	#[cfg(not(docsrs_dummy_build))]
Check warning on line 183 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 183 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_2(out_dir: &Path) -> PathBuf {
	let cargo = std::env::var("CARGO").unwrap_or_else(\|_\| "cargo".into());

	// dummy implementation because docsrs builds have no network access.
	// This will put an empty file in out_dir and return its path.
	#[cfg(docsrs_dummy_build)]
Check warning on line 231 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 231 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_2(out_dir: &Path) -> PathBuf {
	use std::fs::File;
	path
	}

	#[cfg(not(docsrs_dummy_build))]
Check warning on line 249 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 249 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_3(out_dir: &Path) -> PathBuf {
	let cargo = std::env::var("CARGO").unwrap_or_else(\|_\| "cargo".into());

	// dummy implementation because docsrs builds have no network access.
	// This will put an empty file in out_dir and return its path.
	#[cfg(docsrs_dummy_build)]
Check warning on line 293 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 293 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_3(out_dir: &Path) -> PathBuf {
	use std::fs::File;
	path
	}

	#[cfg(not(docsrs_dummy_build))]
Check warning on line 311 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 311 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_4(out_dir: &Path) -> PathBuf {
	let cargo = std::env::var("CARGO").unwrap_or_else(\|_\| "cargo".into());

	// dummy implementation because docsrs builds have no network access.
	// This will put an empty file in out_dir and return its path.
	#[cfg(docsrs_dummy_build)]
Check warning on line 356 in build.rs GitHub Actions / Clippy unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Check unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (macos-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (macos-12) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (ubuntu-latest) unexpected `cfg` condition name: `docsrs_dummy_build` Check warning on line 356 in build.rs GitHub Actions / Test (windows-latest) unexpected `cfg` condition name: `docsrs_dummy_build`
	#[cfg(feature = "bios")]
	async fn build_bios_stage_4(out_dir: &Path) -> PathBuf {
	use std::fs::File;