diff options
author | zhong jiang <zhongjiang@huawei.com> | 2018-08-16 12:37:15 -0400 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2018-10-15 11:23:27 -0400 |
commit | 45128b08f74131753a10a584e3e5c4496b279269 (patch) | |
tree | 55ab4d1310104c9115954a3293f8c74c868fcd32 /fs/btrfs/tree-log.c | |
parent | 556f3ca88ecb7087e7e3144e429414c60a300984 (diff) |
btrfs: change btrfs_pin_log_trans to return void
btrfs_pin_log_trans defines the variable "ret" for return value, but it
is not modified after initialization. Further, I find that none of the
callers do handles the return value, so it is safe to drop the unneeded
"ret" and make it return void.
Signed-off-by: zhong jiang <zhongjiang@huawei.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 | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index 3c2ae0e4f25a..1078c65128c5 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -205,14 +205,11 @@ static int join_running_log_trans(struct btrfs_root *root) | |||
205 | * until you call btrfs_end_log_trans() or it makes any future | 205 | * until you call btrfs_end_log_trans() or it makes any future |
206 | * log transactions wait until you call btrfs_end_log_trans() | 206 | * log transactions wait until you call btrfs_end_log_trans() |
207 | */ | 207 | */ |
208 | int btrfs_pin_log_trans(struct btrfs_root *root) | 208 | void btrfs_pin_log_trans(struct btrfs_root *root) |
209 | { | 209 | { |
210 | int ret = -ENOENT; | ||
211 | |||
212 | mutex_lock(&root->log_mutex); | 210 | mutex_lock(&root->log_mutex); |
213 | atomic_inc(&root->log_writers); | 211 | atomic_inc(&root->log_writers); |
214 | mutex_unlock(&root->log_mutex); | 212 | mutex_unlock(&root->log_mutex); |
215 | return ret; | ||
216 | } | 213 | } |
217 | 214 | ||
218 | /* | 215 | /* |