Commit 79ce12a8 authored by Bart Van Assche's avatar Bart Van Assche Committed by Jens Axboe

skd: Drop second argument of skd_recover_requests()

Since all callers pass zero as second argument to skd_recover_requests(),
drop that second argument.
Signed-off-by: default avatarBart Van Assche <bart.vanassche@wdc.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent c830da8c
...@@ -437,7 +437,7 @@ static void skd_release_special(struct skd_device *skdev, ...@@ -437,7 +437,7 @@ static void skd_release_special(struct skd_device *skdev,
struct skd_special_context *skspcl); struct skd_special_context *skspcl);
static void skd_disable_interrupts(struct skd_device *skdev); static void skd_disable_interrupts(struct skd_device *skdev);
static void skd_isr_fwstate(struct skd_device *skdev); static void skd_isr_fwstate(struct skd_device *skdev);
static void skd_recover_requests(struct skd_device *skdev, int requeue); static void skd_recover_requests(struct skd_device *skdev);
static void skd_soft_reset(struct skd_device *skdev); static void skd_soft_reset(struct skd_device *skdev);
const char *skd_drive_state_to_str(int state); const char *skd_drive_state_to_str(int state);
...@@ -930,7 +930,7 @@ static void skd_timer_tick_not_online(struct skd_device *skdev) ...@@ -930,7 +930,7 @@ static void skd_timer_tick_not_online(struct skd_device *skdev)
skdev->timer_countdown--; skdev->timer_countdown--;
return; return;
} }
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
break; break;
case SKD_DRVR_STATE_BUSY: case SKD_DRVR_STATE_BUSY:
...@@ -1027,13 +1027,13 @@ static void skd_timer_tick_not_online(struct skd_device *skdev) ...@@ -1027,13 +1027,13 @@ static void skd_timer_tick_not_online(struct skd_device *skdev)
/* It never came out of soft reset. Try to /* It never came out of soft reset. Try to
* recover the requests and then let them * recover the requests and then let them
* fail. This is to mitigate hung processes. */ * fail. This is to mitigate hung processes. */
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
else { else {
dev_err(&skdev->pdev->dev, "Disable BusMaster (%x)\n", dev_err(&skdev->pdev->dev, "Disable BusMaster (%x)\n",
skdev->drive_state); skdev->drive_state);
pci_disable_device(skdev->pdev); pci_disable_device(skdev->pdev);
skd_disable_interrupts(skdev); skd_disable_interrupts(skdev);
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
} }
/*start the queue so we can respond with error to requests */ /*start the queue so we can respond with error to requests */
...@@ -2935,7 +2935,7 @@ static void skd_isr_fwstate(struct skd_device *skdev) ...@@ -2935,7 +2935,7 @@ static void skd_isr_fwstate(struct skd_device *skdev)
break; break;
} }
if (skdev->state == SKD_DRVR_STATE_RESTARTING) if (skdev->state == SKD_DRVR_STATE_RESTARTING)
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
if (skdev->state == SKD_DRVR_STATE_WAIT_BOOT) { if (skdev->state == SKD_DRVR_STATE_WAIT_BOOT) {
skdev->timer_countdown = SKD_STARTING_TIMO; skdev->timer_countdown = SKD_STARTING_TIMO;
skdev->state = SKD_DRVR_STATE_STARTING; skdev->state = SKD_DRVR_STATE_STARTING;
...@@ -3009,7 +3009,7 @@ static void skd_isr_fwstate(struct skd_device *skdev) ...@@ -3009,7 +3009,7 @@ static void skd_isr_fwstate(struct skd_device *skdev)
case FIT_SR_DRIVE_FAULT: case FIT_SR_DRIVE_FAULT:
skd_drive_fault(skdev); skd_drive_fault(skdev);
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
blk_start_queue(skdev->queue); blk_start_queue(skdev->queue);
break; break;
...@@ -3018,7 +3018,7 @@ static void skd_isr_fwstate(struct skd_device *skdev) ...@@ -3018,7 +3018,7 @@ static void skd_isr_fwstate(struct skd_device *skdev)
dev_info(&skdev->pdev->dev, "state=0x%x sense=0x%x\n", state, dev_info(&skdev->pdev->dev, "state=0x%x sense=0x%x\n", state,
sense); sense);
skd_drive_disappeared(skdev); skd_drive_disappeared(skdev);
skd_recover_requests(skdev, 0); skd_recover_requests(skdev);
blk_start_queue(skdev->queue); blk_start_queue(skdev->queue);
break; break;
default: default:
...@@ -3032,7 +3032,7 @@ static void skd_isr_fwstate(struct skd_device *skdev) ...@@ -3032,7 +3032,7 @@ static void skd_isr_fwstate(struct skd_device *skdev)
skd_skdev_state_to_str(skdev->state), skdev->state); skd_skdev_state_to_str(skdev->state), skdev->state);
} }
static void skd_recover_requests(struct skd_device *skdev, int requeue) static void skd_recover_requests(struct skd_device *skdev)
{ {
int i; int i;
...@@ -3049,11 +3049,6 @@ static void skd_recover_requests(struct skd_device *skdev, int requeue) ...@@ -3049,11 +3049,6 @@ static void skd_recover_requests(struct skd_device *skdev, int requeue)
if (skreq->n_sg > 0) if (skreq->n_sg > 0)
skd_postop_sg_list(skdev, skreq); skd_postop_sg_list(skdev, skreq);
if (requeue &&
(unsigned long) ++skreq->req->special <
SKD_MAX_RETRIES)
blk_requeue_request(skdev->queue, skreq->req);
else
skd_end_request(skdev, skreq, BLK_STS_IOERR); skd_end_request(skdev, skreq, BLK_STS_IOERR);
skreq->req = NULL; skreq->req = NULL;
......
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