From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josef Bacik Subject: Re: Ceph on btrfs 3.4rc Date: Wed, 23 May 2012 10:12:31 -0400 Message-ID: <20120523141230.GA2127@localhost.localdomain> References: <20120517194358.GA9846@dhcp231-144.rdu.redhat.com> <4FB56B21.7070800@tuxadero.com> <20120518144759.GA1892@localhost.localdomain> <4FB685C9.2050607@tuxadero.com> <20120518190105.GB1892@localhost.localdomain> <4FB9BDBA.2030806@cn.fujitsu.com> <20120522173344.GB1890@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57275 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750915Ab2EWOMw (ORCPT ); Wed, 23 May 2012 10:12:52 -0400 Content-Disposition: inline In-Reply-To: Sender: ceph-devel-owner@vger.kernel.org List-ID: To: Christian Brunner Cc: Josef Bacik , miaox@cn.fujitsu.com, Martin Mailand , Sage Weil , linux-btrfs@vger.kernel.org, ceph-devel@vger.kernel.org On Wed, May 23, 2012 at 02:34:43PM +0200, Christian Brunner wrote: > 2012/5/22 Josef Bacik : > >> > > > > Yeah you would also need to change orphan_meta_reserved. =A0I fixed= this by just > > taking the BTRFS_I(inode)->lock when messing with these since we do= n't want to > > take up all that space in the inode just for a marker. =A0I ran thi= s patch for 3 > > hours with no issues, let me know if it works for you. =A0Thanks, >=20 > Compared to the last runs, I had to run it much longer, but somehow I > managed to hit a BUG_ON again: >=20 Yeah it's because we access other parts of that bitfield with no lock a= t all which is what is likely screwing us. I'm going to have to redo that pa= rt and then do the orphan fix, I'll have a patch shortly. Thanks, Josef -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html