Roland Dreier <[EMAIL PROTECTED]> writes:
> Definitely makes sense. I queued the following version for 2.6.20,
> which gets the max CDB size directly from struct srp_cmd. Does this
> look OK to you?
Looks good to me. I guess there's no way to sneak it into 2.6.19
anymore, right? ;)
Thanks,
Arn
This patch sets the Scsi_Host's max_cmd_len from 12 (default) to
16. Otherwise scsi_dispatch_cmd() won't pass down certain commands such
as READ CAPACITY 16, required for supporting disks > 2TB.
Signed-off-by: Arne Redlich <[EMAIL PROTECTED]>
diff --git a/drivers/infiniband/
Am Freitag, den 21.07.2006, 09:42 -0700 schrieb Sean Hefty:
> Arne Redlich wrote:
> > I only took a rather superficial look at the code. But since the invalid
> > GID REJ is treated with such care, I guess it's intentional and not a
> > bug?
>
> I would lean more t
Am Donnerstag, den 20.07.2006, 18:44 +0300 schrieb Michael S. Tsirkin:
> Quoting r. Arne Redlich <[EMAIL PROTECTED]>:
> > Subject: Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID
> >
> > Am Mittwoch, den 19.07.2006, 20:05 +0300 schrieb Michael S. Tsi
Am Mittwoch, den 19.07.2006, 20:05 +0300 schrieb Michael S. Tsirkin:
> Quoting r. Arne Redlich <[EMAIL PROTECTED]>:
> > Subject: Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID
> >
> > Am Dienstag, den 18.07.2006, 12:21 -0700 schrieb Sean Heft
k->path[0].sgid);
> goto error3;
Won't this lead to a second REJ (IB_CM_REJ_CONSUMER_DEFINED) being
issued by cm_destroy_id()? That's actually what I tried to prevent using
the "reject" param.
Arne
--
Arne Redlich
Xiranet Communications GmbH
_
Am Dienstag, den 18.07.2006, 12:21 -0700 schrieb Sean Hefty:
> Arne Redlich wrote:
> > Yep - the Gen1 SRP initiator does. It sends a REQ with an invalid DGID.
> > If rejected with the correct code (INVALID GID), it will retry after
> > looking up the GID.
>
> Didn
Am Dienstag, den 18.07.2006, 10:09 -0700 schrieb Sean Hefty:
> Arne Redlich wrote:
> > @@ -1354,6 +1354,7 @@ static int cm_req_handler(struct cm_work
> > id.local_id);
> > if (IS_ERR(cm_id_priv->timewait_info
Am Dienstag, den 18.07.2006, 17:43 +0300 schrieb Michael S. Tsirkin:
> Quoting r. Arne Redlich <[EMAIL PROTECTED]>:
> > Reject a REQ containing invalid GID(s) with appropriate reason codes
> > instead of IB_CM_CONSUMER_REJ.
> >
> > Signed-off-by: Arne Redlich &
Am Dienstag, den 18.07.2006, 17:31 +0300 schrieb Michael S. Tsirkin:
> Quoting r. Arne Redlich <[EMAIL PROTECTED]>:
> > Subject: Re: [PATCH 1/2] ib_cm: cm_destroy_id() cleanup
> >
> > Am Dienstag, den 18.07.2006, 06:59 -0700 schrieb Roland Dreier:
> > > > +
laces this magic number with
a more generic "reject" parameter.
Signed-off-by: Arne Redlich <[EMAIL PROTECTED]>
Index: infiniband/core/cm.c
===
--- infiniband/core/cm.c(revision 8565)
+++ infiniband/core
In its current incarnation, cm_destroy_id() will not send a REJ if fed a
magic number (err = -ENOMEM). This patch replaces this magic number with
a more generic "reject" parameter.
Signed-off-by: Arne Redlich <[EMAIL PROTECTED]>
Index: infi
Reject a REQ containing invalid GID(s) with appropriate reason codes
instead of IB_CM_CONSUMER_REJ.
Signed-off-by: Arne Redlich <[EMAIL PROTECTED]>
Index: infiniband/core/cm.c
===
--- infiniband/core/cm.c(revisio
rnel behaves as
expected.
Thanks,
Arne
--
Arne Redlich
Xiranet Communications GmbH
___
openib-general mailing list
openib-general@openib.org
http://openib.org/mailman/listinfo/openib-general
To unsubscribe, please visit http://openib.org/mailman/
Roland Dreier <[EMAIL PROTECTED]> writes:
> >>> I'm afraid it *does* have an effect, unfortunately.
>
> Hmm, go ahead and forward the fix from 2.6.17 to the stable team for
> kernel 2.6.16 if this bug affects your target.
>
> Thanks,
> Roland
It doesn't affect our target during regular oper
Ishai Rabinovitz <[EMAIL PROTECTED]> writes:
> On Tue, May 23, 2006 at 10:10:05AM +0300, Arne Redlich wrote:
>> Roland Dreier <[EMAIL PROTECTED]> writes:
>>
>> > > Roland, maybe this means we need scsi/srp.h in svn for now?
>> > > svn is suppo
sizeof *cmd +
sizeof *buf);
So if a target actually RDMA Reads the indirect descriptor table, it will use a
wrong address.
Arne
--
Arne Redlich
Xiranet Communications GmbH
___
openib-ge
Am Dienstag, den 28.06.2005, 15:32 -0700 schrieb Libor Michalek:
> On Tue, Jun 28, 2005 at 09:15:21AM +0200, Arne Redlich wrote:
> > Am Montag, den 27.06.2005, 11:35 -0700 schrieb Libor Michalek:
> > > On Wed, Jun 22, 2005 at 10:39:43AM +0200, Arne Redlich wrote:
> > >
Am Montag, den 27.06.2005, 11:35 -0700 schrieb Libor Michalek:
> On Wed, Jun 22, 2005 at 10:39:43AM +0200, Arne Redlich wrote:
> > Hi,
> >
> > I'm trying to use SDP from within the kernel. My problem is that the
> > code relies on sk_data_ready() (this callback
Hi,
I'm trying to use SDP from within the kernel. My problem is that the
code relies on sk_data_ready() (this callback is modified to wake up a
Rx thread before executing the original function), but sk_data_ready()
apparently never gets called. Is there any way to fix this?
Thanks,
Arne
--
20 matches
Mail list logo