Re: [PATCH 5/8] opensm: Signal subnet init errors on SubnGet timeouts

2012-07-23 Thread Alex Netes
Hi Jim, On 17:55 Mon 25 Jun , Jim Foraker wrote: A subnet should not be listed as cleanly initialized if CAs fail to respond to SubnGet requests. Signed-off-by: Jim Foraker forak...@llnl.gov --- opensm/osm_sm_mad_ctrl.c |9 + 1 file changed, 9 insertions(+) diff --git

Re: [PATCH V1.1 1/8] opensm: Add guid2mkey cache file support

2012-07-23 Thread Alex Netes
Hi Jim, On 17:25 Tue 03 Jul , Jim Foraker wrote: Adds support for a guid2mkey file, and uses the database to select which mkey to use in outgoing SMPs. One thing that I notice that missing (and I'm not sure how common it's being used, but still I see it as part of SUSE distro) is sldd.sh

Re: [PATCH 0/8] opensm: Improved mkey support

2012-07-23 Thread Alex Netes
Hi Jim, Can you please add short description of the improved M_Key mechanism into the man page? On 17:25 Tue 03 Jul , Jim Foraker wrote: I'm sending new versions of patches 1 and 3 to the list, which correct merge/build issues introduced by other recently accepted patches. They will

[GIT PULL] please pull infiniband.git

2012-07-23 Thread Roland Dreier
Hi Linus, Please pull from git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git tags/rdma-for-3.6 InfiniBand/RDMA changes for the 3.6 merge window: - Updates to the qib low-level driver - First chunk of

[PATCH] IB/qib: correct smatch issue in qib_init.c

2012-07-23 Thread Mike Marciniszyn
Commit 36a8f01c (IB/qib: Add congestion control agent implementation) introduced an smatch warning. This patch corrects the issue. Additionally, Ram Vepa should have been credited as the author for 36a8f01c. Reviewed-by: Ramkrishna Vepa ramkrishna.v...@intel.com Signed-off-by: Mike Marciniszyn

Re: [PATCH net/for-next V1 1/1] IB/ipoib: break linkage to neighbouring system

2012-07-23 Thread Or Gerlitz
On 20/07/2012 18:49, Or Gerlitz wrote: On Thu, Jul 19, 2012 at 4:18 PM, Or Gerlitzogerl...@mellanox.com wrote: From: Shlomo Pongratzshlo...@mellanox.com Dave Millerda...@davemloft.net provided a detailed description of why the way IPoIB is using neighbours for its own ipoib_neigh struct is

Re: [PATCH net/for-next V1 1/1] IB/ipoib: break linkage to neighbouring system

2012-07-23 Thread Eric Dumazet
On Mon, 2012-07-23 at 19:58 +0300, Or Gerlitz wrote: Sorry for the possible spam, but resending (last time forgot to put Eric on the to list so he might missed it, also added Christoph) - this is a fix for very long time bug in IPoIB and we do want it to be reviewed and hopefully

Re: uverbs message alignment

2012-07-23 Thread Jason Gunthorpe
On Sat, Jul 21, 2012 at 05:15:26AM +, Hefty, Sean wrote: struct c4iw_create_raw_qp_req { struct ibv_create_qp ibv_req; __u32 port; __u32 vlan_pri; __u32 nfids; }; struct ibv_create_qp contains a u64, which will force the size of the structure

Re: [PATCH net/for-next V1 1/1] IB/ipoib: break linkage to neighbouring system

2012-07-23 Thread Or Gerlitz
On Mon, Jul 23, 2012 at 8:17 PM, Eric Dumazet eric.duma...@gmail.com wrote: I have no idea of what you are talking about, I have not the patch or a copy of it ;) http://marc.info/?t=13427149132r=1w=2 http://marc.info/?t=13427149135r=1w=2 -- To unsubscribe from this list: send the line

RE: [PATCH 2/5] IB/qib: Added congestion control agent implementation

2012-07-23 Thread Vepa, Ramkrishna
Can you elaborate a little on the role/usage of the sysfs entries this patch adds? I wasn't sure to follow, basically, I thought IBTA CCA defined thing such that one can set/get anything CC related through MADs, isn't that? The PSM library currently implements a static configuration for

Re: [PATCH 5/8] opensm: Signal subnet init errors on SubnGet timeouts

2012-07-23 Thread Jim Foraker
On Mon, 2012-07-23 at 08:43 -0700, Alex Netes wrote: Hi Jim, On 17:55 Mon 25 Jun , Jim Foraker wrote: A subnet should not be listed as cleanly initialized if CAs fail to respond to SubnGet requests. Signed-off-by: Jim Foraker forak...@llnl.gov --- opensm/osm_sm_mad_ctrl.c |

Re: [PATCH 0/8] opensm: Improved mkey support

2012-07-23 Thread Jim Foraker
On Mon, 2012-07-23 at 08:59 -0700, Alex Netes wrote: Hi Jim, Can you please add short description of the improved M_Key mechanism into the man page? Sure. I don't see anything about mkeys in the page now, so I'll write up a section for them. v2, BTW, will also fix some whitespace

Re: [PATCH V1.1 1/8] opensm: Add guid2mkey cache file support

2012-07-23 Thread Jim Foraker
On Mon, 2012-07-23 at 08:55 -0700, Alex Netes wrote: Hi Jim, On 17:25 Tue 03 Jul , Jim Foraker wrote: Adds support for a guid2mkey file, and uses the database to select which mkey to use in outgoing SMPs. One thing that I notice that missing (and I'm not sure how common it's being

[PATCH] infiniband-diags: Fix command line parsing bug in ibccconfig, ibccquery, saquery, and smpquery.

2012-07-23 Thread Albert Chu
Ensure user's inputted operation directly matches an operation name or operation alias. Do not base it on the length of the input, which can lead to conflicts with operation selection. Signed-off-by: Albert Chu ch...@llnl.gov --- src/ibccconfig.c |5 ++--- src/ibccquery.c |5 ++---

Re: [PATCH RESEND net/for-next V1 1/1] IB/ipoib: break linkage to neighbouring system

2012-07-23 Thread David Miller
From: Or Gerlitz ogerl...@mellanox.com Date: Thu, 19 Jul 2012 19:15:02 +0300 -static void neigh_add_path(struct sk_buff *skb, struct neighbour *n, struct net_device *dev) +static void neigh_add_path(struct sk_buff *skb, u8 *daddr, + struct net_device *dev) Please