On 05/05/2017 12:50 AM, Bart Van Assche wrote: > Let the target core check the CMD_T_ABORTED flag instead of the SRP > target driver. Hence remove the transport_check_aborted_status() > call. Since state == SRPT_STATE_CMD_RSP_SENT is something that really > should not happen, do not try to recover if srpt_queue_response() is > called for an I/O context that is in that state. This patch is a bug > fix because the srpt_abort_cmd() call is misplaced - if that function > is called from srpt_queue_response() it should either be called > before the command state is changed or after the response has been > sent. > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Cc: Doug Ledford <dledford@xxxxxxxxxx> > Cc: Hannes Reinecke <hare@xxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxx> > Cc: Andy Grover <agrover@xxxxxxxxxx> > Cc: David Disseldorp <ddiss@xxxxxxx> > --- > drivers/infiniband/ulp/srpt/ib_srpt.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- 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
