RE: [PATCH] iser: fix local conn params being reset to default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



"they" is referred to the user. I'll make it more informative next time.



-----Original Message-----
From: Roi Dayan 
Sent: Wednesday, October 29, 2014 8:37 AM
To: Or Gerlitz
Cc: fujita.tomonori@xxxxxxxxxxxxx; stgt@xxxxxxxxxxxxxxx
Subject: RE: [PATCH] iser: fix local conn params being reset to default

Local params:

MaxXmitDataSegmentLength
MaxQueueCmd





-----Original Message-----
From: Or Gerlitz
Sent: Wednesday, October 29, 2014 7:52 AM
To: Roi Dayan
Cc: fujita.tomonori@xxxxxxxxxxxxx; stgt@xxxxxxxxxxxxxxx
Subject: Re: [PATCH] iser: fix local conn params being reset to default

On 10/28/2014 5:41 PM, Roi Dayan wrote:
> Instead of reseting the params to default if they pass it we should 
> reset to max if they pass it.

who are they? can you remind me what are the local params?

Or.

>
> Signed-off-by: Roi Dayan <roid@xxxxxxxxxxxx>
> ---
>   usr/iscsi/iser_text.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/usr/iscsi/iser_text.c b/usr/iscsi/iser_text.c index
> 8180dbc..b8c5136 100644
> --- a/usr/iscsi/iser_text.c
> +++ b/usr/iscsi/iser_text.c
> @@ -342,8 +342,8 @@ static int iser_login_check_params(struct iscsi_connection *iscsi_conn,
>   		if (p[i].state == KEY_STATE_START && p[i].val != session_keys[i].def) {
>   			if (iscsi_conn->state == STATE_LOGIN) {
>   				if (i >= ISCSI_PARAM_FIRST_LOCAL) {
> -					if (p[i].val > session_keys[i].def)
> -						p[i].val = session_keys[i].def;
> +					if (p[i].val > session_keys[i].max)
> +						p[i].val = session_keys[i].max;
>   					p[i].state = KEY_STATE_DONE;
>   					continue;
>   				}

--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SCSI]     [Linux RAID]     [Linux Clusters]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]

  Powered by Linux