Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 23 Apr 2003 07:43:58 +1000 (EST)
From:      Bruce Evans <bde@zeta.org.au>
To:        John Baldwin <jhb@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/vinum vinumdaemon.c
Message-ID:  <20030423073405.E19214@gamplex.bde.org>
In-Reply-To: <200304221945.h3MJjQSg024837@repoman.freebsd.org>
References:  <200304221945.h3MJjQSg024837@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 22 Apr 2003, John Baldwin wrote:

> jhb         2003/04/22 12:45:26 PDT
>
>   FreeBSD src repository
>
>   Modified files:
>     sys/dev/vinum        vinumdaemon.c
>   Log:
>   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.

I think PS_INMEM is always set for curproc, since curproc couldn't run
without a stack.  vinumdaemon also hacks on P_SYSTEM, which should keep
PS_INMEM from changing later.

Bruce



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030423073405.E19214>