Re: [openib-general] bug report: mad.c: ib_req_notify_cq called without polling cq

2006-06-11 Thread Michael S. Tsirkin
Quoting r. Sean Hefty <[EMAIL PROTECTED]>: > This is arming the CQ _before_ we post MADs to the receive queue of the QP. I > don't think that there's a race here. Good point, thanks. -- MST ___ openib-general mailing list openib-general@openib.org ht

Re: [openib-general] bug report: mad.c: ib_req_notify_cq called without polling cq

2006-06-11 Thread Sean Hefty
>mad.c calls ib_req_notify_cq on hotplug event in ib_mad_port_start, after QPs >are attached to a CQ. Since this function does not poll the CQ, if sufficient >number of MADs arrive at the QP before ib_req_notify_cq is called, RQ might get >empty and no completion events will ever be generated. Th

Re: [openib-general] [PATCH 1/5] ib_addr: retrieve MGID from device address

2006-06-11 Thread Sean Hefty
>dev_addr->broadcast + 4/dev_addr->src_dev_addr + 4 may not be naturally >aligned, >so casting this pointer to structure type may cause compiler to generate >incorrect code. Thanks - I'll update this. - Sean ___ openib-general mailing list openib-gener

Re: [openib-general] [PATCH 0/5] multicast abstraction

2006-06-11 Thread Sean Hefty
>I am planning to use RDMA CM for multicast functionality. It would be >great if you can point me to a simple multicast test program using >RDMA CM? There is a userspace test program (mckey) that will be available, but has not been posted yet. (A kernel test program would look fairly similar.) I

Re: [openib-general] [PATCH] mad: prevent duplicate RMPP sessions on responder side

2006-06-11 Thread Sean Hefty
>Sean, is anyone looking at this? If not, given that Jack's approach does not >touch ABI or API, might it make sense to merge Jack's patch after all and use >that as a starting point? With current code in 2.6.17 large RMPPs often get >aborted because of the problem of the duplicates. On the other

Re: [openib-general] [PATCH 5/5] ucma: export multicast suport to userspace

2006-06-11 Thread Sean Hefty
> > @@ -58,6 +58,8 @@ enum { > > RDMA_USER_CM_CMD_GET_EVENT, > > RDMA_USER_CM_CMD_GET_OPTION, > > RDMA_USER_CM_CMD_SET_OPTION, > > + RDMA_USER_CM_CMD_JOIN_MCAST, > > + RDMA_USER_CM_CMD_LEAVE_MCAST, > > RDMA_USER_CM_CMD_GET_DST_ATTR > > }; > >I think this changes the exported AB

Re: [openib-general] [PATCH] osm: fix num of blocks of GUIDInfo GetTable query

2006-06-11 Thread Hal Rosenstock
On Sun, 2006-06-11 at 20:40, Greg Lindahl wrote: > So this is a _critical_ bugfix ? Depends on one's definition. Anyhow, it's been applied to 1.0. -- Hal > > > Auuuch it is there! > > My mistake. Sp please apply the patch to the OFED 1.0 branch too. > > BTW: Is the osmtest -f a excersizes thi

Re: [openib-general] [PATCH] osm: fix num of blocks of GUIDInfo GetTable query

2006-06-11 Thread Greg Lindahl
So this is a _critical_ bugfix ? > Auuuch it is there! > My mistake. Sp please apply the patch to the OFED 1.0 branch too. > BTW: Is the osmtest -f a excersizes this query on the OFED 1.0 ? > > > Huh ? What's > > > https://openfabrics.org/svn/gen2/branches/1.0/src/userspace/management/o > sm/ope

Re: [openib-general] [PATCH 5/5] ucma: export multicast suport to userspace

2006-06-11 Thread Roland Dreier
> @@ -58,6 +58,8 @@ enum { > RDMA_USER_CM_CMD_GET_EVENT, > RDMA_USER_CM_CMD_GET_OPTION, > RDMA_USER_CM_CMD_SET_OPTION, > +RDMA_USER_CM_CMD_JOIN_MCAST, > +RDMA_USER_CM_CMD_LEAVE_MCAST, > RDMA_USER_CM_CMD_GET_DST_ATTR > }; I think this changes the exported ABI by

Re: [openib-general] [PATCH 2/2] ipoib: handle multicast group reset notification

2006-06-11 Thread Roland Dreier
Sean> Any issue committing this? No, looks fine. - R. ___ 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

[openib-general] [Bug 1] kernel prints out error message for each ib interface

2006-06-11 Thread bugzilla-daemon
http://openib.org/bugzilla/show_bug.cgi?id=1 [EMAIL PROTECTED] changed: What|Removed |Added Status|RESOLVED|CLOSED --- Comment #3 from [EMAIL P

Re: [openib-general] [openfabrics-ewg] RE: OFED-1.0-rc6 is available

