serial: Use external uart_16550 crate

When trying to run this on various hypervisors (QEMU, GCE, etc...), I
noticed that our serial implementation is not very robust.  We can
start in the wrong mode or drop bytes, as we do not properly observe
the status bits or setup the configuration registers.

Instead of doing this ourselevs, we just use an external crate. Note
that this create is made by the same people as x86_64, and only has
that crate as a dependancy.

Signed-off-by: Joe Richey <[email protected]>

Author: josephlr

Cargo.lock

@@ -28,4 +28,4 @@ bitflags = "1.2"
 x86_64 = "0.9"
 atomic_refcell = "0.1"
 r-efi = "2.1.0"
-
+uart_16550 = "0.2.5"

Cargo.toml

@@ -140,6 +140,8 @@ fn boot_from_device(device: &mut block::VirtioBlockDevice, info: &dyn boot::Info
 
 #[no_mangle]
 pub extern "C" fn rust64_start(rdi: Option<&pvh::StartInfo>, rsi: Option<&boot::Params>) -> ! {
+    serial::PORT.borrow_mut().init();
+
     if let Some(start_info) = rdi {
         log!("\nBooting via PVH Boot Protocol");
         run(start_info)

src/main.rs

@@ -18,20 +18,15 @@
 use core::fmt;
 
 use atomic_refcell::AtomicRefCell;
-use x86_64::instructions::port::PortWriteOnly;
+use uart_16550::SerialPort;
 
 // We use COM1 as it is the standard first serial port.
-static PORT: AtomicRefCell<PortWriteOnly<u8>> = AtomicRefCell::new(PortWriteOnly::new(0x3f8));
+pub static PORT: AtomicRefCell<SerialPort> = AtomicRefCell::new(unsafe { SerialPort::new(0x3f8) });
 
 pub struct Serial;
-
 impl fmt::Write for Serial {
     fn write_str(&mut self, s: &str) -> fmt::Result {
-        let mut port = PORT.borrow_mut();
-        for b in s.bytes() {
-            unsafe { port.write(b) }
-        }
-        Ok(())
+        PORT.borrow_mut().write_str(s)
     }
 }