mbox series

[for-next,00/13] Updates for 5.3-rc2

Message ID 1564477010-29804-1-git-send-email-oulijun@huawei.com (mailing list archive)
Headers show
Series Updates for 5.3-rc2 | expand

Message

Lijun Ou July 30, 2019, 8:56 a.m. UTC
Here are some updates for hns driver based 5.3-rc2, mainly
include some codes optimization and comments style modification.

Lang Cheng (6):
  RDMA/hns: Clean up unnecessary initial assignment
  RDMA/hns: Update some comments style
  RDMA/hns: Handling the error return value of hem function
  RDMA/hns: Split bool statement and assign statement
  RDMA/hns: Refactor irq request code
  RDMA/hns: Remove unnecessary kzalloc

Lijun Ou (2):
  RDMA/hns: Encapsulate some lines for setting sq size in user mode
  RDMA/hns: Optimize hns_roce_modify_qp function

Weihang Li (2):
  RDMA/hns: Remove redundant print in hns_roce_v2_ceq_int()
  RDMA/hns: Disable alw_lcl_lpbk of SSU

Yangyang Li (1):
  RDMA/hns: Refactor hns_roce_v2_set_hem for hip08

Yixian Liu (2):
  RDMA/hns: Update the prompt message for creating and destroy qp
  RDMA/hns: Remove unnessary init for cmq reg

 drivers/infiniband/hw/hns/hns_roce_device.h |  65 +++++----
 drivers/infiniband/hw/hns/hns_roce_hem.c    |  15 +-
 drivers/infiniband/hw/hns/hns_roce_hem.h    |   6 +-
 drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 210 ++++++++++++++--------------
 drivers/infiniband/hw/hns/hns_roce_hw_v2.h  |   2 -
 drivers/infiniband/hw/hns/hns_roce_mr.c     |   1 -
 drivers/infiniband/hw/hns/hns_roce_qp.c     | 150 +++++++++++++-------
 7 files changed, 244 insertions(+), 205 deletions(-)

Comments

Dennis Dalessandro July 30, 2019, 12:33 p.m. UTC | #1
On 7/30/2019 4:56 AM, Lijun Ou wrote:
> Here are some updates for hns driver based 5.3-rc2, mainly
> include some codes optimization and comments style modification.
> 
> Lang Cheng (6):
>    RDMA/hns: Clean up unnecessary initial assignment
>    RDMA/hns: Update some comments style
>    RDMA/hns: Handling the error return value of hem function
>    RDMA/hns: Split bool statement and assign statement
>    RDMA/hns: Refactor irq request code
>    RDMA/hns: Remove unnecessary kzalloc
> 
> Lijun Ou (2):
>    RDMA/hns: Encapsulate some lines for setting sq size in user mode
>    RDMA/hns: Optimize hns_roce_modify_qp function
> 
> Weihang Li (2):
>    RDMA/hns: Remove redundant print in hns_roce_v2_ceq_int()
>    RDMA/hns: Disable alw_lcl_lpbk of SSU
> 
> Yangyang Li (1):
>    RDMA/hns: Refactor hns_roce_v2_set_hem for hip08
> 
> Yixian Liu (2):
>    RDMA/hns: Update the prompt message for creating and destroy qp
>    RDMA/hns: Remove unnessary init for cmq reg
> 
>   drivers/infiniband/hw/hns/hns_roce_device.h |  65 +++++----
>   drivers/infiniband/hw/hns/hns_roce_hem.c    |  15 +-
>   drivers/infiniband/hw/hns/hns_roce_hem.h    |   6 +-
>   drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 210 ++++++++++++++--------------
>   drivers/infiniband/hw/hns/hns_roce_hw_v2.h  |   2 -
>   drivers/infiniband/hw/hns/hns_roce_mr.c     |   1 -
>   drivers/infiniband/hw/hns/hns_roce_qp.c     | 150 +++++++++++++-------
>   7 files changed, 244 insertions(+), 205 deletions(-)
> 

A lot of this doesn't seem appropriate for an -rc does it?

-Denny
Lijun Ou July 30, 2019, 1:41 p.m. UTC | #2
在 2019/7/30 20:33, Dennis Dalessandro 写道:
> On 7/30/2019 4:56 AM, Lijun Ou wrote:
>> Here are some updates for hns driver based 5.3-rc2, mainly
>> include some codes optimization and comments style modification.
>>
>> Lang Cheng (6):
>>    RDMA/hns: Clean up unnecessary initial assignment
>>    RDMA/hns: Update some comments style
>>    RDMA/hns: Handling the error return value of hem function
>>    RDMA/hns: Split bool statement and assign statement
>>    RDMA/hns: Refactor irq request code
>>    RDMA/hns: Remove unnecessary kzalloc
>>
>> Lijun Ou (2):
>>    RDMA/hns: Encapsulate some lines for setting sq size in user mode
>>    RDMA/hns: Optimize hns_roce_modify_qp function
>>
>> Weihang Li (2):
>>    RDMA/hns: Remove redundant print in hns_roce_v2_ceq_int()
>>    RDMA/hns: Disable alw_lcl_lpbk of SSU
>>
>> Yangyang Li (1):
>>    RDMA/hns: Refactor hns_roce_v2_set_hem for hip08
>>
>> Yixian Liu (2):
>>    RDMA/hns: Update the prompt message for creating and destroy qp
>>    RDMA/hns: Remove unnessary init for cmq reg
>>
>>   drivers/infiniband/hw/hns/hns_roce_device.h |  65 +++++----
>>   drivers/infiniband/hw/hns/hns_roce_hem.c    |  15 +-
>>   drivers/infiniband/hw/hns/hns_roce_hem.h    |   6 +-
>>   drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 210 ++++++++++++++--------------
>>   drivers/infiniband/hw/hns/hns_roce_hw_v2.h  |   2 -
>>   drivers/infiniband/hw/hns/hns_roce_mr.c     |   1 -
>>   drivers/infiniband/hw/hns/hns_roce_qp.c     | 150 +++++++++++++-------
>>   7 files changed, 244 insertions(+), 205 deletions(-)
>>
>
> A lot of this doesn't seem appropriate for an -rc does it?
>
It mainly optimize some codes and style. if it is some bugs, we should consider -rc?
> -Denny
>
> .
>