Re: iSER + CentOS 5.7 + Xen lockup

2011-11-07 Thread Or Gerlitz
On 11/8/2011 3:14 AM, Mohammed Naser wrote: Hello everyone, we had this server crash a couple of times because of (what seems to me) some iSER problems. This node runs all the default drivers that ship with CentOS 5.7 You mentioned Xen in your subject line, but later say CentOS 5.7, what ker

Re: [PATCH] ib_srpt: Fix Last WQE handling

2011-11-07 Thread Bart Van Assche
On Mon, Nov 7, 2011 at 9:46 PM, Nicholas A. Bellinger wrote: > On Mon, 2011-11-07 at 20:00 +0100, Bart Van Assche wrote: >> On Mon, Nov 7, 2011 at 7:10 PM, Roland Dreier wrote: >> > On Sun, Nov 6, 2011 at 9:31 AM, Bart Van Assche wrote: >> >> Make sure that the last WQE event is handled properly

iSER + CentOS 5.7 + Xen lockup

2011-11-07 Thread Mohammed Naser
Hello everyone, We had this server crash a couple of times because of (what seems to me) some iSER problems. This node runs all the default drivers that ship with CentOS 5.7 BUG: soft lockup - CPU#0 stuck for 60s! [blkback.4.hda:12797] CPU 0: Modules linked in: fuse sch_tbf xt_multiport tun xt_s

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

2011-11-07 Thread Jason Gunthorpe
On Mon, Nov 07, 2011 at 04:54:42PM -0800, Roland Dreier wrote: > On Wed, Nov 2, 2011 at 10:16 AM, Or Gerlitz wrote: > > 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 observations, chang

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

2011-11-07 Thread Roland Dreier
On Wed, Nov 2, 2011 at 10:16 AM, Or Gerlitz wrote: > 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 observations, changes will follow, agree? Let's make sure we learn from our mistakes. Let'

[opensm] [PATCH 3/3] Support source-target-port-guid QoS policy configuration with ULP 'any'

2011-11-07 Thread Albert Chu
For ease of configuration, combines the source-port-guid and target-port-guid options into a single line. Signed-off-by: Albert L. Chu --- doc/QoS_management_in_OpenSM.txt |5 opensm/osm_qos_parser_l.l|2 + opensm/osm_qos_parser_y.y| 44

[opensm] [trivial] [PATCH 1/3] Fix typo in qos-ulps parsing comment

2011-11-07 Thread Albert Chu
"srp target-port-guid" is supported, not "srp port-guid" Signed-off-by: Albert L. Chu --- opensm/osm_qos_parser_y.y |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/opensm/osm_qos_parser_y.y b/opensm/osm_qos_parser_y.y index 8274280..348750d 100644 --- a/opensm/osm_qos_p

[opensm] [PATCH 2/3] Support source-port-guid QoS policy configuration with ULP 'any'

2011-11-07 Thread Albert Chu
Similar to the target-port-guid option, source-port-guid supports the ability to configure a SL based on a specific source port in a path record lookup. Signed-off-by: Albert L. Chu --- doc/QoS_management_in_OpenSM.txt |4 +++ opensm/osm_qos_parser_l.l|2 + opensm/osm_qos_parser_

Re: [PATCH] ib_srpt: Fix Last WQE handling

2011-11-07 Thread Nicholas A. Bellinger
On Mon, 2011-11-07 at 20:00 +0100, Bart Van Assche wrote: > On Mon, Nov 7, 2011 at 7:10 PM, Roland Dreier wrote: > > On Sun, Nov 6, 2011 at 9:31 AM, Bart Van Assche wrote: > >> Make sure that the last WQE event is handled properly if it arrives > >> before the QP has been transitioned into the er

Re: [PATCH] ib_srpt: Fix Last WQE handling

2011-11-07 Thread Bart Van Assche
On Mon, Nov 7, 2011 at 7:10 PM, Roland Dreier wrote: > On Sun, Nov 6, 2011 at 9:31 AM, Bart Van Assche wrote: >> Make sure that the last WQE event is handled properly if it arrives >> before the QP has been transitioned into the error state. > > Do you mean if the event arrives before the modify

[PATCH] opensm/man/torus-2QoS.conf.5.in: Update portgroup_max_ports section

2011-11-07 Thread Hal Rosenstock
for host ports including switch port 0 Signed-off-by: Hal Rosenstock Acked-by: Jim Schutt --- diff --git a/man/torus-2QoS.conf.5.in b/man/torus-2QoS.conf.5.in index df38ad3..d806ab5 100644 --- a/man/torus-2QoS.conf.5.in +++ b/man/torus-2QoS.conf.5.in @@ -1,4 +1,4 @@ -.TH TORUS\-2QOS.CONF 5 "No

Re: [PATCH] ib_srpt: Fix Last WQE handling

2011-11-07 Thread Roland Dreier
On Sun, Nov 6, 2011 at 9:31 AM, Bart Van Assche wrote: > Make sure that the last WQE event is handled properly if it arrives > before the QP has been transitioned into the error state. Do you mean if the event arrives before the modify QP operation returns? If so I agree this is a possible order

[ANNOUNCE] infiniband-diags 1.5.12 tarball release

