From: Mike Snitzer <[email protected]>

Only one user of WRITE_FLAGS so no need to factor it out in an enum
(which causes sparse's 'mixed bitwiseness' warning).  Just use the
flags in the only consumer.

Signed-off-by: Mike Snitzer <[email protected]>
Signed-off-by: Susan LeGendre-McGhee <[email protected]>
Signed-off-by: Matthew Sakai <[email protected]>
---
 drivers/md/dm-vdo/recovery-journal.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/md/dm-vdo/recovery-journal.c 
b/drivers/md/dm-vdo/recovery-journal.c
index a6981e5dd017..1e15bfe42cfc 100644
--- a/drivers/md/dm-vdo/recovery-journal.c
+++ b/drivers/md/dm-vdo/recovery-journal.c
@@ -34,7 +34,6 @@ enum {
         */
        RECOVERY_JOURNAL_RESERVED_BLOCKS =
                (MAXIMUM_VDO_USER_VIOS / RECOVERY_JOURNAL_ENTRIES_PER_BLOCK) + 
2,
-       WRITE_FLAGS = REQ_OP_WRITE | REQ_PRIO | REQ_PREFLUSH | REQ_SYNC | 
REQ_FUA,
 };
 
 /**
@@ -1398,7 +1397,8 @@ static void write_block(struct vdo_waiter *waiter, void 
*context __always_unused
         * block itself is stable before allowing overwrites of the lbn's 
previous data.
         */
        vdo_submit_metadata_vio(&block->vio, journal->origin + 
block->block_number,
-                               complete_write_endio, handle_write_error, 
WRITE_FLAGS);
+                               complete_write_endio, handle_write_error,
+                               REQ_OP_WRITE | REQ_PRIO | REQ_PREFLUSH | 
REQ_SYNC | REQ_FUA);
 }
 
 
-- 
2.42.0


Reply via email to