oops, please ignore this as there is already a same patch from Stefan. Sorry for the trouble.
thanks, liubo On Fri, May 17, 2013 at 04:45:44PM +0800, Liu Bo wrote: > We need to unlock and free extent buffer before the return. > > Signed-off-by: Liu Bo <bo.li....@oracle.com> > --- > fs/btrfs/relocation.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c > index 704a1b8..5c5b8bb 100644 > --- a/fs/btrfs/relocation.c > +++ b/fs/btrfs/relocation.c > @@ -1773,7 +1773,7 @@ again: > if (!eb || !extent_buffer_uptodate(eb)) { > ret = (!eb) ? -ENOMEM : -EIO; > free_extent_buffer(eb); > - return ret; > + break; > } > btrfs_tree_lock(eb); > if (cow) { > -- > 1.7.7.6 > > -- > 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 -- 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