afs: Don't use VL probe running state to make decisions outside probe code
authorDavid Howells <dhowells@redhat.com>
Wed, 19 Aug 2020 14:27:17 +0000 (15:27 +0100)
committerDavid Howells <dhowells@redhat.com>
Thu, 20 Aug 2020 17:21:28 +0000 (18:21 +0100)
commitb95b30940ee46a4d892ca3de3ffb8249c4985b1f
treefcd9268473163fbca8d549a2f78fffa78b4b850e
parentfb72cd3d484ce548597c75ebeecc70483fe8bb6e
afs: Don't use VL probe running state to make decisions outside probe code

Don't use the running state for VL server probes to make decisions about
which server to use as the state is cleared at the start of a probe and
intermediate values might also be misleading.

Instead, add a separate 'latest known' rtt in the afs_vlserver struct and a
flag to indicate if the server is known to be responding and update these
as and when we know what to change them to.

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