Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 1 | /* |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 2 | * Copyright (c) 2009, Microsoft Corporation. |
| 3 | * |
| 4 | * This program is free software; you can redistribute it and/or modify it |
| 5 | * under the terms and conditions of the GNU General Public License, |
| 6 | * version 2, as published by the Free Software Foundation. |
| 7 | * |
| 8 | * This program is distributed in the hope it will be useful, but WITHOUT |
| 9 | * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
| 10 | * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for |
| 11 | * more details. |
| 12 | * |
| 13 | * You should have received a copy of the GNU General Public License along with |
| 14 | * this program; if not, write to the Free Software Foundation, Inc., 59 Temple |
| 15 | * Place - Suite 330, Boston, MA 02111-1307 USA. |
| 16 | * |
| 17 | * Authors: |
Haiyang Zhang | d0e94d1 | 2009-11-23 17:00:22 +0000 | [diff] [blame] | 18 | * Haiyang Zhang <haiyangz@microsoft.com> |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 19 | * Hank Janssen <hjanssen@microsoft.com> |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 20 | */ |
Hank Janssen | eb335bc | 2011-03-29 13:58:48 -0700 | [diff] [blame] | 21 | #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt |
| 22 | |
Greg Kroah-Hartman | 5654e93 | 2009-07-14 15:08:20 -0700 | [diff] [blame] | 23 | #include <linux/kernel.h> |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 24 | #include <linux/sched.h> |
| 25 | #include <linux/wait.h> |
Greg Kroah-Hartman | 0ffa63b | 2009-07-15 11:06:01 -0700 | [diff] [blame] | 26 | #include <linux/mm.h> |
Greg Kroah-Hartman | b4362c9 | 2009-07-16 11:50:41 -0700 | [diff] [blame] | 27 | #include <linux/delay.h> |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 28 | #include <linux/io.h> |
Tejun Heo | 5a0e3ad | 2010-03-24 17:04:11 +0900 | [diff] [blame] | 29 | #include <linux/slab.h> |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 30 | #include <linux/netdevice.h> |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 31 | #include <linux/if_ether.h> |
K. Y. Srinivasan | 3f335ea | 2011-05-12 19:34:15 -0700 | [diff] [blame] | 32 | |
K. Y. Srinivasan | 5ca7252 | 2011-05-12 19:34:37 -0700 | [diff] [blame] | 33 | #include "hyperv_net.h" |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 34 | |
| 35 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 36 | static struct netvsc_device *alloc_net_device(struct hv_device *device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 37 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 38 | struct netvsc_device *net_device; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 39 | struct net_device *ndev = hv_get_drvdata(device); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 40 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 41 | net_device = kzalloc(sizeof(struct netvsc_device), GFP_KERNEL); |
| 42 | if (!net_device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 43 | return NULL; |
| 44 | |
Haiyang Zhang | dc5cd89 | 2012-06-04 06:42:38 +0000 | [diff] [blame^] | 45 | init_waitqueue_head(&net_device->wait_drain); |
Haiyang Zhang | 4d447c9 | 2011-12-15 13:45:17 -0800 | [diff] [blame] | 46 | net_device->start_remove = false; |
K. Y. Srinivasan | c38b9c7 | 2011-08-27 11:31:12 -0700 | [diff] [blame] | 47 | net_device->destroy = false; |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 48 | net_device->dev = device; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 49 | net_device->ndev = ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 50 | |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 51 | hv_set_drvdata(device, net_device); |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 52 | return net_device; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 53 | } |
| 54 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 55 | static struct netvsc_device *get_outbound_net_device(struct hv_device *device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 56 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 57 | struct netvsc_device *net_device; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 58 | |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 59 | net_device = hv_get_drvdata(device); |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 60 | if (net_device && net_device->destroy) |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 61 | net_device = NULL; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 62 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 63 | return net_device; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 64 | } |
| 65 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 66 | static struct netvsc_device *get_inbound_net_device(struct hv_device *device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 67 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 68 | struct netvsc_device *net_device; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 69 | |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 70 | net_device = hv_get_drvdata(device); |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 71 | |
| 72 | if (!net_device) |
| 73 | goto get_in_err; |
| 74 | |
| 75 | if (net_device->destroy && |
| 76 | atomic_read(&net_device->num_outstanding_sends) == 0) |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 77 | net_device = NULL; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 78 | |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 79 | get_in_err: |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 80 | return net_device; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 81 | } |
| 82 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 83 | |
Haiyang Zhang | ec91cd0 | 2011-04-21 12:30:43 -0700 | [diff] [blame] | 84 | static int netvsc_destroy_recv_buf(struct netvsc_device *net_device) |
| 85 | { |
| 86 | struct nvsp_message *revoke_packet; |
| 87 | int ret = 0; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 88 | struct net_device *ndev = net_device->ndev; |
Haiyang Zhang | ec91cd0 | 2011-04-21 12:30:43 -0700 | [diff] [blame] | 89 | |
| 90 | /* |
| 91 | * If we got a section count, it means we received a |
| 92 | * SendReceiveBufferComplete msg (ie sent |
| 93 | * NvspMessage1TypeSendReceiveBuffer msg) therefore, we need |
| 94 | * to send a revoke msg here |
| 95 | */ |
| 96 | if (net_device->recv_section_cnt) { |
| 97 | /* Send the revoke receive buffer */ |
| 98 | revoke_packet = &net_device->revoke_packet; |
| 99 | memset(revoke_packet, 0, sizeof(struct nvsp_message)); |
| 100 | |
| 101 | revoke_packet->hdr.msg_type = |
| 102 | NVSP_MSG1_TYPE_REVOKE_RECV_BUF; |
| 103 | revoke_packet->msg.v1_msg. |
| 104 | revoke_recv_buf.id = NETVSC_RECEIVE_BUFFER_ID; |
| 105 | |
| 106 | ret = vmbus_sendpacket(net_device->dev->channel, |
| 107 | revoke_packet, |
| 108 | sizeof(struct nvsp_message), |
| 109 | (unsigned long)revoke_packet, |
| 110 | VM_PKT_DATA_INBAND, 0); |
| 111 | /* |
| 112 | * If we failed here, we might as well return and |
| 113 | * have a leak rather than continue and a bugchk |
| 114 | */ |
| 115 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 116 | netdev_err(ndev, "unable to send " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 117 | "revoke receive buffer to netvsp\n"); |
K. Y. Srinivasan | a3e0053 | 2011-08-25 09:49:12 -0700 | [diff] [blame] | 118 | return ret; |
Haiyang Zhang | ec91cd0 | 2011-04-21 12:30:43 -0700 | [diff] [blame] | 119 | } |
| 120 | } |
| 121 | |
| 122 | /* Teardown the gpadl on the vsp end */ |
| 123 | if (net_device->recv_buf_gpadl_handle) { |
| 124 | ret = vmbus_teardown_gpadl(net_device->dev->channel, |
| 125 | net_device->recv_buf_gpadl_handle); |
| 126 | |
| 127 | /* If we failed here, we might as well return and have a leak |
| 128 | * rather than continue and a bugchk |
| 129 | */ |
| 130 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 131 | netdev_err(ndev, |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 132 | "unable to teardown receive buffer's gpadl\n"); |
Dan Carpenter | 7f9615e | 2011-08-27 14:06:07 +0300 | [diff] [blame] | 133 | return ret; |
Haiyang Zhang | ec91cd0 | 2011-04-21 12:30:43 -0700 | [diff] [blame] | 134 | } |
| 135 | net_device->recv_buf_gpadl_handle = 0; |
| 136 | } |
| 137 | |
| 138 | if (net_device->recv_buf) { |
| 139 | /* Free up the receive buffer */ |
| 140 | free_pages((unsigned long)net_device->recv_buf, |
| 141 | get_order(net_device->recv_buf_size)); |
| 142 | net_device->recv_buf = NULL; |
| 143 | } |
| 144 | |
| 145 | if (net_device->recv_section) { |
| 146 | net_device->recv_section_cnt = 0; |
| 147 | kfree(net_device->recv_section); |
| 148 | net_device->recv_section = NULL; |
| 149 | } |
| 150 | |
| 151 | return ret; |
| 152 | } |
| 153 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 154 | static int netvsc_init_recv_buf(struct hv_device *device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 155 | { |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 156 | int ret = 0; |
K. Y. Srinivasan | 35abb21 | 2011-05-10 07:55:41 -0700 | [diff] [blame] | 157 | int t; |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 158 | struct netvsc_device *net_device; |
| 159 | struct nvsp_message *init_packet; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 160 | struct net_device *ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 161 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 162 | net_device = get_outbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 163 | if (!net_device) |
K. Y. Srinivasan | 927bc33 | 2011-08-25 09:49:13 -0700 | [diff] [blame] | 164 | return -ENODEV; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 165 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 166 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 167 | net_device->recv_buf = |
K. Y. Srinivasan | df3493e | 2011-02-11 09:59:00 -0800 | [diff] [blame] | 168 | (void *)__get_free_pages(GFP_KERNEL|__GFP_ZERO, |
| 169 | get_order(net_device->recv_buf_size)); |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 170 | if (!net_device->recv_buf) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 171 | netdev_err(ndev, "unable to allocate receive " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 172 | "buffer of size %d\n", net_device->recv_buf_size); |
K. Y. Srinivasan | 927bc33 | 2011-08-25 09:49:13 -0700 | [diff] [blame] | 173 | ret = -ENOMEM; |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 174 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 175 | } |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 176 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 177 | /* |
| 178 | * Establish the gpadl handle for this buffer on this |
| 179 | * channel. Note: This call uses the vmbus connection rather |
| 180 | * than the channel to establish the gpadl handle. |
| 181 | */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 182 | ret = vmbus_establish_gpadl(device->channel, net_device->recv_buf, |
| 183 | net_device->recv_buf_size, |
| 184 | &net_device->recv_buf_gpadl_handle); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 185 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 186 | netdev_err(ndev, |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 187 | "unable to establish receive buffer's gpadl\n"); |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 188 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 189 | } |
| 190 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 191 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 192 | /* Notify the NetVsp of the gpadl handle */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 193 | init_packet = &net_device->channel_init_pkt; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 194 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 195 | memset(init_packet, 0, sizeof(struct nvsp_message)); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 196 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 197 | init_packet->hdr.msg_type = NVSP_MSG1_TYPE_SEND_RECV_BUF; |
| 198 | init_packet->msg.v1_msg.send_recv_buf. |
| 199 | gpadl_handle = net_device->recv_buf_gpadl_handle; |
| 200 | init_packet->msg.v1_msg. |
| 201 | send_recv_buf.id = NETVSC_RECEIVE_BUFFER_ID; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 202 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 203 | /* Send the gpadl notification request */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 204 | ret = vmbus_sendpacket(device->channel, init_packet, |
Greg Kroah-Hartman | 5a4df29 | 2010-10-21 09:43:24 -0700 | [diff] [blame] | 205 | sizeof(struct nvsp_message), |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 206 | (unsigned long)init_packet, |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 207 | VM_PKT_DATA_INBAND, |
Greg Kroah-Hartman | 5a4df29 | 2010-10-21 09:43:24 -0700 | [diff] [blame] | 208 | VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 209 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 210 | netdev_err(ndev, |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 211 | "unable to send receive buffer's gpadl to netvsp\n"); |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 212 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 213 | } |
| 214 | |
K. Y. Srinivasan | 5c5781b3 | 2011-06-16 13:16:35 -0700 | [diff] [blame] | 215 | t = wait_for_completion_timeout(&net_device->channel_init_wait, 5*HZ); |
K. Y. Srinivasan | 35abb21 | 2011-05-10 07:55:41 -0700 | [diff] [blame] | 216 | BUG_ON(t == 0); |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 217 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 218 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 219 | /* Check the response */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 220 | if (init_packet->msg.v1_msg. |
| 221 | send_recv_buf_complete.status != NVSP_STAT_SUCCESS) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 222 | netdev_err(ndev, "Unable to complete receive buffer " |
Haiyang Zhang | 8bff33a | 2011-09-01 12:19:48 -0700 | [diff] [blame] | 223 | "initialization with NetVsp - status %d\n", |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 224 | init_packet->msg.v1_msg. |
| 225 | send_recv_buf_complete.status); |
K. Y. Srinivasan | 927bc33 | 2011-08-25 09:49:13 -0700 | [diff] [blame] | 226 | ret = -EINVAL; |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 227 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 228 | } |
| 229 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 230 | /* Parse the response */ |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 231 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 232 | net_device->recv_section_cnt = init_packet->msg. |
| 233 | v1_msg.send_recv_buf_complete.num_sections; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 234 | |
Haiyang Zhang | c181320 | 2011-11-30 07:19:07 -0800 | [diff] [blame] | 235 | net_device->recv_section = kmemdup( |
| 236 | init_packet->msg.v1_msg.send_recv_buf_complete.sections, |
| 237 | net_device->recv_section_cnt * |
| 238 | sizeof(struct nvsp_1_receive_buffer_section), |
| 239 | GFP_KERNEL); |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 240 | if (net_device->recv_section == NULL) { |
K. Y. Srinivasan | 927bc33 | 2011-08-25 09:49:13 -0700 | [diff] [blame] | 241 | ret = -EINVAL; |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 242 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 243 | } |
| 244 | |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 245 | /* |
| 246 | * For 1st release, there should only be 1 section that represents the |
| 247 | * entire receive buffer |
| 248 | */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 249 | if (net_device->recv_section_cnt != 1 || |
| 250 | net_device->recv_section->offset != 0) { |
K. Y. Srinivasan | 927bc33 | 2011-08-25 09:49:13 -0700 | [diff] [blame] | 251 | ret = -EINVAL; |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 252 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 253 | } |
| 254 | |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 255 | goto exit; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 256 | |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 257 | cleanup: |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 258 | netvsc_destroy_recv_buf(net_device); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 259 | |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 260 | exit: |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 261 | return ret; |
| 262 | } |
| 263 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 264 | |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 265 | /* Negotiate NVSP protocol version */ |
| 266 | static int negotiate_nvsp_ver(struct hv_device *device, |
| 267 | struct netvsc_device *net_device, |
| 268 | struct nvsp_message *init_packet, |
| 269 | u32 nvsp_ver) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 270 | { |
K. Y. Srinivasan | 35abb21 | 2011-05-10 07:55:41 -0700 | [diff] [blame] | 271 | int ret, t; |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 272 | |
| 273 | memset(init_packet, 0, sizeof(struct nvsp_message)); |
| 274 | init_packet->hdr.msg_type = NVSP_MSG_TYPE_INIT; |
| 275 | init_packet->msg.init_msg.init.min_protocol_ver = nvsp_ver; |
| 276 | init_packet->msg.init_msg.init.max_protocol_ver = nvsp_ver; |
| 277 | |
| 278 | /* Send the init request */ |
| 279 | ret = vmbus_sendpacket(device->channel, init_packet, |
| 280 | sizeof(struct nvsp_message), |
| 281 | (unsigned long)init_packet, |
| 282 | VM_PKT_DATA_INBAND, |
| 283 | VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED); |
| 284 | |
| 285 | if (ret != 0) |
| 286 | return ret; |
| 287 | |
| 288 | t = wait_for_completion_timeout(&net_device->channel_init_wait, 5*HZ); |
| 289 | |
| 290 | if (t == 0) |
| 291 | return -ETIMEDOUT; |
| 292 | |
| 293 | if (init_packet->msg.init_msg.init_complete.status != |
| 294 | NVSP_STAT_SUCCESS) |
| 295 | return -EINVAL; |
| 296 | |
| 297 | if (nvsp_ver != NVSP_PROTOCOL_VERSION_2) |
| 298 | return 0; |
| 299 | |
| 300 | /* NVSPv2 only: Send NDIS config */ |
| 301 | memset(init_packet, 0, sizeof(struct nvsp_message)); |
| 302 | init_packet->hdr.msg_type = NVSP_MSG2_TYPE_SEND_NDIS_CONFIG; |
Haiyang Zhang | 4d447c9 | 2011-12-15 13:45:17 -0800 | [diff] [blame] | 303 | init_packet->msg.v2_msg.send_ndis_config.mtu = net_device->ndev->mtu; |
Haiyang Zhang | 1f5f3a7 | 2012-03-12 10:20:50 +0000 | [diff] [blame] | 304 | init_packet->msg.v2_msg.send_ndis_config.capability.ieee8021q = 1; |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 305 | |
| 306 | ret = vmbus_sendpacket(device->channel, init_packet, |
| 307 | sizeof(struct nvsp_message), |
| 308 | (unsigned long)init_packet, |
| 309 | VM_PKT_DATA_INBAND, 0); |
| 310 | |
| 311 | return ret; |
| 312 | } |
| 313 | |
| 314 | static int netvsc_connect_vsp(struct hv_device *device) |
| 315 | { |
| 316 | int ret; |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 317 | struct netvsc_device *net_device; |
| 318 | struct nvsp_message *init_packet; |
| 319 | int ndis_version; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 320 | struct net_device *ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 321 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 322 | net_device = get_outbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 323 | if (!net_device) |
K. Y. Srinivasan | 0f48c72 | 2011-08-25 09:49:14 -0700 | [diff] [blame] | 324 | return -ENODEV; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 325 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 326 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 327 | init_packet = &net_device->channel_init_pkt; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 328 | |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 329 | /* Negotiate the latest NVSP protocol supported */ |
| 330 | if (negotiate_nvsp_ver(device, net_device, init_packet, |
| 331 | NVSP_PROTOCOL_VERSION_2) == 0) { |
| 332 | net_device->nvsp_version = NVSP_PROTOCOL_VERSION_2; |
| 333 | } else if (negotiate_nvsp_ver(device, net_device, init_packet, |
| 334 | NVSP_PROTOCOL_VERSION_1) == 0) { |
| 335 | net_device->nvsp_version = NVSP_PROTOCOL_VERSION_1; |
| 336 | } else { |
K. Y. Srinivasan | 0f48c72 | 2011-08-25 09:49:14 -0700 | [diff] [blame] | 337 | ret = -EPROTO; |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 338 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 339 | } |
Haiyang Zhang | f157e78 | 2011-12-15 13:45:16 -0800 | [diff] [blame] | 340 | |
| 341 | pr_debug("Negotiated NVSP version:%x\n", net_device->nvsp_version); |
| 342 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 343 | /* Send the ndis version */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 344 | memset(init_packet, 0, sizeof(struct nvsp_message)); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 345 | |
Haiyang Zhang | 1f5f3a7 | 2012-03-12 10:20:50 +0000 | [diff] [blame] | 346 | ndis_version = 0x00050001; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 347 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 348 | init_packet->hdr.msg_type = NVSP_MSG1_TYPE_SEND_NDIS_VER; |
| 349 | init_packet->msg.v1_msg. |
| 350 | send_ndis_ver.ndis_major_ver = |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 351 | (ndis_version & 0xFFFF0000) >> 16; |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 352 | init_packet->msg.v1_msg. |
| 353 | send_ndis_ver.ndis_minor_ver = |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 354 | ndis_version & 0xFFFF; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 355 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 356 | /* Send the init request */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 357 | ret = vmbus_sendpacket(device->channel, init_packet, |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 358 | sizeof(struct nvsp_message), |
| 359 | (unsigned long)init_packet, |
| 360 | VM_PKT_DATA_INBAND, 0); |
K. Y. Srinivasan | 0f48c72 | 2011-08-25 09:49:14 -0700 | [diff] [blame] | 361 | if (ret != 0) |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 362 | goto cleanup; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 363 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 364 | /* Post the big receive buffer to NetVSP */ |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 365 | ret = netvsc_init_recv_buf(device); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 366 | |
K. Y. Srinivasan | 0c3b7b2 | 2011-02-11 09:59:43 -0800 | [diff] [blame] | 367 | cleanup: |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 368 | return ret; |
| 369 | } |
| 370 | |
Haiyang Zhang | 648dc59 | 2011-04-21 12:30:47 -0700 | [diff] [blame] | 371 | static void netvsc_disconnect_vsp(struct netvsc_device *net_device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 372 | { |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 373 | netvsc_destroy_recv_buf(net_device); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 374 | } |
| 375 | |
Hank Janssen | 3e18951 | 2010-03-04 22:11:00 +0000 | [diff] [blame] | 376 | /* |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 377 | * netvsc_device_remove - Callback when the root bus device is removed |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 378 | */ |
K. Y. Srinivasan | 905620d | 2011-05-10 07:54:54 -0700 | [diff] [blame] | 379 | int netvsc_device_remove(struct hv_device *device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 380 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 381 | struct netvsc_device *net_device; |
| 382 | struct hv_netvsc_packet *netvsc_packet, *pos; |
K. Y. Srinivasan | c38b9c7 | 2011-08-27 11:31:12 -0700 | [diff] [blame] | 383 | unsigned long flags; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 384 | |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 385 | net_device = hv_get_drvdata(device); |
K. Y. Srinivasan | c38b9c7 | 2011-08-27 11:31:12 -0700 | [diff] [blame] | 386 | spin_lock_irqsave(&device->channel->inbound_lock, flags); |
| 387 | net_device->destroy = true; |
| 388 | spin_unlock_irqrestore(&device->channel->inbound_lock, flags); |
| 389 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 390 | /* Wait for all send completions */ |
Haiyang Zhang | dc5cd89 | 2012-06-04 06:42:38 +0000 | [diff] [blame^] | 391 | wait_event(net_device->wait_drain, |
| 392 | atomic_read(&net_device->num_outstanding_sends) == 0); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 393 | |
Haiyang Zhang | 648dc59 | 2011-04-21 12:30:47 -0700 | [diff] [blame] | 394 | netvsc_disconnect_vsp(net_device); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 395 | |
K. Y. Srinivasan | 3852409 | 2011-08-27 11:31:14 -0700 | [diff] [blame] | 396 | /* |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 397 | * Since we have already drained, we don't need to busy wait |
| 398 | * as was done in final_release_stor_device() |
| 399 | * Note that we cannot set the ext pointer to NULL until |
| 400 | * we have drained - to drain the outgoing packets, we need to |
| 401 | * allow incoming packets. |
K. Y. Srinivasan | 3852409 | 2011-08-27 11:31:14 -0700 | [diff] [blame] | 402 | */ |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 403 | |
| 404 | spin_lock_irqsave(&device->channel->inbound_lock, flags); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 405 | hv_set_drvdata(device, NULL); |
K. Y. Srinivasan | 9d88f33 | 2011-08-27 11:31:16 -0700 | [diff] [blame] | 406 | spin_unlock_irqrestore(&device->channel->inbound_lock, flags); |
K. Y. Srinivasan | 3852409 | 2011-08-27 11:31:14 -0700 | [diff] [blame] | 407 | |
K. Y. Srinivasan | 86c921a | 2011-09-13 10:59:54 -0700 | [diff] [blame] | 408 | /* |
| 409 | * At this point, no one should be accessing net_device |
| 410 | * except in here |
| 411 | */ |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 412 | dev_notice(&device->device, "net device safe to remove\n"); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 413 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 414 | /* Now, we can close the channel safely */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 415 | vmbus_close(device->channel); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 416 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 417 | /* Release all resources */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 418 | list_for_each_entry_safe(netvsc_packet, pos, |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 419 | &net_device->recv_pkt_list, list_ent) { |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 420 | list_del(&netvsc_packet->list_ent); |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 421 | kfree(netvsc_packet); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 422 | } |
| 423 | |
K. Y. Srinivasan | 356c465 | 2011-08-27 11:31:10 -0700 | [diff] [blame] | 424 | kfree(net_device); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 425 | return 0; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 426 | } |
| 427 | |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 428 | |
| 429 | #define RING_AVAIL_PERCENT_HIWATER 20 |
| 430 | #define RING_AVAIL_PERCENT_LOWATER 10 |
| 431 | |
| 432 | /* |
| 433 | * Get the percentage of available bytes to write in the ring. |
| 434 | * The return value is in range from 0 to 100. |
| 435 | */ |
| 436 | static inline u32 hv_ringbuf_avail_percent( |
| 437 | struct hv_ring_buffer_info *ring_info) |
| 438 | { |
| 439 | u32 avail_read, avail_write; |
| 440 | |
| 441 | hv_get_ringbuffer_availbytes(ring_info, &avail_read, &avail_write); |
| 442 | |
| 443 | return avail_write * 100 / ring_info->ring_datasize; |
| 444 | } |
| 445 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 446 | static void netvsc_send_completion(struct hv_device *device, |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 447 | struct vmpacket_descriptor *packet) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 448 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 449 | struct netvsc_device *net_device; |
| 450 | struct nvsp_message *nvsp_packet; |
| 451 | struct hv_netvsc_packet *nvsc_packet; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 452 | struct net_device *ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 453 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 454 | net_device = get_inbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 455 | if (!net_device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 456 | return; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 457 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 458 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 459 | nvsp_packet = (struct nvsp_message *)((unsigned long)packet + |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 460 | (packet->offset8 << 3)); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 461 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 462 | if ((nvsp_packet->hdr.msg_type == NVSP_MSG_TYPE_INIT_COMPLETE) || |
| 463 | (nvsp_packet->hdr.msg_type == |
| 464 | NVSP_MSG1_TYPE_SEND_RECV_BUF_COMPLETE) || |
| 465 | (nvsp_packet->hdr.msg_type == |
| 466 | NVSP_MSG1_TYPE_SEND_SEND_BUF_COMPLETE)) { |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 467 | /* Copy the response back */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 468 | memcpy(&net_device->channel_init_pkt, nvsp_packet, |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 469 | sizeof(struct nvsp_message)); |
K. Y. Srinivasan | 35abb21 | 2011-05-10 07:55:41 -0700 | [diff] [blame] | 470 | complete(&net_device->channel_init_wait); |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 471 | } else if (nvsp_packet->hdr.msg_type == |
| 472 | NVSP_MSG1_TYPE_SEND_RNDIS_PKT_COMPLETE) { |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 473 | int num_outstanding_sends; |
| 474 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 475 | /* Get the send context */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 476 | nvsc_packet = (struct hv_netvsc_packet *)(unsigned long) |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 477 | packet->trans_id; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 478 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 479 | /* Notify the layer above us */ |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 480 | nvsc_packet->completion.send.send_completion( |
| 481 | nvsc_packet->completion.send.send_completion_ctx); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 482 | |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 483 | num_outstanding_sends = |
| 484 | atomic_dec_return(&net_device->num_outstanding_sends); |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 485 | |
Haiyang Zhang | dc5cd89 | 2012-06-04 06:42:38 +0000 | [diff] [blame^] | 486 | if (net_device->destroy && num_outstanding_sends == 0) |
| 487 | wake_up(&net_device->wait_drain); |
| 488 | |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 489 | if (netif_queue_stopped(ndev) && !net_device->start_remove && |
| 490 | (hv_ringbuf_avail_percent(&device->channel->outbound) |
| 491 | > RING_AVAIL_PERCENT_HIWATER || |
| 492 | num_outstanding_sends < 1)) |
| 493 | netif_wake_queue(ndev); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 494 | } else { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 495 | netdev_err(ndev, "Unknown send completion packet type- " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 496 | "%d received!!\n", nvsp_packet->hdr.msg_type); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 497 | } |
| 498 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 499 | } |
| 500 | |
K. Y. Srinivasan | f9819f0 | 2011-05-12 19:34:49 -0700 | [diff] [blame] | 501 | int netvsc_send(struct hv_device *device, |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 502 | struct hv_netvsc_packet *packet) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 503 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 504 | struct netvsc_device *net_device; |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 505 | int ret = 0; |
Greg Kroah-Hartman | 223c1aa | 2009-08-28 16:20:53 -0700 | [diff] [blame] | 506 | struct nvsp_message sendMessage; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 507 | struct net_device *ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 508 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 509 | net_device = get_outbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 510 | if (!net_device) |
K. Y. Srinivasan | ff2bd69 | 2011-08-25 09:49:15 -0700 | [diff] [blame] | 511 | return -ENODEV; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 512 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 513 | |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 514 | sendMessage.hdr.msg_type = NVSP_MSG1_TYPE_SEND_RNDIS_PKT; |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 515 | if (packet->is_data_pkt) { |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 516 | /* 0 is RMC_DATA; */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 517 | sendMessage.msg.v1_msg.send_rndis_pkt.channel_type = 0; |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 518 | } else { |
| 519 | /* 1 is RMC_CONTROL; */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 520 | sendMessage.msg.v1_msg.send_rndis_pkt.channel_type = 1; |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 521 | } |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 522 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 523 | /* Not using send buffer section */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 524 | sendMessage.msg.v1_msg.send_rndis_pkt.send_buf_section_index = |
| 525 | 0xFFFFFFFF; |
| 526 | sendMessage.msg.v1_msg.send_rndis_pkt.send_buf_section_size = 0; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 527 | |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 528 | if (packet->page_buf_cnt) { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 529 | ret = vmbus_sendpacket_pagebuffer(device->channel, |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 530 | packet->page_buf, |
| 531 | packet->page_buf_cnt, |
Greg Kroah-Hartman | ff3f8ee | 2010-10-21 09:32:46 -0700 | [diff] [blame] | 532 | &sendMessage, |
| 533 | sizeof(struct nvsp_message), |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 534 | (unsigned long)packet); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 535 | } else { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 536 | ret = vmbus_sendpacket(device->channel, &sendMessage, |
Haiyang Zhang | e4d59ac | 2011-06-17 07:58:04 -0700 | [diff] [blame] | 537 | sizeof(struct nvsp_message), |
| 538 | (unsigned long)packet, |
| 539 | VM_PKT_DATA_INBAND, |
| 540 | VMBUS_DATA_PACKET_FLAG_COMPLETION_REQUESTED); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 541 | |
| 542 | } |
| 543 | |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 544 | if (ret == 0) { |
| 545 | atomic_inc(&net_device->num_outstanding_sends); |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 546 | if (hv_ringbuf_avail_percent(&device->channel->outbound) < |
| 547 | RING_AVAIL_PERCENT_LOWATER) { |
| 548 | netif_stop_queue(ndev); |
| 549 | if (atomic_read(&net_device-> |
| 550 | num_outstanding_sends) < 1) |
| 551 | netif_wake_queue(ndev); |
| 552 | } |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 553 | } else if (ret == -EAGAIN) { |
| 554 | netif_stop_queue(ndev); |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 555 | if (atomic_read(&net_device->num_outstanding_sends) < 1) { |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 556 | netif_wake_queue(ndev); |
Haiyang Zhang | 33be96e | 2012-03-27 13:20:45 +0000 | [diff] [blame] | 557 | ret = -ENOSPC; |
| 558 | } |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 559 | } else { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 560 | netdev_err(ndev, "Unable to send packet %p ret %d\n", |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 561 | packet, ret); |
Haiyang Zhang | 1d06825 | 2011-12-02 11:56:25 -0800 | [diff] [blame] | 562 | } |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 563 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 564 | return ret; |
| 565 | } |
| 566 | |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 567 | static void netvsc_send_recv_completion(struct hv_device *device, |
| 568 | u64 transaction_id) |
| 569 | { |
| 570 | struct nvsp_message recvcompMessage; |
| 571 | int retries = 0; |
| 572 | int ret; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 573 | struct net_device *ndev; |
| 574 | struct netvsc_device *net_device = hv_get_drvdata(device); |
| 575 | |
| 576 | ndev = net_device->ndev; |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 577 | |
| 578 | recvcompMessage.hdr.msg_type = |
| 579 | NVSP_MSG1_TYPE_SEND_RNDIS_PKT_COMPLETE; |
| 580 | |
| 581 | /* FIXME: Pass in the status */ |
| 582 | recvcompMessage.msg.v1_msg.send_rndis_pkt_complete.status = |
| 583 | NVSP_STAT_SUCCESS; |
| 584 | |
| 585 | retry_send_cmplt: |
| 586 | /* Send the completion */ |
| 587 | ret = vmbus_sendpacket(device->channel, &recvcompMessage, |
| 588 | sizeof(struct nvsp_message), transaction_id, |
| 589 | VM_PKT_COMP, 0); |
| 590 | if (ret == 0) { |
| 591 | /* success */ |
| 592 | /* no-op */ |
K. Y. Srinivasan | d2598f0 | 2011-08-25 09:48:58 -0700 | [diff] [blame] | 593 | } else if (ret == -EAGAIN) { |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 594 | /* no more room...wait a bit and attempt to retry 3 times */ |
| 595 | retries++; |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 596 | netdev_err(ndev, "unable to send receive completion pkt" |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 597 | " (tid %llx)...retrying %d\n", transaction_id, retries); |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 598 | |
| 599 | if (retries < 4) { |
| 600 | udelay(100); |
| 601 | goto retry_send_cmplt; |
| 602 | } else { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 603 | netdev_err(ndev, "unable to send receive " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 604 | "completion pkt (tid %llx)...give up retrying\n", |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 605 | transaction_id); |
| 606 | } |
| 607 | } else { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 608 | netdev_err(ndev, "unable to send receive " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 609 | "completion pkt - %llx\n", transaction_id); |
Haiyang Zhang | 5fa9d3c | 2011-04-21 12:30:42 -0700 | [diff] [blame] | 610 | } |
| 611 | } |
| 612 | |
Haiyang Zhang | 5799115 | 2011-04-21 12:30:41 -0700 | [diff] [blame] | 613 | /* Send a receive completion packet to RNDIS device (ie NetVsp) */ |
| 614 | static void netvsc_receive_completion(void *context) |
| 615 | { |
| 616 | struct hv_netvsc_packet *packet = context; |
| 617 | struct hv_device *device = (struct hv_device *)packet->device; |
| 618 | struct netvsc_device *net_device; |
| 619 | u64 transaction_id = 0; |
| 620 | bool fsend_receive_comp = false; |
| 621 | unsigned long flags; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 622 | struct net_device *ndev; |
Haiyang Zhang | 5799115 | 2011-04-21 12:30:41 -0700 | [diff] [blame] | 623 | |
| 624 | /* |
| 625 | * Even though it seems logical to do a GetOutboundNetDevice() here to |
| 626 | * send out receive completion, we are using GetInboundNetDevice() |
| 627 | * since we may have disable outbound traffic already. |
| 628 | */ |
| 629 | net_device = get_inbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 630 | if (!net_device) |
Haiyang Zhang | 5799115 | 2011-04-21 12:30:41 -0700 | [diff] [blame] | 631 | return; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 632 | ndev = net_device->ndev; |
Haiyang Zhang | 5799115 | 2011-04-21 12:30:41 -0700 | [diff] [blame] | 633 | |
| 634 | /* Overloading use of the lock. */ |
| 635 | spin_lock_irqsave(&net_device->recv_pkt_list_lock, flags); |
| 636 | |
| 637 | packet->xfer_page_pkt->count--; |
| 638 | |
| 639 | /* |
| 640 | * Last one in the line that represent 1 xfer page packet. |
| 641 | * Return the xfer page packet itself to the freelist |
| 642 | */ |
| 643 | if (packet->xfer_page_pkt->count == 0) { |
| 644 | fsend_receive_comp = true; |
| 645 | transaction_id = packet->completion.recv.recv_completion_tid; |
| 646 | list_add_tail(&packet->xfer_page_pkt->list_ent, |
| 647 | &net_device->recv_pkt_list); |
| 648 | |
| 649 | } |
| 650 | |
| 651 | /* Put the packet back */ |
| 652 | list_add_tail(&packet->list_ent, &net_device->recv_pkt_list); |
| 653 | spin_unlock_irqrestore(&net_device->recv_pkt_list_lock, flags); |
| 654 | |
| 655 | /* Send a receive completion for the xfer page packet */ |
| 656 | if (fsend_receive_comp) |
| 657 | netvsc_send_recv_completion(device, transaction_id); |
| 658 | |
Haiyang Zhang | 5799115 | 2011-04-21 12:30:41 -0700 | [diff] [blame] | 659 | } |
| 660 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 661 | static void netvsc_receive(struct hv_device *device, |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 662 | struct vmpacket_descriptor *packet) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 663 | { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 664 | struct netvsc_device *net_device; |
| 665 | struct vmtransfer_page_packet_header *vmxferpage_packet; |
| 666 | struct nvsp_message *nvsp_packet; |
| 667 | struct hv_netvsc_packet *netvsc_packet = NULL; |
Greg Kroah-Hartman | 7e23a6e | 2009-08-27 15:58:15 -0700 | [diff] [blame] | 668 | /* struct netvsc_driver *netvscDriver; */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 669 | struct xferpage_packet *xferpage_packet = NULL; |
Haiyang Zhang | 4532634 | 2011-12-15 13:45:15 -0800 | [diff] [blame] | 670 | int i; |
| 671 | int count = 0; |
Greg Kroah-Hartman | 6436873 | 2009-07-15 14:56:15 -0700 | [diff] [blame] | 672 | unsigned long flags; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 673 | struct net_device *ndev; |
K. Y. Srinivasan | 779b4d1 | 2011-04-26 09:20:22 -0700 | [diff] [blame] | 674 | |
Bill Pemberton | d29274e | 2009-09-11 21:46:43 -0400 | [diff] [blame] | 675 | LIST_HEAD(listHead); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 676 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 677 | net_device = get_inbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 678 | if (!net_device) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 679 | return; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 680 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 681 | |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 682 | /* |
| 683 | * All inbound packets other than send completion should be xfer page |
| 684 | * packet |
| 685 | */ |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 686 | if (packet->type != VM_PKT_DATA_USING_XFER_PAGES) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 687 | netdev_err(ndev, "Unknown packet type received - %d\n", |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 688 | packet->type); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 689 | return; |
| 690 | } |
| 691 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 692 | nvsp_packet = (struct nvsp_message *)((unsigned long)packet + |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 693 | (packet->offset8 << 3)); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 694 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 695 | /* Make sure this is a valid nvsp packet */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 696 | if (nvsp_packet->hdr.msg_type != |
| 697 | NVSP_MSG1_TYPE_SEND_RNDIS_PKT) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 698 | netdev_err(ndev, "Unknown nvsp packet type received-" |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 699 | " %d\n", nvsp_packet->hdr.msg_type); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 700 | return; |
| 701 | } |
| 702 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 703 | vmxferpage_packet = (struct vmtransfer_page_packet_header *)packet; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 704 | |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 705 | if (vmxferpage_packet->xfer_pageset_id != NETVSC_RECEIVE_BUFFER_ID) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 706 | netdev_err(ndev, "Invalid xfer page set id - " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 707 | "expecting %x got %x\n", NETVSC_RECEIVE_BUFFER_ID, |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 708 | vmxferpage_packet->xfer_pageset_id); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 709 | return; |
| 710 | } |
| 711 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 712 | /* |
| 713 | * Grab free packets (range count + 1) to represent this xfer |
| 714 | * page packet. +1 to represent the xfer page packet itself. |
| 715 | * We grab it here so that we know exactly how many we can |
| 716 | * fulfil |
| 717 | */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 718 | spin_lock_irqsave(&net_device->recv_pkt_list_lock, flags); |
| 719 | while (!list_empty(&net_device->recv_pkt_list)) { |
| 720 | list_move_tail(net_device->recv_pkt_list.next, &listHead); |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 721 | if (++count == vmxferpage_packet->range_cnt + 1) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 722 | break; |
| 723 | } |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 724 | spin_unlock_irqrestore(&net_device->recv_pkt_list_lock, flags); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 725 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 726 | /* |
| 727 | * We need at least 2 netvsc pkts (1 to represent the xfer |
| 728 | * page and at least 1 for the range) i.e. we can handled |
| 729 | * some of the xfer page packet ranges... |
| 730 | */ |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 731 | if (count < 2) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 732 | netdev_err(ndev, "Got only %d netvsc pkt...needed " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 733 | "%d pkts. Dropping this xfer page packet completely!\n", |
Hank Janssen | eb335bc | 2011-03-29 13:58:48 -0700 | [diff] [blame] | 734 | count, vmxferpage_packet->range_cnt + 1); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 735 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 736 | /* Return it to the freelist */ |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 737 | spin_lock_irqsave(&net_device->recv_pkt_list_lock, flags); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 738 | for (i = count; i != 0; i--) { |
Milan Dadok | 92ec089 | 2009-10-28 23:23:37 +0100 | [diff] [blame] | 739 | list_move_tail(listHead.next, |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 740 | &net_device->recv_pkt_list); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 741 | } |
Haiyang Zhang | 53d21fd | 2010-12-10 12:03:59 -0800 | [diff] [blame] | 742 | spin_unlock_irqrestore(&net_device->recv_pkt_list_lock, |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 743 | flags); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 744 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 745 | netvsc_send_recv_completion(device, |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 746 | vmxferpage_packet->d.trans_id); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 747 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 748 | return; |
| 749 | } |
| 750 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 751 | /* Remove the 1st packet to represent the xfer page packet itself */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 752 | xferpage_packet = (struct xferpage_packet *)listHead.next; |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 753 | list_del(&xferpage_packet->list_ent); |
Bill Pemberton | d29274e | 2009-09-11 21:46:43 -0400 | [diff] [blame] | 754 | |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 755 | /* This is how much we can satisfy */ |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 756 | xferpage_packet->count = count - 1; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 757 | |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 758 | if (xferpage_packet->count != vmxferpage_packet->range_cnt) { |
Haiyang Zhang | 8bff33a | 2011-09-01 12:19:48 -0700 | [diff] [blame] | 759 | netdev_err(ndev, "Needed %d netvsc pkts to satisfy " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 760 | "this xfer page...got %d\n", |
Hank Janssen | eb335bc | 2011-03-29 13:58:48 -0700 | [diff] [blame] | 761 | vmxferpage_packet->range_cnt, xferpage_packet->count); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 762 | } |
| 763 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 764 | /* Each range represents 1 RNDIS pkt that contains 1 ethernet frame */ |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 765 | for (i = 0; i < (count - 1); i++) { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 766 | netvsc_packet = (struct hv_netvsc_packet *)listHead.next; |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 767 | list_del(&netvsc_packet->list_ent); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 768 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 769 | /* Initialize the netvsc packet */ |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 770 | netvsc_packet->xfer_page_pkt = xferpage_packet; |
| 771 | netvsc_packet->completion.recv.recv_completion = |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 772 | netvsc_receive_completion; |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 773 | netvsc_packet->completion.recv.recv_completion_ctx = |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 774 | netvsc_packet; |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 775 | netvsc_packet->device = device; |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 776 | /* Save this so that we can send it back */ |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 777 | netvsc_packet->completion.recv.recv_completion_tid = |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 778 | vmxferpage_packet->d.trans_id; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 779 | |
Haiyang Zhang | 4532634 | 2011-12-15 13:45:15 -0800 | [diff] [blame] | 780 | netvsc_packet->data = (void *)((unsigned long)net_device-> |
| 781 | recv_buf + vmxferpage_packet->ranges[i].byte_offset); |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 782 | netvsc_packet->total_data_buflen = |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 783 | vmxferpage_packet->ranges[i].byte_count; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 784 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 785 | /* Pass it to the upper layer */ |
K. Y. Srinivasan | ac6f785 | 2011-05-12 19:34:58 -0700 | [diff] [blame] | 786 | rndis_filter_receive(device, netvsc_packet); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 787 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 788 | netvsc_receive_completion(netvsc_packet-> |
Haiyang Zhang | 72a2f5b | 2010-12-10 12:03:58 -0800 | [diff] [blame] | 789 | completion.recv.recv_completion_ctx); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 790 | } |
| 791 | |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 792 | } |
| 793 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 794 | static void netvsc_channel_cb(void *context) |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 795 | { |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 796 | int ret; |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 797 | struct hv_device *device = context; |
| 798 | struct netvsc_device *net_device; |
| 799 | u32 bytes_recvd; |
| 800 | u64 request_id; |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 801 | unsigned char *packet; |
Greg Kroah-Hartman | 8dc0a06 | 2009-08-27 16:02:36 -0700 | [diff] [blame] | 802 | struct vmpacket_descriptor *desc; |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 803 | unsigned char *buffer; |
| 804 | int bufferlen = NETVSC_PACKET_SIZE; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 805 | struct net_device *ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 806 | |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 807 | packet = kzalloc(NETVSC_PACKET_SIZE * sizeof(unsigned char), |
Timo Teräs | d70c673 | 2010-12-17 11:40:24 +0200 | [diff] [blame] | 808 | GFP_ATOMIC); |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 809 | if (!packet) |
| 810 | return; |
| 811 | buffer = packet; |
| 812 | |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 813 | net_device = get_inbound_net_device(device); |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 814 | if (!net_device) |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 815 | goto out; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 816 | ndev = net_device->ndev; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 817 | |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 818 | do { |
Greg Kroah-Hartman | 9f63006 | 2010-10-21 09:09:48 -0700 | [diff] [blame] | 819 | ret = vmbus_recvpacket_raw(device->channel, buffer, bufferlen, |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 820 | &bytes_recvd, &request_id); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 821 | if (ret == 0) { |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 822 | if (bytes_recvd > 0) { |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 823 | desc = (struct vmpacket_descriptor *)buffer; |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 824 | switch (desc->type) { |
| 825 | case VM_PKT_COMP: |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 826 | netvsc_send_completion(device, desc); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 827 | break; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 828 | |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 829 | case VM_PKT_DATA_USING_XFER_PAGES: |
Haiyang Zhang | 5a71ae3 | 2010-12-10 12:03:55 -0800 | [diff] [blame] | 830 | netvsc_receive(device, desc); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 831 | break; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 832 | |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 833 | default: |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 834 | netdev_err(ndev, |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 835 | "unhandled packet type %d, " |
| 836 | "tid %llx len %d\n", |
Haiyang Zhang | 415f228 | 2011-01-26 12:12:13 -0800 | [diff] [blame] | 837 | desc->type, request_id, |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 838 | bytes_recvd); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 839 | break; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 840 | } |
| 841 | |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 842 | /* reset */ |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 843 | if (bufferlen > NETVSC_PACKET_SIZE) { |
Greg Kroah-Hartman | 8c69f52 | 2009-07-15 12:48:29 -0700 | [diff] [blame] | 844 | kfree(buffer); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 845 | buffer = packet; |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 846 | bufferlen = NETVSC_PACKET_SIZE; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 847 | } |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 848 | } else { |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 849 | /* reset */ |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 850 | if (bufferlen > NETVSC_PACKET_SIZE) { |
Greg Kroah-Hartman | 8c69f52 | 2009-07-15 12:48:29 -0700 | [diff] [blame] | 851 | kfree(buffer); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 852 | buffer = packet; |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 853 | bufferlen = NETVSC_PACKET_SIZE; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 854 | } |
| 855 | |
| 856 | break; |
| 857 | } |
K. Y. Srinivasan | 3d5cad9 | 2011-08-25 09:48:59 -0700 | [diff] [blame] | 858 | } else if (ret == -ENOBUFS) { |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 859 | /* Handle large packet */ |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 860 | buffer = kmalloc(bytes_recvd, GFP_ATOMIC); |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 861 | if (buffer == NULL) { |
Bill Pemberton | 454f18a | 2009-07-27 16:47:24 -0400 | [diff] [blame] | 862 | /* Try again next time around */ |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 863 | netdev_err(ndev, |
Greg Kroah-Hartman | 21a80820 | 2009-09-02 10:33:05 -0700 | [diff] [blame] | 864 | "unable to allocate buffer of size " |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 865 | "(%d)!!\n", bytes_recvd); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 866 | break; |
| 867 | } |
| 868 | |
Haiyang Zhang | 85799a3 | 2010-12-10 12:03:54 -0800 | [diff] [blame] | 869 | bufferlen = bytes_recvd; |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 870 | } |
| 871 | } while (1); |
| 872 | |
Bill Pemberton | c6fcf0b | 2010-04-27 16:23:47 -0400 | [diff] [blame] | 873 | out: |
| 874 | kfree(buffer); |
Hank Janssen | fceaf24 | 2009-07-13 15:34:54 -0700 | [diff] [blame] | 875 | return; |
| 876 | } |
Haiyang Zhang | af24ce4 | 2011-04-21 12:30:40 -0700 | [diff] [blame] | 877 | |
| 878 | /* |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 879 | * netvsc_device_add - Callback when the device belonging to this |
| 880 | * driver is added |
| 881 | */ |
K. Y. Srinivasan | 7bd23a4 | 2011-05-10 07:54:53 -0700 | [diff] [blame] | 882 | int netvsc_device_add(struct hv_device *device, void *additional_info) |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 883 | { |
| 884 | int ret = 0; |
| 885 | int i; |
K. Y. Srinivasan | aae2398 | 2011-05-12 19:35:05 -0700 | [diff] [blame] | 886 | int ring_size = |
| 887 | ((struct netvsc_device_info *)additional_info)->ring_size; |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 888 | struct netvsc_device *net_device; |
| 889 | struct hv_netvsc_packet *packet, *pos; |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 890 | struct net_device *ndev; |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 891 | |
| 892 | net_device = alloc_net_device(device); |
| 893 | if (!net_device) { |
K. Y. Srinivasan | ace163a | 2011-08-25 09:49:16 -0700 | [diff] [blame] | 894 | ret = -ENOMEM; |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 895 | goto cleanup; |
| 896 | } |
| 897 | |
K. Y. Srinivasan | 2ddd5e5 | 2011-09-13 10:59:49 -0700 | [diff] [blame] | 898 | /* |
| 899 | * Coming into this function, struct net_device * is |
| 900 | * registered as the driver private data. |
| 901 | * In alloc_net_device(), we register struct netvsc_device * |
| 902 | * as the driver private data and stash away struct net_device * |
| 903 | * in struct netvsc_device *. |
| 904 | */ |
| 905 | ndev = net_device->ndev; |
| 906 | |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 907 | /* Initialize the NetVSC channel extension */ |
| 908 | net_device->recv_buf_size = NETVSC_RECEIVE_BUFFER_SIZE; |
| 909 | spin_lock_init(&net_device->recv_pkt_list_lock); |
| 910 | |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 911 | INIT_LIST_HEAD(&net_device->recv_pkt_list); |
| 912 | |
| 913 | for (i = 0; i < NETVSC_RECEIVE_PACKETLIST_COUNT; i++) { |
| 914 | packet = kzalloc(sizeof(struct hv_netvsc_packet) + |
| 915 | (NETVSC_RECEIVE_SG_COUNT * |
| 916 | sizeof(struct hv_page_buffer)), GFP_KERNEL); |
| 917 | if (!packet) |
| 918 | break; |
| 919 | |
| 920 | list_add_tail(&packet->list_ent, |
| 921 | &net_device->recv_pkt_list); |
| 922 | } |
K. Y. Srinivasan | 35abb21 | 2011-05-10 07:55:41 -0700 | [diff] [blame] | 923 | init_completion(&net_device->channel_init_wait); |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 924 | |
| 925 | /* Open the channel */ |
K. Y. Srinivasan | aae2398 | 2011-05-12 19:35:05 -0700 | [diff] [blame] | 926 | ret = vmbus_open(device->channel, ring_size * PAGE_SIZE, |
| 927 | ring_size * PAGE_SIZE, NULL, 0, |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 928 | netvsc_channel_cb, device); |
| 929 | |
| 930 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 931 | netdev_err(ndev, "unable to open channel: %d\n", ret); |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 932 | goto cleanup; |
| 933 | } |
| 934 | |
| 935 | /* Channel is opened */ |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 936 | pr_info("hv_netvsc channel opened successfully\n"); |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 937 | |
| 938 | /* Connect with the NetVsp */ |
| 939 | ret = netvsc_connect_vsp(device); |
| 940 | if (ret != 0) { |
Haiyang Zhang | d987115 | 2011-09-01 12:19:41 -0700 | [diff] [blame] | 941 | netdev_err(ndev, |
Haiyang Zhang | c909ebb | 2011-09-01 12:19:40 -0700 | [diff] [blame] | 942 | "unable to connect to NetVSP - %d\n", ret); |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 943 | goto close; |
| 944 | } |
| 945 | |
| 946 | return ret; |
| 947 | |
| 948 | close: |
| 949 | /* Now, we can close the channel safely */ |
| 950 | vmbus_close(device->channel); |
| 951 | |
| 952 | cleanup: |
| 953 | |
| 954 | if (net_device) { |
| 955 | list_for_each_entry_safe(packet, pos, |
| 956 | &net_device->recv_pkt_list, |
| 957 | list_ent) { |
| 958 | list_del(&packet->list_ent); |
| 959 | kfree(packet); |
| 960 | } |
| 961 | |
K. Y. Srinivasan | 356c465 | 2011-08-27 11:31:10 -0700 | [diff] [blame] | 962 | kfree(net_device); |
Haiyang Zhang | b637e02 | 2011-04-21 12:30:45 -0700 | [diff] [blame] | 963 | } |
| 964 | |
| 965 | return ret; |
| 966 | } |