cvs commit: src/sys/dev/glxsb glxsb.c glxsb_hash.c

John Baldwin jhb at freebsd.org
Mon Aug 11 21:23:35 UTC 2008


On Monday 11 August 2008 04:41:08 am Pawel Jakub Dawidek wrote:
> pjd         2008-08-11 08:41:08 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     sys/dev/glxsb        glxsb.c glxsb_hash.c 
>   Log:
>   SVN rev 181593 on 2008-08-11 08:41:08Z by pjd
>   
>   - Convert sc_sessions_mtx mutex to a rwlock, so in the fast path
>     (glxsb_process()) we don't block others when looking for our session.
>   - Simplify the loop responsible for freeing sessions on detach.
>   - No need to drop a lock around malloc(M_NOWAIT).
>   - Treat ses_used as boolean.
>   - Avoid gotos where possible.
>   - Various style(9) fixes.
>   
>   Reviewed by:    philip, Patrick Lamaiziere <patfbsd at davenulle.org>

Would it be worthwhile to abstract the session handling out of padlock(4) and 
glxsb(4)?  Are there other CPUs with embedded crypto that will need their own 
drivers that having the abstraction would make writing a driver easier?

-- 
John Baldwin


More information about the cvs-all mailing list