Commit 6ef6f940 authored by Matthew R. Ochs's avatar Matthew R. Ochs Committed by James Bottomley

cxlflash: Remove unnecessary scsi_block_requests

The host reset handler is called with I/O already blocked, thus
there is no need to explicitly block and unblock I/O in the handler.
Signed-off-by: default avatarMatthew R. Ochs <mrochs@linux.vnet.ibm.com>
Signed-off-by: default avatarManoj N. Kumar <manoj@linux.vnet.ibm.com>
Reviewed-by: default avatarBrian King <brking@linux.vnet.ibm.com>
Reviewed-by: default avatarTomas Henzl <thenzl@redhat.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Odin.com>
parent ed486daa
...@@ -1966,7 +1966,6 @@ static int cxlflash_eh_host_reset_handler(struct scsi_cmnd *scp) ...@@ -1966,7 +1966,6 @@ static int cxlflash_eh_host_reset_handler(struct scsi_cmnd *scp)
switch (cfg->state) { switch (cfg->state) {
case STATE_NORMAL: case STATE_NORMAL:
cfg->state = STATE_RESET; cfg->state = STATE_RESET;
scsi_block_requests(cfg->host);
cxlflash_mark_contexts_error(cfg); cxlflash_mark_contexts_error(cfg);
rcr = afu_reset(cfg); rcr = afu_reset(cfg);
if (rcr) { if (rcr) {
...@@ -1975,7 +1974,6 @@ static int cxlflash_eh_host_reset_handler(struct scsi_cmnd *scp) ...@@ -1975,7 +1974,6 @@ static int cxlflash_eh_host_reset_handler(struct scsi_cmnd *scp)
} else } else
cfg->state = STATE_NORMAL; cfg->state = STATE_NORMAL;
wake_up_all(&cfg->reset_waitq); wake_up_all(&cfg->reset_waitq);
scsi_unblock_requests(cfg->host);
break; break;
case STATE_RESET: case STATE_RESET:
wait_event(cfg->reset_waitq, cfg->state != STATE_RESET); wait_event(cfg->reset_waitq, cfg->state != STATE_RESET);
......
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