On Thu, 2014-06-05 at 23:30 +0000, Nicholas A. Bellinger wrote: > From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx> > > Hi Vasu, > > This series generates SAM_STAT_TASK_SET_FULL status for lport->tt.seq_send() > failures in DataIN + response status codepaths, which is done in order to get > the initiator to reduce it's current queue_depth thus reducing the number of > outstanding I/Os permitted in flight on the wire. > > For the DataIN case, once a lport->tt.seq_send() failure occurs it will stop > attempting to send subsequent DataIN payload data, and immediately attempt to > send a response packet with SAM_STAT_TASK_SET_FULL status. > > For the response case, once a lport->tt.seq_send() failure occurs it will > set SAM_STAT_TASK_SET_FULL status and return -ENOMEM to the target, forcing > the response to be requeued by target-core > Good change and it was missing in tcm_fc. > Note this series has been compile tested only, so please review + test. Just did review and basic test but should be stressed to really ensure it does work & fixes issues resulting these patches. Thanks, Vasu > Thank you, > > --nab > > Nicholas Bellinger (2): > tcm_fc: Generate TASK_SET_FULL status for DataIN failures > tcm_fc: Generate TASK_SET_FULL status for response failures > > drivers/target/tcm_fc/tfc_cmd.c | 19 ++++++++++++++++--- > drivers/target/tcm_fc/tfc_io.c | 14 +++++++++++++- > 2 files changed, 29 insertions(+), 4 deletions(-) > -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
