rework architecture and add a "compiler"

Cargo.lock

@@ -19,6 +19,10 @@ version = "1.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "baf1de4339761588bc0619e3cbc0120ee582ebb74b53b4efbf79117bd2da40fd"
 
+[[package]]
+name = "compiler"
+version = "0.1.0"
+
 [[package]]
 name = "hashbrown"
 version = "0.13.2"

Cargo.toml

@@ -1,10 +1,2 @@
-[package]
-name = "holey_bytes"
-version = "0.1.0"
-edition = "2021"
-
-# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
-
-[dependencies]
-log = "*"
-hashbrown = "0.13.2"
+[workspace]
+members = ["hbvm", "compiler"]

compiler/Cargo.toml

@@ -0,0 +1,8 @@
+[package]
+name = "compiler"
+version = "0.1.0"
+edition = "2021"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]

compiler/src/main.rs

@@ -0,0 +1,5 @@
+fn main() {
+    let prog = "load 1, A0
+    jump 0";
+    println!("Hello, world!");
+}

hbvm/Cargo.toml

@@ -0,0 +1,10 @@
+[package]
+name = "holey_bytes"
+version = "0.1.0"
+edition = "2021"
+
+# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
+
+[dependencies]
+log = "*"
+hashbrown = "0.13.2"

hbvm/src/bytecode/ops.rs

@@ -30,6 +30,13 @@ pub enum Operations {
     EnviromentCall = 255,
 }
 
