Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9d696f40 authored by Bart Van Assche's avatar Bart Van Assche Committed by Greg Kroah-Hartman
Browse files

scsi: RDMA/srpt: Fix a credit leak for aborted commands 



commit 40ca8757291ca7a8775498112d320205b2a2e571 upstream.

Make sure that the next time a response is sent to the initiator that the
credit it had allocated for the aborted request gets freed.

Cc: Doug Ledford <dledford@redhat.com>
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: Nicholas Bellinger <nab@linux-iscsi.org>
Cc: Mike Christie <mchristi@redhat.com>
Cc: Hannes Reinecke <hare@suse.com>
Cc: Christoph Hellwig <hch@lst.de>
Fixes: 131e6abc ("target: Add TFO->abort_task for aborted task resources release") # v3.15
Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f16e8317

drivers/infiniband/ulp/srpt/ib_srpt.c

+11 −0
Original line number Diff line number Diff line
@@ -2793,8 +2793,19 @@ static void srpt_queue_tm_rsp(struct se_cmd *cmd) 
	srpt_queue_response(cmd);

}



/*

 * This function is called for aborted commands if no response is sent to the

 * initiator. Make sure that the credits freed by aborting a command are

 * returned to the initiator the next time a response is sent by incrementing

 * ch->req_lim_delta.

 */

static void srpt_aborted_task(struct se_cmd *cmd)

{

	struct srpt_send_ioctx *ioctx = container_of(cmd,

				struct srpt_send_ioctx, cmd);

	struct srpt_rdma_ch *ch = ioctx->ch;



	atomic_inc(&ch->req_lim_delta);

}



static int srpt_queue_status(struct se_cmd *cmd)