afs: Remove afs_vlserver->probe.have_result
authorDavid Howells <dhowells@redhat.com>
Wed, 19 Aug 2020 14:13:44 +0000 (15:13 +0100)
committerDavid Howells <dhowells@redhat.com>
Thu, 20 Aug 2020 17:21:28 +0000 (18:21 +0100)
Remove afs_vlserver->probe.have_result as it's neither read nor waited
upon.

Fixes: 3bf0fb6f33dd ("afs: Probe multiple fileservers simultaneously")
Signed-off-by: David Howells <dhowells@redhat.com>
fs/afs/internal.h
fs/afs/vl_probe.c

index 792ac71..2e6ae63 100644 (file)
@@ -412,7 +412,6 @@ struct afs_vlserver {
                unsigned int    rtt;            /* RTT as ktime/64 */
                u32             abort_code;
                short           error;
-               bool            have_result;
                bool            responded:1;
                bool            is_yfs:1;
                bool            not_yfs:1;
index 081b7e5..ee59188 100644 (file)
@@ -109,11 +109,8 @@ out:
               server_index, index, &alist->addrs[index].transport, rtt_us, ret);
 
        have_result |= afs_vl_probe_done(server);
-       if (have_result) {
-               server->probe.have_result = true;
-               wake_up_var(&server->probe.have_result);
+       if (have_result)
                wake_up_all(&server->probe_wq);
-       }
 }
 
 /*