On Sun, May 06, 2012 at 09:54:48PM +0000, Nicholas A. Bellinger wrote: > From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx> > > This patch changes the received sectors vs. se_dev_attrib.max_sectors > check to fail only when checking against passthrough (pSCSI) backends, > instead of against all existing max_sectors that may be reflecting a > smaller reported value due to underlying HW limitations. > > 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(). max_sectors for iblock now always is UINT_MAX. We still need to limit the request size for the file backend as well for now, so I think this patch is incorrect. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
