Commit e040b7b2 authored by Haiyang Zhang's avatar Haiyang Zhang Committed by Ben Hutchings

hv_vmbus: Add gradually increased delay for retries in vmbus_post_msg()

commit e1c0d82d upstream.

Most of the retries can be done within a millisecond successfully, so we
sleep 1ms before the first retry, then gradually increase the retry
interval to 2^n with max value of 2048ms. Doing so, we will have shorter
overall delay time, because most of the cases succeed within 1-2 attempts.
Signed-off-by: default avatarHaiyang Zhang <haiyangz@microsoft.com>
Reviewed-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Reviewed-by: default avatarDexuan Cui <decui@microsoft.com>
Signed-off-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent fc927717
...@@ -418,6 +418,7 @@ int vmbus_post_msg(void *buffer, size_t buflen) ...@@ -418,6 +418,7 @@ int vmbus_post_msg(void *buffer, size_t buflen)
union hv_connection_id conn_id; union hv_connection_id conn_id;
int ret = 0; int ret = 0;
int retries = 0; int retries = 0;
u32 msec = 1;
conn_id.asu32 = 0; conn_id.asu32 = 0;
conn_id.u.id = VMBUS_MESSAGE_CONNECTION_ID; conn_id.u.id = VMBUS_MESSAGE_CONNECTION_ID;
...@@ -427,7 +428,7 @@ int vmbus_post_msg(void *buffer, size_t buflen) ...@@ -427,7 +428,7 @@ int vmbus_post_msg(void *buffer, size_t buflen)
* insufficient resources. Retry the operation a couple of * insufficient resources. Retry the operation a couple of
* times before giving up. * times before giving up.
*/ */
while (retries < 10) { while (retries < 20) {
ret = hv_post_message(conn_id, 1, buffer, buflen); ret = hv_post_message(conn_id, 1, buffer, buflen);
switch (ret) { switch (ret) {
...@@ -450,7 +451,9 @@ int vmbus_post_msg(void *buffer, size_t buflen) ...@@ -450,7 +451,9 @@ int vmbus_post_msg(void *buffer, size_t buflen)
} }
retries++; retries++;
msleep(1000); msleep(msec);
if (msec < 2048)
msec *= 2;
} }
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment