timeout/callout small step forward
    John Baldwin 
    jhb at freebsd.org
       
    Wed Apr  2 11:17:46 PDT 2008
    
    
  
On Saturday 29 March 2008 03:04:17 am Jeff Roberson wrote:
> http://people.freebsd.org/~jeff/callout.diff
>
> This patch takes the current callout implementation and makes it per-cpu.
> It also hides callout details from the rest of the kernel by making the
> callwheel structure private to kern_timeout.c among other things.
Looks good.  The kern_intr.c diff has a small bug (forgot to remove the return 
(intr_event_create(...)) from swi_add()).  A few style suggestions would be 
to always leave a blank line before a comment (I think I saw this in 
kern_calloutwheel_init()?) and usually there isn't a blank line before a 
SYSINIT().  Maybe make the panic messages when creating softclock threads 
more specific, but that's very minor.
-- 
John Baldwin
    
    
More information about the freebsd-arch
mailing list