connect: fir off by one error in iscsi->reconnect_max_retries
Before b152d26 it was possible to set iscsi->reconnect_max_retries to 0.
This allowed reconnects, but aborted as soon as the reconnect failed.
The same behaviour is currenltly only possible by setting iscsi->reconnect_max_retries
to 1. iscsi->reconnect_max_retries == 0 forbids reconnecting completly.
Signed-off-by: Peter Lieven <pl@kamp.de>
This commit is contained in:
@@ -263,7 +263,7 @@ void iscsi_reconnect_cb(struct iscsi_context *iscsi _U_, int status,
|
||||
backoff = 30;
|
||||
}
|
||||
if (iscsi->reconnect_max_retries != -1 &&
|
||||
iscsi->old_iscsi->retry_cnt >= iscsi->reconnect_max_retries) {
|
||||
iscsi->old_iscsi->retry_cnt > iscsi->reconnect_max_retries) {
|
||||
/* we will exit iscsi_service with -1 the next time we enter it. */
|
||||
backoff = 0;
|
||||
}
|
||||
@@ -375,7 +375,7 @@ int iscsi_reconnect(struct iscsi_context *old_iscsi)
|
||||
}
|
||||
|
||||
if (old_iscsi->reconnect_max_retries != -1 && old_iscsi->old_iscsi &&
|
||||
old_iscsi->old_iscsi->retry_cnt >= old_iscsi->reconnect_max_retries) {
|
||||
old_iscsi->old_iscsi->retry_cnt > old_iscsi->reconnect_max_retries) {
|
||||
iscsi_defer_reconnect(old_iscsi);
|
||||
return -1;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user