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

2013-09-13 Thread Jack Wang
On 09/12/2013 06:30 PM, Bart Van Assche wrote: On 09/12/13 18:16, Jack Wang wrote: On 09/12/2013 12:16 AM, David Dillow wrote: On Tue, 2013-09-10 at 19:44 +0200, Bart Van Assche wrote: If this name was not yet in use in any interface that is visible in user space, I would agree that we

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

2013-09-13 Thread Bart Van Assche
On 09/13/13 10:06, Jack Wang wrote: On 09/12/2013 06:30 PM, Bart Van Assche wrote: On 09/12/13 18:16, Jack Wang wrote: On 09/12/2013 12:16 AM, David Dillow wrote: On Tue, 2013-09-10 at 19:44 +0200, Bart Van Assche wrote: If this name was not yet in use in any interface that is visible in

physical memory registration

2013-09-13 Thread Eva Mishra
I understood following steps for register physical memory region verb as defined in InfiniBand specification step 1: Allocate kernel memory using kmalloc etc. Return address is a kernel virtual address. Allocate multiple buffers. step 2: Get corresponding physical addressess of buffers returned

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

2013-09-13 Thread Bart Van Assche
On 09/13/13 10:40, Bart Van Assche wrote: On 09/13/13 10:06, Jack Wang wrote: On 09/12/2013 06:30 PM, Bart Van Assche wrote: On 09/12/13 18:16, Jack Wang wrote: On 09/12/2013 12:16 AM, David Dillow wrote: On Tue, 2013-09-10 at 19:44 +0200, Bart Van Assche wrote: If this name was not yet in

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

2013-09-13 Thread Jack Wang
On 09/13/2013 11:24 AM, Bart Van Assche wrote: On 09/13/13 10:40, Bart Van Assche wrote: On 09/13/13 10:06, Jack Wang wrote: On 09/12/2013 06:30 PM, Bart Van Assche wrote: On 09/12/13 18:16, Jack Wang wrote: On 09/12/2013 12:16 AM, David Dillow wrote: On Tue, 2013-09-10 at 19:44 +0200, Bart

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

2013-09-13 Thread Bart Van Assche
On 09/13/13 14:25, Jack Wang wrote: I tried your srp-ha branch in github, echo string SRP2=id_ext=${THCA2_GUID},ioc_guid=${THCA2_GUID},dgid=${TGID_P2},pkey=${PKEY},service_id=${THCA2_GUID},can_queue=512 to add_target failed with ib_srp: unknown parameter or missing value 'can_queue=512 [

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

2013-09-13 Thread Jack Wang
On 09/13/2013 03:33 PM, Bart Van Assche wrote: On 09/13/13 14:25, Jack Wang wrote: I tried your srp-ha branch in github, echo string SRP2=id_ext=${THCA2_GUID},ioc_guid=${THCA2_GUID},dgid=${TGID_P2},pkey=${PKEY},service_id=${THCA2_GUID},can_queue=512 to add_target failed with ib_srp:

ibv_poll_cq misbehaving with RoCE 40Gbps

2013-09-13 Thread xrg . dev
Hello list, I am experiencing a strange behavior with ibv_poll_cq on RoCE 40 Gb. When calling the following: int N = ibv_poll_cq (CQ, DEPTH, WC); N sometimes becomes equal to DEPTH, although the number of outstanding WR is well below DEPTH. In our current example, DEPTH = 128 and the # of

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

2013-09-13 Thread Bart Van Assche
On 09/13/13 15:51, Jack Wang wrote: On 09/13/2013 03:33 PM, Bart Van Assche wrote: On 09/13/13 14:25, Jack Wang wrote: I tried your srp-ha branch in github, echo string SRP2=id_ext=${THCA2_GUID},ioc_guid=${THCA2_GUID},dgid=${TGID_P2},pkey=${PKEY},service_id=${THCA2_GUID},can_queue=512 to

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

2013-09-13 Thread Bart Van Assche
On 09/13/13 16:15, Jack Wang wrote: Hello Bart, cat /sys/class/scsi_host/host36/req_lim 64 I just checked srp spec, which do define such behaviour, I wonder in SCST/SRPT, how the request limit is chosen, is it report from low level hardware driver? Hello Jack, The following code probably

Re: [PATCH V2 2/4] IB/usnic: Cisco VIC - driver part 1/3

2013-09-13 Thread Upinder Malhi (umalhi)
Hi Sean, We selected version 8 because it is unused - not a standard version and not in use by anyone. Upinder On Sep 12, 2013, at 10:02 AM, Hefty, Sean sean.he...@intel.com wrote: We are using the RoCE ethertype in the MAC frames, and that is all of the similarity between usNIC

[PATCH 1/1] infiniband: hw: fix uninitialized ‘old_qps’

2013-09-13 Thread Antonio Alecrim Jr
Added missing old_qps initialization in order to fix the following warnings: drivers/infiniband/hw/ocrdma/ocrdma_verbs.c: In function ‘_ocrdma_modify_qp’: drivers/infiniband/hw/ocrdma/ocrdma_verbs.c:1299:31: warning: ‘old_qps’ may be used uninitialized in this function [-Wmaybe-uninitialized]

[patch 1/1] drivers/infiniband/core/cm.c: convert to using idr_alloc_cyclic()

2013-09-13 Thread akpm
From: Zhao Hongjiang zhaohongji...@huawei.com Subject: drivers/infiniband/core/cm.c: convert to using idr_alloc_cyclic() commit 3e6628c4b347 (idr: introduce idr_alloc_cyclic()) adds a new idr_alloc_cyclic routine and converts several of these users to it. This is just a missed one - add it.