Commit 473f9409 authored by K. Y. Srinivasan's avatar K. Y. Srinivasan Committed by Greg Kroah-Hartman

Staging: hv: Get rid of the cdb_len from struct hv_storvsc_request

In preparation for consolidating all I/O request state, get rid of the
cdb_len field from struct hv_storvsc_request and instead use the
equivalent  state in struct vmscsi_request - cdb_length field.
Signed-off-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: default avatarHaiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: default avatarAbhishek Kane <v-abkane@microsoft.com>
Signed-off-by: default avatarHank Janssen <hjanssen@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent fc3967b0
...@@ -948,7 +948,7 @@ static int blkvsc_submit_request(struct blkvsc_request *blkvsc_req, ...@@ -948,7 +948,7 @@ static int blkvsc_submit_request(struct blkvsc_request *blkvsc_req,
vm_srb->target_id = blkdev->target; vm_srb->target_id = blkdev->target;
vm_srb->lun = 0; /* this is not really used at all */ vm_srb->lun = 0; /* this is not really used at all */
storvsc_req->cdb_len = blkvsc_req->cmd_len; vm_srb->cdb_length = blkvsc_req->cmd_len;
storvsc_req->cdb = blkvsc_req->cmnd; storvsc_req->cdb = blkvsc_req->cmnd;
storvsc_req->sense_buffer = blkvsc_req->sense_buffer; storvsc_req->sense_buffer = blkvsc_req->sense_buffer;
......
...@@ -531,9 +531,8 @@ int stor_vsc_on_io_request(struct hv_device *device, ...@@ -531,9 +531,8 @@ int stor_vsc_on_io_request(struct hv_device *device,
"Extension %p", device, stor_device, request, "Extension %p", device, stor_device, request,
request_extension); request_extension);
DPRINT_DBG(STORVSC, "req %p len %d cdblen %d", DPRINT_DBG(STORVSC, "req %p len %d",
request, request->data_buffer.len, request, request->data_buffer.len);
request->cdb_len);
if (!stor_device) { if (!stor_device) {
DPRINT_ERR(STORVSC, "unable to get stor device..." DPRINT_ERR(STORVSC, "unable to get stor device..."
...@@ -553,8 +552,8 @@ int stor_vsc_on_io_request(struct hv_device *device, ...@@ -553,8 +552,8 @@ int stor_vsc_on_io_request(struct hv_device *device,
vstor_packet->vm_srb.sense_info_length = SENSE_BUFFER_SIZE; vstor_packet->vm_srb.sense_info_length = SENSE_BUFFER_SIZE;
/* Copy over the scsi command descriptor block */ /* Copy over the scsi command descriptor block */
vstor_packet->vm_srb.cdb_length = request->cdb_len; memcpy(&vstor_packet->vm_srb.cdb, request->cdb,
memcpy(&vstor_packet->vm_srb.cdb, request->cdb, request->cdb_len); vstor_packet->vm_srb.cdb_length);
vstor_packet->vm_srb.data_transfer_length = request->data_buffer.len; vstor_packet->vm_srb.data_transfer_length = request->data_buffer.len;
......
...@@ -67,7 +67,6 @@ struct storvsc_request_extension { ...@@ -67,7 +67,6 @@ struct storvsc_request_extension {
struct hv_storvsc_request { struct hv_storvsc_request {
u8 *cdb; u8 *cdb;
u32 cdb_len;
u32 status; u32 status;
u32 bytes_xfer; u32 bytes_xfer;
......
...@@ -780,7 +780,7 @@ static int storvsc_queuecommand_lck(struct scsi_cmnd *scmnd, ...@@ -780,7 +780,7 @@ static int storvsc_queuecommand_lck(struct scsi_cmnd *scmnd,
vm_srb->lun = scmnd->device->lun; vm_srb->lun = scmnd->device->lun;
/* ASSERT(scmnd->cmd_len <= 16); */ /* ASSERT(scmnd->cmd_len <= 16); */
request->cdb_len = scmnd->cmd_len; vm_srb->cdb_length = scmnd->cmd_len;
request->cdb = scmnd->cmnd; request->cdb = scmnd->cmnd;
request->sense_buffer = scmnd->sense_buffer; request->sense_buffer = scmnd->sense_buffer;
......
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