Re: [PATCH REPOST FIXES for-3.11 1/4] IB/core: Create QP1 using the pkey index which contains the default pkey

2013-07-18 Thread Or Gerlitz
On 17/07/2013 19:31, Hefty, Sean wrote: diff --git a/drivers/infiniband/core/mad.c b/drivers/infiniband/core/mad.c index dc3fd1e..9be6754 100644 --- a/drivers/infiniband/core/mad.c +++ b/drivers/infiniband/core/mad.c @@ -2663,6 +2663,7 @@ static int ib_mad_port_start(struct ib_mad_port_private

[PATCH V1 FIXES for-3.11 3/4] IB/ipoib: Make sure child devices use valid/proper pkeys

2013-07-18 Thread Or Gerlitz
Make sure that the IB invalid pkey (0x or 0x8000) isn't used for child devices. Also, make sure to always set the full membership bit for the pkey of devices created by rtnl link ops. Signed-off-by: Or Gerlitz ogerl...@mellanox.com --- drivers/infiniband/ulp/ipoib/ipoib_main.c|2 +-

[PATCH V1 FIXES for-3.11 2/4] IB/mlx4: Use default pkey when creating tunnel QPs

2013-07-18 Thread Or Gerlitz
From: Jack Morgenstein ja...@dev.mellanox.co.il When creating tunnel QPs for special QP tunneling, look for the default pkey in the slave's virtual pkey table. If it is present, use the real pkey index where the default pkey is located. If the default pkey is not found in the pkey table, use the

[PATCH V1 FIXES for-3.11 4/4] IB/ipoib: Fix pkey-change flow for Virtualization environments

2013-07-18 Thread Or Gerlitz
From: Erez Shitrit ere...@mellanox.com IPoIB's required behaviour w.r.t to the pkey used by the device is the following: - For parent interfaces (e.g ib0, ib1, etc) who are created automatically as a result of hot-plug events from the IB core, the driver needs to take whatever pkey vlaue it

[PATCH V1 FIXES for-3.11 1/4] IB/core: Create QP1 using the pkey index which contains the default pkey

2013-07-18 Thread Or Gerlitz
From: Jack Morgenstein ja...@dev.mellanox.co.il Currently, QP1 is created using pkey index 0. This patch simply looks for the index containing the default pkey, rather than hard-coding pkey index 0. This change will have no effect in Native mode, since QP0 and QP1 are created before the SM

[PATCH] mlx5: Implement new initialization sequence

2013-07-18 Thread Eli Cohen
Introduce enbale_hca and disable_hca commands to signify when the driver starts or ceases to operate on the device. In addition the driver will use boot and init pages count; boot pages is required to allow firmware to complete boot commands and the other to complete init hca. Command interface

[PATCH for-3.11 1/7] IB/iser: Use proper debug level value for info prints

2013-07-18 Thread Or Gerlitz
Commit 4f363882612 IB/iser: Move informational messages from error to info level was setting info prints to require lower value for the debug level vs warning prints which isn't the common convention, fix that. Also move the prints on unaligned SG from warning to debug level. Signed-off-by: Or

[PATCH for-3.11 5/7] IB/iser: Handle unaligned SG in separate function

2013-07-18 Thread Or Gerlitz
From: Sagi Grimberg sa...@mellanox.com This routine will be shared with other rdma management schemes. The bounce buffer solution for unaligned SG-lists and the sg_to_page_vec routine are likely to be used for other registration schemes and not just FMR. Move them out of the FMR specific code,

[PATCH for-3.11 0/7] Add Fast-Reg support to the iser initiator driver

2013-07-18 Thread Or Gerlitz
Hi Roland, Here's the FRWR (Fast Reg WR) patch series for iser from Sagi Grimberg. This is kind of critical so we can avoid using bounce buffers when running over Connect-IB (the newly introduced mlx5 driver) and ConnectX VF which both don't support FMRs. Patches 1-3 are the other iser updates

[PATCH for-3.11 7/7] IB/iser: Introduce fast memory registration model (FRWR)

2013-07-18 Thread Or Gerlitz
From: Sagi Grimberg sa...@mellanox.com Newer HCAs and Virtual functions may not support FMRs but rather a fast registration model, which we call FRWR - Fast Registration Work Requests. This model was introduced in 00f7ec36c RDMA/core: Add memory management extensions support and works when the

[PATCH for-3.11 3/7] IB/iser: Accept session-cmds_max from user space

2013-07-18 Thread Or Gerlitz
From: Shlomo Pongratz shlo...@mellanox.com Use cmds_max passed from user space to be the number of PDUs to be supported for the session instead of hard-coded ISCSI_DEF_XMIT_CMDS_MAX. Specifically, this allows to control the max number of SCSI commands for the seesion. Also don't ignore the qdepth

[PATCH for-3.11 6/7] IB/iser: Place the fmr pool into a union in iser's IB conn struct

2013-07-18 Thread Or Gerlitz
From: Sagi Grimberg sa...@mellanox.com This is preparation step for other memory registration methods to be added. In addition, change reg/unreg routines signature to indicate they use FMRs. Signed-off-by: Sagi Grimberg sa...@mellanox.com Signed-off-by: Or Gerlitz ogerl...@mellanox.com ---

[PATCH for-3.11 4/7] IB/iser: Generalize rdma memory registration

2013-07-18 Thread Or Gerlitz
From: Sagi Grimberg sa...@mellanox.com Currently the driver uses FMRs as the only means to register the memory pointed by SG provided by the SCSI mid-layer with the RDMA device. As perperation step for adding more methods for fast path memory registrarion, make the alloc/free and reg/unreg calls

[PATCH for-3.11 2/7] IB/iser: Restructure allocation/deallocation of connection resources

2013-07-18 Thread Or Gerlitz
From: Shlomo Pongratz shlo...@mellanox.com This is a preparation step to a patch that accepts the number of max SCSI commands to be supported for the session from the user space iSCSI tools. Move the allocation of the login buffer, FMR pool and its associated page vector from

[PATCH opensm] complib/cl_event_wheel.c: Roundup timeout to nearest msec

2013-07-18 Thread Hal Rosenstock
From: Alex Netes ale...@mellanox.com Signed-off-by: Alex Netes ale...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/complib/cl_event_wheel.c b/complib/cl_event_wheel.c index 24f3e31..27443f6 100644 --- a/complib/cl_event_wheel.c +++ b/complib/cl_event_wheel.c @@

Re: [PATCH V3 for-next 3/4] IB/core: Export ib_create/destroy_flow through uverbs

2013-07-18 Thread Matan Barak
On 18/7/2013 2:31 AM, Hefty, Sean wrote: +ssize_t ib_uverbs_create_flow(struct ib_uverbs_file *file, + const char __user *buf, int in_len, + int out_len) +{ + struct ib_uverbs_create_flow cmd; + struct

Re: [PATCH V2] libibverbs: Allow arbitrary int values for MTU

2013-07-18 Thread Jason Gunthorpe
On Thu, Jul 18, 2013 at 02:32:05AM +, Jeff Squyres (jsquyres) wrote: On Jul 17, 2013, at 5:44 PM, Steve Wise sw...@opengridcomputing.com wrote: The iwarp drivers just report the nearest mtu enum. Apps don't need it for iwarp like they do for ib. For RC, it doesn't matter much. So

RE: [PATCH V3 for-next 3/4] IB/core: Export ib_create/destroy_flow through uverbs

2013-07-18 Thread Hefty, Sean
+ kern_spec = kern_flow_attr + 1; + ib_spec = flow_attr + 1; + for (i = 0; i flow_attr-num_of_specs; i++) { + err = kern_spec_to_ib_spec(kern_spec, ib_spec); + if (err) + goto err_free; + flow_attr-size += + ((struct

non-symmetric MFT

2013-07-18 Thread Michael Di Domenico
i;ve run ibdiagnet -r on my infiniband network, i have a few errors showing up with this Found a non symmetric MFT on MLID:0x0CBB got to node:guid through port:17 which does not point back to node, etc etc... what does this mean? i tried to look at the source but the comment a few lines above

[PATCH librdmacm 3/3] init: Remove USE_IB_ACM configuration option

2013-07-18 Thread sean . hefty
From: Sean Hefty sean.he...@intel.com When the librdmacm is configured, it sets the USE_IB_ACM option if infininband/acm.h is found. We can remove this option with very little overhead, which would allow a user to install ACM after installing the librdmacm, and the librdmacm would be able to

[PATCH librdmacm 2/3] acm: Define needed ACM protocol messages

2013-07-18 Thread sean . hefty
From: Sean Hefty sean.he...@intel.com The librdmacm needs message definitions used to communicate with the ibacm. It currently pulls these from infiniband/acm.h, which is installed by ibacm. This creates an install order dependency on ibacm. However, work on the scalable SA has the ibacm using

[PATCH librdmacm 1/3] cmatose: Allow user to specify address format

2013-07-18 Thread sean . hefty
From: Sean Hefty sean.he...@intel.com Provide an option for the user to indicate the type of addresses used as input. Support hostname, IPv4, IPv6, and GIDs. Signed-off-by: Sean Hefty sean.he...@intel.com --- examples/cmatose.c | 44 +++-

Re: [PATCH V7 libibverbs 0/7] Add extension and XRC QP support

2013-07-18 Thread Or Gerlitz
Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Mon, Jul 15, 2013 at 03:40:47PM +0300, Yishai Hadas wrote: Let me clarify the place holder reservation mentioned in the cover letter. The entries I was referring to are not proprietary vendor verbs but rather user space verbs who are

Re: non-symmetric MFT

2013-07-18 Thread Hal Rosenstock
On 7/18/2013 1:50 PM, Michael Di Domenico wrote: i;ve run ibdiagnet -r on my infiniband network, i have a few errors showing up with this Found a non symmetric MFT on MLID:0x0CBB Is that the real MLID or a typo ? MLIDs start at 0xC000. got to node:guid through port:17 which does not

Re: [PATCH infiniband-diags] infiniband-diags: Eliminate unneeded clean_nodedesc calls

2013-07-18 Thread Ira Weiny
On Mon, 15 Jul 2013 10:32:40 -0400 Hal Rosenstock h...@dev.mellanox.co.il wrote: in ibroute and dump_fts clean_nodedesc is called if needed within remap_node_name Signed-off-by: Hal Rosenstock h...@mellanox.com Thanks, accepted, Ira --- diff --git a/src/dump_fts.c b/src/dump_fts.c