On 06/05/2016 09:32 PM, [email protected] wrote: > From: Mike Christie <[email protected]> > > The bio users should now always be setting up the bio op. This patch > has the block layer copy that to the request. > > Signed-off-by: Mike Christie <[email protected]> > --- > block/blk-core.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/block/blk-core.c b/block/blk-core.c > index 7e943dc..3c45254 100644 > --- a/block/blk-core.c > +++ b/block/blk-core.c > @@ -2976,8 +2976,7 @@ EXPORT_SYMBOL_GPL(__blk_end_request_err); > void blk_rq_bio_prep(struct request_queue *q, struct request *rq, > struct bio *bio) > { > - /* Bit 0 (R/W) is identical in rq->cmd_flags and bio->bi_rw */ > - rq->cmd_flags |= bio->bi_rw & REQ_WRITE; > + req_set_op(rq, bio_op(bio)); > > if (bio_has_data(bio)) > rq->nr_phys_segments = bio_phys_segments(q, bio); > @@ -3062,7 +3061,8 @@ EXPORT_SYMBOL_GPL(blk_rq_unprep_clone); > static void __blk_rq_prep_clone(struct request *dst, struct request *src) > { > dst->cpu = src->cpu; > - dst->cmd_flags |= (src->cmd_flags & REQ_CLONE_MASK) | REQ_NOMERGE; > + req_set_op_attrs(dst, req_op(src), > + (src->cmd_flags & REQ_CLONE_MASK) | REQ_NOMERGE); > dst->cmd_type = src->cmd_type; > dst->__sector = blk_rq_pos(src); > dst->__data_len = blk_rq_bytes(src); > Reviewed-by: Hannes Reinecke <[email protected]>
Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking [email protected] +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- dm-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/dm-devel
