RE: [PATCH libmlx4 1/8] Add raw packet QP support

2012-09-20 Thread Luick, Dean
> -Original Message- > From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- > ow...@vger.kernel.org] On Behalf Of Or Gerlitz > Sent: Thursday, September 20, 2012 3:31 PM > To: rol...@kernel.org > Cc: linux-rdma@vger.kernel.org; Or Gerlitz > Subject: [PATCH libmlx4 1/8] Add raw packe

Re: [PATCH] infiniband-diags/src/perfquery.c: Fix all_ports corner case

2012-09-20 Thread Ira Weiny
On Thu, 13 Sep 2012 23:16:14 -0700 Al Chu wrote: > if AllPortSelect is not supported and --all_ports is specified, it should > only emulate AllPortSelect if no port is specified or the all ports > port number (255) is specified. > > Signed-off-by: Albert Chu Thanks applied, Ira > --- > src/p

Re: [PATCH] infiniband-diags: Allow user to specify multiple ports

2012-09-20 Thread Ira Weiny
On Thu, 13 Sep 2012 16:13:05 -0700 Albert Chu wrote: > Allow user to select multiple ports via comma or range input. May > be particularly useful for gathering aggregate performance counters > for groups of ports, such as all the uplinks or downlinks from a > switch. > > Signed-off-by: Albert C

Re: [PATCH for-next V2 04/22] IB/mlx4: SRIOV IB context objects and proxy/tunnel sqp support

2012-09-20 Thread Or Gerlitz
On Tue, Sep 11, 2012 at 8:10 PM, Doug Ledford wrote: > On 8/3/2012 4:40 AM, Jack Morgenstein wrote: > > struct mlx4_ib_sriov{} is created by the master only. > > It is a container for the following: > > 1. All the info required by the PPF to multiplex and de-multiplex MADs > >(including those

[PATCH 6/6] RDMA/nes: Fix for compilation error when nes_debug is enabled

2012-09-20 Thread Tatyana Nikolova
Removing old variables causing compile error from nes_debug() Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes_hw.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/nes/nes_hw.c b/drivers/infiniband/hw/nes/nes_hw.c index d42c9f4

RE: how to preserve QP over HA events for librdmacm applications

2012-09-20 Thread Hefty, Sean
> Fair enough, I understand one needs to use a different CM id. For the IB > case I was thinking of avoiding APM (since that is limited to a device > -isn't that so?). APM is limited to a single device, as is memory registration, CQs, PDs, SRQs, etc. Migration between devices requires entirely n

[PATCH 7/8] libibverbs: Add man page for ibv_open_qp

2012-09-20 Thread Hefty, Sean
Signed-off-by: Sean Hefty --- man/ibv_open_qp.3 | 50 ++ 1 files changed, 50 insertions(+), 0 deletions(-) create mode 100644 man/ibv_open_qp.3 diff --git a/man/ibv_open_qp.3 b/man/ibv_open_qp.3 new file mode 100644 index 000..0bc5647 --- /d

[PATCH 2/2] libmlx4: Add support for XRC QPs

2012-09-20 Thread Hefty, Sean
Signed-off-by: Sean Hefty --- Note that I have a hack in cq.c. Someone more familiar with the mlx4 HW needs to look at the change. src/buf.c |6 +- src/cq.c | 40 --- src/mlx4-abi.h |6 ++ src/mlx4.c | 19 +++-- src/mlx4.h | 59 src/qp.

[PATCH 1/2] libmlx4: Infra-structure changes to support verbs extensions

2012-09-20 Thread Hefty, Sean
From: Yishai Hadas Signed-off-by: Yishai Hadas Signed-off-by: Tzahi Oved --- src/mlx4.c | 83 +++- src/mlx4.h | 16 2 files changed, 70 insertions(+), 29 deletions(-) diff --git a/src/mlx4.c b/src/mlx4.c index 8cf249a..1

[PATCH 8/8] libibverbs: Add XRC sample application

2012-09-20 Thread Hefty, Sean
From: Jay Sternberg Signed-off-by: Jay Sternberg Signed-off-by: Sean Hefty --- Makefile.am |4 examples/xsrq_pingpong.c | 877 ++ 2 files changed, 880 insertions(+), 1 deletions(-) create mode 100644 examples/xsrq_pingpong.c diff

[PATCH 2/8] libibverbs: Support older providers that do not support extensions

2012-09-20 Thread Hefty, Sean
In order to support providers that do not handle extensions, including providers built against an older version of ibverbs, add a compatibility layer. This allows most of the core ibverbs code to assume that extensions are always available. The compatibility layer is responsible for converting be

[PATCH 5/8] libibverbs: libibverbs: Add support for XRC QPs

2012-09-20 Thread Hefty, Sean
XRC queue pairs: xrc defines two new types of QPs. The initiator, or send-side, xrc qp behaves similar to a send- only RC qp. xrc send qp's are managed through the existing QP functions. The send_wr structure is extended in a back- wards compatible way to support posting sends on a send xrc qp,

[PATCH 6/8] libibverbs: libibverbs: Add ibv_open_qp

2012-09-20 Thread Hefty, Sean
XRC receive QPs are shareable across multiple processes. Allow any process with access to the xrc domain to open an existing QP. After opening the QP, the process will receive events related to the QP and be able to modify the QP. Signed-off-by: Sean Hefty --- include/infiniband/driver.h |

[PATCH 3/8] libibverbs: Introduce XRC domains

2012-09-20 Thread Hefty, Sean
XRC introduces several new concepts and structures, one of which is the XRC domain. XRC domains: xrcd's are a type of protection domain used to associate shared receive queues with xrc queue pairs. Since xrcd are meant to be shared among multiple processes, we introduce new APIs to open/close xrc

[PATCH 4/8] livibverbs: Add support for XRC SRQs

2012-09-20 Thread Hefty, Sean
XRC support requires the use of a new type of SRQ. XRC shared receive queues: xrc srq's are similar to normal srq's, except that they are bound to an xrcd, rather than to a protection domain. Based on the current spec and implementation, they are only usable with xrc qps. To support xrc srq's, w

[PATCH 1/8] libibverbs: Infra-structure changes to support verbs extension

2012-09-20 Thread Hefty, Sean
From: Yishai Hadas Infrastructure to support extended verbs capabilities in a forward/backward manner. The general operation as shown in the following pseudo-code: ibv_open_device() { context = device->ops.alloc_context(); if (context == -1) { context_ex = malloc

[PATCH 5/6] RDMA/nes: Print hardware resource type

2012-09-20 Thread Tatyana Nikolova
Hardware resource types are added and when a resource isn't available, its type is printed. Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes.h | 15 --- drivers/infiniband/hw/nes/nes_utils.c |2 +- drivers/infiniband/hw/nes/nes_verbs.c | 14 +++---

Re: how to preserve QP over HA events for librdmacm applications

2012-09-20 Thread Pradeep Satyanarayana
On 09/20/2012 01:10 PM, Hefty, Sean wrote: What if you say pre-created a second (fail over) QP for HA purposes all under the covers of a single socket? And both QPs were connected before the failure. Not sure if that would work with the same CM id though. If not, we will need to rdma_connect() th

[PATCH 4/6] RDMA/nes: Fix for crash when tx chksum offload is off

2012-09-20 Thread Tatyana Nikolova
When tx checksum offload is disabled for iWarp connection, skb->ip_summed needs to be set to CHECKSUM_NONE Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes_cm.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/hw/nes/nes_cm.c b/drive

[PATCH 3/6] RDMA/nes: Cosmetic changes

2012-09-20 Thread Tatyana Nikolova
Removing unnecessary statement if(1) Refactoring a statement (wqe_misc |= NES_NIC_SQ_WQE_COMPLETION) out of if/else statement, because it is independant of the flow Defining netdev->features in one line for clarity Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes_nic.c | 32 +

[PATCH libmlx4 8/8] When calling ibv_modify_qp() return right value

2012-09-20 Thread Or Gerlitz
From: Dotan Barak When the ibv_query_port() call made by mlx4_modify_qp() fails, the return value from the latter should indicate the error status of the former and not simply -1. Signed-off-by: Dotan Barak Signed-off-by: Or Gerlitz --- src/verbs.c |6 -- 1 files changed, 4 insertions

[PATCH libmlx4 5/8] Allow to use the whole BF buffer

2012-09-20 Thread Or Gerlitz
From: Dotan Barak Increase the maximum size of messages (from 192 to 208) that will use the blue flame buffer. Signed-off-by: Dotan Barak Reviewed-by: Jack Morgenstein Signed-off-by: Or Gerlitz --- src/qp.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qp.c b/

[PATCH libmlx4 6/8] Use BlueFlame for RDMA_WRITE/WITH_IMM without data

2012-09-20 Thread Or Gerlitz
From: Dotan Barak Use Blue-Flame for RDMA Write and RDMA Write with immediate without any data (no s/g). This improves latency for those messages. Signed-off-by: Dotan Barak Reviewed-by: Jack Morgenstein Signed-off-by: Or Gerlitz --- src/qp.c |2 ++ 1 files changed, 2 insertions(+), 0 de

[PATCH libmlx4 7/8] Change enumeration names for masked atomic opcodes

2012-09-20 Thread Or Gerlitz
From: Dotan Barak Change the enumeration names of the masked atomic opcodes to be consistent with the ones used by the mlx4 kernel driver. Signed-off-by: Dotan Barak Signed-off-by: Or Gerlitz --- src/mlx4.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mlx4.h

[PATCH libmlx4 3/8] Limit qp resources accepted for ibv_create_qp()

2012-09-20 Thread Or Gerlitz
From: Sagi Grimberg Use the limits reported in ib_query_device(). Make sure that the limits returned to the caller following qp creation also lie within the reported device limits. Signed-off-by: Sagi Grimberg Signed-off-by: Jack Morgenstein Signed-off-by: Or Gerlitz --- src/mlx4.h | 14 +

[PATCH libmlx4 4/8] Replace sscanf() to strtol()

2012-09-20 Thread Or Gerlitz
From: Dotan Barak When converting a string to a numeric value, strtol() is more safe to use. Signed-off-by: Dotan Barak Signed-off-by: Or Gerlitz --- src/mlx4.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mlx4.c b/src/mlx4.c index 4989c46..0a9139f 100644 --

[PATCH libmlx4 1/8] Add raw packet QP support

2012-09-20 Thread Or Gerlitz
Implement raw packet QPs for Ethernet ports. Signed-off-by: Or Gerlitz --- src/qp.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/qp.c b/src/qp.c index 40a6689..90c4e80 100644 --- a/src/qp.c +++ b/src/qp.c @@ -286,6 +286,10 @@ int mlx4_post_send(struct ibv_qp *i

[PATCH libmlx4 0/8] add raw packet QP, resource limitations, fixes/cleanups

2012-09-20 Thread Or Gerlitz
Roland, This batch of libmlx4 patch contains the patch to support raw packet QP, two patches from Sagi that relate to resource limitations, and few simple fixes/cleanups from Dotan. The first three were submitted pretty long while ago, for this re-submission I made some changes in the change-lo

[PATCH libibverbs 3/3] Fix resource leaks in the pingpong examples present in the failure/error flows.

2012-09-20 Thread Or Gerlitz
From: Dotan Barak Signed-off-by: Dotan Barak Signed-off-by: Or Gerlitz --- examples/rc_pingpong.c | 43 --- examples/srq_pingpong.c | 51 ++ examples/uc_pingpong.c | 43 -

[PATCH libibverbs 2/3] Add helpers to deal with new InfiniBand link speeds

2012-09-20 Thread Or Gerlitz
From: Dotan Barak Introduce support for the following extended speeds: FDR:IBA extended speed 14.0625 Gbps. EDR:IBA extended speed 25.78125 Gbps. Signed-off-by: Dotan Barak Reviewed-by: Hal Rosenstock Signed-off-by: Or Gerlitz --- Makefile.am|6 +++- include/infi

[PATCH libibverbs 1/3] Add raw packet QP type

2012-09-20 Thread Or Gerlitz
IB_QPT_RAW_PACKET allows applications to build a complete packet, including L2 headers, when sending; on the receive side, the HW will not strip any headers. This QP type is designed for userspace direct access to Ethernet; for example by applications that do TCP/IP themselves. Only processes wit

[PATCH libibverbs 0/3] add raw packet QP, new helper and examples cleanups

2012-09-20 Thread Or Gerlitz
Hi Roland, This batch of libibverbs patches include the raw packet QP which was submitted long ago, with new change-log, taken from the kernel, also a helper verbs from Dotan and Hal related to the new IB link speeds, and cleanup patch from Dotan for the examples. Or. Dotan Barak (2): Add help

RE: how to preserve QP over HA events for librdmacm applications

2012-09-20 Thread Hefty, Sean
> What if you say pre-created a second (fail over) QP for HA purposes all > under the covers of a single socket? And both QPs were connected before > the failure. Not sure if that would work with the same CM id though. If > not, we will need to rdma_connect() the second QP after failure. CM IDs ar

[PATCH 2/6] RDMA/nes: Fix for incorrect MSS when tso is on

2012-09-20 Thread Tatyana Nikolova
In nes tso handling code, skb_shared_info is used to get the MSS instead of bool function skb_is_gso() which returns 1 Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes_nic.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/infiniband

[PATCH 1/6] RDMA/nes: Fix for incorrect resolving the loopback MAC address

2012-09-20 Thread Tatyana Nikolova
Loopback code changes to fix incorrect resolving of loopback MAC address. Signed-off-by: Tatyana Nikolova --- drivers/infiniband/hw/nes/nes_cm.c | 32 +++- 1 files changed, 7 insertions(+), 25 deletions(-) diff --git a/drivers/infiniband/hw/nes/nes_cm.c b/drivers/

Re: how to preserve QP over HA events for librdmacm applications

2012-09-20 Thread Atchley, Scott
On Sep 20, 2012, at 1:37 PM, Pradeep Satyanarayana wrote: > On 09/19/2012 11:14 AM, Atchley, Scott wrote: >> On Sep 19, 2012, at 1:05 PM, "Hefty, Sean" wrote: >> I too would be interested in bringing a QP from error back to a usable state. I have been debating whether to reconn

Re: how to preserve QP over HA events for librdmacm applications

2012-09-20 Thread Pradeep Satyanarayana
On 09/19/2012 11:14 AM, Atchley, Scott wrote: On Sep 19, 2012, at 1:05 PM, "Hefty, Sean" wrote: I too would be interested in bringing a QP from error back to a usable state. I have been debating whether to reconnect using the current RDMA calls versus trying to transition the existing RC QP.

RE: [PATCH 25/25] userns: Convert ipathfs to use GLOBAL_ROOT_UID and GLOBAL_ROOT_GID

2012-09-20 Thread Marciniszyn, Mike
> Subject: [PATCH 25/25] userns: Convert ipathfs to use GLOBAL_ROOT_UID and > GLOBAL_ROOT_GID > > From: "Eric W. Biederman" > > Cc: Mike Marciniszyn > Acked-by: Serge Hallyn > Signed-off-by: Eric W. Biederman Thanks for the patch! Acked-by: Mike Marciniszyn -- To unsubscribe from this list

[PATCH] Correct option names in opensm man page

2012-09-20 Thread Bart Van Assche
Long options have to be preceded by a double dash instead of a single. Signed-off-by: Bart Van Assche --- man/opensm.8.in | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/man/opensm.8.in b/man/opensm.8.in index 79ff6a5..4e61c2e 100644 --- a/m

[PATCH] [TRIVIAL] opensm/osm_log.h: fix function documentation

2012-09-20 Thread Yevgeny Kliteynik
Signed-off-by: Yevgeny Kliteynik --- include/opensm/osm_log.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/opensm/osm_log.h b/include/opensm/osm_log.h index 3247296..61ba750 100644 --- a/include/opensm/osm_log.h +++ b/include/opensm/osm_log.h @@ -1,6 +1,6 @@ /