Commit e2e64432 authored by Bill Pemberton's avatar Bill Pemberton Committed by Greg Kroah-Hartman

staging: hv: remove ASSERT()s in StorVsc.c

These ASSERT()s serve no purpose other than for debugging.
Signed-off-by: default avatarBill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 4e5166b5
...@@ -100,7 +100,7 @@ static inline struct storvsc_device *AllocStorDevice(struct hv_device *Device) ...@@ -100,7 +100,7 @@ static inline struct storvsc_device *AllocStorDevice(struct hv_device *Device)
static inline void FreeStorDevice(struct storvsc_device *Device) static inline void FreeStorDevice(struct storvsc_device *Device)
{ {
ASSERT(atomic_read(&Device->RefCount) == 0); /* ASSERT(atomic_read(&Device->RefCount) == 0); */
kfree(Device); kfree(Device);
} }
...@@ -137,10 +137,10 @@ static inline void PutStorDevice(struct hv_device *Device) ...@@ -137,10 +137,10 @@ static inline void PutStorDevice(struct hv_device *Device)
struct storvsc_device *storDevice; struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension; storDevice = (struct storvsc_device *)Device->Extension;
ASSERT(storDevice); /* ASSERT(storDevice); */
atomic_dec(&storDevice->RefCount); atomic_dec(&storDevice->RefCount);
ASSERT(atomic_read(&storDevice->RefCount)); /* ASSERT(atomic_read(&storDevice->RefCount)); */
} }
/* Drop ref count to 1 to effectively disable GetStorDevice() */ /* Drop ref count to 1 to effectively disable GetStorDevice() */
...@@ -149,7 +149,7 @@ static inline struct storvsc_device *ReleaseStorDevice(struct hv_device *Device) ...@@ -149,7 +149,7 @@ static inline struct storvsc_device *ReleaseStorDevice(struct hv_device *Device)
struct storvsc_device *storDevice; struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension; storDevice = (struct storvsc_device *)Device->Extension;
ASSERT(storDevice); /* ASSERT(storDevice); */
/* Busy wait until the ref drop to 2, then set it to 1 */ /* Busy wait until the ref drop to 2, then set it to 1 */
while (atomic_cmpxchg(&storDevice->RefCount, 2, 1) != 2) while (atomic_cmpxchg(&storDevice->RefCount, 2, 1) != 2)
...@@ -165,7 +165,7 @@ static inline struct storvsc_device *FinalReleaseStorDevice( ...@@ -165,7 +165,7 @@ static inline struct storvsc_device *FinalReleaseStorDevice(
struct storvsc_device *storDevice; struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension; storDevice = (struct storvsc_device *)Device->Extension;
ASSERT(storDevice); /* ASSERT(storDevice); */
/* Busy wait until the ref drop to 1, then set it to 0 */ /* Busy wait until the ref drop to 1, then set it to 0 */
while (atomic_cmpxchg(&storDevice->RefCount, 1, 0) != 1) while (atomic_cmpxchg(&storDevice->RefCount, 1, 0) != 1)
...@@ -370,12 +370,12 @@ static void StorVscOnIOCompletion(struct hv_device *Device, ...@@ -370,12 +370,12 @@ static void StorVscOnIOCompletion(struct hv_device *Device,
"completed bytes xfer %u", RequestExt, "completed bytes xfer %u", RequestExt,
VStorPacket->VmSrb.DataTransferLength); VStorPacket->VmSrb.DataTransferLength);
ASSERT(RequestExt != NULL); /* ASSERT(RequestExt != NULL); */
ASSERT(RequestExt->Request != NULL); /* ASSERT(RequestExt->Request != NULL); */
request = RequestExt->Request; request = RequestExt->Request;
ASSERT(request->OnIOCompletion != NULL); /* ASSERT(request->OnIOCompletion != NULL); */
/* Copy over the status...etc */ /* Copy over the status...etc */
request->Status = VStorPacket->VmSrb.ScsiStatus; request->Status = VStorPacket->VmSrb.ScsiStatus;
...@@ -395,8 +395,8 @@ static void StorVscOnIOCompletion(struct hv_device *Device, ...@@ -395,8 +395,8 @@ static void StorVscOnIOCompletion(struct hv_device *Device,
"valid - len %d\n", RequestExt, "valid - len %d\n", RequestExt,
VStorPacket->VmSrb.SenseInfoLength); VStorPacket->VmSrb.SenseInfoLength);
ASSERT(VStorPacket->VmSrb.SenseInfoLength <= /* ASSERT(VStorPacket->VmSrb.SenseInfoLength <= */
request->SenseBufferSize); /* request->SenseBufferSize); */
memcpy(request->SenseBuffer, memcpy(request->SenseBuffer,
VStorPacket->VmSrb.SenseData, VStorPacket->VmSrb.SenseData,
VStorPacket->VmSrb.SenseInfoLength); VStorPacket->VmSrb.SenseInfoLength);
...@@ -451,7 +451,7 @@ static void StorVscOnChannelCallback(void *context) ...@@ -451,7 +451,7 @@ static void StorVscOnChannelCallback(void *context)
DPRINT_ENTER(STORVSC); DPRINT_ENTER(STORVSC);
ASSERT(device); /* ASSERT(device); */
storDevice = MustGetStorDevice(device); storDevice = MustGetStorDevice(device);
if (!storDevice) { if (!storDevice) {
...@@ -474,7 +474,7 @@ static void StorVscOnChannelCallback(void *context) ...@@ -474,7 +474,7 @@ static void StorVscOnChannelCallback(void *context)
request = (struct storvsc_request_extension *) request = (struct storvsc_request_extension *)
(unsigned long)requestId; (unsigned long)requestId;
ASSERT(request); /* ASSERT(request);c */
/* if (vstorPacket.Flags & SYNTHETIC_FLAG) */ /* if (vstorPacket.Flags & SYNTHETIC_FLAG) */
if ((request == &storDevice->InitRequest) || if ((request == &storDevice->InitRequest) ||
...@@ -821,7 +821,7 @@ int StorVscInitialize(struct hv_driver *Driver) ...@@ -821,7 +821,7 @@ int StorVscInitialize(struct hv_driver *Driver)
sizeof(struct vmscsi_request)); sizeof(struct vmscsi_request));
/* Make sure we are at least 2 pages since 1 page is used for control */ /* Make sure we are at least 2 pages since 1 page is used for control */
ASSERT(storDriver->RingBufferSize >= (PAGE_SIZE << 1)); /* ASSERT(storDriver->RingBufferSize >= (PAGE_SIZE << 1)); */
Driver->name = gDriverName; Driver->name = gDriverName;
memcpy(&Driver->deviceType, &gStorVscDeviceType, memcpy(&Driver->deviceType, &gStorVscDeviceType,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment