mbox series

[for-next,v2,00/11] rdmavt/hfi1 updates for next

Message ID 20191126141055.58836.79452.stgit@awfm-01.aw.intel.com (mailing list archive)
Headers show
Series rdmavt/hfi1 updates for next | expand

Message

Dennis Dalessandro Nov. 26, 2019, 2:12 p.m. UTC
Here are some refactoring and code reorg patches for the merge window. Nothing
too scary, no new features. This lays the ground work for stuff coming in
future merge cycles.

There is also one bug fix, from Kaike.

Changes from v1
---------------
Fix Review-by tags with corrupted email address.

---

Grzegorz Andrejczuk (3):
      IB/hfi1: Move common receive IRQ code to function
      IB/hfi1: Decouple IRQ name from type
      IB/hfi1: Return void in packet receiving functions

Kaike Wan (1):
      IB/hfi1: Don't cancel unused work item

Michael J. Ruhl (1):
      IB/hfi1: List all receive contexts from debugfs

Mike Marciniszyn (6):
      IB/hfi1: Add accessor API routines to access context members
      IB/hfi1: Move chip specific functions to chip.c
      IB/hfi1: Add fast and slow handlers for receive context
      IB/hfi1: IB/hfi1: Add an API to handle special case drop
      IB/hfi1: Create API for auto activate
      IB/rdmavt: Correct comments in rdmavt_qp.h header


 drivers/infiniband/hw/hfi1/chip.c        |  187 ++++++++++++++++++------
 drivers/infiniband/hw/hfi1/chip.h        |    6 +
 drivers/infiniband/hw/hfi1/common.h      |    3 
 drivers/infiniband/hw/hfi1/debugfs.c     |    2 
 drivers/infiniband/hw/hfi1/driver.c      |  236 ++++++++++++------------------
 drivers/infiniband/hw/hfi1/file_ops.c    |   12 +-
 drivers/infiniband/hw/hfi1/hfi.h         |  193 ++++++++++++++++++++++++-
 drivers/infiniband/hw/hfi1/init.c        |   87 ++---------
 drivers/infiniband/hw/hfi1/iowait.c      |    4 -
 drivers/infiniband/hw/hfi1/msix.c        |  106 +++++++------
 drivers/infiniband/hw/hfi1/msix.h        |    1 
 drivers/infiniband/hw/hfi1/trace_ctxts.h |    2 
 drivers/infiniband/hw/hfi1/trace_rx.h    |   15 --
 drivers/infiniband/hw/hfi1/vnic_main.c   |    2 
 drivers/infiniband/sw/rdmavt/rc.c        |    9 +
 include/rdma/rdmavt_qp.h                 |   22 ---
 16 files changed, 523 insertions(+), 364 deletions(-)

--
-Denny

Comments

Jason Gunthorpe Nov. 26, 2019, 3:31 p.m. UTC | #1
On Tue, Nov 26, 2019 at 09:12:07AM -0500, Dennis Dalessandro wrote:
> Here are some refactoring and code reorg patches for the merge window. Nothing
> too scary, no new features. This lays the ground work for stuff coming in
> future merge cycles.

This is far too late for this cycle

Jason
Dennis Dalessandro Dec. 11, 2019, 1:34 p.m. UTC | #2
On 11/26/2019 9:12 AM, Dennis Dalessandro wrote:
> Here are some refactoring and code reorg patches for the merge window. Nothing
> too scary, no new features. This lays the ground work for stuff coming in
> future merge cycles.
> 
> There is also one bug fix, from Kaike.
> 
> Changes from v1
> ---------------
> Fix Review-by tags with corrupted email address.
> 
> ---
> 
> Grzegorz Andrejczuk (3):
>        IB/hfi1: Move common receive IRQ code to function
>        IB/hfi1: Decouple IRQ name from type
>        IB/hfi1: Return void in packet receiving functions
> 
> Kaike Wan (1):
>        IB/hfi1: Don't cancel unused work item
> 
> Michael J. Ruhl (1):
>        IB/hfi1: List all receive contexts from debugfs
> 
> Mike Marciniszyn (6):
>        IB/hfi1: Add accessor API routines to access context members
>        IB/hfi1: Move chip specific functions to chip.c
>        IB/hfi1: Add fast and slow handlers for receive context
>        IB/hfi1: IB/hfi1: Add an API to handle special case drop
>        IB/hfi1: Create API for auto activate
>        IB/rdmavt: Correct comments in rdmavt_qp.h header

