mirror of
https://github.com/linuxkit/linuxkit.git
synced 2025-12-08 00:41:36 +00:00
154 lines
5.0 KiB
Diff
154 lines
5.0 KiB
Diff
From b6c9f23164d3e7460d8983d27f2df194ab5e9a0b Mon Sep 17 00:00:00 2001
|
|
From: Dexuan Cui <decui@microsoft.com>
|
|
Date: Wed, 27 Jan 2016 22:29:43 -0800
|
|
Subject: [PATCH 21/25] Drivers: hv: vmbus: add an API
|
|
vmbus_hvsock_device_unregister()
|
|
|
|
The hvsock driver needs this API to release all the resources related
|
|
to the channel.
|
|
|
|
Signed-off-by: Dexuan Cui <decui@microsoft.com>
|
|
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
|
|
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
|
|
(cherry picked from commit 85d9aa705184a4504d0330017e3956fcdae8a9d6)
|
|
---
|
|
drivers/hv/channel_mgmt.c | 33 ++++++++++++++++++++++++++++-----
|
|
drivers/hv/connection.c | 4 ++--
|
|
include/linux/hyperv.h | 2 ++
|
|
3 files changed, 32 insertions(+), 7 deletions(-)
|
|
|
|
diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c
|
|
index 421e3dd..0fe9665 100644
|
|
--- a/drivers/hv/channel_mgmt.c
|
|
+++ b/drivers/hv/channel_mgmt.c
|
|
@@ -195,6 +195,7 @@ void hv_process_channel_removal(struct vmbus_channel *channel, u32 relid)
|
|
vmbus_release_relid(relid);
|
|
|
|
BUG_ON(!channel->rescind);
|
|
+ BUG_ON(!mutex_is_locked(&vmbus_connection.channel_mutex));
|
|
|
|
if (channel->target_cpu != get_cpu()) {
|
|
put_cpu();
|
|
@@ -206,9 +207,7 @@ void hv_process_channel_removal(struct vmbus_channel *channel, u32 relid)
|
|
}
|
|
|
|
if (channel->primary_channel == NULL) {
|
|
- mutex_lock(&vmbus_connection.channel_mutex);
|
|
list_del(&channel->listentry);
|
|
- mutex_unlock(&vmbus_connection.channel_mutex);
|
|
|
|
primary_channel = channel;
|
|
} else {
|
|
@@ -251,6 +250,7 @@ static void vmbus_process_offer(struct vmbus_channel *newchannel)
|
|
struct vmbus_channel *channel;
|
|
bool fnew = true;
|
|
unsigned long flags;
|
|
+ int ret;
|
|
|
|
/* Make sure this is a new offer */
|
|
mutex_lock(&vmbus_connection.channel_mutex);
|
|
@@ -330,7 +330,11 @@ static void vmbus_process_offer(struct vmbus_channel *newchannel)
|
|
* binding which eventually invokes the device driver's AddDevice()
|
|
* method.
|
|
*/
|
|
- if (vmbus_device_register(newchannel->device_obj) != 0) {
|
|
+ mutex_lock(&vmbus_connection.channel_mutex);
|
|
+ ret = vmbus_device_register(newchannel->device_obj);
|
|
+ mutex_unlock(&vmbus_connection.channel_mutex);
|
|
+
|
|
+ if (ret != 0) {
|
|
pr_err("unable to add child device object (relid %d)\n",
|
|
newchannel->offermsg.child_relid);
|
|
kfree(newchannel->device_obj);
|
|
@@ -587,6 +591,8 @@ static void vmbus_onoffer_rescind(struct vmbus_channel_message_header *hdr)
|
|
struct device *dev;
|
|
|
|
rescind = (struct vmbus_channel_rescind_offer *)hdr;
|
|
+
|
|
+ mutex_lock(&vmbus_connection.channel_mutex);
|
|
channel = relid2channel(rescind->child_relid);
|
|
|
|
if (channel == NULL) {
|
|
@@ -595,7 +601,7 @@ static void vmbus_onoffer_rescind(struct vmbus_channel_message_header *hdr)
|
|
* vmbus_process_offer(), we have already invoked
|
|
* vmbus_release_relid() on error.
|
|
*/
|
|
- return;
|
|
+ goto out;
|
|
}
|
|
|
|
spin_lock_irqsave(&channel->lock, flags);
|
|
@@ -605,7 +611,7 @@ static void vmbus_onoffer_rescind(struct vmbus_channel_message_header *hdr)
|
|
if (channel->device_obj) {
|
|
if (channel->chn_rescind_callback) {
|
|
channel->chn_rescind_callback(channel);
|
|
- return;
|
|
+ goto out;
|
|
}
|
|
/*
|
|
* We will have to unregister this device from the
|
|
@@ -620,8 +626,25 @@ static void vmbus_onoffer_rescind(struct vmbus_channel_message_header *hdr)
|
|
hv_process_channel_removal(channel,
|
|
channel->offermsg.child_relid);
|
|
}
|
|
+
|
|
+out:
|
|
+ mutex_unlock(&vmbus_connection.channel_mutex);
|
|
}
|
|
|
|
+void vmbus_hvsock_device_unregister(struct vmbus_channel *channel)
|
|
+{
|
|
+ mutex_lock(&vmbus_connection.channel_mutex);
|
|
+
|
|
+ BUG_ON(!is_hvsock_channel(channel));
|
|
+
|
|
+ channel->rescind = true;
|
|
+ vmbus_device_unregister(channel->device_obj);
|
|
+
|
|
+ mutex_unlock(&vmbus_connection.channel_mutex);
|
|
+}
|
|
+EXPORT_SYMBOL_GPL(vmbus_hvsock_device_unregister);
|
|
+
|
|
+
|
|
/*
|
|
* vmbus_onoffers_delivered -
|
|
* This is invoked when all offers have been delivered.
|
|
diff --git a/drivers/hv/connection.c b/drivers/hv/connection.c
|
|
index 521f48e..09c08b5 100644
|
|
--- a/drivers/hv/connection.c
|
|
+++ b/drivers/hv/connection.c
|
|
@@ -285,7 +285,8 @@ struct vmbus_channel *relid2channel(u32 relid)
|
|
struct list_head *cur, *tmp;
|
|
struct vmbus_channel *cur_sc;
|
|
|
|
- mutex_lock(&vmbus_connection.channel_mutex);
|
|
+ BUG_ON(!mutex_is_locked(&vmbus_connection.channel_mutex));
|
|
+
|
|
list_for_each_entry(channel, &vmbus_connection.chn_list, listentry) {
|
|
if (channel->offermsg.child_relid == relid) {
|
|
found_channel = channel;
|
|
@@ -304,7 +305,6 @@ struct vmbus_channel *relid2channel(u32 relid)
|
|
}
|
|
}
|
|
}
|
|
- mutex_unlock(&vmbus_connection.channel_mutex);
|
|
|
|
return found_channel;
|
|
}
|
|
diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h
|
|
index 09e9ec1..af7ee0a 100644
|
|
--- a/include/linux/hyperv.h
|
|
+++ b/include/linux/hyperv.h
|
|
@@ -1043,6 +1043,8 @@ int __must_check __vmbus_driver_register(struct hv_driver *hv_driver,
|
|
const char *mod_name);
|
|
void vmbus_driver_unregister(struct hv_driver *hv_driver);
|
|
|
|
+void vmbus_hvsock_device_unregister(struct vmbus_channel *channel);
|
|
+
|
|
int vmbus_allocate_mmio(struct resource **new, struct hv_device *device_obj,
|
|
resource_size_t min, resource_size_t max,
|
|
resource_size_t size, resource_size_t align,
|
|
--
|
|
2.8.0.rc3
|
|
|