Commit dcfad4ab authored by Daniel Wagner's avatar Daniel Wagner Committed by Keith Busch
Browse files

nvmet-fcloop: swap the list_add_tail arguments 



The first argument of list_add_tail function is the new element which
should be added to the list which is the second argument. Swap the
arguments to allow processing more than one element at a time.

Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarHannes Reinecke <hare@suse.de>
Signed-off-by: default avatarDaniel Wagner <dwagner@suse.de>
Signed-off-by: default avatarKeith Busch <kbusch@kernel.org>
parent 70fbfc47

drivers/nvme/target/fcloop.c

+3 −3
Original line number Diff line number Diff line
@@ -358,7 +358,7 @@ fcloop_h2t_ls_req(struct nvme_fc_local_port *localport, 
	if (!rport->targetport) {

		tls_req->status = -ECONNREFUSED;

		spin_lock(&rport->lock);

		list_add_tail(&rport->ls_list, &tls_req->ls_list);

		list_add_tail(&tls_req->ls_list, &rport->ls_list);

		spin_unlock(&rport->lock);

		queue_work(nvmet_wq, &rport->ls_work);

		return ret;
@@ -391,7 +391,7 @@ fcloop_h2t_xmt_ls_rsp(struct nvmet_fc_target_port *targetport, 
	if (remoteport) {

		rport = remoteport->private;

		spin_lock(&rport->lock);

		list_add_tail(&rport->ls_list, &tls_req->ls_list);

		list_add_tail(&tls_req->ls_list, &rport->ls_list);

		spin_unlock(&rport->lock);

		queue_work(nvmet_wq, &rport->ls_work);

	}
@@ -446,7 +446,7 @@ fcloop_t2h_ls_req(struct nvmet_fc_target_port *targetport, void *hosthandle, 
	if (!tport->remoteport) {

		tls_req->status = -ECONNREFUSED;

		spin_lock(&tport->lock);

		list_add_tail(&tport->ls_list, &tls_req->ls_list);

		list_add_tail(&tls_req->ls_list, &tport->ls_list);

		spin_unlock(&tport->lock);

		queue_work(nvmet_wq, &tport->ls_work);

		return ret;