细节调整 chyyuu#19

os/src/interrupt/handler.rs

@@ -54,8 +54,9 @@ fn supervisor_timer(context: &mut Context) -> *mut Context {
 }
 
 /// 出现未能解决的异常，终止当前线程
-fn fault(_context: &mut Context, scause: Scause, _stval: usize) -> ! {
+fn fault(_context: &mut Context, scause: Scause, _stval: usize) -> *mut Context {
     println!("{:?} terminated with {:?}", PROCESSOR.get().current_thread(), scause.cause());
     PROCESSOR.get().kill_current_thread();
-    PROCESSOR.get().run();
+    // 跳转到 PROCESSOR 调度的下一个线程
+    PROCESSOR.get().current_thread().run()
 }

os/src/memory/mod.rs

@@ -6,12 +6,12 @@
 // 所以在模块范围内不提示『未使用的函数』等警告
 #![allow(dead_code)]
 
-pub mod address;
-pub mod config;
-pub mod frame;
-pub mod heap;
-pub mod mapping;
-pub mod range;
+mod address;
+mod config;
+mod frame;
+mod heap;
+mod mapping;
+mod range;
 
 /// 一个缩写，模块中一些函数会使用
 pub type MemoryResult<T> = Result<T, &'static str>;

os/src/process/mod.rs

@@ -6,10 +6,10 @@ mod processor;
 mod stack;
 mod thread;
 
-pub(self) use crate::interrupt::*;
-pub(self) use crate::memory::*;
-pub(self) use alloc::{sync::Arc, vec, vec::Vec};
-pub(self) use spin::{Mutex, RwLock};
+use crate::interrupt::*;
+use crate::memory::*;
+use alloc::sync::Arc;
+use spin::{Mutex, RwLock};
 
 pub use config::*;
 pub use process::Process;

os/src/process/processor.rs

@@ -14,7 +14,7 @@ lazy_static! {
 pub struct Processor {
     /// 当前正在执行的线程
     current_thread: Option<Arc<Thread>>,
-    /// 线程调度器，其中不包括正在执行的线程
+    /// 线程调度器，记录所有线程
     scheduler: SchedulerImpl<Arc<Thread>>,
 }
 
@@ -38,7 +38,7 @@ impl Processor {
         // 从 current_thread 中取出 Context
         let thread = self.current_thread();
         let context = thread.run();
-        // 因为这个线程（指的不是 thread，是运行 run 函数的线程）不会回来回收，所以手动 drop 掉 thread 的一个 Arc
+        // 这个函数不会执行到结尾，所以手动 drop 掉 thread 的一个 Arc
         drop(thread);
         // 从此将没有回头
         unsafe {
@@ -64,7 +64,7 @@ impl Processor {
                 next_context
             }
         } else {
-            panic!("no more thread to run");
+            panic!("all threads terminated, shutting down");
         }
     }
 
@@ -87,7 +87,7 @@ impl Processor {
         if let Some(next_thread) = self.scheduler.get_next() {
             self.current_thread.replace(next_thread);
         } else {
-            panic!("no more thread to run");
+            panic!("all threads terminated, shutting down");
         }
     }
 }