btrfs: Remove useless condition in start_log_trans()
authorZhaolei <zhaolei@cn.fujitsu.com>
Mon, 17 Aug 2015 10:44:45 +0000 (18:44 +0800)
committerChris Mason <clm@fb.com>
Wed, 19 Aug 2015 21:24:49 +0000 (14:24 -0700)
commit34eb2a524997e5cd7117569b1fda925516adf6ac
treeb7d17e3f6a4e78fc51acac6a9260ee901536de05
parent46cd28555ffaa40162290dba203daad0ff6f7abd
btrfs: Remove useless condition in start_log_trans()

Dan Carpenter <dan.carpenter@oracle.com> reported a smatch warning
for start_log_trans():
 fs/btrfs/tree-log.c:178 start_log_trans()
 warn: we tested 'root->log_root' before and it was 'false'

 fs/btrfs/tree-log.c
 147          if (root->log_root) {
 We test "root->log_root" here.
 ...

Reason:
 Condition of:
 fs/btrfs/tree-log.c:178: if (!root->log_root) {
 is not necessary after commit: 7237f1833

 It caused a smatch warning, and no functionally error.

Fix:
 Deleting above condition will make smatch shut up,
 but a better way is to do cleanup for start_log_trans()
 to remove duplicated code and make code more readable.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Zhao Lei <zhaolei@cn.fujitsu.com>
Signed-off-by: Chris Mason <clm@fb.com>
fs/btrfs/tree-log.c