Re: [PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

On Wed, Nov 02, 2011 at 11:31:38PM +, Davis, Arlin R wrote: > > > Yes, a user can modify path lifetime via rdma_create_ep() but there > > > is no way for the user to know how much that will be manipulated and > > > increased in the IB CM driver. > > > > Sure there is. It's an open source driv

[ANNOUNCE] dapl-2.0.34

New release for v2.0 (2.0.34) available at http://www.openfabrics.org/downloads/dapl Vlad, please pull into OFED 1.5.4 RC3 Thanks, Arlin -- Latest Package (see ChangeLog for recent changes): md5sum: d8114711c07fa9f4b7e52ab2803a9e8d dapl-2.0.34.tar.gz For 1.2 and 2.0 support on

RE: [PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

> > Yes, a user can modify path lifetime via rdma_create_ep() but there > > is no way for the user to know how much that will be manipulated and > > increased in the IB CM driver. > > Sure there is. It's an open source driver. :) > > The ib_cm calculates the "correct" timeout based on the packe

RE: [PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

> Yes, a user can modify path lifetime via rdma_create_ep() but there > is no way for the user to know how much that will be manipulated and > increased in the IB CM driver. Sure there is. It's an open source driver. :) The ib_cm calculates the "correct" timeout based on the packet lifetime pr

RE: [PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

> > For larger, more congested fabrics, a larger ACK timer is needed. > > Consumers can still change default with environment variable > > DAPL_ACK_TIMER if they need to increase or decrease. > > > > This applies to SCM and UCM providers only. The CMA provider, which > > uses rdma_cm, has no way t

RE: [PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

> For larger, more congested fabrics, a larger ACK timer is needed. > Consumers can still change default with environment variable > DAPL_ACK_TIMER if they need to increase or decrease. > > This applies to SCM and UCM providers only. The CMA provider, which > uses rdma_cm, has no way to control ac

[PATCH] DAPL v2.0: common: increase default IB ack timer from 16 to 20

For larger, more congested fabrics, a larger ACK timer is needed. Consumers can still change default with environment variable DAPL_ACK_TIMER if they need to increase or decrease. This applies to SCM and UCM providers only. The CMA provider, which uses rdma_cm, has no way to control ack timer with

[PATCH] DAPL v2.0: scm: change debug message level for listen/bind errors

reduce to CM_WARN instead of general WARN level. Signed-off-by: Arlin Davis --- dapl/openib_scm/cm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dapl/openib_scm/cm.c b/dapl/openib_scm/cm.c index b9cb1bc..b45a4ab 100644 --- a/dapl/openib_scm/cm.c +++ b/dapl/openib_sc

[PATCH] DAPL v2.0: common: dat_ep_modify seg faults with null ep_param ptr

Add additional NULL ptr check for arg3 ep_param Signed-off-by: Arlin Davis --- dapl/common/dapl_ep_modify.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dapl/common/dapl_ep_modify.c b/dapl/common/dapl_ep_modify.c index 9f0095f..0545870 100644 --- a/dapl/common/da

[PATCH] DAPL v2.0: common: posting events on full queue returns wrong error code

return DAT_QUEUE_FULL instead of DAT_INSUFFICIENT_RESOURCES Signed-off-by: Arlin Davis --- dapl/common/dapl_evd_util.c | 17 - 1 files changed, 8 insertions(+), 9 deletions(-) diff --git a/dapl/common/dapl_evd_util.c b/dapl/common/dapl_evd_util.c index 9171415..41423b2 100644

[PATCH] DAPL v2.0: common: remote ia address null pointer creates seg fault

add NULL ptr check and return DAT_INVALID_PARAMETER, DAT_INVALID_ARG2 Signed-off-by: Arlin Davis --- dapl/common/dapl_ep_connect.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/dapl/common/dapl_ep_connect.c b/dapl/common/dapl_ep_connect.c index 80afead..590d0ed 100

Re: [PATCH 4/5] ib/core: add support for extended performance counters in sysfs

On 11/1/2011 11:42 PM, Roland Dreier wrote: The least bad way forward does seem like it is probably the separate new directory thing Hi Roland, I suggest we go that least bad way along the lines of your comment. If/when on some future point something constructive can be formed from Jason's o

[PATCH-v3] ib_srpt: Initial SRP Target merge for v3.2-rc1

From: Bart Van Assche (Resending w/o the full body as it's exceeding the patch size limit.. ;) This v3 patch adds the kernel module ib_srpt SCSI RDMA Protocol (SRP) target implementation conforming to the SRP r16a specification for the mainline drivers/target infrastructure. This patch has been

[PATCH] ib_srpt: Convert port_guid to use subnet_prefix + interface_id formatting

From: Nicholas Bellinger This patch converts ib_srpt to use srpt_port->port_guid subnet_prefix + interface_id formatting in srpt_add_one() to identify individual HW target ports in /sys/kernel/config/target/srpt/$SRPT_WWPN This layout for SRP Target WWPNs and explict SRP Initiator Node WWPNs wil

Re: [PATCH][TRIVIAL] opensm/osm_sa_mcmember_record.c: Commentary changes relating to SA assigned MGIDs

Hi Hal, On 13:46 Mon 17 Oct , Hal Rosenstock wrote: > > Update one comment to be more accurate > and remove comment which appears to no longer apply > > Signed-off-by: Hal Rosenstock > --- Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body

Re: [PATCH] opensm: Only log SM PORT "state" when it changes

Hi Hal, On 11:41 Fri 30 Sep , Hal Rosenstock wrote: > > rather than every sweep time > so don't spam syslog and opensm.log when port stays down > > Signed-off-by: Hal Rosenstock > --- Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a

Re: [PATCH] opensm/main.c: Change size parameter in setvbuf call from 0 to BUFSIZ

Hi Hal, On 13:34 Mon 17 Oct , Hal Rosenstock wrote: > > to eliminate Win8 assert as this is shared code > > Signed-off-by: Hal Rosenstock > --- Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org Mo

Re: [PATCH][TRIVIAL] opensm: Fix typo in routing section in man page and doc

Hi Hal, On 08:37 Tue 01 Nov , Hal Rosenstock wrote: > > Signed-off-by: Hal Rosenstock > --- Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordom

Re: [PATCH] opensm/osmtest/osmtest.c: inventory file parsing bugfix

Hi Daniel, On 14:35 Wed 02 Nov , Daniel Klein wrote: > Parsing of 'mkey' field from inventory file fails due to precedences. > Instead of parsing 'mkey' field, it parsed 'mkey_violations'. > > Signed-off-by: Daniel Klein > --- Applied, thanks. -- To unsubscribe from this list: send the line

[PATCH] opensm/osmtest/osmtest.c: inventory file parsing bugfix

Parsing of 'mkey' field from inventory file fails due to precedences. Instead of parsing 'mkey' field, it parsed 'mkey_violations'. Signed-off-by: Daniel Klein --- osmtest/osmtest.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/osmtest/osmtest.c b/osmtest/osmtest.c in

Re: [PATCH 4/5] ib/core: add support for extended performance counters in sysfs

On 11/1/2011 11:58 PM, Jason Gunthorpe wrote: maybe you should patch to un-export them until things can be fixed sanely... Jason, I don't think we want to work this way, bug are either opened or attempted to be fixed or fixed. You can't just come out of the the blue and remove functionality

Re: [PATCH 4/5] ib/core: add support for extended performance counters in sysfs

On 11/2/2011 12:46 AM, Ira Weiny wrote: What do you mean "for both"? Ira The sysfs PMA counters are functional for both IB and IBoE, the latter uses a HW counter allocated per device/port for which all the QPs created on that port are reporting their rx/tx bytes/packets, see commits cfcde11c3