From fd638d53882cbc12d6f545e7dcccea7f6da1e0da Mon Sep 17 00:00:00 2001 From: Yifan Wu Date: Sun, 3 Jan 2021 17:38:37 +0800 Subject: [PATCH] Remove meaningless sstatus::set_sie() when initializing. --- os/src/trap/context.rs | 2 -- os/src/trap/mod.rs | 6 ------ 2 files changed, 8 deletions(-) diff --git a/os/src/trap/context.rs b/os/src/trap/context.rs index 665f5fb1..8c5175f7 100644 --- a/os/src/trap/context.rs +++ b/os/src/trap/context.rs @@ -22,8 +22,6 @@ impl TrapContext { let mut sstatus = sstatus::read(); // set CPU privilege to User after trapping back sstatus.set_spp(SPP::User); - // enable Supervisor mode interrupt after trapping back - sstatus.set_spie(true); let mut cx = Self { x: [0; 32], sstatus, diff --git a/os/src/trap/mod.rs b/os/src/trap/mod.rs index 58881b1b..4f5126b2 100644 --- a/os/src/trap/mod.rs +++ b/os/src/trap/mod.rs @@ -10,7 +10,6 @@ use riscv::register::{ Interrupt, }, stval, - sstatus, sie, }; use crate::syscall::syscall; @@ -41,11 +40,6 @@ fn set_user_trap_entry() { } } -#[allow(unused)] -pub fn enable_interrupt() { - unsafe { sstatus::set_sie(); } -} - pub fn enable_timer_interrupt() { unsafe { sie::set_stimer(); } }