These ASSERT()s serve no purpose other than for debugging.
Signed-off-by: Bill Pemberton <wfp5p@virginia.edu>
Cc: Hank Janssen <hjanssen@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
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); */
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
+ /* 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() */
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
+ /* ASSERT(storDevice); */
/* Busy wait until the ref drop to 2, then set it to 1 */
while (atomic_cmpxchg(&storDevice->RefCount, 2, 1) != 2)
/* Busy wait until the ref drop to 2, then set it to 1 */
while (atomic_cmpxchg(&storDevice->RefCount, 2, 1) != 2)
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
struct storvsc_device *storDevice;
storDevice = (struct storvsc_device *)Device->Extension;
+ /* ASSERT(storDevice); */
/* Busy wait until the ref drop to 1, then set it to 0 */
while (atomic_cmpxchg(&storDevice->RefCount, 1, 0) != 1)
/* Busy wait until the ref drop to 1, then set it to 0 */
while (atomic_cmpxchg(&storDevice->RefCount, 1, 0) != 1)
"completed bytes xfer %u", RequestExt,
VStorPacket->VmSrb.DataTransferLength);
"completed bytes xfer %u", RequestExt,
VStorPacket->VmSrb.DataTransferLength);
- ASSERT(RequestExt != NULL);
- ASSERT(RequestExt->Request != NULL);
+ /* ASSERT(RequestExt != NULL); */
+ /* ASSERT(RequestExt->Request != NULL); */
request = RequestExt->Request;
request = RequestExt->Request;
- ASSERT(request->OnIOCompletion != NULL);
+ /* ASSERT(request->OnIOCompletion != NULL); */
/* Copy over the status...etc */
request->Status = VStorPacket->VmSrb.ScsiStatus;
/* Copy over the status...etc */
request->Status = VStorPacket->VmSrb.ScsiStatus;
"valid - len %d\n", RequestExt,
VStorPacket->VmSrb.SenseInfoLength);
"valid - len %d\n", RequestExt,
VStorPacket->VmSrb.SenseInfoLength);
- ASSERT(VStorPacket->VmSrb.SenseInfoLength <=
- request->SenseBufferSize);
+ /* ASSERT(VStorPacket->VmSrb.SenseInfoLength <= */
+ /* request->SenseBufferSize); */
memcpy(request->SenseBuffer,
VStorPacket->VmSrb.SenseData,
VStorPacket->VmSrb.SenseInfoLength);
memcpy(request->SenseBuffer,
VStorPacket->VmSrb.SenseData,
VStorPacket->VmSrb.SenseInfoLength);
storDevice = MustGetStorDevice(device);
if (!storDevice) {
storDevice = MustGetStorDevice(device);
if (!storDevice) {
request = (struct storvsc_request_extension *)
(unsigned long)requestId;
request = (struct storvsc_request_extension *)
(unsigned long)requestId;
+ /* ASSERT(request);c */
/* if (vstorPacket.Flags & SYNTHETIC_FLAG) */
if ((request == &storDevice->InitRequest) ||
/* if (vstorPacket.Flags & SYNTHETIC_FLAG) */
if ((request == &storDevice->InitRequest) ||
sizeof(struct vmscsi_request));
/* Make sure we are at least 2 pages since 1 page is used for control */
sizeof(struct vmscsi_request));
/* 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;
memcpy(&Driver->deviceType, &gStorVscDeviceType,
Driver->name = gDriverName;
memcpy(&Driver->deviceType, &gStorVscDeviceType,