Re: [PATCH 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-13 Thread Jubin John
On Fri, Nov 13, 2015 at 06:45:58PM +0530, Sudip Mukherjee wrote: > On Wed, Nov 11, 2015 at 02:33:21AM -0500, Jubin John wrote: > > This patch fixes the checkpatch issue: > > CHECK: Prefer using the BIT macro > > > > Reviewed-by: Dean Luick <dean.lu...@intel.com> &

Re: [PATCH 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-13 Thread Jubin John
On Fri, Nov 13, 2015 at 01:03:44PM -0500, Jubin John wrote: > On Fri, Nov 13, 2015 at 11:09:22AM -0500, Jubin John wrote: > > On Fri, Nov 13, 2015 at 06:45:58PM +0530, Sudip Mukherjee wrote: > > > On Wed, Nov 11, 2015 at 02:33:21AM -0500, Jubin John wrote: > > > > T

Re: [PATCH 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-13 Thread Jubin John
On Fri, Nov 13, 2015 at 11:09:22AM -0500, Jubin John wrote: > On Fri, Nov 13, 2015 at 06:45:58PM +0530, Sudip Mukherjee wrote: > > On Wed, Nov 11, 2015 at 02:33:21AM -0500, Jubin John wrote: > > > This patch fixes the checkpatch issue: > > > CHECK: Prefer using the BIT

[PATCH 05/18] staging/rdma/hfi1: Clean up comments

2015-11-13 Thread Jubin John
From: Edward Mascarenhas <edward.mascaren...@intel.com> Clean up comments Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Edward Mascarenhas <edward.mascaren...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/r

[PATCH 04/18] staging/rdma/hfi1: Fix qp.h comments

2015-11-13 Thread Jubin John
From: Kaike Wan <kaike@intel.com> This patch fixes a few qp.h comments. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Kaike Wan <kaike@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/qp.h

[PATCH 01/18] staging/rdma/hfi1: Support alternate firmware names

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add support for an automatic fallback for firmware names. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- d

[PATCH 02/18] staging/rdma/hfi1: Decode CNP opcode

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add CNP opcode decode Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/trace.h |3

[PATCH 18/18] staging/rdma/hfi1: Workaround CONFIG_SDMA_VERBOSITY timing issue

2015-11-13 Thread Jubin John
Enabling CONFIG_SDMA_VERBOSITY exposes a timing issue in the driver/DC firmware. Removing the CCE_INT_* CSR reads from sdma_dumpstate and a sdma_dumpstate call from the sdma_sendctrl() works around the issue Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubi

[PATCH 13/18] staging/rdma/hfi1: remove SPC freeze error messages

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> An SPC freeze is not an error. Remove the messages. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/

[PATCH 16/18] staging/rdma/hfi1: Fix Xmit Wait calculation

2015-11-13 Thread Jubin John
d-by: Breyer, Scott J <scott.j.bre...@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/mad.c | 32 1 files changed, 20 insertions(+), 12 deletions(-) diff --g

[PATCH 11/18] staging/rdma/hfi1: Destroy workqueues if hfi1_register_ib_device() call returns error

2015-11-13 Thread Jubin John
ish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/init.c |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/drivers/staging/rdma/hfi1/init.c b/drivers/staging/rdma/hfi1/init.c index

[PATCH 08/18] staging/rdma/hfi1: Add a credit push on diagpkt allocate fail

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> When sending a diagnostic packet, if the send context does not have enough room, force a credit return and try again. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-

[PATCH 07/18] staging/rdma/hfi1: Extend quiet timeout

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> The longest quiet timeout is now 6s. Extend the driver wait. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- d

[PATCH 06/18] staging/rdma/hfi: Add one-time LCB reset

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add one-time LCB reset on driver load to pre-emptively work around any LCB power cycle issues. Reviewed-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH 03/18] staging/rdma/hfi1: Add aeth name syndrome decode

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add aeth name syndrome decode Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/

[PATCH 10/18] staging/rdma/hfi1: Adds software counters for bitfields within various error status fields

2015-11-13 Thread Jubin John
y: Joel Rosenzweig <joel.b.rosenzw...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 3185 +- drivers/staging/rdma/hfi1/chip.h | 269 drivers/staging/rdma/hfi1/hfi.h | 34 + 3 files

[PATCH 12/18] staging/rdma/hfi1: Unexpected link up pkey values are not an error

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Only warn when link up pkeys are not what we expect. Also, allow for the pkey to already be initialized. Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH 14/18] staging/rdma/hfi1: unknown frame messages are not errors

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Change reported unknown frame messages into a counter. These are informational, no errors. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH 09/18] staging/rdma/hfi1: Correctly limit VLs against SDMA engines

