target: Enforce hw_max_sectors for SCF_SCSI_DATA_SG_IO_CDB
authorNicholas Bellinger <nab@linux-iscsi.org>
Sun, 6 May 2012 19:59:45 +0000 (12:59 -0700)
committerNicholas Bellinger <nab@linux-iscsi.org>
Wed, 9 May 2012 22:05:16 +0000 (15:05 -0700)
commit7b4deef5d9a1fa74ccfbd07b4cfe795ee18fceca
tree514d8cf753dd60c5e576226cf36590daa072ac01
parentd5dc28eb92f2a2305a02cb3a5f1ed36542d47512
target: Enforce hw_max_sectors for SCF_SCSI_DATA_SG_IO_CDB

Instead of depending upon a max_sectors value that may be set via
configfs based upon original HW queue limitations, go ahead and convert to using
the hw_max_sectors reported by the backend device in order to determine when
to reject an I/O's who's sector count exceeds what is supported by the backend
with a single se_cmd descriptor.

It addresses a potential case where se_dev_attrib.max_sectors for IBLOCK
backends has already been set via queue_max_sectors() to something small
like max_sectors=32 (LVM, DRBD may do this), resulting typically sized
SCF_SCSI_DATA_SG_IO_CDB to be incorrectly rejected with invalid_cdb_field
in transport_generic_cmd_sequencer().

Reviewed-by: Christoph Hellwig <hch@lst.de>
Cc: Roland Dreier <roland@purestorage.com>
Cc: Andy Grover <agrover@redhat.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_transport.c