On 01/26/2017 12:36 AM, Bart Van Assche wrote: > It is possible that two commands with the same tag are present on > sess_cmd_list because commands are removed from sess_cmd_list after > a response has been sent to the initiator. Hence continue searching > through sess_cmd_list even if a matching tag has already been found. > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxx> > Cc: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx> > Cc: Giridhar Malavali <giridhar.malavali@xxxxxxxxxx> > --- > drivers/target/target_core_tmr.c | 8 +++----- > 1 file changed, 3 insertions(+), 5 deletions(-) > Which arguably is an error in itself; we should return the response after all actions have been taken. But anyway. Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, -- 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
