diff options
author | KY Srinivasan <kys@microsoft.com> | 2015-03-29 21:08:41 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-03-31 14:12:36 -0400 |
commit | cbacec76bcd03ff21b37ac331e652b5a8f3ea644 (patch) | |
tree | 218c105a97d9f3ad742b56c0c26a6feb3b4aaf09 | |
parent | 4e833c596ef5f699479a8ba2d3294a004895f5f4 (diff) | |
download | op-kernel-dev-cbacec76bcd03ff21b37ac331e652b5a8f3ea644.zip op-kernel-dev-cbacec76bcd03ff21b37ac331e652b5a8f3ea644.tar.gz |
hv_netvsc: Cleanup the test for freeing skb when we use sendbuf mechanism
In preparation for embedding the rndis state and other packet state into
the skb, cleanup the test for freeing the skb.
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/hyperv/netvsc.c | 1 | ||||
-rw-r--r-- | drivers/net/hyperv/netvsc_drv.c | 3 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/hyperv/netvsc.c b/drivers/net/hyperv/netvsc.c index b81bd37..ecbd813 100644 --- a/drivers/net/hyperv/netvsc.c +++ b/drivers/net/hyperv/netvsc.c @@ -870,6 +870,7 @@ int netvsc_send(struct hv_device *device, (unsigned long)packet->send_completion_tid; packet->page_buf_cnt = 0; + packet->send_completion_tid = 0; packet->send_buf_index = section_index; packet->total_data_buflen += msd_len; diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv/netvsc_drv.c index 0c99818..75beb89 100644 --- a/drivers/net/hyperv/netvsc_drv.c +++ b/drivers/net/hyperv/netvsc_drv.c @@ -234,11 +234,10 @@ static void netvsc_xmit_completion(void *context) struct hv_netvsc_packet *packet = (struct hv_netvsc_packet *)context; struct sk_buff *skb = (struct sk_buff *) (unsigned long)packet->send_completion_tid; - u32 index = packet->send_buf_index; kfree(packet); - if (skb && (index == NETVSC_INVALID_INDEX)) + if (skb) dev_kfree_skb_any(skb); } |