On Tue, Jan 02 2007, Jens Axboe wrote:On Tue, Jan 02 2007, Rene Herman wrote:Jens Axboe wrote:As expected. The issue is rq_is_sync(rq) takes the data direction into
On Mon, Jan 01 2007, Andrew Morton wrote:(to possibly save others from trying -- no, doesn't fix any)The patch would appear to need this fix:Yeah it is, but I don't think it'll fix it (if anything, it'll be more
--- a/block/cfq-iosched.c~a
+++ a/block/cfq-iosched.c
@@ -592,7 +592,7 @@ static int cfq_allow_merge(request_queue
if (cfqq == RQ_CFQQ(rq))
return 1;
- return 1;
+ return 0;
}
static inline void
_
But that might not fix things...
conservative).
account as well, while bio_sync() only checks the sync bit. This should
fix it.
And here a little more relaxed version, as Mark Lord suggested. We allow
merge of async bio into a sync request, but not vice versa.
Both patches pending testing, will do so now.