[PATCH] opensm: Add update_desc command to opensm console

2010-01-12 Thread Eli Dorfman (Voltaire)
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 ConnectX HCA do not support NodeDescription change trap (as part of trap 144) we need an option to refresh

[PATCH] infiniband-diags: -V, --version: exit with zero status

2010-01-12 Thread Sasha Khapyorsky
When -V, --version command line option is used exit with zero status after a version showing. Signed-off-by: Sasha Khapyorsky sas...@voltaire.com --- On 08:03 Tue 05 Jan , Sean Hefty wrote: Changes to the above components should be made to Sasha's tree first, or they risk being lost

Re: [PATCH] libibverbs: Add huge page support to ibv_madvise_range()

2010-01-12 Thread Alex Vainman
Hi Roland, I would like to ask you if you have an estimation for when you will review the patches below? Thanks, AlexV Alex Vainman wrote: Hi Roland, Have you had a chance to look at this patch and at the patch I've sent with this email: Subject: [PATCH] libibverbs: Add huge page

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

2010-01-12 Thread Sasha Khapyorsky
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_discover.c @@ -40,6 +40,7 @@ static struct node *node_array[32 * 1024];  static

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

2010-01-12 Thread Sasha Khapyorsky
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 ibportstate but don't split it off into a new command. You are right, it

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

2010-01-12 Thread Sasha Khapyorsky
On 11:56 Thu 05 Nov , Ralph Campbell wrote: This patch adds new commands to ibportstate to support initializing the link for CAs connected back-to-back. It also allows more than one field to be changed at the same time such as lid 23 arm or width 1 speed 3 enable. Signed-off-by: Ralph

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

2010-01-12 Thread Sasha Khapyorsky
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 +1,6 @@  /*  * Copyright (c) 2006-2009 Voltaire, Inc. All rights

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

2010-01-12 Thread Sasha Khapyorsky
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 ConnectX HCA do not support

RE: RDMA Read sge errors

2010-01-12 Thread Jack Morgenstein
Adding our correspondence to the openfabrics list: I think it just sort of got dropped. I submitted a 5-patch set to Roland and the list on October 24, 2007. See the thread beginning at: http://lists.openfabrics.org/pipermail/general/2007-October/042351.html We didn't pursue it at the time.

Re: [PATCH] libibverbs: Add huge page support to ibv_madvise_range()

2010-01-12 Thread Eli Cohen
On Sun, Nov 29, 2009 at 07:08:08PM +0200, Alex Vainman wrote: + p_hpage_val = strstr(buf, label); + if (!p_hpage_val) { + errno = EINVAL; + return -1; + } + p_hpage_val += strlen(label); + + errno = 0; + ret_val = strtol(p_hpage_val,

Re: [PATCH] opensm SA DB: dump only if modified

2010-01-12 Thread Sasha Khapyorsky
On 10:47 Mon 11 Jan , Yevgeny Kliteynik wrote: Currently, if dumping SA DB is enabled, the file is dumped at every sweep. This patch adds dirty flag that denotes whether the SA DB needs to be dumped or not. SA DB contains 3 types of data: - informinfo records - service records -

Re: [PATCH] opensm/osm_vl_arb_rcv.c: Dump table after validating block number

2010-01-12 Thread Sasha Khapyorsky
On 14:15 Mon 04 Jan , Hal Rosenstock wrote: rather than before so don't access invalid memory Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com Applied. Thanks. Sasha -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to

OFED-RDMAoE-1.5 GA is available

2010-01-12 Thread Eli Cohen
Hi, I am pleased to announce that OFED-RDMAoE-1.5 GA release is done Notes: The tarball is available on: http://www.openfabrics.org/downloads/OFED/ofed-rdmaoe-1.5/OFED-RDMAoE-1.5.tgz To get BUILD_ID run ofed_info Please report any issues in bugzilla https://bugs.openfabrics.org/ for

Re: [PATCH] mlx4/IB: Add set_4k_mtu module parameter.

2010-01-12 Thread Roland Dreier
I'm not thrilled about this patch. +module_param_named(set_4k_mtu, mlx4_ib_set_4k_mtu, int, 0444); +MODULE_PARM_DESC(set_4k_mtu, attempt to set 4K MTU to all ConnectX ports); What does attempt to set 4K MTU mean? Try to set the MTU cap to 4K? What happens if it fails? Is there any reason

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

2010-01-12 Thread Roland Dreier
RoCEE (pronounced ‘rocky’) allows running the IB transport protocol using Ethernet frames, enabling the deployment of IB semantics on lossless Ethernet fabrics. Just starting to review this, but let me say at the start that I really don't like the term RoCEE for this stuff. The OFED 1.5

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

2010-01-12 Thread Roland Dreier
Note: I do not know of any SRP targets that implement approach (2). As far as I know all SRP targets use approach (1). But no other SRP targets send SRP_CRED_REQ -- and that includes all native IB storage arrays that I know of. So there must be some other way around this (and having

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

2010-01-12 Thread Bart Van Assche
On Tue, Jan 12, 2010 at 6:13 PM, Roland Dreier rdre...@cisco.com wrote:   Note: I do not know of any SRP targets that implement approach (2). As   far as I know all SRP targets use approach (1). But no other SRP targets send SRP_CRED_REQ -- and that includes all native IB storage arrays that

Re: [PATCH] libibmad: Add support for optional PortRcvErrorDetails counter

2010-01-12 Thread Sasha Khapyorsky
On 11:57 Thu 31 Dec , Hal Rosenstock wrote: Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com Applied. Thanks. Sasha -- 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

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

2010-01-12 Thread Sasha Khapyorsky
On 11:58 Thu 31 Dec , Hal Rosenstock wrote: Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com Applied. Thanks. Sasha -- 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

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

2010-01-12 Thread Sasha Khapyorsky
Hi Hal, On 14:10 Thu 31 Dec , Hal Rosenstock wrote: to --details option Signed-off-by: Hal Rosenstock hal.rosenst...@gmail.com --- diff --git a/infiniband-diags/man/ibqueryerrors.8 b/infiniband-diags/man/ibqueryerrors.8 index 83a2b5a..e7feea8 100644 ---

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

2010-01-12 Thread Roland Dreier
Having separate buffer sets for command receives and response sends might seem a sensible way to design a target. But calling ib_post_recv() before sending a response back to the initiator is not acceptable because this increases communication latency. When a target uses separate buffer

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

2010-01-12 Thread Jason Gunthorpe
On Tue, Jan 12, 2010 at 02:57:35PM -0800, Roland Dreier wrote: I doubt you could benchmark the overhead of calling ib_post_recv() in the full SRP protocol. Really, I bet it's less than 100 nanoseconds to form the work request and call ib_post_recv(). Maybe I'm wrong but I really expect the

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

2010-01-12 Thread Bart Van Assche
On Wed, Jan 13, 2010 at 12:24 AM, Jason Gunthorpe jguntho...@obsidianresearch.com 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 dependent on the SQ or the credit level, when using a

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

2010-01-12 Thread Bart Van Assche
On Tue, Jan 12, 2010 at 11:57 PM, Roland Dreier rdre...@cisco.com wrote:   Having separate buffer sets for command receives and response sends   might seem a sensible way to design a target. But calling   ib_post_recv() before sending a response back to the initiator is not   acceptable