looks fine to me.
ming
On Wed, 2006-01-11 at 16:42 +0100, Arne Redlich wrote:
> As previously discussed, this is a slight modification of Ming's CmdSN
> patch. It fixes {Max,Exp}CmdSN values for discovery sessions and also
> incorporates a workaround to the AIX CmdSN issue by incrementing
> MaxCmdSN by 1.
>
> I hope it's alright to send this as one patch, as Ming's original patch
> formatting was corrupted and the changes are small enough.
>
> @K Chapman
> Could you please verify that the AIX issue is fixed by this patch? To do
> so, please use svn rev. 21 and apply this one on top of it. Thanks.
>
> Arne
>
> Index: usr/iscsid.c
> ===================================================================
> --- usr/iscsid.c (Revision 21)
> +++ usr/iscsid.c (Arbeitskopie)
> @@ -385,10 +385,9 @@
> /* } */
>
> ki->param_get(conn->tid, 0, key_session, conn->session_param);
> - conn->exp_cmd_sn = be32_to_cpu(req->cmd_sn);
> - log_debug(1, "exp_cmd_sn: %d,%d", conn->exp_cmd_sn,
> req->cmd_sn);
> - conn->max_cmd_sn = conn->exp_cmd_sn;
> }
> + conn->exp_cmd_sn = be32_to_cpu(req->cmd_sn);
> + log_debug(1, "exp_cmd_sn: %d,%d", conn->exp_cmd_sn, req->cmd_sn);
> text_key_add(conn, "TargetPortalGroupTag", "1");
> }
>
> @@ -572,6 +571,7 @@
> rsp->sid = conn->sid;
> rsp->stat_sn = cpu_to_be32(conn->stat_sn++);
> rsp->exp_cmd_sn = cpu_to_be32(conn->exp_cmd_sn);
> + conn->max_cmd_sn = conn->exp_cmd_sn + 1;
> rsp->max_cmd_sn = cpu_to_be32(conn->max_cmd_sn);
> return;
> init_err:
> @@ -658,6 +658,7 @@
>
> rsp->stat_sn = cpu_to_be32(conn->stat_sn++);
> rsp->exp_cmd_sn = cpu_to_be32(conn->exp_cmd_sn);
> + conn->max_cmd_sn = conn->exp_cmd_sn + 1;
> rsp->max_cmd_sn = cpu_to_be32(conn->max_cmd_sn);
> }
>
> @@ -676,6 +677,7 @@
>
> rsp->stat_sn = cpu_to_be32(conn->stat_sn++);
> rsp->exp_cmd_sn = cpu_to_be32(conn->exp_cmd_sn);
> + conn->max_cmd_sn = conn->exp_cmd_sn + 1;
> rsp->max_cmd_sn = cpu_to_be32(conn->max_cmd_sn);
> }
>
>
>
-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems? Stop! Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
|