diff options
| author | Vitaly Kuznetsov <vkuznets@redhat.com> | 2015-04-12 03:07:40 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-05-24 21:17:40 +0200 |
| commit | 5fa97480b9b125bb9bdb446930289b3274ed7eb7 (patch) | |
| tree | 43368cfa899480101f01d867eaa25da942b8bb6c /drivers/hv | |
| parent | Drivers: hv: util: move kvp/vss function declarations to hyperv_vmbus.h (diff) | |
| download | linux-5fa97480b9b125bb9bdb446930289b3274ed7eb7.tar.xz linux-5fa97480b9b125bb9bdb446930289b3274ed7eb7.zip | |
Drivers: hv: kvp: reset kvp_context
We set kvp_context when we want to postpone receiving a packet from vmbus due
to the previous transaction being unfinished. We, however, never reset this
state, all consequent kvp_respond_to_host() calls will result in poll_channel()
calling hv_kvp_onchannelcallback(). This doesn't cause real issues as:
1) Host is supposed to serialize transactions as well
2) If no message is pending vmbus_recvpacket() will return 0 recvlen.
This is just a cleanup.
Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
Tested-by: Alex Ng <alexng@microsoft.com>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hv')
| -rw-r--r-- | drivers/hv/hv_kvp.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/hv/hv_kvp.c b/drivers/hv/hv_kvp.c index a414c8397f88..caa467dc8351 100644 --- a/drivers/hv/hv_kvp.c +++ b/drivers/hv/hv_kvp.c @@ -621,6 +621,7 @@ void hv_kvp_onchannelcallback(void *context) kvp_transaction.kvp_context = context; return; } + kvp_transaction.kvp_context = NULL; vmbus_recvpacket(channel, recv_buffer, PAGE_SIZE * 4, &recvlen, &requestid); |