Re: [PATCH 0/5] fix bugs of sub transid -- WARNING: at fs/btrfs/ctree.c:432

2011-12-02 Thread David Sterba
On Thu, Dec 01, 2011 at 09:32:19AM +0800, Liu Bo wrote: > The warning is aimed to check whether this ref_cow root is in this > transaction, so we can change it > to WARN_ON(root->ref_cows && trans->transaction->transid > root->last_trans); Ok, I'll add this change to next testing round. > BTW, d

Re: [PATCH 0/5] fix bugs of sub transid -- WARNING: at fs/btrfs/ctree.c:432

2011-11-30 Thread Liu Bo
On 11/30/2011 12:17 AM, David Sterba wrote: > On Tue, Nov 29, 2011 at 09:18:35AM +0800, Liu Bo wrote: >> a) For the first one (last_snapshot bug), >> >> The test involves three processes (derived from Chris): >> >> mkfs.btrfs /dev/xxx >> mount /dev/xxx /mnt >> >> 1) run compilebench -i 30 --makej -

Re: [PATCH 0/5] fix bugs of sub transid -- WARNING: at fs/btrfs/ctree.c:432

2011-11-29 Thread David Sterba
On Tue, Nov 29, 2011 at 09:18:35AM +0800, Liu Bo wrote: > a) For the first one (last_snapshot bug), > > The test involves three processes (derived from Chris): > > mkfs.btrfs /dev/xxx > mount /dev/xxx /mnt > > 1) run compilebench -i 30 --makej -D /mnt > > Let compilebench run until it starts th

Re: [PATCH 0/5] fix bugs of sub transid

2011-11-28 Thread Liu Bo
On 11/28/2011 11:10 PM, David Sterba wrote: > On Mon, Nov 21, 2011 at 06:10:19PM +0800, Liu Bo wrote: >> We've been sufferring two big bugs with sub transid: >> one is a bug related to root's last_snapshot, the other is a bug related to >> disk extent refs' generation. > > Do you have a testcase t

Re: [PATCH 0/5] fix bugs of sub transid

2011-11-28 Thread David Sterba
On Mon, Nov 21, 2011 at 06:10:19PM +0800, Liu Bo wrote: > We've been sufferring two big bugs with sub transid: > one is a bug related to root's last_snapshot, the other is a bug related to > disk extent refs' generation. Do you have a testcase to trigger and check these bugs? > 1) The first patch

[PATCH 0/5] fix bugs of sub transid

2011-11-21 Thread Liu Bo
NOTE: This patchset is based on "danger" branch. We've been sufferring two big bugs with sub transid: one is a bug related to root's last_snapshot, the other is a bug related to disk extent refs' generation. 1) The first patch fixes a warning, 2) the second one fixes the last_snapshot bug, 3) the