[patch -next] mlx5_core: warn if no space left in alloc_4k()

2013-11-01 Thread Dan Carpenter
The warning was unreachable. In the original code, it would print the line number and the function but I have added an error message. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com --- I haven't tested this, hopefully the warning is not annoying. diff --git

[PATCH opensm] Fix handling of get P_KeyTable on timeout

2013-11-01 Thread Hal Rosenstock
From: Dan Ben Yosef da...@mellanox.com if the get of pkey tbl times out, remove the node from the subnet and indicate subnet initialization error so subnet is reswept Signed-off-by: Dan Ben Yosef da...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git

[PATCH opensm] Fix possible use of lid 0 when sending set PortInfo after failure of the first PortInfo set

2013-11-01 Thread Hal Rosenstock
From: Dan Ben Yosef da...@mellanox.com Signed-off-by: Dan Ben Yosef da...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/include/opensm/osm_port.h b/include/opensm/osm_port.h index 8e34b84..f1edcf9 100644 --- a/include/opensm/osm_port.h +++

Re: [PATCH RFC v2 00/10] Introduce Signature feature

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: In T10-DIF, when a series of 512-byte data blocks are transferred, each block is followed by an 8-byte guard. The guard consists of CRC that protects the integrity of the data in the block, and some other tags that protects against mis-directed IOs.

Re: [PATCH RFC v2 08/10] IB/mlx5: Support IB_WR_REG_SIG_MR

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: +static u8 bs_selector(int block_size) +{ + switch (block_size) { + case 512: return 0x1; + case 520: return 0x2; + case 4096: return 0x3; + case 4160: return 0x4; + case

Re: [PATCH RFC v2 02/10] IB/core: Introduce Signature Verbs API

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: +/** + * struct ib_sig_domain - Parameters specific for T10-DIF + * domain. + * @sig_type: specific signauture type + * @sig: union of all signature domain attributes that may + * be used to set domain layout. + * @dif: + *

Re: [PATCH RFC v2 01/10] IB/core: Introduce protected memory regions

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: +/** + * ib_mr_init_attr - Memory region init attributes passed to routine + * ib_create_mr. + * @max_reg_descriptors: max number of registration units that + * may be used with UMR work requests. + * @flags: MR creation flags bit mask. + */ +struct

Re: [PATCH RFC v2 08/10] IB/mlx5: Support IB_WR_REG_SIG_MR

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: This patch implements IB_WR_REG_SIG_MR posted by the user. Baisically this WR involvs 3 WQEs in order to prepare and properly register the signature layout: 1. post UMR WR to register the sig_mr in one of two possible ways: * In case the user

Re: [PATCH RFC v2 07/10] IB/mlx5: Keep mlx5 MRs in a radix tree under device

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: + /* connect to MR tree */ + write_lock_irq(table-lock); + err = radix_tree_insert(table-tree, mr-key 0xff00, mr); + write_unlock_irq(table-lock); The conversion from MR key into radix tree index occurs three times so maybe

[PATCH] rdma/cm: Discard events for IDs not yet claimed by user space

2013-11-01 Thread sean . hefty
From: Sean Hefty sean.he...@intel.com Problem reported by Avneesh Pant avneesh.p...@oracle.com: It looks like we are triggering a bug in RDMA CM/UCM interaction. The bug specifically hits when we have an incoming connection request and the connecting process dies BEFORE the passive end of the

Re: [PATCH RFC v2 02/10] IB/core: Introduce Signature Verbs API

2013-11-01 Thread Bart Van Assche
On 31/10/2013 5:24, Sagi Grimberg wrote: + * @type3_inc_reftag: T10-DIF type 3 does not state + * about the reference tag, it is the user + * choice to increment it or not. Can you explain this further ? Does this mean that the HCA can check

Re: [PATCH RFC v2 00/10] Introduce Signature feature

2013-11-01 Thread Nicholas A. Bellinger
On Fri, 2013-11-01 at 08:03 -0700, Bart Van Assche wrote: On 31/10/2013 5:24, Sagi Grimberg wrote: In T10-DIF, when a series of 512-byte data blocks are transferred, each block is followed by an 8-byte guard. The guard consists of CRC that protects the integrity of the data in the block,