git: kernel -- ffs: Replace softupdates critical section locks with lockmgr.

Venkatesh Srinivas vsrinivas at crater.dragonflybsd.org
Thu Mar 29 02:28:19 PDT 2012


commit 8e90f899fdf61479c5e76faa87e7ff716982ed08
Author: Venkatesh Srinivas <me at endeavour.zapto.org>
Date:   Wed Mar 28 11:09:15 2012 -0700

    kernel -- ffs: Replace softupdates critical section locks  with lockmgr.
    
    ffs softupdates was using a combination of critical sections and the mplock
    to construct its acquire_lock/free_lock/interlocked_sleep primitives.
    
    The softupdates I/O completion callback had a few points where it could block,
    causing it to lose the mplock and any critical sections it held. When it did
    so, front-end softupdates code would try to grab the lock but panic on seeing
    the I/O completion callback in progress. This was not a problem in older
    systems, as splx() would prevent the I/O callback from starting while
    softdep code was executing.
    
    This patch converts to using hard locks (lockmgr), which are held even while
    a thread is blocked.
    
    getdirtybuf(), locking a dirty buffer via BUF_LOCK, was changed to not
    deadlock against the I/O completion path; specifically it drops the softdep
    lock when its non-blocking attempt to lock a dirty buffer fails. One caller
    of getdirtybuf() also required changes to retry locking buffers.
    
    Closes-bug: 2291
    Partially-from: FreeBSD r140709
    Reported-by: tuxillo, marino, Rumko, vsrinivas
    Discussed-with: dillon

Summary of changes:
 sys/vfs/ufs/ffs_softdep.c |  356 +++++++++++++++------------------------------
 1 files changed, 117 insertions(+), 239 deletions(-)

http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/8e90f899fdf61479c5e76faa87e7ff716982ed08


-- 
DragonFly BSD source repository





More information about the Commits mailing list