I guess these didn't catch the train for 5.5 merge window. Do I need to 
resubmit to break this into two series, one for RC and one for next?

I think 1, 9, 10, and 11 could/should probably go for RC.

-Denny
Jason Gunthorpe Dec. 11, 2019, 4:21 p.m. UTC | #3
On Wed, Dec 11, 2019 at 08:34:40AM -0500, Dennis Dalessandro wrote:
> On 11/26/2019 9:12 AM, Dennis Dalessandro wrote:
> > Here are some refactoring and code reorg patches for the merge window. Nothing
> > too scary, no new features. This lays the ground work for stuff coming in
> > future merge cycles.
> > 
> > There is also one bug fix, from Kaike.
> > 
> > Changes from v1
> > Fix Review-by tags with corrupted email address.
> > 
> > 
> > Grzegorz Andrejczuk (3):
> >        IB/hfi1: Move common receive IRQ code to function
> >        IB/hfi1: Decouple IRQ name from type
> >        IB/hfi1: Return void in packet receiving functions
> > 
> > Kaike Wan (1):
> >        IB/hfi1: Don't cancel unused work item
> > 
> > Michael J. Ruhl (1):
> >        IB/hfi1: List all receive contexts from debugfs
> > 
> > Mike Marciniszyn (6):
> >        IB/hfi1: Add accessor API routines to access context members
> >        IB/hfi1: Move chip specific functions to chip.c
> >        IB/hfi1: Add fast and slow handlers for receive context
> >        IB/hfi1: IB/hfi1: Add an API to handle special case drop
> >        IB/hfi1: Create API for auto activate
> >        IB/rdmavt: Correct comments in rdmavt_qp.h header
> 
> I guess these didn't catch the train for 5.5 merge window. Do I need to
> resubmit to break this into two series, one for RC and one for next?

It is more reliable if it shows in patchworks as you want

> I think 1, 9, 10, and 11 could/should probably go for RC.

Make sure they have suitable commit messages please

Jason
Dennis Dalessandro Dec. 11, 2019, 4:33 p.m. UTC | #4
On 12/11/2019 11:21 AM, Jason Gunthorpe wrote:
> On Wed, Dec 11, 2019 at 08:34:40AM -0500, Dennis Dalessandro wrote:
>> On 11/26/2019 9:12 AM, Dennis Dalessandro wrote:
>>> Here are some refactoring and code reorg patches for the merge window. Nothing
>>> too scary, no new features. This lays the ground work for stuff coming in
>>> future merge cycles.
>>>
>>> There is also one bug fix, from Kaike.
>>>
>>> Changes from v1
>>> Fix Review-by tags with corrupted email address.
>>>
>>>
>>> Grzegorz Andrejczuk (3):
>>>         IB/hfi1: Move common receive IRQ code to function
>>>         IB/hfi1: Decouple IRQ name from type
>>>         IB/hfi1: Return void in packet receiving functions
>>>
>>> Kaike Wan (1):
>>>         IB/hfi1: Don't cancel unused work item
>>>
>>> Michael J. Ruhl (1):
>>>         IB/hfi1: List all receive contexts from debugfs
>>>
>>> Mike Marciniszyn (6):
>>>         IB/hfi1: Add accessor API routines to access context members
>>>         IB/hfi1: Move chip specific functions to chip.c
>>>         IB/hfi1: Add fast and slow handlers for receive context
>>>         IB/hfi1: IB/hfi1: Add an API to handle special case drop
>>>         IB/hfi1: Create API for auto activate
>>>         IB/rdmavt: Correct comments in rdmavt_qp.h header
>>
>> I guess these didn't catch the train for 5.5 merge window. Do I need to
>> resubmit to break this into two series, one for RC and one for next?
> 
> It is more reliable if it shows in patchworks as you want

Ok, you can toss this series then if you haven't yet and I'll respin and 
resubmit.

> 
>> I think 1, 9, 10, and 11 could/should probably go for RC.
> 
> Make sure they have suitable commit messages please

Will do.

-Denny