On 1/27/21 11:27 PM, Damien Le Moal wrote:

+
+       bio_set_dev(bio, bdev);
+       bio->bi_iter.bi_sector = sector;
+       bio_set_op_attrs(bio, op, op_flags);


This function is obsolete. Open code this.


And that also mean that you could remove one argument to bio_new(): combine op
and op_flags into "unsigned int opf"



I did that initially but kept it separate for RFC, that is much easier than having

an extra arg, will change it in V1.