Re: [ceph-users] Help needed porting Ceph to RSockets

2013-08-20 Thread Andreas Bluemle
Hi Sean, I will re-check until the end of the week; there is some test scheduling issue with our test system, which affects my access times. Thanks Andreas On Mon, 19 Aug 2013 17:10:11 + Hefty, Sean sean.he...@intel.com wrote: Can you see if the patch below fixes the hang?

Re: [ceph-users] Help needed porting Ceph to RSockets

2013-08-20 Thread Andreas Bluemle
Hi, I have added the patch and re-tested: I still encounter hangs of my application. I am not quite sure whether the I hit the same error on the shutdown because now I don't hit the error always, but only every now and then. WHen adding the patch to my code base (git tag v1.0.17) I notice an

[PATCH 0/8] IB SRP initiator patches for kernel 3.12

2013-08-20 Thread Bart Van Assche
The purpose of this InfiniBand SRP initiator patch series is as follows: - Make the SRP initiator driver better suited for use in a H.A. setup. Add fast_io_fail_tmo, dev_loss_tmo and reconnect_delay parameters. These can be used either to speed up failover or to avoid device removal when

[PATCH 1/8] IB/srp: Keep rport as long as the IB transport layer

2013-08-20 Thread Bart Van Assche
Keep the rport data structure around after srp_remove_host() has finished until cleanup of the IB transport layer has finished completely. This is necessary because later patches use the rport pointer inside the queuecommand callback. Without this patch accessing the rport from inside a

[PATCH 2/8] scsi_transport_srp: Add transport layer error handling

2013-08-20 Thread Bart Van Assche
Add the necessary functions in the SRP transport module to allow an SRP initiator driver to implement transport layer error handling similar to the functionality already provided by the FC transport layer. This includes: - Support for implementing fast_io_fail_tmo, the time that should elapse

[PATCH 3/8] IB/srp: Add srp_terminate_io()

2013-08-20 Thread Bart Van Assche
Finish all outstanding I/O requests after fast_io_fail_tmo expired, which speeds up failover in a multipath setup. This patch is a reworked version of a patch from Sebastian Riemer. Reported-by: Sebastian Riemer sebastian.rie...@profitbricks.com Signed-off-by: Bart Van Assche bvanass...@acm.org

[PATCH 4/8] IB/srp: Use SRP transport layer error recovery

2013-08-20 Thread Bart Van Assche
Enable reconnect_delay, fast_io_fail_tmo and dev_loss_tmo functionality for the IB SRP initiator. Add kernel module parameters that allow to specify default values for these three parameters. Signed-off-by: Bart Van Assche bvanass...@acm.org Acked-by: David Dillow dillo...@ornl.gov Cc: Roland

[PATCH 5/8] IB/srp: Start timers if a transport layer error occurs

2013-08-20 Thread Bart Van Assche
Start the reconnect timer, fast_io_fail timer and dev_loss timers if a transport layer error occurs. Signed-off-by: Bart Van Assche bvanass...@acm.org Acked-by: David Dillow dillo...@ornl.gov Cc: Roland Dreier rol...@kernel.org Cc: Vu Pham v...@mellanox.com Cc: Sebastian Riemer

[PATCH 7/8] IB/srp: Introduce srp_alloc_req_data()

2013-08-20 Thread Bart Van Assche
This patch does not change any functionality. Signed-off-by: Bart Van Assche bvanass...@acm.org Cc: Roland Dreier rol...@purestorage.com Cc: David Dillow dillo...@ornl.gov Cc: Vu Pham v...@mellanox.com Cc: Sebastian Riemer sebastian.rie...@profitbricks.com --- drivers/infiniband/ulp/srp/ib_srp.c

[PATCH 8/8] IB/srp: Make queue size configurable

2013-08-20 Thread Bart Van Assche
Certain storage configurations, e.g. a sufficiently large array of hard disks in a RAID configuration, need a queue depth above 64 to achieve optimal performance. Hence make the queue depth configurable. Signed-off-by: Bart Van Assche bvanass...@acm.org Cc: Roland Dreier rol...@purestorage.com

Re: [PATCH] osm_port_info_rcv.c Issue a log message if we cannot read the MKey of a port

2013-08-20 Thread Hal Rosenstock
On 8/19/2013 6:46 AM, Line Holen wrote: On 08/16/13 15:47, Hal Rosenstock wrote: On 8/14/2013 6:26 AM, Line Holen wrote: Signed-off-by: Line Holenline.ho...@oracle.com --- diff --git a/opensm/osm_port_info_rcv.c b/opensm/osm_port_info_rcv.c index 7dcd15e..961b376 100644 ---

[PATCH opensm] osm_subnet.c: Fix memory leak caused by commit dc0760cb8088fbe079e19682570a884ba01e94ff

