[announce] librdmacm release 1.0.11

2010-01-13 Thread Sean Hefty
I've published a 1.0.11 release of librdmacm. It contains updates to some of the sample programs. Changes from the 1.0.10 release: librdmacm/mckey: report multicast events to the user librdmacm/cmatose: add support for ipv6 librdmacm/mckey: enforce local binding for unmapped multicast address Up

RE: [PATCHv7 7/9] ib_core: Add API to support RoCEE from userspace

2010-01-13 Thread Sean Hefty
>Add ib_uverbs_get_mac() to be used by ibv_create_ah() to retirieve the remote >port's MAC address from the remote port's GID. Port link layer is also returned >by ibv_query_port(). Is this really to map the GID of a remote port to its MAC address, or is the mapping for a local port? If it's the

RE: [PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Sean Hefty
>Wouldn't trying to use RDMA_PS_SCTP hit the > > default: > return -EPROTONOSUPPORT; > >case in cma_get_port()? So one couldn't actually do anything with >RDMA_PS_SCTP? Or is there some way to avoid binding? You're right. There isn't any way to actually do anything with it.

Re: [PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Roland Dreier
> Technically, I believe it's available to userspace, though the librdmacm > doesn't > define it. I'd be surprised if anything uses it. Wouldn't trying to use RDMA_PS_SCTP hit the default: return -EPROTONOSUPPORT; case in cma_get_port()? So one couldn't actually do

Re: [PATCHv7 0/9] RoCEE support to Infiniband

2010-01-13 Thread Roland Dreier
> RoCEE was chosen for these patches in accordance with the terminology > used in the proposal discussed by the OFA. We have no problem > resubmitting these patches using "iboe" so that you can push them into > 2.6.34. We will need to stay aware of any future name recommendations > by relev

RE: [PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Sean Hefty
>The old wrong value has been there since we first merged this. So >changing it means anything using this in userspace would be broken. But >is PS_SCTP exported to userspace? And as far as I can tell PS_SCTP is >not implemented (this line is the only reference to RDMA_PS_SCTP in the >whole kerne

RE: [PATCHv7 0/9] RoCEE support to Infiniband

2010-01-13 Thread Liran Liss
RoCEE was chosen for these patches in accordance with the terminology used in the proposal discussed by the OFA. We have no problem resubmitting these patches using "iboe" so that you can push them into 2.6.34. We will need to stay aware of any future name recommendations by relevant industry bod

Re: [PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Roland Dreier
> -RDMA_PS_SCTP = 0x0183 > +RDMA_PS_SCTP = 0x0184 The old wrong value has been there since we first merged this. So changing it means anything using this in userspace would be broken. But is PS_SCTP exported to userspace? And as far as I can tell PS_SCTP is not implemented (this li

[PATCH] infiniband-diags/ibportstate.8: Add new supported ops

2010-01-13 Thread Hal Rosenstock
Added by Ralph Campbell in commit 3af8e05e01a92bb504c8e806c0c81dc5ef246f7f Signed-off-by: Hal Rosenstock --- diff --git a/infiniband-diags/man/ibportstate.8 b/infiniband-diags/man/ibportstate.8 index fea860e..c4c1b2a 100644 --- a/infiniband-diags/man/ibportstate.8 +++ b/infiniband-diags/man/ib

Re: [PATCH] infiniband-diags/ibportstate: allow changes to CA portinfo parameters

2010-01-13 Thread Hal Rosenstock
On Tue, Jan 12, 2010 at 5:11 AM, Sasha Khapyorsky wrote: > Hi Ralph, > > On 13:43 Mon 11 Jan , Ralph Campbell wrote: >> Its been 2 months since I posted this. >> What is the current status? Last comment I saw was from Ira >> saying either add this functionality to ibportstate or >> rename ibpo

Re: [PATCH] opensm/osm_qos.c: split switch external and end ports setup

2010-01-13 Thread Hal Rosenstock
On Tue, Jan 12, 2010 at 5:43 AM, Sasha Khapyorsky wrote: > On 08:40 Thu 07 Jan , Hal Rosenstock wrote: >> On Tue, Jan 5, 2010 at 6:10 AM, Sasha Khapyorsky wrote: >> > >> > This splits QoS related port parameters setup over switch external ports >> > and end ports. Such separation leaves us wi

Re: [PATCHv3] opensm: Add support for optimized SLtoVLMappingTable programming

2010-01-13 Thread Hal Rosenstock
On Tue, Jan 12, 2010 at 5:55 AM, Sasha Khapyorsky wrote: > On 08:35 Thu 07 Jan , Hal Rosenstock wrote: >> > >> > diff --git a/opensm/opensm/osm_qos.c b/opensm/opensm/osm_qos.c >> > index f54e995..6bbbfa2 100644 >> > --- a/opensm/opensm/osm_qos.c >> > +++ b/opensm/opensm/osm_qos.c >> > @@ -1,5

Re: [PATCH] opensm/osm_qos.c: merge SL2VL mapping capability check

2010-01-13 Thread Hal Rosenstock
On Tue, Jan 12, 2010 at 5:35 AM, Sasha Khapyorsky wrote: > On 08:38 Thu 07 Jan , Hal Rosenstock wrote: >> >> Change appears to be for switches to always rely on this bit rather >> than only when VLCap is 1. I wonder if there are any switches with >> VLCap > 1 that don't set the IsSLMappingSupp

Re: [PATCH] tests/subnet_discover: discover test utility

2010-01-13 Thread Hal Rosenstock
Hi Sasha, On Tue, Jan 12, 2010 at 4:31 AM, Sasha Khapyorsky wrote: > Hi Hal, > > On 08:56 Mon 11 Jan , Hal Rosenstock wrote: >> > >> > diff --git a/tests/subnet_discover.c b/tests/subnet_discover.c >> > index 7f8a85c..42e7aee 100644 >> > --- a/tests/subnet_discover.c >> > +++ b/tests/subnet_d

Re: [PATCH] infiniband-diags/ibqueryerrors: Add support for optional PortRcvErrorDetails counter

2010-01-13 Thread Hal Rosenstock
Hi Sasha, On Tue, Jan 12, 2010 at 2:14 PM, Sasha Khapyorsky wrote: > Hi Hal, > > On 14:10 Thu 31 Dec , Hal Rosenstock wrote: >> >> to --details option >> >> Signed-off-by: Hal Rosenstock >> --- >> diff --git a/infiniband-diags/man/ibqueryerrors.8 >> b/infiniband-diags/man/ibqueryerrors.8 >>

Re: [PATCH] IB/srp: Fix initiator lockup

2010-01-13 Thread Jason Gunthorpe
On Wed, Jan 13, 2010 at 07:57:26PM +0100, Bart Van Assche wrote: > On Wed, Jan 13, 2010 at 7:16 PM, Jason Gunthorpe > wrote: > > > > On Wed, Jan 13, 2010 at 08:23:27AM +0100, Bart Van Assche wrote: > > > On Wed, Jan 13, 2010 at 12:24 AM, Jason Gunthorpe > > > wrote: > > > > [ ... ] > > > > > > >

RE: [PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Sean Hefty
>Should be 132 rather than 131 according to IANA IP protocol IDs > >Signed-off-by: Hal Rosenstock Acked-by: Sean Hefty >--- Thanks. I just noticed this this morning after reviewing the luster patches. You beat me to the patch. >diff --git a/include/rdma/rdma_cm.h b/include/rdma/rdma_cm.h >in

[PATCH] rdma_cm.h: Fix SCTP protocol ID

2010-01-13 Thread Hal Rosenstock
Should be 132 rather than 131 according to IANA IP protocol IDs Signed-off-by: Hal Rosenstock --- diff --git a/include/rdma/rdma_cm.h b/include/rdma/rdma_cm.h index c6b2962..1c5e589 100644 --- a/include/rdma/rdma_cm.h +++ b/include/rdma/rdma_cm.h @@ -67,7 +67,7 @@ enum rdma_port_space {

Re: Documentation for libiverbs?

2010-01-13 Thread Bryan Green
Werther Pirani writes: > Hello, > > I've been redirected to this list from OpenFabrics' web site, so let me apo= > logise=20 > right away if this is not the right place for this question. > > We are writing an application that requires high throughput and low latency= > . More=20 > specifical

Re: [PATCH] IB/srp: Fix initiator lockup

2010-01-13 Thread Bart Van Assche
On Wed, Jan 13, 2010 at 7:16 PM, Jason Gunthorpe wrote: > > On Wed, Jan 13, 2010 at 08:23:27AM +0100, Bart Van Assche wrote: > > On Wed, Jan 13, 2010 at 12:24 AM, Jason Gunthorpe > > wrote: > > > [ ... ] > > > > > > Also, I couldn't tell for sure from a cursory examination of the > > > patch, but

Re: [PATCH] IB/srp: Fix initiator lockup

2010-01-13 Thread Jason Gunthorpe
On Wed, Jan 13, 2010 at 08:23:27AM +0100, Bart Van Assche wrote: > On Wed, Jan 13, 2010 at 12:24 AM, Jason Gunthorpe > wrote: > > [ ... ] > > > > Also, I couldn't tell for sure from a cursory examination of the > > patch, but the initiator must be designed to not stall processing the > > RQ depend

Re: [PATCH v2] opensm: Add update_desc command to opensm console

2010-01-13 Thread Sasha Khapyorsky
On 17:34 Wed 13 Jan , Eli Dorfman (Voltaire) wrote: > > Sometimes nodes are discovered by the opensm with their default > name before real host name was resolves and updated in the HCA. > Since Mellanox ConnectX HCA do not support NodeDescription > change trap (as part of trap 144) we need an

Re: [PATCH 1/3] rdma_cm: Add support for a new RDMA_PS_LUSTRE Lustre port space

2010-01-13 Thread Roland Dreier
> In general, I don't like the idea of application port spaces for QoS > support. Can't this be done using port numbers in the existing port > space? I agree. If setting QoS requires a kernel patch for each application, I think we've messed up QoS somehow. There needs to be a way to control

Re: [PATCH] IB/srp: Fix initiator lockup

2010-01-13 Thread Roland Dreier
> The impact of the ib_post_recv() call is small but measurable. I'll > publish measurement results soon. Interesting. If there is overhead you could amortize it by keeping some number of spare buffers posting, and batching your calls to ib_post_recv() (eg post a list of 8 buffers every 8 comm

RE: [PATCH 1/3] rdma_cm: Add support for a new RDMA_PS_LUSTRE Lustre port space

2010-01-13 Thread Sean Hefty
> This patch adds a new port space for use by Lustre traffic. > > This, along with patches to OpenSM and Lustre, allow to define a >specific QoS for lustre. In general, I don't like the idea of application port spaces for QoS support. Can't this be done using port numbers in the existing port s

[PATCH v2] opensm: Add update_desc command to opensm console

2010-01-13 Thread Eli Dorfman (Voltaire)
Sometimes nodes are discovered by the opensm with their default name before real host name was resolves and updated in the HCA. Since Mellanox ConnectX HCA do not support NodeDescription change trap (as part of trap 144) we need an option to refresh NodeDescription from console. Signed-off-by: El

Re: [PATCH] opensm: Add update_desc command to opensm console

2010-01-13 Thread Eli Dorfman (Voltaire)
Sasha Khapyorsky wrote: > Hi Eli, > > On 10:32 Tue 12 Jan , Eli Dorfman (Voltaire) wrote: >> Add update_desc command to opensm console >> >> Sometimes nodes are discovered by the opensm with their default >> name before real host name was resolves and updated in the HCA. >> Since Mellanox Conn

[PATCH 2/3] opensm/qos_policy: Add a new service ID and keyword for Lustre QoS

2010-01-13 Thread sebastien dugue
This patch adds a new service ID for the Lustre ULP as well as the lexer and parser bits for the policy definition simplified syntax. The QoS policy parser now supports the following 'qos-ulps' keywords: lustre : 0 #default SL for Lustre lustre, target-port-gui

[PATCH 1/3] rdma_cm: Add support for a new RDMA_PS_LUSTRE Lustre port space

2010-01-13 Thread sebastien dugue
This patch adds a new port space for use by Lustre traffic. This, along with patches to OpenSM and Lustre, allow to define a specific QoS for lustre. Signed-off-by: Sebastien Dugue --- drivers/infiniband/core/cma.c |5 + include/rdma/rdma_cm.h| 11 ++- 2 files cha

[PATCH 3/3] Add new port_space parameter to ko2iblnd module

2010-01-13 Thread sebastien dugue
This patch is only given as a reference. Index: b/lnet/klnds/o2iblnd/o2iblnd.c === --- a/lnet/klnds/o2iblnd/o2iblnd.c +++ b/lnet/klnds/o2iblnd/o2iblnd.c @@ -2503,7 +2503,7 @@ kiblnd_startup (lnet_ni_t *ni) ibdev->

[PATCH 0/3] Add support for specific Lustre Qos

2010-01-13 Thread sebastien dugue
These patches allow to have a specific QoS for Lustre traffic. - Patch 1 adds a new Lustre port space to the rdma_cm kernel module - Patch 2 defines a new Lustre service ID (matching the new RDMA CMA port space) and a new 'lustre' keyword in the OpenSM QoS policy parser. - Patch

Re: Documentation for libiverbs?

2010-01-13 Thread Tziporet Koren
On 1/13/2010 12:06 PM, Werther Pirani wrote: Hello, I've been redirected to this list from OpenFabrics' web site, so let me apologise right away if this is not the right place for this question. We are writing an application that requires high throughput and low latency. More specifically

Documentation for libiverbs?

2010-01-13 Thread Werther Pirani
Hello, I've been redirected to this list from OpenFabrics' web site, so let me apologise right away if this is not the right place for this question. We are writing an application that requires high throughput and low latency. More specifically, we need to take advantage of features like R

Re: [PATCH] IB/srp: Fix initiator lockup

2010-01-13 Thread Bart Van Assche
On Wed, Jan 13, 2010 at 12:24 AM, Jason Gunthorpe wrote: > I'd actually think that the decline caused by exhausting credit on > the initiator side is far worse than ib_post_recv.. If ib_post_recv > really is noticable then maybe just force loading the RQ after XX > unposted are outstanding. The p