diff options
author | Douglas Gilbert <dgilbert@interlog.com> | 2014-06-04 10:58:30 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-07-17 16:07:34 -0400 |
commit | 16070cc189c5e343696c29c8cff779e692cfcb8d (patch) | |
tree | 00edc84093c62557235b118fe8e3e28441c07dce /drivers/scsi/sg.c | |
parent | 65c26a0f39695ba01d9693754f27ca76cc8a3ab5 (diff) |
sg: add SG_FLAG_Q_AT_TAIL flag
When the SG_IO ioctl was copied into the block layer and
later into the bsg driver, subtle differences emerged.
One difference is the way injected commands are queued through
the block layer (i.e. this is not SCSI device queueing nor SATA
NCQ). Summarizing:
- SG_IO in the block layer: blk_exec*(at_head=false)
- sg SG_IO: at_head=true
- bsg SG_IO: at_head=true
Some time ago Boaz Harrosh introduced a sg v4 flag called
BSG_FLAG_Q_AT_TAIL to override the bsg driver default.
This patch does the equivalent for the sg driver.
ChangeLog:
Introduce SG_FLAG_Q_AT_TAIL flag to cause commands
to be injected into the block layer with
at_head=false.
Signed-off-by: Douglas Gilbert <dgilbert@interlog.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Reviewed-by: Ewan D. Milne <emilne@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/sg.c')
-rw-r--r-- | drivers/scsi/sg.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c index 37fb44b0074b..32425ac61096 100644 --- a/drivers/scsi/sg.c +++ b/drivers/scsi/sg.c | |||
@@ -741,7 +741,7 @@ static int | |||
741 | sg_common_write(Sg_fd * sfp, Sg_request * srp, | 741 | sg_common_write(Sg_fd * sfp, Sg_request * srp, |
742 | unsigned char *cmnd, int timeout, int blocking) | 742 | unsigned char *cmnd, int timeout, int blocking) |
743 | { | 743 | { |
744 | int k, data_dir; | 744 | int k, data_dir, at_head; |
745 | Sg_device *sdp = sfp->parentdp; | 745 | Sg_device *sdp = sfp->parentdp; |
746 | sg_io_hdr_t *hp = &srp->header; | 746 | sg_io_hdr_t *hp = &srp->header; |
747 | 747 | ||
@@ -785,11 +785,16 @@ sg_common_write(Sg_fd * sfp, Sg_request * srp, | |||
785 | break; | 785 | break; |
786 | } | 786 | } |
787 | hp->duration = jiffies_to_msecs(jiffies); | 787 | hp->duration = jiffies_to_msecs(jiffies); |
788 | if (hp->interface_id != '\0' && /* v3 (or later) interface */ | ||
789 | (SG_FLAG_Q_AT_TAIL & hp->flags)) | ||
790 | at_head = 0; | ||
791 | else | ||
792 | at_head = 1; | ||
788 | 793 | ||
789 | srp->rq->timeout = timeout; | 794 | srp->rq->timeout = timeout; |
790 | kref_get(&sfp->f_ref); /* sg_rq_end_io() does kref_put(). */ | 795 | kref_get(&sfp->f_ref); /* sg_rq_end_io() does kref_put(). */ |
791 | blk_execute_rq_nowait(sdp->device->request_queue, sdp->disk, | 796 | blk_execute_rq_nowait(sdp->device->request_queue, sdp->disk, |
792 | srp->rq, 1, sg_rq_end_io); | 797 | srp->rq, at_head, sg_rq_end_io); |
793 | return 0; | 798 | return 0; |
794 | } | 799 | } |
795 | 800 | ||