diff options
| author | Alexander Aring <aahringo@redhat.com> | 2023-05-29 23:44:33 +0200 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2023-06-14 17:17:33 +0200 |
| commit | f8bce79d9d9edb8d2302a216f298f4839fb0adb6 (patch) | |
| tree | 1c7629a9d8c5a5f624b459cb6cdee8dca4e2c1dd /fs | |
| parent | fs: dlm: unregister memory at the very last (diff) | |
| download | linux-f8bce79d9d9edb8d2302a216f298f4839fb0adb6.tar.xz linux-f8bce79d9d9edb8d2302a216f298f4839fb0adb6.zip | |
fs: dlm: don't check othercon twice
This patch removes an another check if con->othercon set inside the
branch which already does that.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/dlm/lowcomms.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c index 5aad4d4842eb..b28505b8b23b 100644 --- a/fs/dlm/lowcomms.c +++ b/fs/dlm/lowcomms.c @@ -1497,8 +1497,7 @@ int dlm_lowcomms_close(int nodeid) call_srcu(&connections_srcu, &con->rcu, connection_release); if (con->othercon) { clean_one_writequeue(con->othercon); - if (con->othercon) - call_srcu(&connections_srcu, &con->othercon->rcu, connection_release); + call_srcu(&connections_srcu, &con->othercon->rcu, connection_release); } srcu_read_unlock(&connections_srcu, idx); |