Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 4 Mar 2021 14:39:48 GMT
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: a00bf7d9bba1 - stable/13 - Make DataSN counter of solicited Data-Out local.
Message-ID:  <202103041439.124Edm4S006548@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by mav:

URL: https://cgit.FreeBSD.org/src/commit/?id=a00bf7d9bba1dd359f521b3d11b483e32fb4de2a

commit a00bf7d9bba1dd359f521b3d11b483e32fb4de2a
Author:     Alexander Motin <mav@FreeBSD.org>
AuthorDate: 2021-02-02 18:37:13 +0000
Commit:     Alexander Motin <mav@FreeBSD.org>
CommitDate: 2021-03-04 14:39:43 +0000

    Make DataSN counter of solicited Data-Out local.
    
    DataSN for solicited Data-Out is per-R2T.  Since we handle whole R2T
    in one go, we don't need to store it anywhere, especially in global
    per-command structure.  This may allow us to handle multiple R2T per
    command at once, if we decide, or may be relax locking.
    
    Rename the second use of that field to io_referenced_task_tag.
    
    MFC after:      1 month
    
    (cherry picked from commit 3dd2a7a5ea2f1641c7525f692eed416fa02c28e6)
---
 sys/dev/iscsi/iscsi.c | 9 ++++-----
 sys/dev/iscsi/iscsi.h | 2 +-
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/sys/dev/iscsi/iscsi.c b/sys/dev/iscsi/iscsi.c
index 5e6c80255bff..c4123d9f4aa7 100644
--- a/sys/dev/iscsi/iscsi.c
+++ b/sys/dev/iscsi/iscsi.c
@@ -1018,7 +1018,7 @@ iscsi_pdu_handle_task_response(struct icl_pdu *response)
 		ISCSI_SESSION_WARN(is, "task response 0x%x",
 		    bhstmr->bhstmr_response);
 	} else {
-		aio = iscsi_outstanding_find(is, io->io_datasn);
+		aio = iscsi_outstanding_find(is, io->io_referenced_task_tag);
 		if (aio != NULL && aio->io_ccb != NULL)
 			iscsi_session_terminate_task(is, aio, CAM_REQ_ABORTED);
 	}
@@ -1157,6 +1157,7 @@ iscsi_pdu_handle_r2t(struct icl_pdu *response)
 	struct ccb_scsiio *csio;
 	size_t off, len, total_len;
 	int error;
+	uint32_t datasn = 0;
 
 	is = PDU_SESSION(response);
 
@@ -1183,8 +1184,6 @@ iscsi_pdu_handle_r2t(struct icl_pdu *response)
 	 * XXX: Verify R2TSN.
 	 */
 
-	io->io_datasn = 0;
-
 	off = ntohl(bhsr2t->bhsr2t_buffer_offset);
 	if (off > csio->dxfer_len) {
 		ISCSI_SESSION_WARN(is, "target requested invalid offset "
@@ -1234,7 +1233,7 @@ iscsi_pdu_handle_r2t(struct icl_pdu *response)
 		    bhsr2t->bhsr2t_initiator_task_tag;
 		bhsdo->bhsdo_target_transfer_tag =
 		    bhsr2t->bhsr2t_target_transfer_tag;
-		bhsdo->bhsdo_datasn = htonl(io->io_datasn++);
+		bhsdo->bhsdo_datasn = htonl(datasn++);
 		bhsdo->bhsdo_buffer_offset = htonl(off);
 		error = icl_pdu_append_data(request, csio->data_ptr + off, len,
 		    M_NOWAIT);
@@ -2204,7 +2203,7 @@ iscsi_action_abort(struct iscsi_session *is, union ccb *ccb)
 		xpt_done(ccb);
 		return;
 	}
-	io->io_datasn = aio->io_initiator_task_tag;
+	io->io_referenced_task_tag = aio->io_initiator_task_tag;
 
 	bhstmr = (struct iscsi_bhs_task_management_request *)request->ip_bhs;
 	bhstmr->bhstmr_opcode = ISCSI_BHS_OPCODE_TASK_REQUEST;
diff --git a/sys/dev/iscsi/iscsi.h b/sys/dev/iscsi/iscsi.h
index ddd88ffc5289..80ac9d877107 100644
--- a/sys/dev/iscsi/iscsi.h
+++ b/sys/dev/iscsi/iscsi.h
@@ -45,7 +45,7 @@ struct iscsi_outstanding {
 	union ccb			*io_ccb;
 	size_t				io_received;
 	uint32_t			io_initiator_task_tag;
-	uint32_t			io_datasn;
+	uint32_t			io_referenced_task_tag;
 	void				*io_icl_prv;
 };
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202103041439.124Edm4S006548>