143aa3132SThomas Gleixner // SPDX-License-Identifier: GPL-2.0-only
214b50f80SVitaly Kuznetsov /*
314b50f80SVitaly Kuznetsov  * Kernel/userspace transport abstraction for Hyper-V util driver.
414b50f80SVitaly Kuznetsov  *
514b50f80SVitaly Kuznetsov  * Copyright (C) 2015, Vitaly Kuznetsov <vkuznets@redhat.com>
614b50f80SVitaly Kuznetsov  */
714b50f80SVitaly Kuznetsov 
814b50f80SVitaly Kuznetsov #include <linux/slab.h>
914b50f80SVitaly Kuznetsov #include <linux/fs.h>
1014b50f80SVitaly Kuznetsov #include <linux/poll.h>
1114b50f80SVitaly Kuznetsov 
1214b50f80SVitaly Kuznetsov #include "hyperv_vmbus.h"
1314b50f80SVitaly Kuznetsov #include "hv_utils_transport.h"
1414b50f80SVitaly Kuznetsov 
1514b50f80SVitaly Kuznetsov static DEFINE_SPINLOCK(hvt_list_lock);
16*ffc58bc4SCai Huoqing static LIST_HEAD(hvt_list);
1714b50f80SVitaly Kuznetsov 
hvt_reset(struct hvutil_transport * hvt)1814b50f80SVitaly Kuznetsov static void hvt_reset(struct hvutil_transport *hvt)
1914b50f80SVitaly Kuznetsov {
2014b50f80SVitaly Kuznetsov 	kfree(hvt->outmsg);
2114b50f80SVitaly Kuznetsov 	hvt->outmsg = NULL;
2214b50f80SVitaly Kuznetsov 	hvt->outmsg_len = 0;
2314b50f80SVitaly Kuznetsov 	if (hvt->on_reset)
2414b50f80SVitaly Kuznetsov 		hvt->on_reset();
2514b50f80SVitaly Kuznetsov }
2614b50f80SVitaly Kuznetsov 
hvt_op_read(struct file * file,char __user * buf,size_t count,loff_t * ppos)2714b50f80SVitaly Kuznetsov static ssize_t hvt_op_read(struct file *file, char __user *buf,
2814b50f80SVitaly Kuznetsov 			   size_t count, loff_t *ppos)
2914b50f80SVitaly Kuznetsov {
3014b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
3114b50f80SVitaly Kuznetsov 	int ret;
3214b50f80SVitaly Kuznetsov 
3314b50f80SVitaly Kuznetsov 	hvt = container_of(file->f_op, struct hvutil_transport, fops);
3414b50f80SVitaly Kuznetsov 
35a1502566SVitaly Kuznetsov 	if (wait_event_interruptible(hvt->outmsg_q, hvt->outmsg_len > 0 ||
36a1502566SVitaly Kuznetsov 				     hvt->mode != HVUTIL_TRANSPORT_CHARDEV))
3714b50f80SVitaly Kuznetsov 		return -EINTR;
3814b50f80SVitaly Kuznetsov 
39a72f3a4cSVitaly Kuznetsov 	mutex_lock(&hvt->lock);
40a1502566SVitaly Kuznetsov 
41a1502566SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_DESTROY) {
42a1502566SVitaly Kuznetsov 		ret = -EBADF;
43a1502566SVitaly Kuznetsov 		goto out_unlock;
44a1502566SVitaly Kuznetsov 	}
45a1502566SVitaly Kuznetsov 
4614b50f80SVitaly Kuznetsov 	if (!hvt->outmsg) {
4714b50f80SVitaly Kuznetsov 		ret = -EAGAIN;
4814b50f80SVitaly Kuznetsov 		goto out_unlock;
4914b50f80SVitaly Kuznetsov 	}
5014b50f80SVitaly Kuznetsov 
5114b50f80SVitaly Kuznetsov 	if (count < hvt->outmsg_len) {
5214b50f80SVitaly Kuznetsov 		ret = -EINVAL;
5314b50f80SVitaly Kuznetsov 		goto out_unlock;
5414b50f80SVitaly Kuznetsov 	}
5514b50f80SVitaly Kuznetsov 
5614b50f80SVitaly Kuznetsov 	if (!copy_to_user(buf, hvt->outmsg, hvt->outmsg_len))
5714b50f80SVitaly Kuznetsov 		ret = hvt->outmsg_len;
5814b50f80SVitaly Kuznetsov 	else
5914b50f80SVitaly Kuznetsov 		ret = -EFAULT;
6014b50f80SVitaly Kuznetsov 
6114b50f80SVitaly Kuznetsov 	kfree(hvt->outmsg);
6214b50f80SVitaly Kuznetsov 	hvt->outmsg = NULL;
6314b50f80SVitaly Kuznetsov 	hvt->outmsg_len = 0;
6414b50f80SVitaly Kuznetsov 
65e0fa3e5eSVitaly Kuznetsov 	if (hvt->on_read)
66e0fa3e5eSVitaly Kuznetsov 		hvt->on_read();
67e0fa3e5eSVitaly Kuznetsov 	hvt->on_read = NULL;
68e0fa3e5eSVitaly Kuznetsov 
6914b50f80SVitaly Kuznetsov out_unlock:
70a72f3a4cSVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
7114b50f80SVitaly Kuznetsov 	return ret;
7214b50f80SVitaly Kuznetsov }
7314b50f80SVitaly Kuznetsov 
hvt_op_write(struct file * file,const char __user * buf,size_t count,loff_t * ppos)7414b50f80SVitaly Kuznetsov static ssize_t hvt_op_write(struct file *file, const char __user *buf,
7514b50f80SVitaly Kuznetsov 			    size_t count, loff_t *ppos)
7614b50f80SVitaly Kuznetsov {
7714b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
7814b50f80SVitaly Kuznetsov 	u8 *inmsg;
791f75338bSVitaly Kuznetsov 	int ret;
8014b50f80SVitaly Kuznetsov 
8114b50f80SVitaly Kuznetsov 	hvt = container_of(file->f_op, struct hvutil_transport, fops);
8214b50f80SVitaly Kuznetsov 
83b0035964SOlaf Hering 	inmsg = memdup_user(buf, count);
84b0035964SOlaf Hering 	if (IS_ERR(inmsg))
85b0035964SOlaf Hering 		return PTR_ERR(inmsg);
86b0035964SOlaf Hering 
87a1502566SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_DESTROY)
88a1502566SVitaly Kuznetsov 		ret = -EBADF;
89a1502566SVitaly Kuznetsov 	else
901f75338bSVitaly Kuznetsov 		ret = hvt->on_msg(inmsg, count);
911f75338bSVitaly Kuznetsov 
9214b50f80SVitaly Kuznetsov 	kfree(inmsg);
9314b50f80SVitaly Kuznetsov 
941f75338bSVitaly Kuznetsov 	return ret ? ret : count;
9514b50f80SVitaly Kuznetsov }
9614b50f80SVitaly Kuznetsov 
hvt_op_poll(struct file * file,poll_table * wait)97afc9a42bSAl Viro static __poll_t hvt_op_poll(struct file *file, poll_table *wait)
9814b50f80SVitaly Kuznetsov {
9914b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
10014b50f80SVitaly Kuznetsov 
10114b50f80SVitaly Kuznetsov 	hvt = container_of(file->f_op, struct hvutil_transport, fops);
10214b50f80SVitaly Kuznetsov 
10314b50f80SVitaly Kuznetsov 	poll_wait(file, &hvt->outmsg_q, wait);
104a1502566SVitaly Kuznetsov 
105a1502566SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_DESTROY)
106a9a08845SLinus Torvalds 		return EPOLLERR | EPOLLHUP;
107a1502566SVitaly Kuznetsov 
10814b50f80SVitaly Kuznetsov 	if (hvt->outmsg_len > 0)
109a9a08845SLinus Torvalds 		return EPOLLIN | EPOLLRDNORM;
11014b50f80SVitaly Kuznetsov 
11114b50f80SVitaly Kuznetsov 	return 0;
11214b50f80SVitaly Kuznetsov }
11314b50f80SVitaly Kuznetsov 
hvt_op_open(struct inode * inode,struct file * file)11414b50f80SVitaly Kuznetsov static int hvt_op_open(struct inode *inode, struct file *file)
11514b50f80SVitaly Kuznetsov {
11614b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
117a1502566SVitaly Kuznetsov 	int ret = 0;
118a1502566SVitaly Kuznetsov 	bool issue_reset = false;
11914b50f80SVitaly Kuznetsov 
12014b50f80SVitaly Kuznetsov 	hvt = container_of(file->f_op, struct hvutil_transport, fops);
12114b50f80SVitaly Kuznetsov 
122a1502566SVitaly Kuznetsov 	mutex_lock(&hvt->lock);
123a1502566SVitaly Kuznetsov 
124a1502566SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_DESTROY) {
125a1502566SVitaly Kuznetsov 		ret = -EBADF;
126a1502566SVitaly Kuznetsov 	} else if (hvt->mode == HVUTIL_TRANSPORT_INIT) {
12714b50f80SVitaly Kuznetsov 		/*
128a1502566SVitaly Kuznetsov 		 * Switching to CHARDEV mode. We switch bach to INIT when
129a1502566SVitaly Kuznetsov 		 * device gets released.
13014b50f80SVitaly Kuznetsov 		 */
13114b50f80SVitaly Kuznetsov 		hvt->mode = HVUTIL_TRANSPORT_CHARDEV;
132a1502566SVitaly Kuznetsov 	}
13314b50f80SVitaly Kuznetsov 	else if (hvt->mode == HVUTIL_TRANSPORT_NETLINK) {
13414b50f80SVitaly Kuznetsov 		/*
13514b50f80SVitaly Kuznetsov 		 * We're switching from netlink communication to using char
13614b50f80SVitaly Kuznetsov 		 * device. Issue the reset first.
13714b50f80SVitaly Kuznetsov 		 */
138a1502566SVitaly Kuznetsov 		issue_reset = true;
13914b50f80SVitaly Kuznetsov 		hvt->mode = HVUTIL_TRANSPORT_CHARDEV;
140a1502566SVitaly Kuznetsov 	} else {
141a1502566SVitaly Kuznetsov 		ret = -EBUSY;
142a1502566SVitaly Kuznetsov 	}
14314b50f80SVitaly Kuznetsov 
144a1502566SVitaly Kuznetsov 	if (issue_reset)
145a1502566SVitaly Kuznetsov 		hvt_reset(hvt);
146a1502566SVitaly Kuznetsov 
147a1502566SVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
148a1502566SVitaly Kuznetsov 
149a1502566SVitaly Kuznetsov 	return ret;
15014b50f80SVitaly Kuznetsov }
15114b50f80SVitaly Kuznetsov 
hvt_transport_free(struct hvutil_transport * hvt)1529420098aSVitaly Kuznetsov static void hvt_transport_free(struct hvutil_transport *hvt)
1539420098aSVitaly Kuznetsov {
1549420098aSVitaly Kuznetsov 	misc_deregister(&hvt->mdev);
1559420098aSVitaly Kuznetsov 	kfree(hvt->outmsg);
1569420098aSVitaly Kuznetsov 	kfree(hvt);
1579420098aSVitaly Kuznetsov }
1589420098aSVitaly Kuznetsov 
hvt_op_release(struct inode * inode,struct file * file)15914b50f80SVitaly Kuznetsov static int hvt_op_release(struct inode *inode, struct file *file)
16014b50f80SVitaly Kuznetsov {
16114b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
1629420098aSVitaly Kuznetsov 	int mode_old;
16314b50f80SVitaly Kuznetsov 
16414b50f80SVitaly Kuznetsov 	hvt = container_of(file->f_op, struct hvutil_transport, fops);
16514b50f80SVitaly Kuznetsov 
166a1502566SVitaly Kuznetsov 	mutex_lock(&hvt->lock);
1679420098aSVitaly Kuznetsov 	mode_old = hvt->mode;
168a1502566SVitaly Kuznetsov 	if (hvt->mode != HVUTIL_TRANSPORT_DESTROY)
16914b50f80SVitaly Kuznetsov 		hvt->mode = HVUTIL_TRANSPORT_INIT;
17014b50f80SVitaly Kuznetsov 	/*
17114b50f80SVitaly Kuznetsov 	 * Cleanup message buffers to avoid spurious messages when the daemon
17214b50f80SVitaly Kuznetsov 	 * connects back.
17314b50f80SVitaly Kuznetsov 	 */
17414b50f80SVitaly Kuznetsov 	hvt_reset(hvt);
17514b50f80SVitaly Kuznetsov 
1769420098aSVitaly Kuznetsov 	if (mode_old == HVUTIL_TRANSPORT_DESTROY)
177e9c18ae6SVitaly Kuznetsov 		complete(&hvt->release);
178e9c18ae6SVitaly Kuznetsov 
179e9c18ae6SVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
1809420098aSVitaly Kuznetsov 
18114b50f80SVitaly Kuznetsov 	return 0;
18214b50f80SVitaly Kuznetsov }
18314b50f80SVitaly Kuznetsov 
hvt_cn_callback(struct cn_msg * msg,struct netlink_skb_parms * nsp)18414b50f80SVitaly Kuznetsov static void hvt_cn_callback(struct cn_msg *msg, struct netlink_skb_parms *nsp)
18514b50f80SVitaly Kuznetsov {
18614b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt, *hvt_found = NULL;
18714b50f80SVitaly Kuznetsov 
18814b50f80SVitaly Kuznetsov 	spin_lock(&hvt_list_lock);
18914b50f80SVitaly Kuznetsov 	list_for_each_entry(hvt, &hvt_list, list) {
19014b50f80SVitaly Kuznetsov 		if (hvt->cn_id.idx == msg->id.idx &&
19114b50f80SVitaly Kuznetsov 		    hvt->cn_id.val == msg->id.val) {
19214b50f80SVitaly Kuznetsov 			hvt_found = hvt;
19314b50f80SVitaly Kuznetsov 			break;
19414b50f80SVitaly Kuznetsov 		}
19514b50f80SVitaly Kuznetsov 	}
19614b50f80SVitaly Kuznetsov 	spin_unlock(&hvt_list_lock);
19714b50f80SVitaly Kuznetsov 	if (!hvt_found) {
19814b50f80SVitaly Kuznetsov 		pr_warn("hvt_cn_callback: spurious message received!\n");
19914b50f80SVitaly Kuznetsov 		return;
20014b50f80SVitaly Kuznetsov 	}
20114b50f80SVitaly Kuznetsov 
20214b50f80SVitaly Kuznetsov 	/*
20314b50f80SVitaly Kuznetsov 	 * Switching to NETLINK mode. Switching to CHARDEV happens when someone
20414b50f80SVitaly Kuznetsov 	 * opens the device.
20514b50f80SVitaly Kuznetsov 	 */
206a1502566SVitaly Kuznetsov 	mutex_lock(&hvt->lock);
20714b50f80SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_INIT)
20814b50f80SVitaly Kuznetsov 		hvt->mode = HVUTIL_TRANSPORT_NETLINK;
20914b50f80SVitaly Kuznetsov 
21014b50f80SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_NETLINK)
21114b50f80SVitaly Kuznetsov 		hvt_found->on_msg(msg->data, msg->len);
21214b50f80SVitaly Kuznetsov 	else
21314b50f80SVitaly Kuznetsov 		pr_warn("hvt_cn_callback: unexpected netlink message!\n");
214a1502566SVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
21514b50f80SVitaly Kuznetsov }
21614b50f80SVitaly Kuznetsov 
hvutil_transport_send(struct hvutil_transport * hvt,void * msg,int len,void (* on_read_cb)(void))217e0fa3e5eSVitaly Kuznetsov int hvutil_transport_send(struct hvutil_transport *hvt, void *msg, int len,
218e0fa3e5eSVitaly Kuznetsov 			  void (*on_read_cb)(void))
21914b50f80SVitaly Kuznetsov {
22014b50f80SVitaly Kuznetsov 	struct cn_msg *cn_msg;
22114b50f80SVitaly Kuznetsov 	int ret = 0;
22214b50f80SVitaly Kuznetsov 
223a1502566SVitaly Kuznetsov 	if (hvt->mode == HVUTIL_TRANSPORT_INIT ||
224a1502566SVitaly Kuznetsov 	    hvt->mode == HVUTIL_TRANSPORT_DESTROY) {
22514b50f80SVitaly Kuznetsov 		return -EINVAL;
22614b50f80SVitaly Kuznetsov 	} else if (hvt->mode == HVUTIL_TRANSPORT_NETLINK) {
22714b50f80SVitaly Kuznetsov 		cn_msg = kzalloc(sizeof(*cn_msg) + len, GFP_ATOMIC);
2289dd6a064SDan Carpenter 		if (!cn_msg)
22914b50f80SVitaly Kuznetsov 			return -ENOMEM;
23014b50f80SVitaly Kuznetsov 		cn_msg->id.idx = hvt->cn_id.idx;
23114b50f80SVitaly Kuznetsov 		cn_msg->id.val = hvt->cn_id.val;
23214b50f80SVitaly Kuznetsov 		cn_msg->len = len;
23314b50f80SVitaly Kuznetsov 		memcpy(cn_msg->data, msg, len);
23414b50f80SVitaly Kuznetsov 		ret = cn_netlink_send(cn_msg, 0, 0, GFP_ATOMIC);
23514b50f80SVitaly Kuznetsov 		kfree(cn_msg);
236e0fa3e5eSVitaly Kuznetsov 		/*
237e0fa3e5eSVitaly Kuznetsov 		 * We don't know when netlink messages are delivered but unlike
238e0fa3e5eSVitaly Kuznetsov 		 * in CHARDEV mode we're not blocked and we can send next
239e0fa3e5eSVitaly Kuznetsov 		 * messages right away.
240e0fa3e5eSVitaly Kuznetsov 		 */
241e0fa3e5eSVitaly Kuznetsov 		if (on_read_cb)
242e0fa3e5eSVitaly Kuznetsov 			on_read_cb();
24314b50f80SVitaly Kuznetsov 		return ret;
24414b50f80SVitaly Kuznetsov 	}
24514b50f80SVitaly Kuznetsov 	/* HVUTIL_TRANSPORT_CHARDEV */
246a72f3a4cSVitaly Kuznetsov 	mutex_lock(&hvt->lock);
247a1502566SVitaly Kuznetsov 	if (hvt->mode != HVUTIL_TRANSPORT_CHARDEV) {
248a1502566SVitaly Kuznetsov 		ret = -EINVAL;
249a1502566SVitaly Kuznetsov 		goto out_unlock;
250a1502566SVitaly Kuznetsov 	}
251a1502566SVitaly Kuznetsov 
25214b50f80SVitaly Kuznetsov 	if (hvt->outmsg) {
25314b50f80SVitaly Kuznetsov 		/* Previous message wasn't received */
25414b50f80SVitaly Kuznetsov 		ret = -EFAULT;
25514b50f80SVitaly Kuznetsov 		goto out_unlock;
25614b50f80SVitaly Kuznetsov 	}
25714b50f80SVitaly Kuznetsov 	hvt->outmsg = kzalloc(len, GFP_KERNEL);
258cdc0c0c9SOlaf Hering 	if (hvt->outmsg) {
25914b50f80SVitaly Kuznetsov 		memcpy(hvt->outmsg, msg, len);
26014b50f80SVitaly Kuznetsov 		hvt->outmsg_len = len;
261e0fa3e5eSVitaly Kuznetsov 		hvt->on_read = on_read_cb;
26214b50f80SVitaly Kuznetsov 		wake_up_interruptible(&hvt->outmsg_q);
263cdc0c0c9SOlaf Hering 	} else
264cdc0c0c9SOlaf Hering 		ret = -ENOMEM;
26514b50f80SVitaly Kuznetsov out_unlock:
266a72f3a4cSVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
26714b50f80SVitaly Kuznetsov 	return ret;
26814b50f80SVitaly Kuznetsov }
26914b50f80SVitaly Kuznetsov 
hvutil_transport_init(const char * name,u32 cn_idx,u32 cn_val,int (* on_msg)(void *,int),void (* on_reset)(void))27014b50f80SVitaly Kuznetsov struct hvutil_transport *hvutil_transport_init(const char *name,
27114b50f80SVitaly Kuznetsov 					       u32 cn_idx, u32 cn_val,
27214b50f80SVitaly Kuznetsov 					       int (*on_msg)(void *, int),
27314b50f80SVitaly Kuznetsov 					       void (*on_reset)(void))
27414b50f80SVitaly Kuznetsov {
27514b50f80SVitaly Kuznetsov 	struct hvutil_transport *hvt;
27614b50f80SVitaly Kuznetsov 
27714b50f80SVitaly Kuznetsov 	hvt = kzalloc(sizeof(*hvt), GFP_KERNEL);
27814b50f80SVitaly Kuznetsov 	if (!hvt)
27914b50f80SVitaly Kuznetsov 		return NULL;
28014b50f80SVitaly Kuznetsov 
28114b50f80SVitaly Kuznetsov 	hvt->cn_id.idx = cn_idx;
28214b50f80SVitaly Kuznetsov 	hvt->cn_id.val = cn_val;
28314b50f80SVitaly Kuznetsov 
28414b50f80SVitaly Kuznetsov 	hvt->mdev.minor = MISC_DYNAMIC_MINOR;
28514b50f80SVitaly Kuznetsov 	hvt->mdev.name = name;
28614b50f80SVitaly Kuznetsov 
28714b50f80SVitaly Kuznetsov 	hvt->fops.owner = THIS_MODULE;
28814b50f80SVitaly Kuznetsov 	hvt->fops.read = hvt_op_read;
28914b50f80SVitaly Kuznetsov 	hvt->fops.write = hvt_op_write;
29014b50f80SVitaly Kuznetsov 	hvt->fops.poll = hvt_op_poll;
29114b50f80SVitaly Kuznetsov 	hvt->fops.open = hvt_op_open;
29214b50f80SVitaly Kuznetsov 	hvt->fops.release = hvt_op_release;
29314b50f80SVitaly Kuznetsov 
29414b50f80SVitaly Kuznetsov 	hvt->mdev.fops = &hvt->fops;
29514b50f80SVitaly Kuznetsov 
29614b50f80SVitaly Kuznetsov 	init_waitqueue_head(&hvt->outmsg_q);
297a72f3a4cSVitaly Kuznetsov 	mutex_init(&hvt->lock);
298e9c18ae6SVitaly Kuznetsov 	init_completion(&hvt->release);
29914b50f80SVitaly Kuznetsov 
30014b50f80SVitaly Kuznetsov 	spin_lock(&hvt_list_lock);
30114b50f80SVitaly Kuznetsov 	list_add(&hvt->list, &hvt_list);
30214b50f80SVitaly Kuznetsov 	spin_unlock(&hvt_list_lock);
30314b50f80SVitaly Kuznetsov 
30414b50f80SVitaly Kuznetsov 	hvt->on_msg = on_msg;
30514b50f80SVitaly Kuznetsov 	hvt->on_reset = on_reset;
30614b50f80SVitaly Kuznetsov 
30714b50f80SVitaly Kuznetsov 	if (misc_register(&hvt->mdev))
30814b50f80SVitaly Kuznetsov 		goto err_free_hvt;
30914b50f80SVitaly Kuznetsov 
31014b50f80SVitaly Kuznetsov 	/* Use cn_id.idx/cn_id.val to determine if we need to setup netlink */
31114b50f80SVitaly Kuznetsov 	if (hvt->cn_id.idx > 0 && hvt->cn_id.val > 0 &&
31214b50f80SVitaly Kuznetsov 	    cn_add_callback(&hvt->cn_id, name, hvt_cn_callback))
31314b50f80SVitaly Kuznetsov 		goto err_free_hvt;
31414b50f80SVitaly Kuznetsov 
31514b50f80SVitaly Kuznetsov 	return hvt;
31614b50f80SVitaly Kuznetsov 
31714b50f80SVitaly Kuznetsov err_free_hvt:
318e66853b0SAlex Ng 	spin_lock(&hvt_list_lock);
319e66853b0SAlex Ng 	list_del(&hvt->list);
320e66853b0SAlex Ng 	spin_unlock(&hvt_list_lock);
32114b50f80SVitaly Kuznetsov 	kfree(hvt);
32214b50f80SVitaly Kuznetsov 	return NULL;
32314b50f80SVitaly Kuznetsov }
32414b50f80SVitaly Kuznetsov 
hvutil_transport_destroy(struct hvutil_transport * hvt)32514b50f80SVitaly Kuznetsov void hvutil_transport_destroy(struct hvutil_transport *hvt)
32614b50f80SVitaly Kuznetsov {
3279420098aSVitaly Kuznetsov 	int mode_old;
3289420098aSVitaly Kuznetsov 
329a1502566SVitaly Kuznetsov 	mutex_lock(&hvt->lock);
3309420098aSVitaly Kuznetsov 	mode_old = hvt->mode;
331a1502566SVitaly Kuznetsov 	hvt->mode = HVUTIL_TRANSPORT_DESTROY;
332a1502566SVitaly Kuznetsov 	wake_up_interruptible(&hvt->outmsg_q);
333a1502566SVitaly Kuznetsov 	mutex_unlock(&hvt->lock);
334a1502566SVitaly Kuznetsov 
3359420098aSVitaly Kuznetsov 	/*
3369420098aSVitaly Kuznetsov 	 * In case we were in 'chardev' mode we still have an open fd so we
3379420098aSVitaly Kuznetsov 	 * have to defer freeing the device. Netlink interface can be freed
3389420098aSVitaly Kuznetsov 	 * now.
3399420098aSVitaly Kuznetsov 	 */
34014b50f80SVitaly Kuznetsov 	spin_lock(&hvt_list_lock);
34114b50f80SVitaly Kuznetsov 	list_del(&hvt->list);
34214b50f80SVitaly Kuznetsov 	spin_unlock(&hvt_list_lock);
34314b50f80SVitaly Kuznetsov 	if (hvt->cn_id.idx > 0 && hvt->cn_id.val > 0)
34414b50f80SVitaly Kuznetsov 		cn_del_callback(&hvt->cn_id);
3459420098aSVitaly Kuznetsov 
346e9c18ae6SVitaly Kuznetsov 	if (mode_old == HVUTIL_TRANSPORT_CHARDEV)
347e9c18ae6SVitaly Kuznetsov 		wait_for_completion(&hvt->release);
348e9c18ae6SVitaly Kuznetsov 
3499420098aSVitaly Kuznetsov 	hvt_transport_free(hvt);
35014b50f80SVitaly Kuznetsov }
351