RE: [PATCH] IB/qib: add optional numa affinity

2013-06-27 Thread Marciniszyn, Mike
Or, Sorry for the delay. The maximum performance differs from CPU to CPU. The module parameter was chosen to avoid CPU family tests, which didn't seem like a good idea. Mike -Original Message- From: Or Gerlitz [mailto:or.gerl...@gmail.com] Sent: Saturday, June 22, 2013 4:46 PM

[PATCHv2 ibacm 10/15] Add the ability to preload the destination GID and LID caches

2013-06-27 Thread Hal Rosenstock
Preloading of these caches is supported via a file which is produced by OpenSM by the dump_pr plugin which contains sufficient SA PathRecord information. Details on this file format and configuring OpenSM for this are found in dump_pr_notes.txt in dump_pr. File format is specified in

[PATCHv2 13/16 ibacm] ib_acme.c: Support IPv6 addressing in resolve_ip

2013-06-27 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Rebased src/acme.c | 33 - 1 files changed, 28 insertions(+), 5 deletions(-) diff --git a/src/acme.c b/src/acme.c index ec1d3d2..5ba5c0a 100644 --- a/src/acme.c +++ b/src/acme.c @@ -476,29

[PATCHv2 11/16 ibacm] Add description of preloading GID and LID destination caches to acm_notes.txt

2013-06-27 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Updated for changes in previous patch v2 10/16 acm_notes.txt | 10 ++ man/ibacm.1 |2 +- 2 files changed, 11 insertions(+), 1 deletions(-) diff --git a/acm_notes.txt b/acm_notes.txt index 99eaf9a..50f0286

[PATCH v2 16/16 ibacm] Add option to allow for ACM cache lookup performance measurement

2013-06-27 Thread Hal Rosenstock
-C option is added for resolution repetition count Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Rebased man/ib_acme.1 |8 ++-- src/acme.c| 52 +++- src/libacm.c | 20 +++- src/libacm.h |

[PATCHv2 14/16 ibacm] Convert linux header files from DOS to unix format

2013-06-27 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: New patch since v1 patches linux/dlist.h | 162 ++-- linux/osd.h | 344 2 files changed, 253 insertions(+), 253 deletions(-) diff --git

[PATCHv2 12/16 ibacm] Increase buffer space for inet_ntop from 32 to INET6_ADDRSTRLEN

2013-06-27 Thread Hal Rosenstock
Signed-off-by: Hal Rosenstock h...@mellanox.com --- Change since v1: Rebased linux/acme_linux.c |2 +- src/acm.c |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/linux/acme_linux.c b/linux/acme_linux.c index 1039645..ef5b107 100644 ---

[PATCH v2 15/16 ibacm] Add support for preloading IPv4 and IPv6 ACM caches

2013-06-27 Thread Hal Rosenstock
A hosts file (ibacm_hosts.cfg) is configured with tuples of IP addresses (either IPv4 or IPv6) and IB GID. This information in conjunction with the path record preloading allows for the IPv4 and IPv6 ACM caches to be preloaded. IPv4 was tested with both rping and ucmatose as well as acme. IPv6

[PATCH v3 24/45] infiniband: ehca: Use get/put_online_cpus_atomic() to prevent CPU offline

2013-06-27 Thread Srivatsa S. Bhat
Once stop_machine() is gone from the CPU offline path, we won't be able to depend on disabling preemption to prevent CPUs from going offline from under us. Use the get/put_online_cpus_atomic() APIs to prevent CPUs from going offline, while invoking from atomic context. Cc: Hoang-Nam Nguyen

RE: [PATCH V2 for-next 1/4] IB/core: Add receive Flow Steering support

2013-06-27 Thread Hefty, Sean
The TCP/IP filters are broken into separate filters based in L4/L3. It would seem to make sense if the IB filters were similarly divided into L2/L3/L4 filters. IB and IPv6 could probably share the same filter definition. IPv6 filters wasn't defined through this submission, but as I

Re: [PATCH 03/14] IB/srp: Avoid that srp_reset_host() is skipped after a TL error

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:23 +0200, Bart Van Assche wrote: The SCSI error handler assumes that the transport layer is operational if an eh_abort_handler() returns SUCCESS. Hence let srp_abort() only return SUCCESS if sending the ABORT TASK task management function succeeded. This patch avoids

