ARM: rename x86_64 crate to arch

Also,
* move the autogenerated code in a different crate named `arch_sys`
* x86_64 code code inside `arch` and `arch_sys` is labeled as such

Signed-off-by: Diana Popa <[email protected]>

Author: dianpopa

Cargo.lock

@@ -26,7 +26,7 @@ kernel = { path = "../kernel" }
 memory_model = { path = "../memory_model" }
 net_util = { path = "../net_util" }
 rate_limiter = { path = "../rate_limiter" }
-x86_64 = { path = "../x86_64" }
+arch = { path = "../arch" }
 
 [features]
 vsock = ["vmm/vsock"]

api_server/Cargo.toml

@@ -97,11 +97,11 @@ impl PartialEq for ParsedRequest {
 
 #[cfg(test)]
 mod tests {
+    extern crate arch;
     extern crate devices;
     extern crate kernel;
     extern crate memory_model;
     extern crate net_util;
-    extern crate x86_64;
 
     use self::devices::virtio::net::Error as VirtioNetError;
     use self::memory_model::GuestMemoryError;
@@ -299,7 +299,7 @@ mod tests {
         check_error_response(vmm_resp, StatusCode::InternalServerError);
         let vmm_resp = VmmActionError::StartMicrovm(
             ErrorKind::Internal,
-            StartMicrovmError::ConfigureSystem(x86_64::Error::E820Configuration),
+            StartMicrovmError::ConfigureSystem(arch::Error::ZeroPagePastRamEnd),
         );
         check_error_response(vmm_resp, StatusCode::InternalServerError);
         let vmm_resp = VmmActionError::StartMicrovm(

api_server/src/request/mod.rs

@@ -1,14 +1,15 @@
 [package]
-name = "x86_64"
+name = "arch"
 version = "0.1.0"
 authors = ["The Chromium OS Authors"]
 
 [dependencies]
 byteorder = "=1.2.1"
 libc = ">=0.2.39"
 
-kvm_gen = { path = "../kvm_gen" }
+arch_gen = { path = "../arch_gen" }
 kvm = { path = "../kvm" }
+kvm_gen = { path = "../kvm_gen" }
 memory_model = { path = "../memory_model" }
 sys_util = { path = "../sys_util" }
 

x86_64/Cargo.toml renamed to arch/Cargo.toml

@@ -0,0 +1,36 @@
+// Copyright 2018 Amazon.com, Inc. or its affiliates. All Rights Reserved.
+// SPDX-License-Identifier: Apache-2.0
+
+extern crate byteorder;
+extern crate libc;
+
+extern crate arch_gen;
+extern crate kvm;
+extern crate kvm_gen;
+extern crate memory_model;
+extern crate sys_util;
+
+use std::result;
+
+#[derive(Debug)]
+pub enum Error {
+    /// The zero page extends past the end of guest_mem.
+    ZeroPagePastRamEnd,
+    /// Error writing the zero page of guest memory.
+    ZeroPageSetup,
+    #[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
+    /// X86_64 specific error triggered during system configuration.
+    X86_64Setup(x86_64::Error),
+}
+pub type Result<T> = result::Result<T, Error>;
+
+// 1MB.  We don't put anything above here except the kernel itself.
+pub const HIMEM_START: usize = 0x100000;
+
+#[cfg(any(target_arch = "x86", target_arch = "x86_64"))]
+pub mod x86_64;
+impl From<x86_64::Error> for Error {
+    fn from(e: x86_64::Error) -> Error {
+        Error::X86_64Setup(e)
+    }
+}

arch/src/lib.rs

@@ -5,67 +5,46 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the THIRD-PARTY file.
 
-extern crate byteorder;
-extern crate kvm;
-extern crate kvm_gen;
-extern crate libc;
-extern crate memory_model;
-extern crate sys_util;
-
-#[allow(dead_code)]
-#[allow(non_upper_case_globals)]
-#[allow(non_camel_case_types)]
-#[allow(non_snake_case)]
-mod bootparam;
-// boot_params is just a series of ints, it is safe to initialize it.
-unsafe impl memory_model::DataInit for bootparam::boot_params {}
-
-#[allow(dead_code)]
-#[allow(non_upper_case_globals)]
-mod msr_index;
-
-#[allow(dead_code)]
-#[allow(non_upper_case_globals)]
-#[allow(non_camel_case_types)]
-mod mpspec;
-// These mpspec types are only data, reading them from data is a safe initialization.
-unsafe impl memory_model::DataInit for mpspec::mpc_bus {}
-unsafe impl memory_model::DataInit for mpspec::mpc_cpu {}
-unsafe impl memory_model::DataInit for mpspec::mpc_intsrc {}
-unsafe impl memory_model::DataInit for mpspec::mpc_ioapic {}
-unsafe impl memory_model::DataInit for mpspec::mpc_table {}
-unsafe impl memory_model::DataInit for mpspec::mpc_lintsrc {}
-unsafe impl memory_model::DataInit for mpspec::mpf_intel {}
-
 mod gdt;
 pub mod interrupts;
-pub mod layout;
 mod mptable;
 pub mod regs;
 
 use std::mem;
-use std::result;
 
-use bootparam::boot_params;
-use bootparam::E820_RAM;
+use arch_gen::x86::bootparam::{boot_params, E820_RAM};
 use memory_model::{GuestAddress, GuestMemory};
 
-pub use interrupts::Error as IntError;
-pub use mptable::Error as MpTableError;
-pub use regs::Error as RegError;
-
 #[derive(Debug)]
 pub enum Error {
     /// Invalid e820 setup params.
     E820Configuration,
     /// Error writing MP table to memory.
-    MpTableSetup(MpTableError),
-    /// The zero page extends past the end of guest_mem.
-    ZeroPagePastRamEnd,
-    /// Error writing the zero page of guest memory.
-    ZeroPageSetup,
+    MpTableSetup(mptable::Error),
 }
-pub type Result<T> = result::Result<T, Error>;
+
+// Magic addresses used to lay out x86_64 VMs.
+
+// Initial stack for the boot CPU.
+pub const BOOT_STACK_POINTER: usize = 0x8ff0;
+
+// Kernel command line.
+pub const CMDLINE_START: usize = 0x20000;
+pub const CMDLINE_MAX_SIZE: usize = 0x10000;
+
+// The 'zero page', a.k.a linux kernel bootparams.
+pub const ZERO_PAGE_START: usize = 0x7000;
+
+// Where BIOS/VGA magic would live on a real PC.
+const EBDA_START: u64 = 0x9fc00;
+
+// MPTABLE, describing VCPUS.
+const MPTABLE_START: usize = 0x9fc00;
+
+// Initial pagetables.
+const PDE_START: usize = 0xb000;
+const PDPTE_START: usize = 0xa000;
+const PML4_START: usize = 0x9000;
 
 const FIRST_ADDR_PAST_32BITS: usize = (1 << 32);
 const MEM_32BIT_GAP_SIZE: usize = (768 << 20);
@@ -97,7 +76,6 @@ pub fn arch_memory_regions(size: usize) -> Vec<(GuestAddress, usize)> {
 }
 
 /// X86 specific memory hole/ memory mapped devices/ reserved area.
-///
 pub fn get_32bit_gap_start() -> usize {
     FIRST_ADDR_PAST_32BITS - MEM_32BIT_GAP_SIZE
 }
@@ -115,15 +93,15 @@ pub fn configure_system(
     cmdline_addr: GuestAddress,
     cmdline_size: usize,
     num_cpus: u8,
-) -> Result<()> {
+) -> super::Result<()> {
     const KERNEL_BOOT_FLAG_MAGIC: u16 = 0xaa55;
     const KERNEL_HDR_MAGIC: u32 = 0x53726448;
     const KERNEL_LOADER_OTHER: u8 = 0xff;
     const KERNEL_MIN_ALIGNMENT_BYTES: u32 = 0x1000000; // Must be non-zero.
     let first_addr_past_32bits = GuestAddress(FIRST_ADDR_PAST_32BITS);
     let end_32bit_gap_start = GuestAddress(get_32bit_gap_start());
 
-    let himem_start = GuestAddress(layout::HIMEM_START);
+    let himem_start = GuestAddress(super::HIMEM_START);
 
     // Note that this puts the mptable at the last 1k of Linux's 640k base RAM
     mptable::setup_mptable(guest_mem, num_cpus).map_err(Error::MpTableSetup)?;
@@ -137,7 +115,7 @@ pub fn configure_system(
     params.hdr.cmdline_size = cmdline_size as u32;
     params.hdr.kernel_alignment = KERNEL_MIN_ALIGNMENT_BYTES;
 
-    add_e820_entry(&mut params, 0, layout::EBDA_START, E820_RAM)?;
+    add_e820_entry(&mut params, 0, EBDA_START, E820_RAM)?;
 
     let mem_end = guest_mem.end_addr();
     if mem_end < end_32bit_gap_start {
@@ -164,20 +142,25 @@ pub fn configure_system(
         }
     }
 
-    let zero_page_addr = GuestAddress(layout::ZERO_PAGE_START);
+    let zero_page_addr = GuestAddress(ZERO_PAGE_START);
     guest_mem
         .checked_offset(zero_page_addr, mem::size_of::<boot_params>())
-        .ok_or(Error::ZeroPagePastRamEnd)?;
+        .ok_or(super::Error::ZeroPagePastRamEnd)?;
     guest_mem
         .write_obj_at_addr(params, zero_page_addr)
-        .map_err(|_| Error::ZeroPageSetup)?;
+        .map_err(|_| super::Error::ZeroPageSetup)?;
 
     Ok(())
 }
 
 /// Add an e820 region to the e820 map.
 /// Returns Ok(()) if successful, or an error if there is no space left in the map.
-fn add_e820_entry(params: &mut boot_params, addr: u64, size: u64, mem_type: u32) -> Result<()> {
+fn add_e820_entry(
+    params: &mut boot_params,
+    addr: u64,
+    size: u64,
+    mem_type: u32,
+) -> Result<(), Error> {
     if params.e820_entries >= params.e820_map.len() as u8 {
         return Err(Error::E820Configuration);
     }
@@ -193,7 +176,7 @@ fn add_e820_entry(params: &mut boot_params, addr: u64, size: u64, mem_type: u32)
 #[cfg(test)]
 mod tests {
     use super::*;
-    use bootparam::e820entry;
+    use arch_gen::x86::bootparam::e820entry;
 
     #[test]
     fn regions_lt_4gb() {
@@ -223,7 +206,12 @@ mod tests {
     fn test_system_configuration() {
         let no_vcpus = 4;
         let gm = GuestMemory::new(&vec![(GuestAddress(0), 0x10000)]).unwrap();
-        assert!(configure_system(&gm, GuestAddress(0), 0, 1).is_err());
+        let config_err = configure_system(&gm, GuestAddress(0), 0, 1);
+        assert!(config_err.is_err());
+        assert_eq!(
+            format!("{:?}", config_err),
+            "Err(X86_64Setup(MpTableSetup(NotEnoughMemory)))"
+        );
 
         // Now assigning some memory that falls before the 32bit memory hole.
         let mem_size = 128 << 20;