> -----Original Message----- > From: linux-block-owner@xxxxxxxxxxxxxxx [mailto:linux-block- > owner@xxxxxxxxxxxxxxx] On Behalf Of Bart Van Assche ... > The intention of commit 6077c2d706097c0 was to address the last mentioned > case. It may be possible to move the delayed queue rerun from the > dm_queue_rq() into dm_requeue_original_request(). But I think it would be > wrong to rerun the queue immediately in case a SCSI target system returns > "BUSY". Seeing SCSI BUSY mentioned here... On its own, patch 6077c2d706 seems to be adding an arbitrarily selected magic value of 100 ms without explanation in the patch description or in the added code. But, dm_request_original_request() already seems to have chosen that value for similar purposes: unsigned long delay_ms = delay_requeue ? 100 : 0; So, making them share a #define would indicate there's a reason for that particular value. Any change to the value would be picked up everywhere. All the other callers of blk_mq_delay_run_hw_queue() use macros: drivers/md/dm-rq.c: blk_mq_delay_run_hw_queue(hctx, 100/*ms*/); drivers/nvme/host/fc.c: blk_mq_delay_run_hw_queue(queue->hctx, NVMEFC_QUEUE_DELAY); drivers/scsi/scsi_lib.c: blk_mq_delay_run_hw_queue(hctx, SCSI_QUEUE_DELAY); drivers/scsi/scsi_lib.c: blk_mq_delay_run_hw_queue(hctx, SCSI_QUEUE_DELAY); Those both happen to be set to 3 (ms). --- Robert Elliott, HPE Persistent Memory -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