Re: [PATCH 04/14] IB/srp: Skip host settle delay

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:24 +0200, Bart Van Assche wrote: The SRP initiator implements host reset by reconnecting to the SRP target. That means that communication with the target is possible as soon as host reset finished. Hence skip the host settle delay. Acked-by: David Dillow

Re: [PATCH 01/14] IB/srp: Fix remove_one crash due to resource exhaustion

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:20 +0200, Bart Van Assche wrote: If the add_one callback fails during driver load no resources are allocated so there isn't a need to release any resources. Trying to clean the resource may lead to the following kernel panic: Acked-by: David Dillow dillo...@ornl.gov

Re: [PATCH 05/14] IB/srp: Maintain a single connection per I_T nexus

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:25 +0200, Bart Van Assche wrote: An SRP target is required to maintain a single connection between initiator and target. This means that if the 'add_target' attribute is used to create a second connection to a target that the first connection will be logged out and

Re: [PATCH 02/14] IB/srp: Fix race between srp_queuecommand() and srp_claim_req()

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 17:14 +0200, Bart Van Assche wrote: Not sure how I missed that ... I will drop this version and replace it with the proper fix. I expect the merge window will be opening soon; I should be able to make time on Sunday to review anything you post by then -- do you think the

Re: [PATCH 13/14] IB/srp: Make transport layer retry count configurable

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:36 +0200, Bart Van Assche wrote: From: Vu Pham vuhu...@mellanox.com Allow the InfiniBand RC retry count to be configured by the user as an option in the target login string. The transport layer timeout in nanoseconds is computed as follows from the retry count:

Re: [PATCH 12/14] IB/srp: Make HCA completion vector configurable

2013-06-27 Thread David Dillow
On Wed, 2013-06-12 at 15:35 +0200, Bart Van Assche wrote: Several InfiniBand HCA's allow to configure the completion vector per queue pair. This allows to spread the workload created by IB completion interrupts over multiple MSI-X vectors and hence over multiple CPU cores. In other words,

Re: [PATCH V2 for-next 1/4] IB/core: Add receive Flow Steering support

2013-06-27 Thread Or Gerlitz
On Thu, Jun 27, 2013 at 12:33 AM, Steve Wise sw...@opengridcomputing.com wrote: On 6/26/2013 4:13 PM, Or Gerlitz wrote: On Wed, Jun 26, 2013 at 10:56 PM, Hefty, Sean sean.he...@intel.com Steering rules processing order is according to rules priority. The user sets the 12 low-order bits from

Re: [PATCH V2 for-next 1/4] IB/core: Add receive Flow Steering support

2013-06-27 Thread Or Gerlitz
On Thu, Jun 27, 2013 at 11:55 PM, Hefty, Sean sean.he...@intel.com wrote: My point was that the IPv6 filter should be defined and used here. The following basic filters were defined: ethernet - src/dst mac ... ip -src/dst ip tcp/udp - src/dst port These are at

Re: [PATCH 01/14] IB/srp: Fix remove_one crash due to resource exhaustion

2013-06-27 Thread Roland Dreier
On Thu, Jun 27, 2013 at 2:01 PM, David Dillow dillo...@ornl.gov wrote: On Wed, 2013-06-12 at 15:20 +0200, Bart Van Assche wrote: If the add_one callback fails during driver load no resources are allocated so there isn't a need to release any resources. Trying to clean the resource may lead to

RE: [PATCH V2 for-next 1/4] IB/core: Add receive Flow Steering support

2013-06-27 Thread Hefty, Sean
Still for the initial set of patches that goes in I tend to just remove the IB filter structure and define the different IB filters along your proposal in a follow-up patches/es, OK? That sounds fine to me. -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a

RE: [PATCH 00/15 ibacm] ACM Cache Preloading (and some cosmetic cleanups)

2013-06-27 Thread Hefty, Sean
Patches 1-9 clean up documentation and sample config files. Rather than trying to keep the sample ibacm_addr/opts files in sync, I elected to remove them from the git tree. There's really no use for them, and I don't think they're included with the released package anyway. Doing this drops

RE: [PATCH 03/15 ibacm] acm.c: Cosmetic formatting change to acm log message

2013-06-27 Thread Hefty, Sean
diff --git a/src/acm.c b/src/acm.c index e956b09..a124e93 100644 --- a/src/acm.c +++ b/src/acm.c @@ -1277,7 +1277,7 @@ static void acm_process_acm_recv(struct acm_ep *ep, struct ibv_wc *wc, struct ac rec = (struct acm_resolve_rec *) mad-data; acm_format_name(2, log_data, sizeof