diff options
| author |  Heinz Mauelshagen <heinzm@redhat.com>
| 2023-02-07 22:07:22 +0100 |
|---|---|---|
| committer |  Mike Snitzer <snitzer@kernel.org>
| 2023-02-14 14:23:07 -0500 |
| commit | 6cc435fa76cc1786ef460c08c31b1d27c13e9cff (patch) | |
| tree | e6eef4e16e342101dc1d6437abf889f4dd25a1bc /drivers | |
| parent | dm log: avoid multiple line dereference (diff) | |
| download | linux-6cc435fa76cc1786ef460c08c31b1d27c13e9cff.tar.gz linux-6cc435fa76cc1786ef460c08c31b1d27c13e9cff.tar.bz2 linux-6cc435fa76cc1786ef460c08c31b1d27c13e9cff.zip | |
dm: avoid 'do {} while(0)' loop in single statement macros
Signed-off-by: Heinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/md/dm-integrity.c | 4 | ||||
| -rw-r--r-- | drivers/md/dm-mpath.c | 4 |
2 files changed, 3 insertions, 5 deletions
diff --git a/drivers/md/dm-integrity.c b/drivers/md/dm-integrity.c index 8e3d31a7fe9f..868b158846dd 100644 --- a/drivers/md/dm-integrity.c +++ b/drivers/md/dm-integrity.c @@ -113,9 +113,9 @@ struct journal_entry { #endif #define journal_entry_get_sector(je) le64_to_cpu((je)->u.sector) #define journal_entry_is_unused(je) ((je)->u.s.sector_hi == cpu_to_le32(-1)) -#define journal_entry_set_unused(je) do { ((je)->u.s.sector_hi = cpu_to_le32(-1)); } while (0) +#define journal_entry_set_unused(je) ((je)->u.s.sector_hi = cpu_to_le32(-1)) #define journal_entry_is_inprogress(je) ((je)->u.s.sector_hi == cpu_to_le32(-2)) -#define journal_entry_set_inprogress(je) do { ((je)->u.s.sector_hi = cpu_to_le32(-2)); } while (0) +#define journal_entry_set_inprogress(je) ((je)->u.s.sector_hi = cpu_to_le32(-2)) #define JOURNAL_BLOCK_SECTORS 8 #define JOURNAL_SECTOR_DATA ((1 << SECTOR_SHIFT) - sizeof(commit_id_t)) diff --git a/drivers/md/dm-mpath.c b/drivers/md/dm-mpath.c index 0ea168b63db2..371a38bc0c61 100644 --- a/drivers/md/dm-mpath.c +++ b/drivers/md/dm-mpath.c @@ -472,13 +472,11 @@ failed: * it has been invoked. */ #define dm_report_EIO(m) \ -do { \ DMDEBUG_LIMIT("%s: returning EIO; QIFNP = %d; SQIFNP = %d; DNFS = %d", \ dm_table_device_name((m)->ti->table), \ test_bit(MPATHF_QUEUE_IF_NO_PATH, &(m)->flags), \ test_bit(MPATHF_SAVED_QUEUE_IF_NO_PATH, &(m)->flags), \ - dm_noflush_suspending((m)->ti)); \ -} while (0) + dm_noflush_suspending((m)->ti)) /* * Check whether bios must be queued in the device-mapper core rather |