Preempt a running proc even if there is no other process/thread queued
authorclaudio <claudio@openbsd.org>
Wed, 30 Aug 2023 09:02:38 +0000 (09:02 +0000)
committerclaudio <claudio@openbsd.org>
Wed, 30 Aug 2023 09:02:38 +0000 (09:02 +0000)
on that CPU's runqueue. This way mi_switch() is invoked which is necessary
to a) signal srm that the cpu changed context b) runtime stats are updated
c) requests to stop the CPU are checked.
This should fix the issue reported by Eric Wong (e at 80x24 org) that
RLIMIT_CPU is unreliable on idle systems.
OK kettenis@ cheloha@

sys/kern/sched_bsd.c

index 465624e..2f9ef56 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sched_bsd.c,v 1.84 2023/08/29 16:19:34 claudio Exp $  */
+/*     $OpenBSD: sched_bsd.c,v 1.85 2023/08/30 09:02:38 claudio Exp $  */
 /*     $NetBSD: kern_synch.c,v 1.37 1996/04/22 01:38:37 christos Exp $ */
 
 /*-
@@ -106,7 +106,7 @@ roundrobin(struct clockintr *cl, void *cf)
                }
        }
 
-       if (spc->spc_nrun)
+       if (spc->spc_nrun || spc->spc_schedflags & SPCF_SHOULDYIELD)
                need_resched(ci);
 }