block: clear all of bi_opf in bio_set_op_attrs
authorChristoph Hellwig <hch@lst.de>
Mon, 21 Nov 2016 15:18:18 +0000 (16:18 +0100)
committerJens Axboe <axboe@fb.com>
Mon, 21 Nov 2016 16:35:05 +0000 (09:35 -0700)
Since commit 87374179 ("block: add a proper block layer data direction
encoding") we only or the new op and flags into bi_opf in bio_set_op_attrs
instead of clearing the old value.  I've not seen any breakage with the
new behavior, but it seems dangerous.

Also convert it to an inline function to make the argument passing
safer.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@fb.com>
include/linux/blk_types.h

index 4d0044d0998410599649f16bb84a692c564cbd0b..f57458a6a93bc3cebf146fda6d339b0b38bfdfe4 100644 (file)
@@ -207,8 +207,11 @@ enum req_flag_bits {
        ((req)->cmd_flags & REQ_OP_MASK)
 
 /* obsolete, don't use in new code */
-#define bio_set_op_attrs(bio, op, op_flags) \
-       ((bio)->bi_opf |= (op | op_flags))
+static inline void bio_set_op_attrs(struct bio *bio, unsigned op,
+               unsigned op_flags)
+{
+       bio->bi_opf = op | op_flags;
+}
 
 static inline bool op_is_write(unsigned int op)
 {