From 7b5e3e1239a7740e3f407a2d8936482aed2f99ac Mon Sep 17 00:00:00 2001 From: John Baldwin Date: Tue, 22 Apr 2003 19:45:26 +0000 Subject: [PATCH] Keep the proc locked while we set PS_INMEM so that either of the proc lock or sched_lock are sufficient to test this flag. XXX: vinum should really be using a kernel process via kthread_create() instead of this hack. I'm not even sure PS_INMEM can be clear at this point anyways. --- sys/dev/vinum/vinumdaemon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/dev/vinum/vinumdaemon.c b/sys/dev/vinum/vinumdaemon.c index a98977eb2c34..3ae09c0d43f0 100644 --- a/sys/dev/vinum/vinumdaemon.c +++ b/sys/dev/vinum/vinumdaemon.c @@ -71,10 +71,10 @@ vinum_daemon(void) PROC_LOCK(curproc); curproc->p_flag |= P_SYSTEM; /* we're a system process */ - PROC_UNLOCK(curproc); mtx_lock_spin(&sched_lock); curproc->p_sflag |= PS_INMEM; mtx_unlock_spin(&sched_lock); + PROC_UNLOCK(curproc); daemon_save_config(); /* start by saving the configuration */ daemonpid = curproc->p_pid; /* mark our territory */ while (1) {