On Wed, 2006-01-11 at 10:20 -0800, K Chapman wrote:
> --- Arne Redlich <arne.redlich@xxxxxxxxxxx> 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);
> > }
>
> ok. with rev21 + cmdsn patch + event.c patch (and
> other back compat patches) + ISCSI_CONN_IOV_MAX patch
> (to get 512k)
could u skip that ISCSI_CONN_IOV_MAX patch and see if works?
>
> iet builds fine, and aix still works.
as you know, what other OS/ini we should try? now we handled hp-ux and
aix. windows is ok, nw works from report of list. linux works.
>
> aaarrrggghhh!!!!
> FreeBSD rocks
>
> __________________________________________________
> Do You Yahoo!?
> Tired of spam? Yahoo! Mail has the best spam protection around
> http://mail.yahoo.com
>
>
> -------------------------------------------------------
> 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
> _______________________________________________
> Iscsitarget-devel mailing list
> Iscsitarget-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/iscsitarget-devel
-------------------------------------------------------
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
|
