Hi Chris,
On Thu, 20 Jun 2013, Chris Mason wrote:
> Quoting Sage Weil (2013-06-20 17:56:19)
> > On Wed, 19 Jun 2013, Sage Weil wrote:
> > > Hi Chris,
> > >
> > > On Tue, 18 Jun 2013, Chris Mason wrote:
> > > > [...]
> > > > Very long way of saying I think we're one release_path short. Sage, I
> > > > haven't tested this at all yet, I was hoping to trigger it first.
> > > >
> > > > diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c
> > > > index c276ac9..c1954b3 100644
> > > > --- a/fs/btrfs/tree-log.c
> > > > +++ b/fs/btrfs/tree-log.c
> > > > @@ -3730,6 +3730,7 @@ next_slot:
> > > > log_extents:
> > > > if (fast_search) {
> > > > btrfs_release_path(dst_path);
> > > > + btrfs_release_path(path);
> > > > ret = btrfs_log_changed_extents(trans, root, inode,
> > > > dst_path);
> > > > if (ret) {
> > > > err = ret;
> > >
> > > This seems to be doing the trick. I'll keep testing overnight, but so
> > > far
> > > so good!
> >
> > ...and it's still holding up well in QA.
>
> Awesome, thanks for getting the traces for us. Looks like this one has
> been around since v3.7, so I'm not going to try and sneak it into the
> 3.10 final. I'll have it in the next merge window and for stable.
I was just pulling in the current -rc for testing and realized that this
isn't upstream yet. Was this missed? Sorry I didn't check earlier.
Thanks!
sage
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
