[PATCH 1/2] staging/rdma/hfi1: add common routine for queuing acks

2015-11-10 Thread Mike Marciniszyn
This patch is a prelimary patch required to coalesce acks. The routine to "schedule" a QP for sending a NAK is now centralized in rc_defer_ack(). The flag is changed for clarity since the all acks will potentially use the deferral mechanism. Reviewed-by: Dennis Dalessandro Signed-of

[PATCH 2/2] staging/rdma/hfi1: add ACK coalescing logic

2015-11-10 Thread Mike Marciniszyn
Implement ACK coalesing logic using a 8 bit counter. The algorithm is send pio ack when: - fecn present - this is the first packet in an interrupt session - counter is >= HFI1_PSN_CREDIT Otherwise the ack is defered. Reviewed-by: Dennis Dalessandro Signed-off-by: Mike Marciniszyn --- driv

[PATCH] staging/rdma/hfi1: fix pio progress routine race with allocator

2015-12-03 Thread Mike Marciniszyn
. Reviewed-by: Mark Debbage Signed-off-by: Mike Marciniszyn --- drivers/staging/rdma/hfi1/pio.c |9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rdma/hfi1/pio.c b/drivers/staging/rdma/hfi1/pio.c index eab58c1..8e10857 100644 --- a/drivers/staging/rdma/hfi1

[PATCH] staging/rdma/hfi1: fix sdma build failures to always clean up

2015-12-03 Thread Mike Marciniszyn
There are holes in the sdma build support routines that do not clean any partially built sdma descriptors after mapping or allocate failures. This patch corrects these issues. Reviewed-by: Dennis Dalessandro Signed-off-by: Mike Marciniszyn --- drivers/staging/rdma/hfi1/sdma.c | 10

[PATCH] staging/rdma/hfi1: convert buffers allocated atomic to per cpu

2015-12-07 Thread Mike Marciniszyn
Profiling has shown the the atomic is a performance issue for the pio hot path. If multiple cpus allocated an sc's buffer, the cacheline containing the atomic will bounce from L0 to L0. Convert the atomic to a percpu variable. Reviewed-by: Jubin John Signed-off-by: Mike Marcin

[PATCH 1/2] staging/rdma/hfi1: add definitions for OPA traps

2015-12-10 Thread Mike Marciniszyn
From: Jubin John These new definitions will be used by follow-on patches for formating and sending OPA traps. Reviewed-by: Ira Weiny Signed-off-by: Jubin John --- drivers/staging/rdma/hfi1/mad.h | 114 +++ 1 file changed, 114 insertions(+) diff --git a/dr

[PATCH 2/2] staging/rdma/hfi1: HFI now sends OPA Traps instead of IBTA

2015-12-10 Thread Mike Marciniszyn
From: Erik E. Kahn send_trap() was still using old ib_smp instead of opa_smp for formatting and sending traps. Reviewed-by: Arthur Kepner Reviewed-by: Ira Weiny Reviewed-by: Dennis Dalessandro Signed-off-by: Jubin John Signed-off-by: Erik E. Kahn --- drivers/staging/rdma/hfi1/mad.c | 12

[PATCH 0/2] Convert IBTA traps to OPA traps

2015-12-10 Thread Mike Marciniszyn
This two patch series gets rid of the vestigal use of IBTA traps in the hfi1 driver. --- Erik E. Kahn (1): staging/rdma/hfi1: HFI now sends OPA Traps instead of IBTA Jubin John (1): staging/rdma/hfi1: add definitions for OPA traps drivers/staging/rdma/hfi1/mad.c | 121 ++

[PATCH] staging/rdma/hfi1: Fix a possible null pointer dereference

2015-12-10 Thread Mike Marciniszyn
From: Easwar Hariharan A code inspection pointed out that kmalloc_array may return NULL and memset doesn't check the input pointer for NULL, resulting in a possible NULL dereference. This patch fixes this. Reviewed-by: Mike Marciniszyn Signed-off-by: Easwar Hariharan --- drivers/staging

[PATCH] staging/rdma/hfi1: Fix for module parameter hdrq_entsize when it's 0

2015-12-10 Thread Mike Marciniszyn
From: Sebastian Sanchez If driver is loaded with parameter hdrq_entsize=0, then there's a NULL dereference when the driver gets unloaded. This causes a kernel Oops and prevents the module from being unloaded. This patch fixes this issue by making sure -EINVAL gets returned when hdrq_entsize=0.

[PATCH] staging/rdma/hfi1: Change num_rcv_contexts to num_user_contexts and its meaning

2015-12-11 Thread Mike Marciniszyn
variable num_rcv_contexts to num_user_contexts, and it also makes any negative value for this variable default to the number of CPU cores, so if num_user_contexts is set >= 0, the value will number of contexts. Reviewed-by: Mike Marciniszyn Signed-off-by: Sebastian Sanchez --- drivers/stag

[PATCH] MAINTAINERS: Add maintainer section for hfi1

2015-08-18 Thread Mike Marciniszyn
Signed-off-by: Mike Marciniszyn --- 0 files changed diff --git a/MAINTAINERS b/MAINTAINERS index b3c1a56..45953e9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9892,6 +9892,13 @@ M: Arnaud Patard S: Odd Fixes F: drivers/staging/xgifb/ +HFI1 DRIVER +M: Mike Marciniszyn

[PATCH] Kconfig: add temporary PCI dependency

2015-08-18 Thread Mike Marciniszyn
The move from infiniband to staging requires a temporary PCI dependency to fix 0-day build issues. The drivers/infiniband/hw/Kconfig gratuitously added it for all drivers. Signed-off-by: Mike Marciniszyn --- drivers/staging/hfi1/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] IB/hfi1: fix sdma_descq_cnt parameter parsing

2015-09-15 Thread Mike Marciniszyn
The boolean tests should have been or-ed. Reported-by: David Binderman Reviewed-by: Jubin John Signed-off-by: Mike Marciniszyn --- drivers/staging/rdma/hfi1/sdma.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rdma/hfi1/sdma.c b/drivers/staging/rdma