[lustre-devel] [PATCH 0/5] staging: lustre: ldlm: remove ldlm typedef usage from code

James Simmons jsimmons at infradead.org
Wed Nov 23 14:59:46 PST 2016


Replace usage of ldlm_policy_data_t, ldlm_wire_policy_t, ldlm_side_t
with named enums to conform to upstream coding style.

Fix users of flags that were using "int" instead of named enum.
Rename some "flags" variables to distinguish between different flags.

Rename LDLM_CANCEL_* flags (used with enum ldlm_lru_flags) to
LDLM_LRU_FLAGS_* to avoid confusion with enum ldlm_cancel_flags.

Signed-off-by: Andreas Dilger <andreas.dilger at intel.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6142
Reviewed-on: http://review.whamcloud.com/15300
Reviewed-on: http://review.whamcloud.com/15301
Reviewed-by: James Simmons <uja.ornl at yahoo.com>
Reviewed-by: Bob Glossman <bob.glossman at intel.com>
Reviewed-by: frank zago <fzago at cray.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin at intel.com>
Reviewed-by: John L. Hammond <john.hammond at intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin at intel.com>
Signed-off-by: James Simmons <jsimmons at infradead.org>

Andreas Dilger (5):
  staging: lustre: ldlm: remove ldlm_policy_data_t typedef usage from code
  staging: lustre: ldlm: remove ldlm_side_t typedef usage from code
  staging: lustre: ldlm: remove ldlm_wire_policy_data_t typedef usage from code
  staging: lustre: ldlm: rename LDLM_CANCEL_* flags
  staging: lustre: ldlm: change "int" to proper enum type

 .../lustre/lustre/include/lustre/lustre_idl.h      |    6 +-
 drivers/staging/lustre/lustre/include/lustre_dlm.h |   44 +++++-----
 drivers/staging/lustre/lustre/include/obd.h        |    6 +-
 drivers/staging/lustre/lustre/include/obd_class.h  |    6 +-
 drivers/staging/lustre/lustre/ldlm/ldlm_extent.c   |    8 +-
 drivers/staging/lustre/lustre/ldlm/ldlm_flock.c    |    8 +-
 .../staging/lustre/lustre/ldlm/ldlm_inodebits.c    |    8 +-
 drivers/staging/lustre/lustre/ldlm/ldlm_internal.h |   85 ++++++++++----------
 drivers/staging/lustre/lustre/ldlm/ldlm_lock.c     |   33 ++++----
 drivers/staging/lustre/lustre/ldlm/ldlm_plain.c    |    8 +-
 drivers/staging/lustre/lustre/ldlm/ldlm_pool.c     |   13 ++--
 drivers/staging/lustre/lustre/ldlm/ldlm_request.c  |   50 ++++++------
 drivers/staging/lustre/lustre/ldlm/ldlm_resource.c |   22 +++---
 drivers/staging/lustre/lustre/llite/file.c         |   10 ++-
 .../staging/lustre/lustre/llite/llite_internal.h   |    2 +-
 drivers/staging/lustre/lustre/llite/llite_mmap.c   |    2 +-
 drivers/staging/lustre/lustre/llite/vvp_io.c       |    2 +-
 drivers/staging/lustre/lustre/lmv/lmv_obd.c        |   11 ++-
 drivers/staging/lustre/lustre/mdc/mdc_internal.h   |    7 +-
 drivers/staging/lustre/lustre/mdc/mdc_locks.c      |   26 +++---
 drivers/staging/lustre/lustre/mdc/mdc_reint.c      |    2 +-
 drivers/staging/lustre/lustre/mgc/mgc_request.c    |    2 +-
 .../staging/lustre/lustre/osc/osc_cl_internal.h    |    5 +-
 drivers/staging/lustre/lustre/osc/osc_internal.h   |    4 +-
 drivers/staging/lustre/lustre/osc/osc_lock.c       |    6 +-
 drivers/staging/lustre/lustre/osc/osc_object.c     |    2 +-
 drivers/staging/lustre/lustre/osc/osc_page.c       |    3 +-
 drivers/staging/lustre/lustre/osc/osc_request.c    |    4 +-
 .../staging/lustre/lustre/ptlrpc/pack_generic.c    |    2 +-
 29 files changed, 200 insertions(+), 187 deletions(-)



More information about the lustre-devel mailing list