2013-08-20 Thread Hal Rosenstock
From: Vladimir Koushnir vladim...@mellanox.com double strdup for p_opt-dump_files_dir is causing memory leak Signed-off-by: Vladimir Koushnir vladim...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_subnet.c b/opensm/osm_subnet.c index 7ab1671..4b5ef38

[PATCH v2 opensm] osm_subnet.c: Fix memory leak caused by commit dc0760cb8088fbe079e19682570a884ba01e94ff

2013-08-20 Thread Hal Rosenstock
From: Vladimir Koushnir vladim...@mellanox.com double strdup for p_opt-dump_files_dir is causing memory leak Approach from Bart Van Assche bvanass...@acm.org Signed-off-by: Vladimir Koushnir vladim...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Eliminate

RE: [ceph-users] Help needed porting Ceph to RSockets

2013-08-20 Thread Hefty, Sean
I have added the patch and re-tested: I still encounter hangs of my application. I am not quite sure whether the I hit the same error on the shutdown because now I don't hit the error always, but only every now and then. I guess this is at least some progress... :/ WHen adding the patch to

RE: [PATCH] Add new verb: uv_query_port_max_datagram()

2013-08-20 Thread Hefty, Sean
Where is the documentation for this? Multiple people have referred to it, but I don't see any mention of it in libibverbs.git. This is an unmerged, yet to be accepted patch set. Extensions were added as part of adding support for XRC. Yishai Hadas posted v9 of the series on 8/1 - Add

[PATCH] infiniband-diags: fail configure if glib2 is not found

2013-08-20 Thread Ira Weiny
Signed-off-by: Ira Weiny ira.we...@intel.com --- README |1 + configure.ac |2 ++ 2 files changed, 3 insertions(+), 0 deletions(-) diff --git a/README b/README index d19e3e9..10a11e2 100644 --- a/README +++ b/README @@ -14,6 +14,7 @@ Dependencies: 2) libibumad = 1.3.7

[PATCH] IB/qib: Move COUNTER_MASK definition within qib_mad.h header guards

2013-08-20 Thread Ira Weiny
Commit 36a8f01c (IB/qib: Add congestion control agent implementation) caused statements to leak pass the header guard. Correct with this update. Reviewed-by: Marciniszyn, Mike mike.marcinis...@intel.com Signed-off-by: Ira Weiny ira.we...@intel.com --- drivers/infiniband/hw/qib/qib_mad.h |

Re: [PATCH 8/8] IB/srp: Make queue size configurable

2013-08-20 Thread Sagi Grimberg
On 8/20/2013 3:50 PM, Bart Van Assche wrote: Certain storage configurations, e.g. a sufficiently large array of hard disks in a RAID configuration, need a queue depth above 64 to achieve optimal performance. Hence make the queue depth configurable. Signed-off-by: Bart Van Assche

[PATCH opensm] osm_db_files.c: Fix memory leak when deleting entries from osm db

2013-08-20 Thread Hal Rosenstock
From: Alex Netes ale...@mellanox.com The key also should be freed. Signed-off-by: Alex Netes ale...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- opensm/osm_db_files.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/opensm/osm_db_files.c

Re: [PATCH 8/8] IB/srp: Make queue size configurable

2013-08-20 Thread Bart Van Assche
On 08/20/13 17:34, Sagi Grimberg wrote: On 8/20/2013 3:50 PM, Bart Van Assche wrote: Certain storage configurations, e.g. a sufficiently large array of hard disks in a RAID configuration, need a queue depth above 64 to achieve optimal performance. Hence make the queue depth configurable. [ ...

celina2john...@hotmail.com

2013-08-20 Thread celina2joh555n4...@libero.it
celina2john...@hotmail.com Hello, My name is Celina Johnson. i saw your profile today and become interesting to know more about you. please i will like you respond to me at my private e-mail address (celina2john...@hotmail.com) so that i will tell you more about my self and also give you my

[PATCH opensm] osm_db_files.c : Fix resource leak guid2lid parser

2013-08-20 Thread Hal Rosenstock
From: Dan Ben Yosef da...@dev.mellanox.co.il leaks the storage that p_accum_val and p_key points to. Signed-off-by: Dan Ben Yosef da...@dev.mellanox.co.il Reviewed-by: Vladimir Koushnir vladim...@mellanox.com Signed-off-by: Hal Rosenstock h...@mellanox.com --- diff --git a/opensm/osm_db_files.c

Re: [PATCH 8/8] IB/srp: Make queue size configurable

2013-08-20 Thread David Dillow
On Tue, 2013-08-20 at 17:55 +0200, Bart Van Assche wrote: On 08/20/13 17:34, Sagi Grimberg wrote: Question, If srp now will allow larger queues while using a single global FMR pool of size 1024, isn't it more likely now that in stress environment srp will run out of FMRs to handle IO