Date: Sat, 25 Jan 2003 23:02:24 -0500 (EST) From: Jeff Roberson <jroberson@chesapeake.net> To: Julian Elischer <julian@elischer.org> Cc: Matthew Dillon <dillon@apollo.backplane.com>, Steve Kargl <sgk@troutmask.apl.washington.edu>, Robert Watson <rwatson@FreeBSD.ORG>, Gary Jennejohn <garyj@jennejohn.org>, <arch@FreeBSD.ORG> Subject: Re: New scheduler (#3) Message-ID: <20030125230123.W7994-100000@mail.chesapeake.net> In-Reply-To: <Pine.BSF.4.21.0301251920520.89171-100000@InterJet.elischer.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> > I want the sleep time to ONLY reflect voluntary sleep time. Thats why it > > was only done in sched_sleep(). It looks like I might have to add it to > > either select or the condvar code. I think that perhaps adding it to > > condvar would be correct since that is a voluntary sleep. > > > > Good catch. thanks. > > Select uses cv_wait() > > Both jhb and I have been considering passing a parameter to mi_switch() > that indicates if the switch was voluntary or not.. > in the meanwhile you can do: > if (TD_IS_ON_SLEEPQ(td)) > in mi_switch (sched_switchout()) > to catch both cases.. or just call sched_sleep() from the cv code > as it's almost identical to the sleep code.. I added sched_sleep() to the cv code since it was doing the old way manually. Now it is correct for both schedulers. I do like the idea of vol/invol parameter to mi_switch(). Cheers, Jeff To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030125230123.W7994-100000>