2006-06-11 Thread Pavel Shamis (Pasha)
We also did performance checks on different platforms, and the default MTU was changed to 1K (The 2k is more optimal for ddr platforms) Thank you for pointing the issue. Pavel Shamis (Pasha) Scott Weitzenkamp (sweitzen) wrote: > The MTU change undos the changes for bug 81, so I have reopened bug

Re: [openib-general] [PATCH] mad: prevent duplicate RMPP sessions on responder side

2006-06-11 Thread Michael S. Tsirkin
Quoting r. Sean Hefty <[EMAIL PROTECTED]>: > Subject: Re: [PATCH] mad: prevent duplicate RMPP sessions on responder side > > Jack Morgenstein wrote: > >Prevent opening multiple RMPP MAD transaction sessions at responder side > >with the same TID, GID/LID, class. > > > >Could happen if RMPP request

[openib-general] bug report: mad.c: ib_req_notify_cq called without polling cq

2006-06-11 Thread Michael S. Tsirkin
Hello, Sean! mad.c calls ib_req_notify_cq on hotplug event in ib_mad_port_start, after QPs are attached to a CQ. Since this function does not poll the CQ, if sufficient number of MADs arrive at the QP before ib_req_notify_cq is called, RQ might get empty and no completion events will ever be gene

Re: [openib-general] [PATCH 0/5] multicast abstraction

2006-06-11 Thread amith rajith mamidala
Hi Sean, I am planning to use RDMA CM for multicast functionality. It would be great if you can point me to a simple multicast test program using RDMA CM? Thanks, Amith On Fri, 9 Jun 2006, Sean Hefty wrote: > Sean Hefty wrote: > > This patch series enhances support for joining and leaving multi

Re: [openib-general] [openfabrics-ewg] OFED 1.0-rc6 tarball available with working ipath driver

2006-06-11 Thread Tziporet Koren
Bryan O'Sullivan wrote: > Due to unfortunate timing, the ipath driver in OFED 1.0-rc6 does not > work correctly. You can download an updated tarball from here, for > which the ipath driver works fine: > > http://openib.red-bean.com/OFED-1.0-rc6+ipath.tar.bz2 > > Alternatively, pull the necessary p

[openib-general] [Bug 131] New: working with huge pages may crash the kernel on Suse10

2006-06-11 Thread bugzilla-daemon
http://openib.org/bugzilla/show_bug.cgi?id=131 Summary: working with huge pages may crash the kernel on Suse10 Product: OpenFabrics Linux Version: 1.0rc6 Platform: X86-64 OS/Version: Other Status: NEW Severity: normal

Re: [openib-general] [PATCH] osm: fix num of blocks of GUIDInfo GetTable query

2006-06-11 Thread Hal Rosenstock
On Sun, 2006-06-11 at 06:46, Eitan Zahavi wrote: > Auuuch it is there! > My mistake. Sp please apply the patch to the OFED 1.0 branch too. > BTW: Is the osmtest -f a excersizes this query on the OFED 1.0 ? Yes. -- Hal > > Huh ? What's > > > https://openfabrics.org/svn/gen2/branches/1.0/src/use

Re: [openib-general] [PATCH 1/5] ib_addr: retrieve MGID from device address

2006-06-11 Thread Michael S. Tsirkin
Quoting r. Sean Hefty <[EMAIL PROTECTED]>: > Subject: [PATCH 1/5] ib_addr: retrieve MGID from device address > > Extract the MGID used by ipoib for broadcast traffic from the device > address. > > Signed-off-by: Sean Hefty <[EMAIL PROTECTED]> > --- > This will be used to get the MCMemberRecord fo

Re: [openib-general] [PATCH] uDAPL openib-cma provider - add support for IB_CM_REQ_OPTIONS

2006-06-11 Thread Tziporet Koren
Jack put the bug fix to OFED 1.0. Tziporet -Original Message- From: James Lentini [mailto:[EMAIL PROTECTED] Sent: Saturday, June 10, 2006 1:12 AM To: Tziporet Koren Cc: Jack Morgenstein; openib-general; Arlin Davis Subject: Re: [openib-general] Re: [PATCH] uDAPL openib-cma provider - ad

Re: [openib-general] [PATCH] osm: fix num of blocks of GUIDInfo GetTable query

2006-06-11 Thread Eitan Zahavi
Auuuch it is there! My mistake. Sp please apply the patch to the OFED 1.0 branch too. BTW: Is the osmtest -f a excersizes this query on the OFED 1.0 ? > Huh ? What's > https://openfabrics.org/svn/gen2/branches/1.0/src/userspace/management/o sm/opens > m/osm_sa_guidinfo_record.c > > -- Hal > > >

Re: [openib-general] [PATCH] osm: fix num of blocks of GUIDInfo GetTable query

2006-06-11 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-06-11 at 02:07, Eitan Zahavi wrote: > Hi Hal, > > As the 1.0 does not support GUIDInfo I do not this patch is relevant to > 1.0 Huh ? What's https://openfabrics.org/svn/gen2/branches/1.0/src/userspace/management/osm/opensm/osm_sa_guidinfo_record.c -- Hal > > Eitan Zahav