2011-11-07 Thread Ira Weiny
Hello, There is a new release of infiniband-diags. Tarball available at: http://www.openfabrics.org/downloads/management/infiniband-diags-1.5.12.tar.gz (listed in http://www.openfabrics.org/downloads/management/latest.txt) 7a823a3f6d9cfa3d19e1ca6889f3c122 infiniband-diags-1.5.12.tar.gz This i

Re: [PATCH][MINOR] opensm/osm_torus.c: Use "OpenSM standard" error codes

2011-11-07 Thread Alex Netes
Hi Hal, On 10:20 Mon 07 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/majordo

[PATCH][MINOR] opensm/osm_torus.c: Use "OpenSM standard" error codes

2011-11-07 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock --- diff --git a/opensm/osm_torus.c b/opensm/osm_torus.c index a2d7b15..df46fb8 100644 --- a/opensm/osm_torus.c +++ b/opensm/osm_torus.c @@ -170,8 +170,7 @@ struct link { */ struct port_grp { enum endpt_type type; - size_t port_cnt;/* number

[PATCH V2] RDMA/cm: destroy idr as part of the cleanup in the error flow

2011-11-07 Thread Dotan Barak
Clean the idr as part of the cleanup flow since it is a resource too. Signed-off-by: Dotan Barak Reviewed-by: Jack Morgenstein Reviewed-by: Sean Hefty --- V2 - don't check if idr_pre_get() failed, , we'll just try again later. drivers/infiniband/core/cm.c | 16 ++-- 1 files cha

[PATCH 11/11] mthca: use the new cache in mthca driver

2011-11-07 Thread Goldwyn Rodrigues
Sample usage of the new caching functions incorporated in the mthca driver. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c |7 +++ drivers/infiniband/hw/mthca/mthca_av.c |1 - drivers/infiniband/hw/mthca/mthca_dev.h |1 + drivers/infi

[PATCH 10/11] Remove cache finding functions

2011-11-07 Thread Goldwyn Rodrigues
Cache finding functions are no longer required since the query functions take care of the caching. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 72 --- include/rdma/ib_cache.h | 34 -- 2 files changed, 0 in

[PATCH 09/11] Remove old cache setup functions

2011-11-07 Thread Goldwyn Rodrigues
Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 193 -- drivers/infiniband/core/device.c | 10 -- 2 files changed, 0 insertions(+), 203 deletions(-) diff --git a/drivers/infiniband/core/cache.c b/drivers/infiniband/core/cache.c index

[PATCH 08/11] Functions to drop caches

2011-11-07 Thread Goldwyn Rodrigues
These functions are called in case of port changes or device resets etc. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 26 ++ include/rdma/ib_cache.h | 15 +++ 2 files changed, 41 insertions(+), 0 deletions(-) diff --git a

[PATCH 07/11] Functions to setup and free cache

2011-11-07 Thread Goldwyn Rodrigues
Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 104 +++ include/rdma/ib_cache.h | 13 + 2 files changed, 117 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/core/cache.c b/drivers/infiniband/core/cache.c ind

[PATCH 05/11] Use cache structure instead of device (pkey)

2011-11-07 Thread Goldwyn Rodrigues
Changes all ib_xxx_cached_pkey funcitons to use device structure instead of cache structure. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 37 - include/rdma/ib_cache.h |4 ++-- 2 files changed, 22 insertions(+), 19 delet

[PATCH 06/11] Add cache update functions

2011-11-07 Thread Goldwyn Rodrigues
Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 54 +++ include/rdma/ib_cache.h | 25 ++ 2 files changed, 79 insertions(+), 0 deletions(-) diff --git a/drivers/infiniband/core/cache.c b/drivers/infiniband/core

[PATCH 04/11] Use cache structure instead of device (gid)

2011-11-07 Thread Goldwyn Rodrigues
Changes all ib_xxx_cached_gid funcitons to use device structure instead of cache structure. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 41 +- include/rdma/ib_cache.h |4 +- include/rdma/ib_verbs.h |2 + 3 f

[PATCH 03/11] Remove lmc from cache

2011-11-07 Thread Goldwyn Rodrigues
The lmc cache is removed because it cannot be used in this structure. Since, there is no call for querying lmc specifically, we cannot use it in the device driver. TODO: Find a better place for lmc cache, perhaps the ib_device structure? Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/c

[PATCH 02/11] Get rid of caching function usage

2011-11-07 Thread Goldwyn Rodrigues
Remove all old caching functions. Note: Since lmc cache could not be removed, in_query_lmc() has been added which performs a regular query and extracts the lmc and returns it. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cm.c |6 +++--- drivers/infiniband/core/cma.

[PATCH 01/11] Enable bitmaps for valid cache entries

2011-11-07 Thread Goldwyn Rodrigues
Add a bitmap to check if the entry is valid or not. This keeps a record of which entries have been updated and which have not. Signed-off-by: Goldwyn Rodrigues --- drivers/infiniband/core/cache.c | 26 -- 1 files changed, 24 insertions(+), 2 deletions(-) diff --git a/d

[PATCH 00/11] New Caching mechanism for ib_core

2011-11-07 Thread Goldwyn Rodrigues
Hi, This series of patches introduces caching as a library in the ib_core as compared to the previous of implementation as a part of the ib_device calls. Each driver can make use of this by including the ib_cache data structure in the individual device structure. The updation of the cache is the