Re: [PATCH] ib_ehca: fix compile failure on ppc64

2012-02-28 Thread Kyle McMartin
On Mon, Feb 27, 2012 at 09:18:35PM -0800, Roland Dreier wrote: On Mon, Feb 27, 2012 at 2:02 PM, Kyle McMartin kmcmar...@redhat.com wrote:  static u64 ehca_get_max_hwpage_size(struct ehca_shca *shca)  { -       return 1UL ilog2(shca-hca_cap_mr_pgsize); +       u32 pgsize =

Re: [PATCH 2/6] IB/core: fix wrong display of rate in sysfs

2012-02-28 Thread Or Gerlitz
If no one has a problem, I'll just go ahead with the following... Roland, here's your patch fixed to work, this function is something... you've missed a break in the SDR switch case.. I added it and tested Or. -- IB/core: Fix SDR rates in sysfs Commit 71eeba16 (IB: Add new InfiniBand

[PATCH V3 1/6] IB: use central enum for speed instead of hard-coded values

2012-02-28 Thread Or Gerlitz
The kernel IB stack uses one enumeration for IB speed, which wasn't explicitly specified in the verbs header file. Add that enum, and use it all over the code. Note that the IB speed/width notation is also used by iWARP and IBoE hw drivers who apply the convention of rate = speed X width, to

Re: [Patch] iser: free ib connection resources in the proper place

2012-02-28 Thread Or Gerlitz
On 2/25/2012 9:36 PM, Doug Ledford wrote: We allocate the login dma buffers in iser_verbs.c as part of alloc_ib_conn_resources(), however we are freeing them in iser_initiator.c as part of iser_free_rx_descriptors(). This is needlessly confusing. We have an alloc_rx_descriptors() and it

[PATCH] mlx4: prevent the device from being removed concurrently

2012-02-28 Thread Thadeu Lima de Souza Cascardo
When a EEH happens, the catas poll code will try to restart the device, removing it and adding it back again. The EEH code will try to do the same. One of the threads ends up accessing memory that was freed by the other thread and we get a crash. The EEH backtrace: 4Call Trace:

Re: [PATCH] mlx4: prevent the device from being removed concurrently

2012-02-28 Thread David Miller
From: Thadeu Lima de Souza Cascardo casca...@linux.vnet.ibm.com Date: Tue, 28 Feb 2012 15:36:16 -0300 When a EEH happens, the catas poll code will try to restart the device, removing it and adding it back again. The EEH code will try to do the same. One of the threads ends up accessing memory

[PATCH] opensm: osm_subnet.c: Patch to add error-reporting to the parsing of opensm.conf

2012-02-28 Thread Mike Heinz
I've recently seen two different users on different fabrics make the same mistake: They made a syntax error in their opensm.conf file and then couldn't figure out why their changes didn't take effect. Looking at the code that parses the opensm.conf file, it appears to treat any line it cannot

Re: [PATCH] mlx4: prevent the device from being removed concurrently

2012-02-28 Thread David Miller
From: Thadeu Lima de Souza Cascardo casca...@linux.vnet.ibm.com Date: Tue, 28 Feb 2012 17:34:38 -0300 On Tue, Feb 28, 2012 at 02:30:51PM -0500, David Miller wrote: From: Thadeu Lima de Souza Cascardo casca...@linux.vnet.ibm.com Date: Tue, 28 Feb 2012 15:36:16 -0300 When a EEH happens, the

ibacm fixes/updates

2012-02-28 Thread Doug Ledford
Hi Sean, We were asked to integrate this into our product for scalability purposes. While working on that, I ran across a number of issues that needed fixed up (typical for a new package). So, here's a number of patches and fix ups for those things. First, the package is ibacm but the binary

Re: ibacm fixes/updates

2012-02-28 Thread Doug Ledford
On 02/28/2012 04:08 PM, Doug Ledford wrote: By default ibacm expects to find its configuration files in /etc/ibacm. This adds to the proliferation of directories in /etc/ needlessly. We already have a number of RDMA related directories to choose from depending on your install (OFED ==

RE: ibacm fixes/updates

2012-02-28 Thread Hefty, Sean
Thanks - I'll try to add these patches to the next release and work to get things fixed up. - Sean -- 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/majordomo-info.html

[Patch opensm] Allow for easily configuring multiple fabrics on one opensm server

2012-02-28 Thread Doug Ledford
There are two things that stand in the way of opensm being run on redundant fabrics easily: 1) The opensm init script only starts one instance of opensm and opensm will only work on one fabric per instance 2) Even if you start multiple instances, you have to hand modify config files for each