cvs commit: src/sys/vfs/hammer hammer_btree.c hammer_cursor.c
Matthew Dillon
dillon at crater.dragonflybsd.org
Fri Mar 21 19:09:23 PDT 2008
dillon 2008/03/21 19:06:55 PDT
DragonFly src repository
Modified files:
sys/vfs/hammer hammer_btree.c hammer_cursor.c
Log:
HAMMER 34/many: Stabilization pass
* Fix a node reference count leak in hammer_btree_lock_children(). This
was resulting in a panic at umount time.
* Fix a misordered lock in hammer_cursor_up() which could result in
an assertion in the B-Tree iteration code. The cursor-up code was
resolving the parent index prior to acquiring a shared lock on the
parent node. An insertion by another thread could do a split and
change the parent index. The shared lock must be acquired first.
* Add additional debugging output and add additional assertions. HAMMER
now has 205 assert lines in the code.
Reported-by: YONETANI Tomokazu <qhwt+dfly at les.ath.cx> (umount panic)
Revision Changes Path
1.35 +29 -4 src/sys/vfs/hammer/hammer_btree.c
1.20 +6 -4 src/sys/vfs/hammer/hammer_cursor.c
http://www.dragonflybsd.org/cvsweb/src/sys/vfs/hammer/hammer_btree.c.diff?r1=1.34&r2=1.35&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/vfs/hammer/hammer_cursor.c.diff?r1=1.19&r2=1.20&f=u
More information about the Commits
mailing list