On Tue, Aug 14, 2018 at 10:47:09AM +0800, Liu Bo wrote:

The btrfs_release_path() is just useless as path is only used in error handling.

Where is it duplicated? And I don't think it's useless, while the
changelog does not explain why and it's not obvious from the context. If
the path is locked, then releasing it right after it's not needed makes
sense.  There are several potentially heavyweight operations between the
release and final free.