Skip to content

IB: cleanup debugfs usage

Greg Kroah-Hartman linux-rdma lkml 1 version Applied

Status from this message: Applied to rdma for-next

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest8 patches 5 repliesApplied

Patches in v1 Applied

#Subject and filesSizeRepliesApplied
1/8infiniband: cxgb4: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/cxgb4/device.c+1 -72 replies
2/8infiniband: hfi1: drop crazy DEBUGFS_SEQ_FILE_CREATE() macrodrivers/infiniband/hw/hfi1/debugfs.c drivers/infiniband/hw/hfi1/debugfs.h drivers/infiniband/hw/hfi1/fault.c+24 -410 replies
3/8infiniband: hfi1: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/hfi1/debugfs.c drivers/infiniband/hw/hfi1/fault.c+18 -340 replies
4/8infiniband: qib: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/qib/qib_debugfs.c+7 -190 replies
5/8infiniband: mlx5: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/mlx5/cong.c drivers/infiniband/hw/mlx5/main.c drivers/infiniband/hw/mlx5/mlx5_ib.h +1 more+18 -641 reply
6/8infiniband: ocrdma: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/ocrdma/ocrdma_stats.c+22 -450 replies
7/8infiniband: usnic: no need to check return value of debugfs_create functionsdrivers/infiniband/hw/usnic/usnic_debugfs.c+0 -261 reply
8/8infiniband: ipoib: no need to check return value of debugfs_create functionsdrivers/infiniband/ulp/ipoib/ipoib.h drivers/infiniband/ulp/ipoib/ipoib_fs.c drivers/infiniband/ulp/ipoib/ipoib_main.c+4 -110 replies

Reviews trailers per patch

3 of 8 patches have a Reviewed-by or Acked-by.

PatchSteve WiseLeon RomanovskyParvi Kaustubhi
1/8A-b
2/8
3/8
4/8
5/8A-b
6/8
7/8A-b
8/8

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 0/8] IB: cleanup debugfs usage

When calling debugfs code, there is no need to ever check the return value of the call, as no logic should ever change if a call works properly or not. Fix up a bunch of infiniband-specific code to not care about the results of debugfs.

Greg Kroah-Hartman (8):
infiniband: cxgb4: no need to check return value of debugfs_create functions
infiniband: hfi1: drop crazy DEBUGFS_SEQ_FILE_CREATE() macro
infiniband: hfi1: no need to check return value of debugfs_create functions
infiniband: qib: no need to check return value of debugfs_create functions
infiniband: mlx5: no need to check return value of debugfs_create functions
infiniband: ocrdma: no need to check return value of debugfs_create functions
infiniband: usnic: no need to check return value of debugfs_create functions
infiniband: ipoib: no need to check return value of debugfs_create functions

 drivers/infiniband/hw/cxgb4/device.c        |  8 +--
 drivers/infiniband/hw/hfi1/debugfs.c        | 52 +++++++---------
 drivers/infiniband/hw/hfi1/debugfs.h        | 12 ----
 drivers/infiniband/hw/hfi1/fault.c          | 53 ++++++----------
 drivers/infiniband/hw/mlx5/cong.c           | 15 ++---
 drivers/infiniband/hw/mlx5/main.c           |  8 +--
 drivers/infiniband/hw/mlx5/mlx5_ib.h        |  8 +--
 drivers/infiniband/hw/mlx5/mr.c             | 51 +++-------------
 drivers/infiniband/hw/ocrdma/ocrdma_stats.c | 67 +++++++--------------
 drivers/infiniband/hw/qib/qib_debugfs.c     | 26 +++-----
 drivers/infiniband/hw/usnic/usnic_debugfs.c | 26 --------
 drivers/infiniband/ulp/ipoib/ipoib.h        |  4 +-
 drivers/infiniband/ulp/ipoib/ipoib_fs.c     |  7 +--
 drivers/infiniband/ulp/ipoib/ipoib_main.c   |  4 +-
 14 files changed, 94 insertions(+), 247 deletions(-)
Signature · 1 line
2.20.1