mbox series

[PATCHv2,for-next,00/12] rtrs: misc fix and cleanup

Message ID 20201023074353.21946-1-jinpu.wang@cloud.ionos.com (mailing list archive)
Headers show
Series rtrs: misc fix and cleanup | expand

Message

Jinpu Wang Oct. 23, 2020, 7:43 a.m. UTC 
Please consider to include following changes to upstream.

I re-ordered the patch sequence so bugfix comes first, then cleanup.
First 5 patches are bugfixes, we also added the Fixes tag accordingly.

Thanks!

Changelog:
v2 -> v1:
Droped "RDMA/rtrs: removed unused filed list of rtrs_iu" merged alreay.
Droped "RDMA/rtrs-clt: remove unnecessary dev_ref of rtrs_sess", buggy.
New "RDMA/rtrs-srv: don't guard the whole __alloc_srv with srv_mutex"

v1:
https://lore.kernel.org/linux-rdma/20201012131814.121096-1-jinpu.wang@cloud.ionos.com/

Danil Kipnis (1):
  RDMA/rtrs-clt: remove destroy_con_cq_qp in case route resolving failed

Gioh Kim (4):
  RDMA/rtrs-clt: missing error from rtrs_rdma_conn_established
  RDMA/rtrs: remove unnecessary argument dir of rtrs_iu_free
  RDMA/rtrs-clt: remove duplicated switch-case handling for CM error
    events
  RDMA/rtrs-clt: remove duplicated code

Guoqing Jiang (5):
  RDMA/rtrs-srv: don't guard the whole __alloc_srv with srv_mutex
  RDMA/rtrs-srv: fix typo
  RDMA/rtrs-srv: kill rtrs_srv_change_state_get_old
  RDMA/rtrs: introduce rtrs_post_send
  RDMA/rtrs-clt: remove 'addr' from rtrs_clt_add_path_to_arr

Jack Wang (2):
  RDMA/rtrs-clt: remove outdated comment in create_con_cq_qp
  RDMA/rtrs-clt: avoid run destroy_con_cq_qp/create_con_cq_qp in
    parallel

 drivers/infiniband/ulp/rtrs/rtrs-clt.c |  74 ++++++++-------
 drivers/infiniband/ulp/rtrs/rtrs-clt.h |   1 +
 drivers/infiniband/ulp/rtrs/rtrs-pri.h |   3 +-
 drivers/infiniband/ulp/rtrs/rtrs-srv.c | 119 ++++++++++---------------
 drivers/infiniband/ulp/rtrs/rtrs-srv.h |   2 +-
 drivers/infiniband/ulp/rtrs/rtrs.c     |  61 +++++--------
 6 files changed, 108 insertions(+), 152 deletions(-)

Comments

Jason Gunthorpe Oct. 28, 2020, 4:28 p.m. UTC | #1 
On Fri, Oct 23, 2020 at 09:43:41AM +0200, Jack Wang wrote:
> Please consider to include following changes to upstream.
> 
> I re-ordered the patch sequence so bugfix comes first, then cleanup.
> First 5 patches are bugfixes, we also added the Fixes tag accordingly.
> 
> Thanks!
> 
> Changelog:
> v2 -> v1:
> Droped "RDMA/rtrs: removed unused filed list of rtrs_iu" merged alreay.
> Droped "RDMA/rtrs-clt: remove unnecessary dev_ref of rtrs_sess", buggy.
> New "RDMA/rtrs-srv: don't guard the whole __alloc_srv with srv_mutex"
> 
> v1:
> https://lore.kernel.org/linux-rdma/20201012131814.121096-1-jinpu.wang@cloud.ionos.com/
> 
> Danil Kipnis (1):
>   RDMA/rtrs-clt: remove destroy_con_cq_qp in case route resolving failed
> 
> Gioh Kim (4):
>   RDMA/rtrs-clt: missing error from rtrs_rdma_conn_established
>   RDMA/rtrs: remove unnecessary argument dir of rtrs_iu_free
>   RDMA/rtrs-clt: remove duplicated switch-case handling for CM error
>     events
>   RDMA/rtrs-clt: remove duplicated code
> 
> Guoqing Jiang (5):
>   RDMA/rtrs-srv: don't guard the whole __alloc_srv with srv_mutex
>   RDMA/rtrs-srv: fix typo
>   RDMA/rtrs-srv: kill rtrs_srv_change_state_get_old
>   RDMA/rtrs: introduce rtrs_post_send
>   RDMA/rtrs-clt: remove 'addr' from rtrs_clt_add_path_to_arr
> 
> Jack Wang (2):
>   RDMA/rtrs-clt: remove outdated comment in create_con_cq_qp
>   RDMA/rtrs-clt: avoid run destroy_con_cq_qp/create_con_cq_qp in
>     parallel

Applied to for next

Please note subjects in RDMA should lead with a capital letter:

   RDMA/rtrs-clt: Remove outdated comment in create_con_cq_qp

Thanks,
Jason