Shaun Tancheff <shaun@xxxxxxxxxxxx> writes: > In a few places a temporary value smaller than a cmd_flags > is used to test for bits and or build up a new cmd_flags. > > Change to use explicit u64 values where appropriate. This is not a bug fix, so please fix your subject. I'm not against cleaning up the mixing of 32 vs 64 bit variables, but if you're going to go down that path, then you might as well fix the mixing of signed vs unsigned use as well. And now that I look at it, bio->bi_rw is unsigned long whereas req->cmd_flags is u64. That could make for fun bugs in the future. We should at least add a comment in the rq_flag_bits enum to the effect of bio flags need to stay in the bottom 32 bits. Cheers, Jeff -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
