Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756751Ab1F2Oap (ORCPT ); Wed, 29 Jun 2011 10:30:45 -0400 Received: from p3plsmtps2ded01.prod.phx3.secureserver.net ([208.109.80.58]:48311 "HELO p3plsmtps2ded01-01.prod.phx3.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1755823Ab1F2OWx (ORCPT ); Wed, 29 Jun 2011 10:22:53 -0400 From: "K. Y. Srinivasan" To: gregkh@suse.de, linux-kernel@vger.kernel.org, devel@linuxdriverproject.org, virtualization@lists.osdl.org Cc: "K. Y. Srinivasan" , Haiyang Zhang , Abhishek Kane , Hank Janssen Subject: [PATCH 11/40] Staging: hv: storvsc: Leverage the spinlock to manage ref_cnt Date: Wed, 29 Jun 2011 07:39:08 -0700 Message-Id: <1309358377-8537-11-git-send-email-kys@microsoft.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1309358377-8537-1-git-send-email-kys@microsoft.com> References: <1309358301-8488-1-git-send-email-kys@microsoft.com> <1309358377-8537-1-git-send-email-kys@microsoft.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3532 Lines: 96 Now that we have a spin lock protecting access to the stor device pointer, use it manage the reference count as well. Signed-off-by: K. Y. Srinivasan Signed-off-by: Haiyang Zhang Signed-off-by: Abhishek Kane Signed-off-by: Hank Janssen --- drivers/staging/hv/hyperv_storage.h | 8 ++++---- drivers/staging/hv/storvsc.c | 10 +++++----- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/hv/hyperv_storage.h b/drivers/staging/hv/hyperv_storage.h index 53b65be..d946211 100644 --- a/drivers/staging/hv/hyperv_storage.h +++ b/drivers/staging/hv/hyperv_storage.h @@ -265,7 +265,7 @@ struct storvsc_device { struct hv_device *device; /* 0 indicates the device is being destroyed */ - atomic_t ref_count; + int ref_count; bool destroy; bool drain_notify; atomic_t num_outstanding_req; @@ -296,9 +296,9 @@ static inline struct storvsc_device *get_out_stor_device( spin_lock_irqsave(&device->ext_lock, flags); stor_device = (struct storvsc_device *)device->ext; - if (stor_device && (atomic_read(&stor_device->ref_count) > 1) && + if (stor_device && (stor_device->ref_count > 1) && !stor_device->destroy) - atomic_inc(&stor_device->ref_count); + stor_device->ref_count++; else stor_device = NULL; spin_unlock_irqrestore(&device->ext_lock, flags); @@ -315,7 +315,7 @@ static inline void put_stor_device(struct hv_device *device) spin_lock_irqsave(&device->ext_lock, flags); stor_device = (struct storvsc_device *)device->ext; - atomic_dec(&stor_device->ref_count); + stor_device->ref_count--; spin_unlock_irqrestore(&device->ext_lock, flags); } diff --git a/drivers/staging/hv/storvsc.c b/drivers/staging/hv/storvsc.c index c4cb170..a41be2a 100644 --- a/drivers/staging/hv/storvsc.c +++ b/drivers/staging/hv/storvsc.c @@ -42,7 +42,7 @@ static inline struct storvsc_device *alloc_stor_device(struct hv_device *device) /* Set to 2 to allow both inbound and outbound traffics */ /* (ie get_out_stor_device() and get_in_stor_device()) to proceed. */ - atomic_set(&stor_device->ref_count, 2); + stor_device->ref_count = 2; stor_device->destroy = false; init_waitqueue_head(&stor_device->waiting_to_drain); stor_device->device = device; @@ -61,8 +61,8 @@ static inline struct storvsc_device *get_in_stor_device( spin_lock_irqsave(&device->ext_lock, flags); stor_device = (struct storvsc_device *)device->ext; - if (stor_device && atomic_read(&stor_device->ref_count)) - atomic_inc(&stor_device->ref_count); + if (stor_device && stor_device->ref_count) + stor_device->ref_count++; else stor_device = NULL; spin_unlock_irqrestore(&device->ext_lock, flags); @@ -379,7 +379,7 @@ int storvsc_dev_remove(struct hv_device *device) * pointer. */ stor_device = (struct storvsc_device *)device->ext; - atomic_dec(&stor_device->ref_count); + stor_device->destroy = true; /* @@ -391,7 +391,7 @@ int storvsc_dev_remove(struct hv_device *device) storvsc_wait_to_drain(stor_device); spin_lock_irqsave(&device->ext_lock, flags); - atomic_set(&stor_device->ref_count, 0); + stor_device->ref_count = 0; device->ext = NULL; spin_unlock_irqrestore(&device->ext_lock, flags); -- 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/