mq-deadline: Make sure to always unlock zones
authorDamien Le Moal <damien.lemoal@wdc.com>
Wed, 28 Feb 2018 17:35:29 +0000 (09:35 -0800)
committerJens Axboe <axboe@kernel.dk>
Thu, 1 Mar 2018 15:39:24 +0000 (08:39 -0700)
commitf3bc78d2d4b489590540ab2788d5376583e28173
tree0a6dd46bd1feef14581a7fd0ddc7d704a54c4952
parent468f098734ab095b72d8af26bcd4bcb2ed31a3be
mq-deadline: Make sure to always unlock zones

In case of a failed write request (all retries failed) and when using
libata, the SCSI error handler calls scsi_finish_command(). In the
case of blk-mq this means that scsi_mq_done() does not get called,
that blk_mq_complete_request() does not get called and also that the
mq-deadline .completed_request() method is not called. This results in
the target zone of the failed write request being left in a locked
state, preventing that any new write requests are issued to the same
zone.

Fix this by replacing the .completed_request() method with the
.finish_request() method as this method is always called whether or
not a request completes successfully. Since the .finish_request()
method is only called by the blk-mq core if a .prepare_request()
method exists, add a dummy .prepare_request() method.

Fixes: 5700f69178e9 ("mq-deadline: Introduce zone locking support")
Cc: Hannes Reinecke <hare@suse.com>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com>
[ bvanassche: edited patch description ]
Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
block/mq-deadline.c