diff options
| author | Ilya Dryomov <idryomov@gmail.com> | 2016-01-13 14:32:57 +0100 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@gmail.com> | 2016-01-21 19:36:08 +0100 |
| commit | f6330cc1f04b7dcb84b572d402cdacf7e275a022 (patch) | |
| tree | c6460bb777e8be11bf521ff5624fb4159b06c512 /net/dns_resolver | |
| parent | 67645d7619738e51c668ca69f097cb90b5470422 (diff) | |
| download | linux-f6330cc1f04b7dcb84b572d402cdacf7e275a022.tar.gz linux-f6330cc1f04b7dcb84b572d402cdacf7e275a022.tar.bz2 linux-f6330cc1f04b7dcb84b572d402cdacf7e275a022.zip | |
libceph: clear messenger auth_retry flag if we fault
Commit 20e55c4cc758 ("libceph: clear messenger auth_retry flag when we
authenticate") got us only half way there. We clear the flag if the
second attempt succeeds, but it also needs to be cleared if that
attempt fails, to allow for the exponential backoff to kick in.
Otherwise, if ->should_authenticate() thinks our keys are valid, we
will busy loop, incrementing auth_retry to no avail:
process_connect ffff880079a63830 got BADAUTHORIZER attempt 1
process_connect ffff880079a63830 got BADAUTHORIZER attempt 2
process_connect ffff880079a63830 got BADAUTHORIZER attempt 3
process_connect ffff880079a63830 got BADAUTHORIZER attempt 4
process_connect ffff880079a63830 got BADAUTHORIZER attempt 5
...
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Sage Weil <sage@redhat.com>
Diffstat (limited to 'net/dns_resolver')
0 files changed, 0 insertions, 0 deletions