* [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation
@ 2018-07-11 16:45 Varun Prakash
2018-07-12 16:57 ` Mike Christie
2018-07-13 3:05 ` Martin K. Petersen
0 siblings, 2 replies; 3+ messages in thread
From: Varun Prakash @ 2018-07-11 16:45 UTC (permalink / raw)
To: target-devel
- rounddown CXGBIT_MAX_ISO_PAYLOAD by csk->emss before
calculating max_iso_npdu to get max TCP payload
in multiple of mss.
- call cxgbit_set_digest() before cxgbit_set_iso_npdu()
to set csk->submode, it is used in calculating
number of iso pdus.
Signed-off-by: Varun Prakash <varun@chelsio.com>
---
drivers/target/iscsi/cxgbit/cxgbit_target.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/drivers/target/iscsi/cxgbit/cxgbit_target.c b/drivers/target/iscsi/cxgbit/cxgbit_target.c
index 514986b..25eb389 100644
--- a/drivers/target/iscsi/cxgbit/cxgbit_target.c
+++ b/drivers/target/iscsi/cxgbit/cxgbit_target.c
@@ -652,6 +652,7 @@ static int cxgbit_set_iso_npdu(struct cxgbit_sock *csk)
struct iscsi_param *param;
u32 mrdsl, mbl;
u32 max_npdu, max_iso_npdu;
+ u32 max_iso_payload;
if (conn->login->leading_connection) {
param = iscsi_find_param_from_key(MAXBURSTLENGTH,
@@ -670,8 +671,10 @@ static int cxgbit_set_iso_npdu(struct cxgbit_sock *csk)
mrdsl = conn_ops->MaxRecvDataSegmentLength;
max_npdu = mbl / mrdsl;
- max_iso_npdu = CXGBIT_MAX_ISO_PAYLOAD /
- (ISCSI_HDR_LEN + mrdsl +
+ max_iso_payload = rounddown(CXGBIT_MAX_ISO_PAYLOAD, csk->emss);
+
+ max_iso_npdu = max_iso_payload /
+ (ISCSI_HDR_LEN + mrdsl +
cxgbit_digest_len[csk->submode]);
csk->max_iso_npdu = min(max_npdu, max_iso_npdu);
@@ -741,6 +744,9 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
if (conn_ops->MaxRecvDataSegmentLength > cdev->mdsl)
conn_ops->MaxRecvDataSegmentLength = cdev->mdsl;
+ if (cxgbit_set_digest(csk))
+ return -1;
+
if (conn->login->leading_connection) {
param = iscsi_find_param_from_key(ERRORRECOVERYLEVEL,
conn->param_list);
@@ -764,7 +770,7 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
if (is_t5(cdev->lldi.adapter_type))
goto enable_ddp;
else
- goto enable_digest;
+ return 0;
}
if (test_bit(CDEV_ISO_ENABLE, &cdev->flags)) {
@@ -781,10 +787,6 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
}
}
-enable_digest:
- if (cxgbit_set_digest(csk))
- return -1;
-
return 0;
}
--
2.0.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation
2018-07-11 16:45 [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation Varun Prakash
@ 2018-07-12 16:57 ` Mike Christie
2018-07-13 3:05 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Mike Christie @ 2018-07-12 16:57 UTC (permalink / raw)
To: target-devel
On 07/11/2018 11:33 AM, Varun Prakash wrote:
> - rounddown CXGBIT_MAX_ISO_PAYLOAD by csk->emss before
> calculating max_iso_npdu to get max TCP payload
> in multiple of mss.
>
> - call cxgbit_set_digest() before cxgbit_set_iso_npdu()
> to set csk->submode, it is used in calculating
> number of iso pdus.
>
> Signed-off-by: Varun Prakash <varun@chelsio.com>
> ---
> drivers/target/iscsi/cxgbit/cxgbit_target.c | 16 +++++++++-------
> 1 file changed, 9 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/target/iscsi/cxgbit/cxgbit_target.c b/drivers/target/iscsi/cxgbit/cxgbit_target.c
> index 514986b..25eb389 100644
> --- a/drivers/target/iscsi/cxgbit/cxgbit_target.c
> +++ b/drivers/target/iscsi/cxgbit/cxgbit_target.c
> @@ -652,6 +652,7 @@ static int cxgbit_set_iso_npdu(struct cxgbit_sock *csk)
> struct iscsi_param *param;
> u32 mrdsl, mbl;
> u32 max_npdu, max_iso_npdu;
> + u32 max_iso_payload;
>
> if (conn->login->leading_connection) {
> param = iscsi_find_param_from_key(MAXBURSTLENGTH,
> @@ -670,8 +671,10 @@ static int cxgbit_set_iso_npdu(struct cxgbit_sock *csk)
> mrdsl = conn_ops->MaxRecvDataSegmentLength;
> max_npdu = mbl / mrdsl;
>
> - max_iso_npdu = CXGBIT_MAX_ISO_PAYLOAD /
> - (ISCSI_HDR_LEN + mrdsl +
> + max_iso_payload = rounddown(CXGBIT_MAX_ISO_PAYLOAD, csk->emss);
> +
> + max_iso_npdu = max_iso_payload /
> + (ISCSI_HDR_LEN + mrdsl +
> cxgbit_digest_len[csk->submode]);
>
> csk->max_iso_npdu = min(max_npdu, max_iso_npdu);
> @@ -741,6 +744,9 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
> if (conn_ops->MaxRecvDataSegmentLength > cdev->mdsl)
> conn_ops->MaxRecvDataSegmentLength = cdev->mdsl;
>
> + if (cxgbit_set_digest(csk))
> + return -1;
> +
> if (conn->login->leading_connection) {
> param = iscsi_find_param_from_key(ERRORRECOVERYLEVEL,
> conn->param_list);
> @@ -764,7 +770,7 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
> if (is_t5(cdev->lldi.adapter_type))
> goto enable_ddp;
> else
> - goto enable_digest;
> + return 0;
> }
>
> if (test_bit(CDEV_ISO_ENABLE, &cdev->flags)) {
> @@ -781,10 +787,6 @@ static int cxgbit_set_params(struct iscsi_conn *conn)
> }
> }
>
> -enable_digest:
> - if (cxgbit_set_digest(csk))
> - return -1;
> -
> return 0;
> }
>
Looks ok to me.
Reviewed-by: Mike Christie <mchristi@redhat.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation
2018-07-11 16:45 [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation Varun Prakash
2018-07-12 16:57 ` Mike Christie
@ 2018-07-13 3:05 ` Martin K. Petersen
1 sibling, 0 replies; 3+ messages in thread
From: Martin K. Petersen @ 2018-07-13 3:05 UTC (permalink / raw)
To: target-devel
Varun,
> - rounddown CXGBIT_MAX_ISO_PAYLOAD by csk->emss before
> calculating max_iso_npdu to get max TCP payload
> in multiple of mss.
>
> - call cxgbit_set_digest() before cxgbit_set_iso_npdu()
> to set csk->submode, it is used in calculating
> number of iso pdus.
Applied to 4.18/scsi-fixes. Thank you!
--
Martin K. Petersen Oracle Linux Engineering
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-07-13 3:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-11 16:45 [PATCH] target: iscsi: cxgbit: fix max iso npdu calculation Varun Prakash
2018-07-12 16:57 ` Mike Christie
2018-07-13 3:05 ` Martin K. Petersen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.