From 957e767dfc30acff5570af8b78ebe2cbbdd9ac02 Mon Sep 17 00:00:00 2001
From: Julien CLEMENT <julien.clement@epita.fr>
Date: Sat, 7 Jan 2023 16:06:05 +0100
Subject: [PATCH] add todos

Signed-off-by: Julien CLEMENT <julien.clement@epita.fr>
---
 src/interrupts/pic/pit.rs | 1 +
 src/proc/scheduler/mod.rs | 2 ++
 src/utils/mutex.rs        | 1 +
 3 files changed, 4 insertions(+)

diff --git a/src/interrupts/pic/pit.rs b/src/interrupts/pic/pit.rs
index 23a1cc1..50acdd8 100644
--- a/src/interrupts/pic/pit.rs
+++ b/src/interrupts/pic/pit.rs
@@ -9,6 +9,7 @@ pub fn gettick() -> u64 {
 }
 
 pub extern "x86-interrupt" fn timer_interrupt_handler(_stack_frame: InterruptStackFrame) {
+    // TODO: thread preemption
     unsafe {
         TICKS += 1;
         PICS.lock()
diff --git a/src/proc/scheduler/mod.rs b/src/proc/scheduler/mod.rs
index 7d09de8..a5eef06 100644
--- a/src/proc/scheduler/mod.rs
+++ b/src/proc/scheduler/mod.rs
@@ -15,6 +15,7 @@ use futures_util::task::AtomicWaker;
 use lazy_static::lazy_static;
 
 lazy_static! {
+    // TODO: put this in a spin mutex instead
     pub static ref SCHEDULER: AsyncMutex<Scheduler> = AsyncMutex::new(Scheduler::new());
 }
 
@@ -23,6 +24,7 @@ pub type Threadt = Arc<RefCell<Thread>>;
 pub const K_THREAD_ID: ThreadId = ThreadId(0); // Kernel main thread identifier
 
 struct ThreadStream {
+    // TODO: add the double scheduler queue
     ids: ArrayQueue<ThreadId>,
     waker: AtomicWaker,
 }
diff --git a/src/utils/mutex.rs b/src/utils/mutex.rs
index 70ffb13..24f4bef 100644
--- a/src/utils/mutex.rs
+++ b/src/utils/mutex.rs
@@ -11,6 +11,7 @@ use futures_util::task::AtomicWaker;
 #[derive(Clone)]
 struct Lock {
     lock: Arc<AtomicBool>,
+    // TODO: Make a queue of wakers
     waker: Arc<AtomicWaker>,
 }