On 15.05.2018 20:52, Liu Bo wrote: > @tmp must be NULL at this point, free_extent_buffer is not needed at all.
You need to explain that this code is executed only if find_extent_buffer didn't return anything, meaning tmp is null, otherwise it's hard to review this change without looking at the code. Codewise it's correct: Reviewed-by: Nikolay Borisov <nbori...@suse.com> > > Signed-off-by: Liu Bo <bo....@linux.alibaba.com> > --- > fs/btrfs/ctree.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/fs/btrfs/ctree.c b/fs/btrfs/ctree.c > index b3f6f300e492..9fa3d77c98d4 100644 > --- a/fs/btrfs/ctree.c > +++ b/fs/btrfs/ctree.c > @@ -2432,7 +2432,6 @@ noinline void btrfs_unlock_up_safe(struct btrfs_path > *path, int level) > btrfs_unlock_up_safe(p, level + 1); > btrfs_set_path_blocking(p); > > - free_extent_buffer(tmp); > if (p->reada != READA_NONE) > reada_for_search(fs_info, p, level, slot, key->objectid); > > -- 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