[azure 4.13][PATCH 12/19] Revert "UBUNTU: SAUCE: vmbus: add vmbus onoffer/onoffer_rescind sync."

[Marcelo Henrique Cerri]

From: Dexuan Cui <decui at microsoft.com>

BugLink: http://bugs.launchpad.net/bugs/1736283

This reverts commit 7439b4e7ae5dfbf8d11d1ec5917c7d7d603bf521.

Signed-off-by: Dexuan Cui <decui at microsoft.com>
Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri at canonical.com>
---
 drivers/hv/channel_mgmt.c | 24 +++++-------------------
 drivers/hv/hyperv_vmbus.h |  1 -
 drivers/hv/vmbus_drv.c    |  1 -
 3 files changed, 5 insertions(+), 21 deletions(-)

diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c
index 4f64561eac7a..509019dc7df9 100644
--- a/drivers/hv/channel_mgmt.c
+++ b/drivers/hv/channel_mgmt.c
@@ -449,7 +449,6 @@ void vmbus_free_channels(void)
 static void vmbus_process_offer(struct vmbus_channel *newchannel)
 {
 	struct vmbus_channel *channel;
-	struct hv_device *device_obj;
 	bool fnew = true;
 	unsigned long flags;
 	u16 dev_type;
@@ -531,7 +530,6 @@ static void vmbus_process_offer(struct vmbus_channel *newchannel)
 		if (channel->sc_creation_callback != NULL)
 			channel->sc_creation_callback(newchannel);
 		newchannel->probe_done = true;
-		atomic_dec(&vmbus_connection.register_in_progress);
 		return;
 	}
 
@@ -540,37 +538,33 @@ static void vmbus_process_offer(struct vmbus_channel *newchannel)
 	 * We need to set the DeviceObject field before calling
 	 * vmbus_child_dev_add()
 	 */
-	device_obj = vmbus_device_create(
+	newchannel->device_obj = vmbus_device_create(
 		&newchannel->offermsg.offer.if_type,
 		&newchannel->offermsg.offer.if_instance,
 		newchannel);
-	if (!device_obj)
+	if (!newchannel->device_obj)
 		goto err_deq_chan;
 
-	device_obj->device_id = dev_type;
+	newchannel->device_obj->device_id = dev_type;
 	/*
 	 * Add the new device to the bus. This will kick off device-driver
 	 * binding which eventually invokes the device driver's AddDevice()
 	 * method.
 	 */
-	atomic_dec(&vmbus_connection.offer_in_progress);
-	ret = vmbus_device_register(device_obj);
+	ret = vmbus_device_register(newchannel->device_obj);
 
 	if (ret != 0) {
 		pr_err("unable to add child device object (relid %d)\n",
 			newchannel->offermsg.child_relid);
-		kfree(device_obj);
+		kfree(newchannel->device_obj);
 		goto err_deq_chan;
 	}
-	newchannel->device_obj = device_obj;
-	atomic_dec(&vmbus_connection.register_in_progress);
 
 	newchannel->probe_done = true;
 	return;
 
 err_deq_chan:
 	atomic_dec(&vmbus_connection.offer_in_progress);
-	atomic_dec(&vmbus_connection.register_in_progress);
 	mutex_lock(&vmbus_connection.channel_mutex);
 	list_del(&newchannel->listentry);
 	mutex_unlock(&vmbus_connection.channel_mutex);
@@ -913,14 +907,6 @@ static void vmbus_onoffer_rescind(struct vmbus_channel_message_header *hdr)
 	 * At this point, the rescind handling can proceed safely.
 	 */
 
-	while (atomic_read(&vmbus_connection.register_in_progress) != 0) {
-		/*
-		 * We wait here until any channel offer is currently
-		 * being processed.
-		 */
-		msleep(1);
-	}
-
 	if (channel->device_obj) {
 		if (channel->chn_rescind_callback) {
 			channel->chn_rescind_callback(channel);
diff --git a/drivers/hv/hyperv_vmbus.h b/drivers/hv/hyperv_vmbus.h
index 2ae3df330909..0256a3e19552 100644
--- a/drivers/hv/hyperv_vmbus.h
+++ b/drivers/hv/hyperv_vmbus.h
@@ -298,7 +298,6 @@ struct vmbus_connection {
 	int connect_cpu;
 
 	atomic_t offer_in_progress;
-	atomic_t register_in_progress;
 
 	enum vmbus_connect_state conn_state;
 
diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
index 6edf611e276e..eb5dbeb31b17 100644
--- a/drivers/hv/vmbus_drv.c
+++ b/drivers/hv/vmbus_drv.c
@@ -841,7 +841,6 @@ static void vmbus_dispatch_msg_work(struct work_struct *work)
 		flush_workqueue(vmbus_connection.work_queue_rescind);
 
 		atomic_inc(&vmbus_connection.offer_in_progress);
-		atomic_inc(&vmbus_connection.register_in_progress);
 		queue_work_on(vmbus_connection.connect_cpu,
 			      vmbus_connection.work_queue,
 			      &context->work);
-- 
2.7.4