Re: CVS commit: src/sys/kern

[christos%zoulas.com@localhost (Christos Zoulas)]

On Apr 1,  6:20pm, martin%duskware.de@localhost (Martin Husemann) wrote:
-- Subject: Re: CVS commit: src/sys/kern

| On Mon, Apr 01, 2013 at 11:36:57AM -0400, Christos Zoulas wrote:
| > Probably cleaner to check for it there?
| 
| Maybe, but I wasn't sure about the other callers. There is similar
| code in itimerfix() and changing it there symetrically made a lot
| of things fail imediately, so I went for the simple fix.
| 
| > Anyway, the question is really if
| > we should bother to sleep at all for 0, 0 or shortcut the return. I think
| > we should shortcut the return.
| 
| Shortcut is what I did, or do you mean something else?

I meant fix it in itim{er,spec}fix. Yes, I will move the check out again,
because 0 means disable the timer.


christos