From 5f130d43e1dece0fb35e6007256cb55b25550194 Mon Sep 17 00:00:00 2001 From: fekhesk Date: Thu, 27 Oct 2022 10:29:58 -0700 Subject: [PATCH] put memory mapper and frame allocator into mutex --- src/internals/cpu.rs | 32 ++++++++++++++++---------------- src/lib.rs | 13 ++++++------- src/memory/mod.rs | 26 ++++++++++++++++---------- src/serial/mod.rs | 11 +++++++++++ 4 files changed, 49 insertions(+), 33 deletions(-) diff --git a/src/internals/cpu.rs b/src/internals/cpu.rs index 8fdd444..d6eb27a 100644 --- a/src/internals/cpu.rs +++ b/src/internals/cpu.rs @@ -47,23 +47,23 @@ pub fn get_apic_base() -> usize { ((edx as usize) << 32) | (eax as usize) } -pub fn enable_apic(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootInfoFrameAllocator) { +pub fn enable_apic() { // PIC should be disabled by now // now enable local apic // 1. set bit 8 of spurious interrupt vector register let sivr_addr = 0xfee000f0; - let sivr = read_phys_memory32(mem_mapper, frame_alloc, sivr_addr); - write_phys_memory32(mem_mapper, frame_alloc, sivr_addr, sivr | (1 << 8)); + let sivr = read_phys_memory32(sivr_addr); + write_phys_memory32(sivr_addr, sivr | (1 << 8)); } -pub fn apic_read_io(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootInfoFrameAllocator, ioapicaddr: usize, reg: u32) -> u32 { - write_phys_memory32(mem_mapper, frame_alloc, ioapicaddr as u32, reg); - read_phys_memory32(mem_mapper, frame_alloc, ioapicaddr as u32 + 0x10) +pub fn apic_read_io(ioapicaddr: usize, reg: u32) -> u32 { + write_phys_memory32(ioapicaddr as u32, reg); + read_phys_memory32(ioapicaddr as u32 + 0x10) } -pub fn apic_write_io(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootInfoFrameAllocator, ioapicaddr: usize, reg: u32, val: u32) { - write_phys_memory32(mem_mapper, frame_alloc, ioapicaddr as u32, reg); - write_phys_memory32(mem_mapper, frame_alloc, ioapicaddr as u32 + 0x10, val); +pub fn apic_write_io(ioapicaddr: usize, reg: u32, val: u32) { + write_phys_memory32(ioapicaddr as u32, reg); + write_phys_memory32(ioapicaddr as u32 + 0x10, val); } pub fn disable_pic() { @@ -83,17 +83,17 @@ pub fn disable_pic() { command(0xa1, 0xff); } -pub fn ioapic_set_irq(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootInfoFrameAllocator, ioapicaddr: usize, irq: u8, apic_id: u64, vector:u8) { +pub fn ioapic_set_irq(ioapicaddr: usize, irq: u8, apic_id: u64, vector:u8) { let lo_index: u32 = (0x10 + irq*2 ) as u32; let hi_index: u32 = (0x10 + irq*2 + 1) as u32; - let mut high = apic_read_io(mem_mapper, frame_alloc, ioapicaddr, hi_index); + let mut high = apic_read_io(ioapicaddr, hi_index); // set apic id high &= !(0xff000000); high |= (apic_id as u32) << 24; - apic_write_io(mem_mapper, frame_alloc, ioapicaddr, hi_index, high); + apic_write_io(ioapicaddr, hi_index, high); - let mut low = apic_read_io(mem_mapper, frame_alloc, ioapicaddr, lo_index); + let mut low = apic_read_io( ioapicaddr, lo_index); // unmask low &= !(1 << 16); @@ -105,7 +105,7 @@ pub fn ioapic_set_irq(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootIn low &= !(0xff); low |= vector as u32; - apic_write_io(mem_mapper, frame_alloc, ioapicaddr, lo_index, low); + apic_write_io(ioapicaddr, lo_index, low); } pub fn apic_eoi() { @@ -136,7 +136,7 @@ pub extern "x86-interrupt" fn keyboard_irq(stack_frame: InterruptStackFrame) { } // todo! we should abstract this away -pub fn setup_apic_interrupts(mem_mapper: &mut OffsetPageTable, frame_alloc: &mut BootInfoFrameAllocator, ioapicaddr: usize) { +pub fn setup_apic_interrupts(ioapicaddr: usize) { // set keyboard irq to interrupt 40 - ioapic_set_irq(mem_mapper, frame_alloc, ioapicaddr, 1, 0, 40); + ioapic_set_irq(ioapicaddr, 1, 0, 40); } \ No newline at end of file diff --git a/src/lib.rs b/src/lib.rs index 35409fe..52c66c6 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -25,6 +25,7 @@ use x86_64::registers::segmentation::{CS, Segment, SS}; use x86_64::structures::paging::Translate; use crate::boot::KernelInfo; use crate::internals::WhyDoTheyCallItOvenWhenYouOfInTheColdFoodOfOutHotEatTheFood::*; +use crate::memory::{FRAME_ALLOC, MEM_MAPPER}; use crate::serial::terminal::ST; mod font; @@ -165,17 +166,15 @@ pub extern fn kernel_main(args: KernelArgs) -> ! { let kern_info = Mutex::new(KernelInfo::init_from_kernel_args(args)); // memory stuff - let mut mapper = None; - let mut frame_allocator = None; { print!("initialising mapper..."); - mapper = Some(unsafe { memory::init(VirtAddr::new(0)) }); + MEM_MAPPER.lock().replace(unsafe { memory::init(VirtAddr::new(0)) }); println!("[OK]"); print!("initialising frame allocator..."); - frame_allocator = Some(unsafe { memory::BootInfoFrameAllocator::init(kern_info) }); + FRAME_ALLOC.lock().replace(unsafe { memory::BootInfoFrameAllocator::init(kern_info) }); println!("[OK]"); print!("initialising heap..."); - memory::allocator::init_heap(mapper.as_mut().unwrap(), frame_allocator.as_mut().unwrap()).expect("heap init failed"); + memory::allocator::init_heap(MEM_MAPPER.lock().as_mut().unwrap(), FRAME_ALLOC.lock().as_mut().unwrap()).expect("heap init failed"); println!("[OK]"); print!("testing heap..."); @@ -206,10 +205,10 @@ pub extern fn kernel_main(args: KernelArgs) -> ! { unsafe { internals::cpu::disable_pic() }; println!("[OK]"); print!("initialising apic..."); - unsafe { internals::cpu::enable_apic(mapper.as_mut().unwrap(), frame_allocator.as_mut().unwrap()) }; + unsafe { internals::cpu::enable_apic() }; println!("[OK]"); print!("setting up apic interrupts..."); - unsafe { internals::cpu::setup_apic_interrupts(mapper.as_mut().unwrap(), frame_allocator.as_mut().unwrap()) }; + unsafe { internals::cpu::setup_apic_interrupts() }; println!("[OK]"); // enable interrupts x86_64::instructions::interrupts::enable(); diff --git a/src/memory/mod.rs b/src/memory/mod.rs index 9b2a305..1755dec 100644 --- a/src/memory/mod.rs +++ b/src/memory/mod.rs @@ -2,9 +2,15 @@ pub mod allocator; use alloc::boxed::Box; use alloc::sync::Arc; +use lazy_static::lazy_static; use x86_64::structures::paging::{FrameAllocator, Mapper, OffsetPageTable, PageTable, PhysFrame, Size4KiB, Translate}; use x86_64::{PhysAddr, VirtAddr}; +lazy_static!{ + pub static ref MEM_MAPPER: Mutex> = Mutex::new(None); + pub static ref FRAME_ALLOC: Mutex> = Mutex::new(None); +} + pub struct Locked { inner: spin::Mutex, } @@ -86,26 +92,26 @@ unsafe impl FrameAllocator for BootInfoFrameAllocator { } } -pub fn read_phys_memory32(mem_mapper: &mut OffsetPageTable, frame_allocator: &mut BootInfoFrameAllocator, addr: u32) -> u32 { +pub fn read_phys_memory32(addr: u32) -> u32 { let initaladdr = VirtAddr::new(addr as u64); - let addr = unsafe { mem_mapper.translate_addr(initaladdr) }; + let addr = unsafe { MEM_MAPPER.lock().as_mut().unwrap().translate_addr(initaladdr) }; if let Some(addr) = addr { let addr = addr.as_u64() as *const u32; unsafe { *addr } } else { debug!("read_phys_memory32: addr {:x} not mapped", initaladdr.as_u64()); // map the page - let frame = frame_allocator.allocate_frame().unwrap(); + let frame = FRAME_ALLOC.lock().as_mut().unwrap().allocate_frame().unwrap(); debug!("allocated frame: {:?}", frame); let flags = x86_64::structures::paging::PageTableFlags::PRESENT | x86_64::structures::paging::PageTableFlags::WRITABLE; let page = x86_64::structures::paging::Page::containing_address(initaladdr); debug!("mapped page: {:?}", page); - let map_to_result = unsafe { mem_mapper.map_to(page, frame, flags, frame_allocator) }; + let map_to_result = unsafe { MEM_MAPPER.lock().as_mut().unwrap().map_to(page, frame, flags, FRAME_ALLOC.lock().as_mut().unwrap()) }; debug!("map_to_result: {:?}", map_to_result); if map_to_result.is_err() { panic!("Failed to map page"); } - let addr = unsafe { mem_mapper.translate_addr(initaladdr) }; + let addr = unsafe { MEM_MAPPER.lock().as_mut().unwrap().translate_addr(initaladdr) }; if let Some(addr) = addr { let addr = addr.as_u64() as *const u32; unsafe { *addr } @@ -115,26 +121,26 @@ pub fn read_phys_memory32(mem_mapper: &mut OffsetPageTable, frame_allocator: &mu } } -pub fn write_phys_memory32(mem_mapper: &mut OffsetPageTable, frame_allocator: &mut BootInfoFrameAllocator, addr: u32, value: u32) { +pub fn write_phys_memory32(addr: u32, value: u32) { let initaladdr = VirtAddr::new(addr as u64); - let addr = unsafe { mem_mapper.translate_addr(initaladdr) }; + let addr = unsafe { MEM_MAPPER.lock().as_mut().unwrap().translate_addr(initaladdr) }; if let Some(addr) = addr { let addr = addr.as_u64() as *mut u32; unsafe { *addr = value }; } else { debug!("write_phys_memory32: addr {:x} not mapped", initaladdr.as_u64()); // map the page - let frame = frame_allocator.allocate_frame().unwrap(); + let frame = FRAME_ALLOC.lock().as_mut().unwrap().allocate_frame().unwrap(); debug!("allocated frame: {:?}", frame); let flags = x86_64::structures::paging::PageTableFlags::PRESENT | x86_64::structures::paging::PageTableFlags::WRITABLE; let page = x86_64::structures::paging::Page::containing_address(initaladdr); debug!("mapped page: {:?}", page); - let map_to_result = unsafe { mem_mapper.map_to(page, frame, flags, frame_allocator) }; + let map_to_result = unsafe { MEM_MAPPER.lock().as_mut().unwrap().map_to(page, frame, flags, FRAME_ALLOC.lock().as_mut().unwrap()) }; debug!("map_to_result: {:?}", map_to_result); if map_to_result.is_err() { panic!("Failed to map page"); } - let addr = unsafe { mem_mapper.translate_addr(initaladdr) }; + let addr = unsafe { MEM_MAPPER.lock().as_mut().unwrap().translate_addr(initaladdr) }; if let Some(addr) = addr { let addr = addr.as_u64() as *mut u32; unsafe { *addr = value }; diff --git a/src/serial/mod.rs b/src/serial/mod.rs index a2628a4..ab39655 100644 --- a/src/serial/mod.rs +++ b/src/serial/mod.rs @@ -74,6 +74,17 @@ pub fn read(port: u16) -> u8 { data } +// dummy functions for non-x86 +#[cfg(not(any(target_arch="x86", target_arch="x86_64")))] +pub fn command(port: u16, data: u8) { + unimplemented!() +} + +#[cfg(not(any(target_arch="x86", target_arch="x86_64")))] +pub fn read(port: u16) -> u8 { + unimplemented!() +} + impl Port { fn is_transmit_empty(&self) -> bool { let status = read(self.base as u16 + serial_offsets::LINE_STATUS as u16);