+pub enum PageMapTypes {
+    // Have the host make a new VMPage
+    VMPage = 0,
+    // Ask the host to map a RealPage into memory
+    RealPage = 1,
+}
+
 pub enum SubTypes {
     EightBit = 1,
     SixtyFourBit = 2,

hbvm/src/engine/mod.rs

@@ -3,27 +3,32 @@ pub mod config;
 pub mod enviroment_calls;
 pub mod regs;
 
-use crate::bytecode::{
+use {
+    self::call_stack::CallStack,
+    crate::{
+        bytecode::{
             ops::{Operations::*, *},
             types::*,
-};
-
-use {
-    crate::{engine::call_stack::FnCall, memory, HaltStatus, RuntimeErrors},
+        },
+        engine::call_stack::FnCall,
+        memory, HaltStatus, RuntimeErrors,
+    },
     alloc::vec::Vec,
     config::EngineConfig,
     log::trace,
     regs::Registers,
 };
-
-use self::call_stack::CallStack;
 // pub const PAGE_SIZE: usize = 8192;
 
+pub struct RealPage {
+    pub ptr: *mut u8,
+}
+
 #[derive(Debug, Clone, Copy)]
-pub struct Page {
+pub struct VMPage {
     pub data: [u8; 8192],
 }
-impl Page {
+impl VMPage {
     pub fn new() -> Self {
         Self {
             data: [0; 4096 * 2],
@@ -31,6 +36,21 @@ impl Page {
     }
 }
 
+pub enum Page {
+    VMPage(VMPage),
+    RealPage(RealPage),
+}
+impl Page {
+    pub fn data(&self) -> [u8; 4096 * 2] {
+        match self {
+            Page::VMPage(vmpage) => vmpage.data,
+            Page::RealPage(_) => {
+                unimplemented!("Memmapped hw page not yet supported")
+            }
+        }
+    }
+}
+
 pub fn empty_enviroment_call(engine: &mut Engine) -> Result<&mut Engine, u64> {
     trace!("Registers {:?}", engine.registers);
     Err(0)
@@ -38,16 +58,16 @@ pub fn empty_enviroment_call(engine: &mut Engine) -> Result<&mut Engine, u64> {
 
 pub struct Engine {
     pub index:     usize,
-    program:   Vec<u8>,
-    registers: Registers,
-    config:    EngineConfig,
+    pub program:   Vec<u8>,
+    pub registers: Registers,
+    pub config:    EngineConfig,
 
     /// BUG: This DOES NOT account for overflowing
-    last_timer_count: u32,
-    timer_callback: Option<fn() -> u32>,
-    memory: memory::Memory,
+    pub last_timer_count: u32,
+    pub timer_callback: Option<fn() -> u32>,
+    pub memory: memory::Memory,
     pub enviroment_call_table: [EnviromentCall; 256],
-    call_stack: CallStack,
+    pub call_stack: CallStack,
 }
 use crate::engine::enviroment_calls::EnviromentCall;
 impl Engine {
@@ -319,8 +339,49 @@ F5-F9 {:016X} {:016X} {:016X} {:016X} {:016X}",
                 // TODO: Implement 64 bit register to register subtraction
                 (2, 4) => {
                     // 64 bit
-                    self.index += 19;
+
+                    let mut lhs_array = [0; 8];
+                    let mut rhs_array = [0; 8];
+
+                    for (index, byte) in self.program[self.index + 2..self.index + 10]
+                        .into_iter()
+                        .enumerate()
+                    {
+                        lhs_array[index] = *byte;
                     }
+                    let lhs = u64::from_be_bytes(lhs_array);
+
+                    for (index, byte) in self.program[self.index + 10..self.index + 18]
+                        .into_iter()
+                        .enumerate()
+                    {
+                        rhs_array[index] = *byte;
+                    }
+
+                    let rhs = u64::from_be_bytes(rhs_array);
+
+                    // println!("RHS 64BIT {}", rhs);
+
+                    let ret = lhs - rhs;
+
+                    let reg = self.program[self.index + 18];
+                    // println!("Store {} in {:02X}", ret, reg);
+
+                    match reg {
+                        0xA0..=0xC9 => {
+                            panic!("Register undersized")
+                        }
+                        0xD0..=0xF9 => {
+                            self.set_register_64(reg, ret);
+                        }
+                        _ => {
+                            panic!("Not a register.")
+                        }
+                    }
+
+                    self.index += 18;
+                }
+
                 // Read from address to register
                 (5, 0) => {
                     let mut addr_array = [0; 8];

hbvm/src/lib.rs

@@ -1,14 +1,10 @@
-// #![no_std]
+#![no_std]
 extern crate alloc;
 
 pub mod bytecode;
 pub mod engine;
 pub mod memory;
 
-use bytecode::ops::*;
-use bytecode::types::{CONST_F64, CONST_U8};
-use engine::Engine;
-
 #[derive(Debug)]
 pub enum RuntimeErrors {
     InvalidOpcode(u8),
@@ -22,10 +18,3 @@ pub enum HaltStatus {
     Halted,
     Running,
 }
-
-pub struct HandSide {
-    signed: bool,
-    float: bool,
-    num8: Option<u8>,
-    num64: Option<u64>,
-}

hbvm/src/main.rs

@@ -31,6 +31,7 @@ fn main() -> Result<(), RuntimeErrors> {
     eng.enviroment_call_table[10] = print_fn;
     eng.run()?;
     eng.dump();
+    println!("{:#?}", eng.registers);
 
     Ok(())
 }

hbvm/src/memory.rs

@@ -1,10 +1,13 @@
-use hashbrown::HashMap;
-use log::trace;
+use crate::engine::VMPage;
 
-use crate::{engine::Page, RuntimeErrors};
+use {
+    crate::{engine::Page, RuntimeErrors},
+    alloc::vec::Vec,
+    hashbrown::HashMap,
+    log::trace,
+};
 
 pub struct Memory {
-    //TODO: hashmap with the start bytes as key and end bytes as offset
     inner: HashMap<u64, Page>,
 }
 
@@ -13,9 +16,12 @@ impl Memory {
         Self {
             inner: HashMap::new(),
         }
+        //
     }
 
-    pub fn map_vec(&mut self, address: u64, vec: Vec<u8>) {}
+    pub fn map_vec(&mut self, address: u64, vec: Vec<u8>) {
+        panic!("Mapping vectors into pages is not supported yet");
+    }
 }
 
 impl Memory {
@@ -24,7 +30,7 @@ impl Memory {
         trace!("page {} offset {}", page, offset);
         match self.inner.get(&page) {
             Some(page) => {
-                let val = page.data[offset as usize];
+                let val = page.data()[offset as usize];
                 trace!("Value {}", val);
                 Ok(val)
             }
@@ -43,13 +49,13 @@ impl Memory {
         let ret: Option<(&u64, &mut Page)> = self.inner.get_key_value_mut(&page);
         match ret {
             Some((_, page)) => {
-                page.data[offset as usize] = value;
+                page.data()[offset as usize] = value;
             }
             None => {
-                let mut pg = Page::new();
+                let mut pg = VMPage::new();
                 pg.data[offset as usize] = value;
-                self.inner.insert(page, pg);
-                println!("Mapped page {}", page);
+                self.inner.insert(page, Page::VMPage(pg));
+                trace!("Mapped page {}", page);
             }
         }
         Ok(())