mbox series

[for-next,v2,00/16] Use ibdev_dbg instead of pr_xxx

Message ID 20221103171013.20659-1-rpearsonhpe@gmail.com (mailing list archive)
Headers show
Series Use ibdev_dbg instead of pr_xxx | expand

Message

Bob Pearson Nov. 3, 2022, 5:09 p.m. UTC 
This patch series replaces calls to pr_xxx with calls to ibdev_dbg
using macros for rxe objects adapted from similar macros from siw,
except in situations where the rdma device is not in scope.

This patch series is based on the current for-next branch.

v2:
  Fixed a typo in 01/16 in rxe_dbg_uc().
  Made fixes to 01/16 from a comment by Jason Gunthorp to enclose
  macro parameters in parentheses.
  Fixed an error in 06/16 caught by the kernel test robot <lkp@intel.com> 

Bob Pearson (16):
  RDMA/rxe: Add ibdev_dbg macros for rxe
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_comp.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_cq.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mr.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mw.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_net.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_qp.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_req.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_resp.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_srq.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_verbs.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_av.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_task.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_icrc.c
  RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mmap.c

 drivers/infiniband/sw/rxe/rxe.c       |  4 +-
 drivers/infiniband/sw/rxe/rxe.h       | 19 ++++++++
 drivers/infiniband/sw/rxe/rxe_av.c    | 43 ++++++++++++++---
 drivers/infiniband/sw/rxe/rxe_comp.c  | 12 ++---
 drivers/infiniband/sw/rxe/rxe_cq.c    |  8 ++--
 drivers/infiniband/sw/rxe/rxe_icrc.c  |  4 +-
 drivers/infiniband/sw/rxe/rxe_loc.h   |  8 +---
 drivers/infiniband/sw/rxe/rxe_mmap.c  |  6 +--
 drivers/infiniband/sw/rxe/rxe_mr.c    | 40 +++++++---------
 drivers/infiniband/sw/rxe/rxe_mw.c    | 20 ++++----
 drivers/infiniband/sw/rxe/rxe_net.c   | 38 ++++++++-------
 drivers/infiniband/sw/rxe/rxe_qp.c    | 69 +++++++++++++--------------
 drivers/infiniband/sw/rxe/rxe_req.c   | 10 ++--
 drivers/infiniband/sw/rxe/rxe_resp.c  | 17 +++----
 drivers/infiniband/sw/rxe/rxe_srq.c   | 20 ++++----
 drivers/infiniband/sw/rxe/rxe_task.c  |  7 +--
 drivers/infiniband/sw/rxe/rxe_verbs.c | 18 ++++---
 17 files changed, 190 insertions(+), 153 deletions(-)


base-commit: 692373d186205dfb1b56f35f22702412d94d9420

Comments

Jason Gunthorpe Nov. 10, 2022, 7:36 p.m. UTC | #1 
On Thu, Nov 03, 2022 at 12:09:58PM -0500, Bob Pearson wrote:
> This patch series replaces calls to pr_xxx with calls to ibdev_dbg
> using macros for rxe objects adapted from similar macros from siw,
> except in situations where the rdma device is not in scope.
> 
> This patch series is based on the current for-next branch.
> 
> v2:
>   Fixed a typo in 01/16 in rxe_dbg_uc().
>   Made fixes to 01/16 from a comment by Jason Gunthorp to enclose
>   macro parameters in parentheses.
>   Fixed an error in 06/16 caught by the kernel test robot <lkp@intel.com> 
> 
> Bob Pearson (16):
>   RDMA/rxe: Add ibdev_dbg macros for rxe
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_comp.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_cq.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mr.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mw.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_net.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_qp.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_req.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_resp.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_srq.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_verbs.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_av.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_task.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_icrc.c
>   RDMA/rxe: Replace pr_xxx by rxe_dbg_xxx in rxe_mmap.c

Applied to for-next, thanks

Jason