refactor: migrate to pvh crate

Signed-off-by: Martin Kröning <martin.kroening@eonerc.rwth-aachen.de>

Author: mkroening

Cargo.lock

@@ -39,6 +39,7 @@ fdt = "0.1.5"
 chrono = { version = "0.4", default-features = false }
 
 [target.'cfg(target_arch = "x86_64")'.dependencies]
+pvh = { version = "0.1", default-features = false, features = ["elfnote"] }
 uart_16550 = "0.4.0"
 x86_64 = { version = "0.15.4", default-features = false, features = [
     "instructions",

Cargo.toml

@@ -179,7 +179,9 @@ fn boot_from_device(
 
 #[cfg(target_arch = "x86_64")]
 #[no_mangle]
-pub extern "C" fn rust64_start(#[cfg(not(feature = "coreboot"))] pvh_info: &pvh::StartInfo) -> ! {
+pub extern "C" fn rust64_start(
+    #[cfg(not(feature = "coreboot"))] pvh_info: &::pvh::start_info::StartInfo,
+) -> ! {
     serial::PORT.borrow_mut().init();
     logger::init();
 

src/main.rs

@@ -1,45 +1,20 @@
 // SPDX-License-Identifier: Apache-2.0
 // Copyright 2020 Google LLC
 
-use core::mem::size_of;
+use pvh::start_info::{MemmapTableEntry, StartInfo};
 
 use crate::{
     bootinfo::{EntryType, Info, MemoryEntry},
     common,
     layout::MemoryDescriptor,
 };
 
-// Structures from xen/include/public/arch-x86/hvm/start_info.h
-#[derive(Debug)]
-#[repr(C)]
-pub struct StartInfo {
-    magic: [u8; 4],
-    version: u32,
-    flags: u32,
-    nr_modules: u32,
-    modlist_paddr: u64,
-    cmdline_paddr: u64,
-    rsdp_paddr: u64,
-    memmap_paddr: u64,
-    memmap_entries: u32,
-    _pad: u32,
-}
-
-#[derive(Clone, Copy, Debug)]
-#[repr(C)]
-struct MemMapEntry {
-    addr: u64,
-    size: u64,
-    entry_type: u32,
-    _pad: u32,
-}
-
-impl From<MemMapEntry> for MemoryEntry {
-    fn from(value: MemMapEntry) -> Self {
+impl From<MemmapTableEntry> for MemoryEntry {
+    fn from(value: MemmapTableEntry) -> Self {
         Self {
             addr: value.addr,
             size: value.size,
-            entry_type: EntryType::from(value.entry_type),
+            entry_type: EntryType::from(value.ty),
         }
     }
 }
@@ -63,7 +38,7 @@ impl Info for StartInfo {
     }
     fn entry(&self, idx: usize) -> MemoryEntry {
         assert!(idx < self.num_entries());
-        let ptr = self.memmap_paddr as *const MemMapEntry;
+        let ptr = self.memmap_paddr as *const MemmapTableEntry;
         let entry = unsafe { *ptr.add(idx) };
         MemoryEntry::from(entry)
     }
@@ -77,34 +52,7 @@ impl Info for StartInfo {
 
 // The PVH Boot Protocol starts at the 32-bit entrypoint to our firmware.
 extern "C" {
-    fn ram32_start();
+    fn ram32_start() -> !;
 }
 
-// The kind/name/desc of the PHV ELF Note are from xen/include/public/elfnote.h.
-// This is the "Physical entry point into the kernel".
-const XEN_ELFNOTE_PHYS32_ENTRY: u32 = 18;
-type Name = [u8; 4];
-type Desc = unsafe extern "C" fn();
-
-// We make sure our ELF Note has an alignment of 4 for maximum compatibility.
-// Some software (QEMU) calculates padding incorectly if alignment != 4.
-#[repr(C, packed(4))]
-struct Note {
-    name_size: u32,
-    desc_size: u32,
-    kind: u32,
-    name: Name,
-    desc: Desc,
-}
-
-// This is: ELFNOTE(Xen, XEN_ELFNOTE_PHYS32_ENTRY, .quad ram32_start)
-#[cfg(not(test))]
-#[link_section = ".note"]
-#[used]
-static PVH_NOTE: Note = Note {
-    name_size: size_of::<Name>() as u32,
-    desc_size: size_of::<Desc>() as u32,
-    kind: XEN_ELFNOTE_PHYS32_ENTRY,
-    name: *b"Xen\0",
-    desc: ram32_start,
-};
+pvh::xen_elfnote_phys32_entry!(ram32_start);

src/pvh.rs

@@ -14,7 +14,7 @@ SECTIONS
   /* Mapping the program headers and note into RAM makes the file smaller. */
   . = ram_min;
   . += SIZEOF_HEADERS;
-  .note : { *(.note) } :note :ram
+  .note : { *(.note .note.*) } :note :ram
 
   /* These sections are mapped into RAM from the file. Omitting :ram from
      later sections avoids emitting empty sections in the final binary.       */