Skip to content

Commit

Permalink
Staging: hv: Get rid of the cdb_len from struct hv_storvsc_request
Browse files Browse the repository at this point in the history
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: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Abhishek Kane <v-abkane@microsoft.com>
Signed-off-by: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
K. Y. Srinivasan authored and Greg Kroah-Hartman committed Apr 5, 2011
1 parent fc3967b commit 473f940
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 8 deletions.
2 changes: 1 addition & 1 deletion drivers/staging/hv/blkvsc_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -948,7 +948,7 @@ static int blkvsc_submit_request(struct blkvsc_request *blkvsc_req,
vm_srb->target_id = blkdev->target;
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->sense_buffer = blkvsc_req->sense_buffer;
Expand Down
9 changes: 4 additions & 5 deletions drivers/staging/hv/storvsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -531,9 +531,8 @@ int stor_vsc_on_io_request(struct hv_device *device,
"Extension %p", device, stor_device, request,
request_extension);

DPRINT_DBG(STORVSC, "req %p len %d cdblen %d",
request, request->data_buffer.len,
request->cdb_len);
DPRINT_DBG(STORVSC, "req %p len %d",
request, request->data_buffer.len);

if (!stor_device) {
DPRINT_ERR(STORVSC, "unable to get stor device..."
Expand All @@ -553,8 +552,8 @@ int stor_vsc_on_io_request(struct hv_device *device,
vstor_packet->vm_srb.sense_info_length = SENSE_BUFFER_SIZE;

/* Copy over the scsi command descriptor block */
vstor_packet->vm_srb.cdb_length = request->cdb_len;
memcpy(&vstor_packet->vm_srb.cdb, request->cdb, request->cdb_len);
memcpy(&vstor_packet->vm_srb.cdb, request->cdb,
vstor_packet->vm_srb.cdb_length);

vstor_packet->vm_srb.data_transfer_length = request->data_buffer.len;

Expand Down
1 change: 0 additions & 1 deletion drivers/staging/hv/storvsc_api.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ struct storvsc_request_extension {

struct hv_storvsc_request {
u8 *cdb;
u32 cdb_len;
u32 status;
u32 bytes_xfer;

Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/hv/storvsc_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -780,7 +780,7 @@ static int storvsc_queuecommand_lck(struct scsi_cmnd *scmnd,
vm_srb->lun = scmnd->device->lun;

/* ASSERT(scmnd->cmd_len <= 16); */
request->cdb_len = scmnd->cmd_len;
vm_srb->cdb_length = scmnd->cmd_len;
request->cdb = scmnd->cmnd;

request->sense_buffer = scmnd->sense_buffer;
Expand Down

0 comments on commit 473f940

Please sign in to comment.