diff options
| author | Fabian Huegel <fabian_huegel@web.de> | 2018-01-11 18:16:56 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-01-15 16:01:57 +0100 |
| commit | 6b1833a3de5ef91714fe56c234a476f0596762d9 (patch) | |
| tree | 7408386f4b31fffab423799fc361865ee8f8431e | |
| parent | 454431c61e559723657c8880b6896c0303f787af (diff) | |
| download | linux-stable-6b1833a3de5ef91714fe56c234a476f0596762d9.tar.gz linux-stable-6b1833a3de5ef91714fe56c234a476f0596762d9.tar.bz2 linux-stable-6b1833a3de5ef91714fe56c234a476f0596762d9.zip | |
staging: lustre: Enclose complex macros in do-while loops
Some complex multi-line macros were not enclosed by a do-while(0),
so we fixed that.
Signed-off-by: Fabian Huegel <fabian_huegel@web.de>
Signed-off-by: Christoph Volkert <linux@christoph-volkert.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/lustre/lustre/include/obd_class.h | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/drivers/staging/lustre/lustre/include/obd_class.h b/drivers/staging/lustre/lustre/include/obd_class.h index 25db7ec6ecd0..d2a3680dcb2c 100644 --- a/drivers/staging/lustre/lustre/include/obd_class.h +++ b/drivers/staging/lustre/lustre/include/obd_class.h @@ -332,22 +332,26 @@ static inline int obd_check_dev_active(struct obd_device *obd) / sizeof(((struct obd_ops *)(0))->iocontrol)) #define OBD_COUNTER_INCREMENT(obdx, op) \ +do { \ if ((obdx)->obd_stats) { \ unsigned int coffset; \ coffset = (unsigned int)((obdx)->obd_cntr_base) + \ OBD_COUNTER_OFFSET(op); \ LASSERT(coffset < (obdx)->obd_stats->ls_num); \ lprocfs_counter_incr((obdx)->obd_stats, coffset); \ - } + } \ +} while (0) #define EXP_COUNTER_INCREMENT(export, op) \ +do { \ if ((export)->exp_obd->obd_stats) { \ unsigned int coffset; \ coffset = (unsigned int)((export)->exp_obd->obd_cntr_base) + \ OBD_COUNTER_OFFSET(op); \ LASSERT(coffset < (export)->exp_obd->obd_stats->ls_num); \ lprocfs_counter_incr((export)->exp_obd->obd_stats, coffset); \ - } + } \ +} while (0) #define MD_COUNTER_OFFSET(op) \ ((offsetof(struct md_ops, op) - \ @@ -355,15 +359,18 @@ static inline int obd_check_dev_active(struct obd_device *obd) / sizeof(((struct md_ops *)(0))->getstatus)) #define MD_COUNTER_INCREMENT(obdx, op) \ +do { \ if ((obd)->md_stats) { \ unsigned int coffset; \ coffset = (unsigned int)((obdx)->md_cntr_base) + \ MD_COUNTER_OFFSET(op); \ LASSERT(coffset < (obdx)->md_stats->ls_num); \ lprocfs_counter_incr((obdx)->md_stats, coffset); \ - } + } \ +} while (0) #define EXP_MD_COUNTER_INCREMENT(export, op) \ +do { \ if ((export)->exp_obd->obd_stats) { \ unsigned int coffset; \ coffset = (unsigned int)((export)->exp_obd->md_cntr_base) + \ @@ -373,7 +380,8 @@ static inline int obd_check_dev_active(struct obd_device *obd) if ((export)->exp_md_stats) \ lprocfs_counter_incr( \ (export)->exp_md_stats, coffset); \ - } + } \ +} while (0) #define EXP_CHECK_MD_OP(exp, op) \ do { \ |