Commit 92ae4ebd authored by Olaf Hering's avatar Olaf Hering Committed by Greg Kroah-Hartman

staging: hv: storvsc: ignore SET_WINDOW scsi command

Some commands sent by smartd will offline the device.
With this change applied, smartd sill not monitor the device anymore.
Signed-off-by: default avatarOlaf Hering <olaf@aepfle.de>
Acked-by: default avatarK. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent e2bb6537
...@@ -1122,6 +1122,22 @@ static void storvsc_command_completion(struct hv_storvsc_request *request) ...@@ -1122,6 +1122,22 @@ static void storvsc_command_completion(struct hv_storvsc_request *request)
kmem_cache_free(host_dev->request_pool, cmd_request); kmem_cache_free(host_dev->request_pool, cmd_request);
} }
static bool storvsc_check_scsi_cmd(struct scsi_cmnd *scmnd)
{
bool allowed = true;
u8 scsi_op = scmnd->cmnd[0];
switch (scsi_op) {
/* smartd sends this command, which will offline the device */
case SET_WINDOW:
scmnd->result = DID_ERROR << 16;
allowed = false;
break;
default:
break;
}
return allowed;
}
/* /*
* storvsc_queuecommand - Initiate command processing * storvsc_queuecommand - Initiate command processing
...@@ -1141,6 +1157,10 @@ static int storvsc_queuecommand_lck(struct scsi_cmnd *scmnd, ...@@ -1141,6 +1157,10 @@ static int storvsc_queuecommand_lck(struct scsi_cmnd *scmnd,
unsigned int sg_count = 0; unsigned int sg_count = 0;
struct vmscsi_request *vm_srb; struct vmscsi_request *vm_srb;
if (storvsc_check_scsi_cmd(scmnd) == false) {
done(scmnd);
return 0;
}
/* If retrying, no need to prep the cmd */ /* If retrying, no need to prep the cmd */
if (scmnd->host_scribble) { if (scmnd->host_scribble) {
......
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