2015-11-13 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Correctly reduce the number of VLs when limited by the number of SDMA engines. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com&

[PATCH 15/18] staging/rdma/hfi1: Consider VL15 MTU also when calculating the maximum VL MTU

2015-11-13 Thread Jubin John
affect the VL15 traffic. Reviewed-by: Dean Luick <dean.lu...@intel.com> Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Harish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c

[PATCH 17/18] staging/rdma/hfi1: Adding counter resolutions for DataPortCounters

2015-11-13 Thread Jubin John
unters request. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Andrea Lowe <andrea.l.l...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/mad.c | 38 ++ 1 files changed, 26 ins

Re: [PATCH 10/13] staging/rdma/hfi1: adding per SDMA engine stats to hfistats

2015-11-16 Thread Jubin John
On Wed, Nov 11, 2015 at 11:22:15AM +0300, Dan Carpenter wrote: > On Wed, Nov 11, 2015 at 02:33:30AM -0500, Jubin John wrote: > > @@ -8288,6 +8367,21 @@ static int init_cntrs(struct hfi1_devdata *dd) > > dd->ndevcntrs++; > >

Re: [PATCH 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-16 Thread Jubin John
On Sat, Nov 14, 2015 at 12:41:02PM +0530, Sudip Mukherjee wrote: > On Fri, Nov 13, 2015 at 11:09:22AM -0500, Jubin John wrote: > > On Fri, Nov 13, 2015 at 06:45:58PM +0530, Sudip Mukherjee wrote: > > > On Wed, Nov 11, 2015 at 02:33:21AM -0500, Jubin John wrote: > &

[PATCH v2 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-16 Thread Jubin John
lt;ira.we...@intel.com> Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.h | 48 ++-- drivers/staging/rdma/hfi1/common.h |4 +- drivers/staging/r

[PATCH v2 02/13] staging/rdma/hfi1: Fix downgrade race

2015-11-16 Thread Jubin John
Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 31 --- 1 files changed, 24 insertions(+), 7 deletions(-) diff

[PATCH v2 08/13] staging/rdma/hfi1: change krcvqs module parameter type from byte to uint

2015-11-16 Thread Jubin John
-by: Mark F. Brown <mark.f.br...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/hfi.h |2 +- drivers/staging/rdma/hfi1/init.c |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rdma/hfi1/hfi.h b/d

[PATCH v2 05/13] staging/rdma/hfi1: Add space between concatenated string elements

2015-11-16 Thread Jubin John
Space between concantenated string elements is more human readable and fixes the checkpatch issue: CHECK: Concatenated strings should use spaces between elements Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/

[PATCH v2 04/13] staging/rdma/hfi1: Remove rcv bubbles code

2015-11-16 Thread Jubin John
From: Ira Weiny <ira.we...@intel.com> Rcv bubbles were improperly calculated for HFIs, fix that here. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off

[PATCH v2 00/13] staging/rdma/hfi1: Driver cleanup and misc fixes

2015-11-16 Thread Jubin John
a/hfi1: Adjust EPROM partitions, add EPROM commands Harish Chegondi (1): staging/rdma/hfi1: Move s_sde to the read mostly portion of the hfi1_qp structure Ira Weiny (1): staging/rdma/hfi1: Remove rcv bubbles code Jubin John (3): staging/rdma/hfi1: Use BIT macro staging/rdma/hfi1: Add space

[PATCH v2 06/13] staging/rdma/hfi1: Move s_sde to the read mostly portion of the hfi1_qp structure

2015-11-16 Thread Jubin John
ish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/verbs.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/rdma/hfi1/verbs.h b/drivers/staging/rdma/hfi1/verbs.h index 600bd1b..6380

[PATCH v2 03/13] staging/rdma/hfi1: remove RxCtxRHQS from hfi1stats

2015-11-16 Thread Jubin John
From: Vennila Megavannan <vennila.megavan...@intel.com> Removed the RxCtxRHQS counter being dumped into dev_cntrs Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Vennila Megavannan <vennila.megavan...@intel.com> Signed-off-by: Jubin John <

[PATCH v2 10/13] staging/rdma/hfi1: adding per SDMA engine stats to hfistats

2015-11-16 Thread Jubin John
Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Vennila Megavannan <vennila.megavan...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdm

[PATCH v2 12/13] staging/rdma/hfi1: Read EFI variable for device description

2015-11-16 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Read an EFI variable for the device description. Create the infrastructure for additional variable reads. Reviewed-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: J

[PATCH v2 11/13] staging/rdma/hfi1: Remove unneeded variable index

2015-11-16 Thread Jubin John
alessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 19 +++ 1 files changed, 7 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b

[PATCH v2 13/13] staging/rdma/hfi1: Adjust EPROM partitions, add EPROM commands

2015-11-16 Thread Jubin John
s when possible. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/eprom.c| 119 +++--- drivers/staging/rdm

[PATCH v2 07/13] staging/rdma/hfi1: rework is_a0() and is_bx()

2015-11-16 Thread Jubin John
From: Mike Marciniszyn <mike.marcinis...@intel.com> The current is_bx() will incorrectly match on other steppings. is_a0() is removed in favor of is_ax(). Reviewed-by: Mark Debbage <mark.debb...@intel.com> Signed-off-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-

[PATCH v2 09/13] staging/rdma/hfi1: Change default krcvqs

2015-11-16 Thread Jubin John
Change the default number of krcvqs to number of numa nodes + 1 based on the performance data collected. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c |2 +- 1 files changed, 1 inse

[PATCH 01/13] staging/rdma/hfi1: Use BIT macro

2015-11-10 Thread Jubin John
This patch fixes the checkpatch issue: CHECK: Prefer using the BIT macro Reviewed-by: Dean Luick <dean.lu...@intel.com> Reviewed-by: Ira Weiny <ira.we...@intel.com> Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com&

[PATCH 06/13] staging/rdma/hfi1: Move s_sde to the read mostly portion of the hfi1_qp structure

2015-11-10 Thread Jubin John
ish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/verbs.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/rdma/hfi1/verbs.h b/drivers/staging/rdma/hfi1/verbs.h index 600bd1b..6380

[PATCH 10/13] staging/rdma/hfi1: adding per SDMA engine stats to hfistats

2015-11-10 Thread Jubin John
Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Vennila Megavannan <vennila.megavan...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdm

[PATCH 12/13] staging/rdma/hfi1: Read EFI variable for device description

2015-11-10 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Read an EFI variable for the device description. Create the infrastructure for additional variable reads. Reviewed-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: J

[PATCH 11/13] staging/rdma/hfi1: Remove unneeded variable index

2015-11-10 Thread Jubin John
alessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 19 +++ 1 files changed, 7 insertions(+), 12 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b

[PATCH 13/13] staging/rdma/hfi1: Adjust EPROM partitions, add EPROM commands

2015-11-10 Thread Jubin John
s when possible. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/eprom.c| 119 +++--- drivers/staging/rdm

[PATCH 04/13] staging/rdma/hfi: Remove rcv bubbles code

2015-11-10 Thread Jubin John
From: Ira Weiny <ira.we...@intel.com> Rcv bubbles were improperly calculated for HFIs, fix that here. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off

[PATCH 03/13] staging/rdma/hfi1: remove RxCtxRHQS from hfi1stats

2015-11-10 Thread Jubin John
From: Vennila Megavannan <vennila.megavan...@intel.com> Removed the RxCtxRHQS counter being dumped into dev_cntrs Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Vennila Megavannan <vennila.megavan...@intel.com> Signed-off-by: Jubin John <

[PATCH 08/13] staging/rdma/hfi1: change krcvqs module parameter type from byte to uint

2015-11-10 Thread Jubin John
-by: Mark F. Brown <mark.f.br...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/hfi.h |2 +- drivers/staging/rdma/hfi1/init.c |4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rdma/hfi1/hfi.h b/d

[PATCH 09/13] staging/rdma/hfi1: Change default krcvqs

2015-11-10 Thread Jubin John
Change the default number of krcvqs to number of numa nodes + 1 based on the performance data collected. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c |2 +- 1 files changed, 1 inse

[PATCH 02/13] staging/rdma/hfi1: Fix downgrade race

2015-11-10 Thread Jubin John
Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 31 --- 1 files changed, 24 insertions(+), 7 deletions(-) diff

[PATCH 07/13] staging/rdma/hfi1: rework is_a0() and is_bx()

2015-11-10 Thread Jubin John
From: Mike Marciniszyn <mike.marcinis...@intel.com> The current is_bx() will incorrectly match on other steppings. is_a0() is removed in favor of is_ax(). Reviewed-by: Mark Debbage <mark.debb...@intel.com> Signed-off-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-

[PATCH 05/13] staging/rdma/hfi1: Add space between concatenated string elements

2015-11-10 Thread Jubin John
Space between concantenated string elements is more human readable and fixes the checkpatch issue: CHECK: Concatenated strings should use spaces between elements Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/

Re: [PATCH RESEND 11/11] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-09 Thread Jubin John
On Fri, Nov 06, 2015 at 07:25:31PM -0800, Greg KH wrote: > On Fri, Nov 06, 2015 at 08:07:06PM -0500, Jubin John wrote: > > --- a/drivers/staging/rdma/hfi1/hfi.h > > +++ b/drivers/staging/rdma/hfi1/hfi.h > > @@ -1084,6 +1084,10 @@ struct hfi1_devdata { > > /* Save

Re: [PATCH RESEND 10/11] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-09 Thread Jubin John
f (unlikely(vl >= num_vls)) { > > Can you prove that unlikely() makes a measured difference here? If not, > please remove it. If you can, please provide the proof in the changelog > when you resend it. Unfortunately, I can't prove this so will remove in v2. Thanks, Jubin John __

[PATCH v2 0/2] staging/rdma/hfi1: Miscellaneous Fixes

2015-11-09 Thread Jubin John
Updated these 2 patches based on feedback received. Changes in v2: - Removed volatile for receive context tail dummy address - Removed unlikely() in sdma_select_engine_vl() Ira Weiny (1): staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid Mark F. Brown (1):

[PATCH v2 1/2] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-09 Thread Jubin John
gt; Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Mitko Haralanov <mitko.harala...@intel.com> Signed-off-by: Mark F. Brown <mark.f.br...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 18 +

[PATCH v2 2/2] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-09 Thread Jubin John
Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Kaike Wan <kaike@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/sdma.c | 12 ++-- 1 files changed, 10 insertions(+),

[PATCH] staging/rdma/hfi1: Clear the QSFP reset that is asserted on FLR

2015-11-04 Thread jubin . john
; Signed-off-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 13 + 1 files changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/stagin

[PATCH] staging/rdma/hfi1: Enable WFR PCIe extended tags from the driver

2015-11-04 Thread jubin . john
wed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Ashutosh Dixit <ashutosh.di...@intel.com> Signed-off-by: Vennila Megavannan <vennila.megavan...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/pcie.c | 12 +++-

[PATCH] staging/rdma/hfi1: Disable thermal polling before sensor initialization

2015-11-04 Thread jubin . john
hermal polling from reset_asic_csrs() function. Reviewed by: Dennis Dalessandro <dennis.dalessan...@intel.com> Reviewed by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Jareer Abdel-Qader <jareer.h.abdel-qa...@intel.com> Signed-off-by: Ira Weiny <ira.we...@inte

[PATCH] staging/rdma/hfi1: Convert dd_dev_info() to hfi1_cdbg() in process startup

2015-11-04 Thread jubin . john
tel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/file_ops.c | 10 +- drivers/staging/rdma/hfi1/init.c | 31 ++- drivers/staging/rdma/hfi1/pio.c | 19 ++- 3 files changed, 33 insertions(+

[PATCH] staging/rdma/hfi1: Always download SBus firmware

2015-11-04 Thread jubin . john
; Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/firmware.c |2 +- drivers/staging/rdma/hfi1/pcie.c | 12 +--- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/drivers/stag

[PATCH RESEND 11/11] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-06 Thread Jubin John
gt; Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Mitko Haralanov <mitko.harala...@intel.com> Signed-off-by: Mark F. Brown <mark.f.br...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 18 +

[PATCH RESEND 08/11] staging/rdma/hfi1: Remove spurious error messages

2015-11-06 Thread Jubin John
gt; Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Ignacio Hernandez <ignacio.hernan...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/driver.c | 26

[PATCH RESEND 05/11] staging/rdma/hfi1: Disable thermal polling before sensor initialization

2015-11-06 Thread Jubin John
hermal polling from reset_asic_csrs() function. Reviewed by: Dennis Dalessandro <dennis.dalessan...@intel.com> Reviewed by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Jareer Abdel-Qader <jareer.h.abdel-qa...@intel.com> Signed-off-by: Ira Weiny <ira.we...@inte

[PATCH RESEND 09/11] staging/rdma/hfi1: use one-shot LCB write

2015-11-06 Thread Jubin John
tel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 42 + drivers/staging/rdma/hfi1/chip.h |1 + 2 files changed, 38 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/

[PATCH RESEND 06/11] staging/rdma/hfi1: Select only devices with active links

2015-11-06 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> When looking for or validating a user device, only use devices that are currently active. Reviewed-by: Mitko Haralanov <mitko.harala...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH RESEND 10/11] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-06 Thread Jubin John
Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Kaike Wan <kaike@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/sdma.c | 12 ++-- 1 files changed, 10 insertions(+),

[PATCH RESEND 01/11] staging/rdma/hfi1: Convert dd_dev_info() to hfi1_cdbg() in process startup

2015-11-06 Thread Jubin John
tel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/file_ops.c | 10 +- drivers/staging/rdma/hfi1/init.c | 31 ++- drivers/staging/rdma/hfi1/pio.c | 19 ++- 3 files changed, 33 insertions(+

[PATCH RESEND 00/11] staging/rdma/hfi1: Miscellaneous Fixes

2015-11-06 Thread Jubin John
Resending these previously sent hfi1 driver patches as a series, instead of individual patches, based on feedback about order of patch application from a friendly patch-bot of Greg KH. Dean Luick (3): staging/rdma/hfi1: Always download SBus firmware staging/rdma/hfi1: Select only devices

[PATCH RESEND 04/11] staging/rdma/hfi1: Always download SBus firmware

2015-11-06 Thread Jubin John
; Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/firmware.c |2 +- drivers/staging/rdma/hfi1/pcie.c | 12 +--- 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/drivers/stag

[PATCH RESEND 07/11] staging/rdma/hfi1: Fix for opaportconfig ledon by not checking for portNum

2015-11-06 Thread Jubin John
ar.hariha...@intel.com> Signed-off-by: Sebastian Sanchez <sebastian.sanc...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/mad.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rdma/hfi1/mad.c b/drivers

[PATCH RESEND 02/11] staging/rdma/hfi1: Clear the QSFP reset that is asserted on FLR

2015-11-06 Thread Jubin John
; Signed-off-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 13 + 1 files changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/stagin

Re: [PATCH] staging/rdma/hfi1: Disable thermal polling before sensor initialization

2015-11-05 Thread Jubin John
> This should be "Jareer Abdel-Qader <jareer.h.abdel-qa...@intel.com>". ok. Will fix in V2. Thanks, Jubin John ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH v2] staging/rdma/hfi1: Disable thermal polling before sensor initialization

2015-11-05 Thread Jubin John
hermal polling from reset_asic_csrs() function. Reviewed by: Dennis Dalessandro <dennis.dalessan...@intel.com> Reviewed by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Jareer Abdel-Qader <jareer.h.abdel-qa...@intel.com> Signed-off-by: Ira Weiny <ira.we...@inte

[PATCH] staging/rdma/hfi1: Fix for opaportconfig ledon by not checking for portNum

2015-11-05 Thread Jubin John
ar.hariha...@intel.com> Signed-off-by: Sebastian Sanchez <sebastian.sanc...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/mad.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rdma/hfi1/mad.c b/drivers

[PATCH] staging/rdma/hfi1: Remove spurious error messages

2015-11-05 Thread Jubin John
gt; Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Ignacio Hernandez <ignacio.hernan...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/driver.c | 26

[PATCH] staging/rdma/hfi1: Workaround to prevent corruption during packet delivery

2015-11-05 Thread Jubin John
gt; Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Mitko Haralanov <mitko.harala...@intel.com> Signed-off-by: Mark F. Brown <mark.f.br...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 18 +

[PATCH] staging/rdma/hfi1: Select only devices with active links

2015-11-05 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> When looking for or validating a user device, only use devices that are currently active. Reviewed-by: Mitko Haralanov <mitko.harala...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH] staging/rdma/hfi1: use one-shot LCB write

2015-11-05 Thread Jubin John
tel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/chip.c | 42 + drivers/staging/rdma/hfi1/chip.h |1 + 2 files changed, 38 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/

[PATCH] staging/rdma/hfi1: Eliminate WARN_ON when VL is invalid

2015-11-05 Thread Jubin John
Marciniszyn <mike.marcinis...@intel.com> Reviewed-by: Kaike Wan <kaike@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- drivers/staging/rdma/hfi1/sdma.c | 12 ++-- 1 files changed, 10 insertions(+),

[PATCH v2 00/17] Driver cleanup and misc fixes series 3

2015-12-01 Thread Jubin John
Changes in v2: - Added more information in commit messages of patches 01, 02 03, 04, 05, 07 and 09 - Fixed driver name to hfi1 in subject line of patch 06 - Refreshed patch 10 on top of staging-next - Dropped patch 18 (staging/rdma/hfi1: Workaround

[PATCH v2 05/17] staging/rdma/hfi1: Clean up comments

2015-12-01 Thread Jubin John
edward.mascaren...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Added more information in commit message drivers/staging/rdma/hfi1/chip.c |1 - drivers/staging/rdma/hfi1/driver.c |2 +- drivers/staging/rdma/hfi1/hfi.h|4 ++-- drivers/

[PATCH v2 07/17] staging/rdma/hfi1: Extend quiet timeout

2015-12-01 Thread Jubin John
: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Added more information in commit message drivers/staging/rdma/hfi1/chip.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rd

[PATCH v2 08/17] staging/rdma/hfi1: Add a credit push on diagpkt allocate fail

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> When sending a diagnostic packet, if the send context does not have enough room, force a credit return and try again. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-

[PATCH v2 10/17] staging/rdma/hfi1: Adds software counters for bitfields within various error status fields

2015-12-01 Thread Jubin John
y: Joel Rosenzweig <joel.b.rosenzw...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Refreshed patch on top of latest staging-next drivers/staging/rdma/hfi1/chip.c | 3185 +- drivers/staging/rdma/hfi1/chip.

[PATCH v2 14/17] staging/rdma/hfi1: unknown frame messages are not errors

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Change reported unknown frame messages into a counter. These are informational, no errors. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH v2 04/17] staging/rdma/hfi1: Fix qp.h comments

2015-12-01 Thread Jubin John
From: Kaike Wan <kaike@intel.com> This patch fixes a few incorrect header file comments in qp.h Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Kaike Wan <kaike@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in

[PATCH v2 13/17] staging/rdma/hfi1: remove SPC freeze error messages

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> An SPC freeze is not an error. Remove the messages. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in

[PATCH v2 12/17] staging/rdma/hfi1: Unexpected link up pkey values are not an error

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Only warn when link up pkeys are not what we expect. Also, allow for the pkey to already be initialized. Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH v2 17/17] staging/rdma/hfi1: Adding counter resolutions for DataPortCounters

2015-12-01 Thread Jubin John
unters request. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Andrea Lowe <andrea.l.l...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - No changes drivers/staging/rdma/hfi1/mad.c | 38 +++

[PATCH v2 06/17] staging/rdma/hfi1: Add one-time LCB reset

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add one-time LCB reset on driver load to pre-emptively work around any LCB power cycle issues. Reviewed-by: Easwar Hariharan <easwar.hariha...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <

[PATCH v2 15/17] staging/rdma/hfi1: Consider VL15 MTU also when calculating the maximum VL MTU

2015-12-01 Thread Jubin John
affect the VL15 traffic. Reviewed-by: Dean Luick <dean.lu...@intel.com> Reviewed-by: Arthur Kepner <arthur.kep...@intel.com> Signed-off-by: Harish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - No changes dr

[PATCH v2 02/17] staging/rdma/hfi1: Decode CNP opcode

2015-12-01 Thread Jubin John
p 0x80,CNP" Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Added more information in commit message drivers/staging/rdma/hfi1/trace

[PATCH v2 01/17] staging/rdma/hfi1: Support alternate firmware names

2015-12-01 Thread Jubin John
From: Dean Luick <dean.lu...@intel.com> Add support for an automatic fallback for firmware names to support debug-signed and production-signed firmware images. Reviewed-by: Dennis Dalessandro <dennis.dalessan...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Si

[PATCH v2 16/17] staging/rdma/hfi1: Fix Xmit Wait calculation

2015-12-01 Thread Jubin John
d-by: Breyer, Scott J <scott.j.bre...@intel.com> Signed-off-by: Ira Weiny <iwe...@gmail.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - No changes drivers/staging/rdma/hfi1/mad.c | 32 1 files changed, 20 insert

[PATCH v2 11/17] staging/rdma/hfi1: Destroy workqueues if hfi1_register_ib_device() call returns error

2015-12-01 Thread Jubin John
ish Chegondi <harish.chego...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - No changes drivers/staging/rdma/hfi1/init.c |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/drivers/staging/rdma/hfi1/init.c b/drivers/stagin

[PATCH v2 09/17] staging/rdma/hfi1: Correctly limit VLs against SDMA engines

2015-12-01 Thread Jubin John
th one of the platforms that doesn't have enough SDMA (4) mechanisms for the typical number of VLs. Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2:

[PATCH v2 03/17] staging/rdma/hfi1: Add aeth name syndrome decode

2015-12-01 Thread Jubin John
decode: aeth syn 0x00 Here is the decode with the fix: aeth syn 0x00 ACK Reviewed-by: Mike Marciniszyn <mike.marcinis...@intel.com> Signed-off-by: Dean Luick <dean.lu...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Added more infor

Re: [PATCH v2 00/13] staging/rdma/hfi1: Driver cleanup and misc fixes

2015-12-08 Thread Jubin John
Hi Greg, Is this patch series still in your queue? I just wanted to make sure it didn't get dropped. Thanks, Jubin John ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH v2] staging/rdma/hfi1: check for ARMED->ACTIVE transition in receive interrupt

2016-01-05 Thread Jubin John
On Mon, Jan 04, 2016 at 10:23:46PM +0200, Leon Romanovsky wrote: > On Mon, Jan 04, 2016 at 11:21:19AM -0500, Jubin John wrote: > > From: Jim Snow <jim.m.s...@intel.com> > > > > } else { > > + /* Auto activat

[PATCH v3] staging/rdma/hfi1: check for ARMED->ACTIVE transition in receive interrupt

2016-01-05 Thread Jubin John
ned-off-by: Brendan Cunningham <brendan.cunning...@intel.com> Signed-off-by: Jubin John <jubin.j...@intel.com> --- Changes in v2: - Fixed whitespace - Converted armed->active transition to inline function - Added comment to document reason for skipping HFI1_CTRL_

  1   2   >