diff options
| author | Filipe Manana <fdmanana@gmail.com> | 2014-03-31 14:52:14 +0100 |
|---|---|---|
| committer | Chris Mason <clm@fb.com> | 2014-04-07 09:08:46 -0700 |
| commit | c715e155c94ba0b3657820d676ec3c7213a5ce81 (patch) | |
| tree | 2d90c2a1db76bbaeb6074e4ac9f9c2818fd5e136 /fs | |
| parent | 9a40f1222a372de77344d85d31f8fe0e1c0e60e7 (diff) | |
| download | op-kernel-dev-c715e155c94ba0b3657820d676ec3c7213a5ce81.zip op-kernel-dev-c715e155c94ba0b3657820d676ec3c7213a5ce81.tar.gz | |
Btrfs: send, build path string only once in send_hole
There's no point building the path string in each iteration of the
send_hole loop, as it produces always the same string.
Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
Signed-off-by: Chris Mason <clm@fb.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/btrfs/send.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index e8e9f35..1ac3ca9 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -4436,6 +4436,9 @@ static int send_hole(struct send_ctx *sctx, u64 end) p = fs_path_alloc(); if (!p) return -ENOMEM; + ret = get_cur_path(sctx, sctx->cur_ino, sctx->cur_inode_gen, p); + if (ret < 0) + goto tlv_put_failure; memset(sctx->read_buf, 0, BTRFS_SEND_READ_SIZE); while (offset < end) { len = min_t(u64, end - offset, BTRFS_SEND_READ_SIZE); @@ -4443,9 +4446,6 @@ static int send_hole(struct send_ctx *sctx, u64 end) ret = begin_cmd(sctx, BTRFS_SEND_C_WRITE); if (ret < 0) break; - ret = get_cur_path(sctx, sctx->cur_ino, sctx->cur_inode_gen, p); - if (ret < 0) - break; TLV_PUT_PATH(sctx, BTRFS_SEND_A_PATH, p); TLV_PUT_U64(sctx, BTRFS_SEND_A_FILE_OFFSET, offset); TLV_PUT(sctx, BTRFS_SEND_A_DATA, sctx->read_buf, len); |
