sys_alarm系统调用初步实现 #698

smallcjy · 2024-04-06T08:44:13Z

实现思想及过程

Alarm定时器封装，创建alarm结构体，包含timer和timeout（停止时间），方法：new函数和timer方法的封装；
创建timerfunc函数，创建SYS_ALARM信号并在定时器触发时使用signal_send发送信号；
在pcb创建一个新的对象alarm，供存放旧alarm定时器的信息，类型为Arc<Mutex<Option>>
进程初始化时，把alarm初始化为包含None的锁的Arc指针
sys_alarm系统调用设计思想：首先获取当前进程的pcb的alarm定时器的引用，查看其Option值是否为None，若是运行alarm_init初始化，然后启动定时器；若不是，获取其截止时间和当前时间的差值（即剩余时间）并将其设置为None值（删除旧的定时器），初始化并启动新定时器。最后返回剩余时间
jiffies和秒数之间的转换暂时按照固定的形式转换
setitimer系统调用和alarm系统调用共用一个alarm定时器，setitimer尚未实现。
测试程序
设置三个测试用例，第一个创建一个5秒的alarm，查看是否接受到alarm发送的信号；第二个创建一个5秒的alarm，查看其返回值是否为上一个alarm的剩余时间，第三个测试在创建一个2秒的alarm后创建一个5秒的alarm，查看其返回值是否为上个闹钟的剩余时间并取消未执行的闹钟，查看新闹钟是否执行。

fslongjin

提交之前要先make fmt

fslongjin · 2024-04-06T09:30:47Z

kernel/src/process/mod.rs

@@ -693,6 +709,7 @@ impl ProcessControlBlock {
            children: RwLock::new(Vec::new()),
            wait_queue: WaitQueue::default(),
            thread: RwLock::new(ThreadInfo::new()),
+            alarm_timer: Arc::new(Mutex::new(None)),


为什么不是Option<Arc<AlarmTimer>>?
目前这种写法的封装性不好。这些处理是否可变的操作，应当封装到里面去。

感觉会不会存在多线程同时访问alarm产生数据竞争，Arc<Mutex<>>能支持内部可变

要把Mutex封装到AlarmTimer里面去。而不是暴露出来。

fslongjin · 2024-04-06T09:31:12Z

kernel/src/process/timer.rs

+}
+
+impl AlarmTimer {
+    pub fn new(timer_func: Box<dyn TimerFunction>, expire_time: u64) -> Mutex<Option<Self>>{


这个expire的是jiffies吗？如果是的话把变量名改为expired_jifffies

fslongjin · 2024-04-06T09:32:03Z

user/apps/alarm-test/README.md

@@ -0,0 +1,14 @@
+# DragonOS Rust-Application Template


fslongjin · 2024-04-06T09:33:02Z

user/apps/alarm-test/src/main.rs

+
+    //test1: alarm系统调用能否正常运行
+    unsafe {
+        syscall(37,5);


不要直接写系统调用号。
原因： #682 (comment)

fslongjin · 2024-04-06T14:30:12Z

#662

smallcjy · 2024-04-06T14:49:13Z

okok马上修改

chiichen

我在想这些持续时间都用Duration表示是不是方便一点，不用搞这些s, ms, ns的转换了。后面可以考虑移植一下 std 里的 instatnt 和duaration，结合起来就方便很多了，这种东西还是挺容易出错的

chiichen · 2024-04-07T16:12:24Z

kernel/src/time/syscall.rs

+                //这里的second是以jiddies为单位
+                //Todo：秒转换成jiddies


打错了？应该是jiffies

chiichen · 2024-04-07T16:12:28Z

kernel/src/process/timer.rs

+}
+
+//初始化目标进程的alarm定时器
+pub fn alarm_timer_init(pid: Pid, time_out: u64) {


同样的，这里如果是jiffies的话要体现出来，如果不是的话就写time_out_sec

chiichen · 2024-04-07T16:22:19Z

与主线代码有冲突，要解决一下。还有，就是开发不要直接用自己仓库的master开发，要新建一个分支

smallcjy · 2024-04-08T00:12:40Z

确实直接用秒在计算剩余秒数时可能会有误差

smallcjy and others added 12 commits March 31, 2024 22:06

alarm定时器封装

08ef4d5

alarm定时器封装bug修改

b749e30

进程闹钟初始化

267aab7

alarm系统调用实现

1c752d8

完整alarm系统调用实现第一版（未debug）

3bfa14c

Merge branch 'DragonOS-Community:master' into master

0b34ee5

fix重启alarm定时器没有重新插入定时器列表的问题

56d161a

alarm系统调用第二版,修改了一些bug

8617e99

修正错误的调用号

a3a2f2b

初步实现sys_alarm系统调用

f9d67e1

解决conflict

ecfa35b

Merge branch 'master' into master

aa5c514

fslongjin requested changes Apr 6, 2024

View reviewed changes

chiichen reviewed Apr 7, 2024

View reviewed changes

smallcjy closed this Apr 9, 2024

		//这里的second是以jiddies为单位
		//Todo：秒转换成jiddies

Uh oh!

sys_alarm系统调用初步实现 #698

sys_alarm系统调用初步实现 #698

Uh oh!

Conversation

smallcjy commented Apr 6, 2024

Uh oh!

fslongjin left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

fslongjin commented Apr 6, 2024

Uh oh!

smallcjy commented Apr 6, 2024

Uh oh!

chiichen left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

chiichen commented Apr 7, 2024

Uh oh!

smallcjy commented Apr 8, 2024

Uh oh!

Uh oh!