diff options
author | Filipe David Borba Manana <fdmanana@gmail.com> | 2013-10-07 16:20:44 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2013-11-11 21:59:30 -0500 |
commit | efd0c4055a26e88689b3c0cf95492dfe147cba67 (patch) | |
tree | 9c92856460fc0f1309f14ffd36a74b644505070c | |
parent | 452c75c3d2187089f6e846710e6ea7883bf30f8a (diff) |
Btrfs: remove unnecessary key copy when logging inode
The btrfs_insert_empty_item() function doesn't modify its
key argument.
Signed-off-by: Filipe David Borba Manana <fdmanana@gmail.com>
Reviewed-by: Zach Brown <zab@redhat.com>
Signed-off-by: Josef Bacik <jbacik@fusionio.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
-rw-r--r-- | fs/btrfs/tree-log.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 1134aa40b6d1..f2e0531dc719 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -3170,11 +3170,10 @@ static int log_inode_item(struct btrfs_trans_handle *trans, | |||
3170 | struct inode *inode) | 3170 | struct inode *inode) |
3171 | { | 3171 | { |
3172 | struct btrfs_inode_item *inode_item; | 3172 | struct btrfs_inode_item *inode_item; |
3173 | struct btrfs_key key; | ||
3174 | int ret; | 3173 | int ret; |
3175 | 3174 | ||
3176 | memcpy(&key, &BTRFS_I(inode)->location, sizeof(key)); | 3175 | ret = btrfs_insert_empty_item(trans, log, path, |
3177 | ret = btrfs_insert_empty_item(trans, log, path, &key, | 3176 | &BTRFS_I(inode)->location, |
3178 | sizeof(*inode_item)); | 3177 | sizeof(*inode_item)); |
3179 | if (ret && ret != -EEXIST) | 3178 | if (ret && ret != -EEXIST) |
3180 | return ret; | 3179 | return ret; |