Re: [PATCH v2] ibsim: allocate mft according to number of switch ports

2011-01-04 Thread Eli Dorfman (Voltaire)
Sasha, It seems that you've missed this patch. Please apply. Thanks, Eli Eli Dorfman (Voltaire) wrote: allocate mft according to number of switch ports calculate number of port masks according to number of switch ports and allocate MFT accordingly Signed-off-by: Eli Dorfman

Re: [PATCH 24/24] IB/qib: Improve SERDES tunning on QMH boards

2011-01-04 Thread Roland Dreier
@@ -7596,8 +7616,6 @@ static int serdes_7322_init_new(struct qib_pportdata *ppd) /* VGA output common mode */ ibsd_wr_allchans(ppd, 12, (3 2), BMASK(3, 2)); -/* Add a delay to allow the SERDES to init */ -msleep(3000); return 0; } Am I confused or is

Re: [PATCH 0/24] various fixes for QIB driver

2011-01-04 Thread Roland Dreier
The following patches (ordered first to last) are for various bug fixes found during internal testing. OK, but in the future please send patches in smaller sets (say, no more than 10 at a time). It's hard to review and apply big patch bombs. Also when I apply all this stuff, you will be

RE: [PATCH 24/24] IB/qib: Improve SERDES tunning on QMH boards

2011-01-04 Thread Mike Marciniszyn
I noticed that, but was concerned with conflicts, which there are none in this case. 23 and 24 can combined to get rid of the hunk below. I have no issue with losing the history for 23. Mike -Original Message- From: Roland Dreier [mailto:rdre...@cisco.com] Sent: Tuesday, January

Re: [PATCH 18/24] IB/qib: adding fix missing from earlier patch

2011-01-04 Thread Roland Dreier
rcv/error race fix was missing part of a fix that makes merges impossible. Which fix? What merges are impossible? Is it a good thing or a bad thing for merges to be impossible? - R. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to

RE: [PATCH 0/24] various fixes for QIB driver

2011-01-04 Thread Mike Marciniszyn
Understood on the volume. We hadn't had any fixes for a bit, there was a considerable backlog. Mike -Original Message- From: Roland Dreier [mailto:rdre...@cisco.com] Sent: Tuesday, January 04, 2011 1:59 PM To: Mike Marciniszyn Cc: linux-rdma@vger.kernel.org Subject: Re: [PATCH 0/24]

Re: [PATCHv10 1/4] libibverbs: Add link layer field to ibv_port_attr

2011-01-04 Thread Roland Dreier
Are you planning to look into completing the IBoE support for user space any time soon? with v10 of the patch Eli posted four patches to libibverbs and two patch to libmlx4 Yes, I just got back from a long vacation and I want to try and get some things into my 2.6.38 tree first, and then

Re: [PATCH] mlx4: Change a warning message to debug

2011-01-04 Thread Roland Dreier
OK, I just got rid of the message entirely. -- 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

Re: [PATCHv10 1/4] libibverbs: Add link layer field to ibv_port_attr

2011-01-04 Thread Or Gerlitz
Roland Dreier rdre...@cisco.com wrote: I just got back from a long vacation and I want to try and get some things into my 2.6.38 tree first, and then work on userspace stuff in a few days. sounds great, so what's on the plate for 2.6.38? Or. -- To unsubscribe from this list: send the line

Re: [PATCH] mlx4/ib: Fix WARNING: at lib/swiotlb.c:562

2011-01-04 Thread Roland Dreier
Thanks, applied (although your mailer caused some whitespace damage to the patch that I had to fix by hand) -- 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

Re: [PATCH] mlx4_core: avoid vunmap of invalid pointer in allocation bad flow

2011-01-04 Thread Roland Dreier
thanks, applied. -- 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

Re: double implementation of create QP in the stack.

2011-01-04 Thread Roland Dreier
I see that the /drivers/infiniband/core contains two identical implementations of the qp create implementation. One is in verbs.c ... And ib_uverbs_cmd.c contains a second one. Is there any reason for such double implementation of create_qp(). No good reason I don't think... the two

Re: [PATCHv10 1/4] libibverbs: Add link layer field to ibv_port_attr

2011-01-04 Thread Roland Dreier
sounds great, so what's on the plate for 2.6.38? Don't know for sure, just working through the backlog at patchwork, taking the easier/more obvious stuff first and seeing how far I get. - R. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to

[PATCH] uDAPL v2.0: common: qp modify RTR using wrong ep attribute parameter for dest_rd_atomic

2011-01-04 Thread Davis, Arlin R
max_rdma_read_in should be used instead of max_rdma_read_out Signed-off-by: Arlin Davis arlin.r.da...@intel.com --- dapl/openib_common/qp.c | 12 +++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git a/dapl/openib_common/qp.c b/dapl/openib_common/qp.c index

[PATCH] uDAPL v2.0: scm: retry socket connect on ECONNREFUSED under heavy load

2011-01-04 Thread Davis, Arlin R
with large scale workloads a linux server starts rejecting socket connect requests. Add retry logic for connection refused errors. increasing net.ipv4.tcp_max_syn_backlog to 2048 will also reduce the chance of these errors when scaling up. Signed-off-by: Arlin Davis arlin.r.da...@intel.com ---

RE: [PATCH] uDAPL v2.0: scm: retry socket connect on ECONNREFUSED under heavy load

2011-01-04 Thread Hefty, Sean
+ if ((err == ETIMEDOUT) || (ECONNREFUSED --cm_ptr-retry)) { This is missing 'err ==' on the right-hand side. -- 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

[PATCH v2] uDAPL v2.0: scm: retry socket connect on ECONNREFUSED under heavy load

2011-01-04 Thread Davis, Arlin R
+if ((err == ETIMEDOUT) || (ECONNREFUSED --cm_ptr-retry)) { This is missing 'err ==' on the right-hand side. Just making sure you are paying attention. Thanks! --- with large scale workloads a linux server starts rejecting socket connect requests. Add retry logic for connection