Skip to content

Commit

Permalink
Staging: hv: storvsc: Get rid of final_release_stor_device() by inlin…
Browse files Browse the repository at this point in the history
…ing code

Get rid of final_release_stor_device() by inlining code.

Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
K. Y. Srinivasan authored and Greg Kroah-Hartman committed Aug 29, 2011
1 parent c352ee8 commit a5b2359
Showing 1 changed file with 6 additions and 17 deletions.
23 changes: 6 additions & 17 deletions drivers/staging/hv/storvsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -67,22 +67,6 @@ static inline struct storvsc_device *get_in_stor_device(
return stor_device;
}

/* Drop ref count to 0. No one can use stor_device object. */
static inline struct storvsc_device *final_release_stor_device(
struct hv_device *device)
{
struct storvsc_device *stor_device;

stor_device = (struct storvsc_device *)device->ext;

/* Busy wait until the ref drop to 1, then set it to 0 */
while (atomic_cmpxchg(&stor_device->ref_count, 1, 0) != 1)
udelay(100);

device->ext = NULL;
return stor_device;
}

static int storvsc_channel_init(struct hv_device *device)
{
struct storvsc_device *stor_device;
Expand Down Expand Up @@ -401,7 +385,12 @@ int storvsc_dev_remove(struct hv_device *device)

storvsc_wait_to_drain(stor_device);

stor_device = final_release_stor_device(device);
/*
* Since we have already drained, we don't need to busy wait
* as was done in final_release_stor_device()
*/
atomic_set(&stor_device->ref_count, 0);
device->ext = NULL;

/* Close the channel */
vmbus_close(device->channel);
Expand Down

0 comments on commit a5b2359

Please sign in to comment.