Hi Amon, On Wed, 30 Nov 2011, Sage Weil wrote: > On Wed, 30 Nov 2011, Amon Ott wrote: > > Hi! > > > > With some kernel debug options for soft and hard lockup detection, I got > > some > > fine traces. My kernel is a 3.1.4 to which I have ported from ceph-client > > for-linus branch what is suitable for 3.1. If needed, I can make my exact > > ceph code available. > > > > Traces are attached. It seems that two depending locks can be acquired in > > different order at different parts of the code, and thus lead to a > > deadlock. > > Additionally, I am still trying to reproduce a partial lockup of single > > dirs > > with this debugging. Those are likely to be related to mutex locking dirs > > without unlocking properly. > > Thanks, put these in the tracker at > http://tracker.newdream.net/issues/1762.
I pushed a wip-i-ceph-lock branch to ceph-client.git that replaces our (ab?)use of i_lock with a new i_ceph_lock in the ceph inode. This avoids being bitten by the lock ordering constraint imposed by igrab(), which requires i_lock to safely take a reference to an inode without racing with inode destruction. This lets us keep two inode list locks logically ordered inside i_ceph_lock (with i_lock as an inner lock). I did some very basic testing and it didn't blow up. If can give it a try, that would be very helpful. Also, we need to enable lockdep in our qa environment and make sure teuthology is erroring out on lockdep warnings. (#1763) Thanks! sage -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
