diff options
author | Anand Jain <anand.jain@oracle.com> | 2015-09-25 02:43:01 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2015-09-29 10:30:00 -0400 |
commit | a4553fefb59cb0336f543fa567170b47e90142a9 (patch) | |
tree | 103dcab4eeecae6d9ee4e796bcdb4209aa340ca4 /fs/btrfs/tree-log.c | |
parent | 57d816a15ba2c2690c57635134bc01cf4da4623c (diff) |
Btrfs: consolidate btrfs_error() to btrfs_std_error()
btrfs_error() and btrfs_std_error() does the same thing
and calls _btrfs_std_error(), so consolidate them together.
And the main motivation is that btrfs_error() is closely
named with btrfs_err(), one handles error action the other
is to log the error, so don't closely name them.
Signed-off-by: Anand Jain <anand.jain@oracle.com>
Suggested-by: David Sterba <dsterba@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/tree-log.c')
-rw-r--r-- | fs/btrfs/tree-log.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 1bbaace73383..c3f9a9c71f28 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -5314,7 +5314,7 @@ int btrfs_recover_log_trees(struct btrfs_root *log_root_tree) | |||
5314 | 5314 | ||
5315 | ret = walk_log_tree(trans, log_root_tree, &wc); | 5315 | ret = walk_log_tree(trans, log_root_tree, &wc); |
5316 | if (ret) { | 5316 | if (ret) { |
5317 | btrfs_error(fs_info, ret, "Failed to pin buffers while " | 5317 | btrfs_std_error(fs_info, ret, "Failed to pin buffers while " |
5318 | "recovering log root tree."); | 5318 | "recovering log root tree."); |
5319 | goto error; | 5319 | goto error; |
5320 | } | 5320 | } |
@@ -5328,7 +5328,7 @@ again: | |||
5328 | ret = btrfs_search_slot(NULL, log_root_tree, &key, path, 0, 0); | 5328 | ret = btrfs_search_slot(NULL, log_root_tree, &key, path, 0, 0); |
5329 | 5329 | ||
5330 | if (ret < 0) { | 5330 | if (ret < 0) { |
5331 | btrfs_error(fs_info, ret, | 5331 | btrfs_std_error(fs_info, ret, |
5332 | "Couldn't find tree log root."); | 5332 | "Couldn't find tree log root."); |
5333 | goto error; | 5333 | goto error; |
5334 | } | 5334 | } |
@@ -5346,7 +5346,7 @@ again: | |||
5346 | log = btrfs_read_fs_root(log_root_tree, &found_key); | 5346 | log = btrfs_read_fs_root(log_root_tree, &found_key); |
5347 | if (IS_ERR(log)) { | 5347 | if (IS_ERR(log)) { |
5348 | ret = PTR_ERR(log); | 5348 | ret = PTR_ERR(log); |
5349 | btrfs_error(fs_info, ret, | 5349 | btrfs_std_error(fs_info, ret, |
5350 | "Couldn't read tree log root."); | 5350 | "Couldn't read tree log root."); |
5351 | goto error; | 5351 | goto error; |
5352 | } | 5352 | } |
@@ -5361,7 +5361,7 @@ again: | |||
5361 | free_extent_buffer(log->node); | 5361 | free_extent_buffer(log->node); |
5362 | free_extent_buffer(log->commit_root); | 5362 | free_extent_buffer(log->commit_root); |
5363 | kfree(log); | 5363 | kfree(log); |
5364 | btrfs_error(fs_info, ret, "Couldn't read target root " | 5364 | btrfs_std_error(fs_info, ret, "Couldn't read target root " |
5365 | "for tree log recovery."); | 5365 | "for tree log recovery."); |
5366 | goto error; | 5366 | goto error; |
5367 | } | 5367 | } |