Re: [openib-general] [PATCH] IB/SRP - increase supported CDB size

2006-11-17 Thread Arne Redlich
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, Arne

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-24 Thread Arne Redlich
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 towards it being a bug. The data

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-21 Thread Arne Redlich
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. Tsirkin: Quoting r. Arne Redlich

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-20 Thread Arne Redlich
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 Hefty: Arne Redlich wrote: Yep - the Gen1

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-19 Thread Arne Redlich
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)) { ret = PTR_ERR

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-19 Thread Arne Redlich
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't it have a DGID from a path record

Re: [openib-general] [PATCH] ib_cm: fix REJ due to invalid GID

2006-07-19 Thread Arne Redlich
the reject param. 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/listinfo/openib

[openib-general] [PATCH 0/2] ib_cm: fix REJ on cm_id destruction

2006-07-18 Thread Arne Redlich
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/listinfo/openib-general

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-18 Thread Arne Redlich
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(revision 8556

Re: [openib-general] [PATCH 1/2] ib_cm: cm_destroy_id() cleanup

2006-07-18 Thread Arne Redlich
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: infiniband/core/cm.c

Re: [openib-general] [PATCH 1/2] ib_cm: cm_destroy_id() cleanup

2006-07-18 Thread Arne Redlich
. Signed-off-by: Arne Redlich [EMAIL PROTECTED] Index: infiniband/core/cm.c === --- infiniband/core/cm.c(revision 8565) +++ infiniband/core/cm.c(working copy) @@ -702,7 +702,7 @@ static void cm_reset_to_idle(struct

Re: [openib-general] [PATCH 1/2] ib_cm: cm_destroy_id() cleanup

2006-07-18 Thread 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: + cm_destroy_id(cm_id_priv-id, (ret

Re: [openib-general] [PATCH 2/2] ib_cm: fix REJ due to invalid GID

2006-07-18 Thread Arne Redlich
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 [EMAIL PROTECTED] Are there actual

Re: [openib-general] Re: Fwd: [Bug 91] sizeof(srp_indirect_buf) wrongon 64-bit platforms

2006-05-24 Thread Arne Redlich
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 operation, as

Re: [openib-general] Re: Fwd: [Bug 91] sizeof(srp_indirect_buf) wrong on 64-bit platforms

2006-05-23 Thread Arne Redlich
+ 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-general mailing list openib-general@openib.org http://openib.org

Re: [openib-general] Re: Fwd: [Bug 91] sizeof(srp_indirect_buf) wrongon 64-bit platforms

2006-05-23 Thread Arne Redlich
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 supposed to work on 2.6.16 ... As far as I can tell the bug

Re: [openib-general] SDP sk_data_ready() callback

2005-06-30 Thread Arne Redlich
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: Hi, I'm trying to use SDP

Re: [openib-general] SDP sk_data_ready() callback

2005-06-28 Thread Arne Redlich
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 is modified to wake up a Rx thread before

[openib-general] SDP sk_data_ready() callback

2005-06-22 Thread Arne Redlich
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 -- Arne