Re: [PATCH 3/4] block: implement mixed merge of different failfastrequests

From: Tejun Heo
Date: Wed Jul 08 2009 - 20:48:19 EST


Hello,

Boaz Harrosh wrote:
>> @@ -1165,6 +1165,7 @@ static int __make_request(struct request_queue *q, struct bio *bio)
>> const unsigned short prio = bio_prio(bio);
>> const int sync = bio_sync(bio);
>> const int unplug = bio_unplug(bio);
>> + const unsigned int ff = bio->bi_rw & REQ_FAILFAST_MASK;
>
> Perhaps a bio_fail_fast(bio)
> and also an blk_failfast(rq).

Me not being a big fan of those simple accessors, I want to avoid
adding those especially the use of bio ones are mostly confined to
block layer proper.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/