On Mon, 31 Oct 2005, Jens Axboe wrote:
>
> So either we disable merging for noop by setting REQ_NOMERGE in
> elevator_noop_add_request(), or we add a noop_list and do the
> dispatching like in the other io schedulers. I'd prefer the latter,
> merging is still beneficial for noop (and it has always done it).
>
> For now, we should add the former.
>
> Signed-off-by: Jens Axboe <[email protected]>
Btw, Jens, I appreciate seeing the discussion history when applying a
patch, but at the same time I do _not_ want to use it as a commit message,
it's just too confusing and worthless in that context.
And yet, your final comments don't much make sense without the background,
so I can't just use them either.
So, I rewrote the explanation. Which is fine, but I wish people who sent
patches would think more about what message they want to have in the
commit logs, so that (a) Lazy-Linus doesn't have to write his own message
and (b) so that the message is correct when Lazy-and-Stupid-Linus
sometimes doesn't necessarily see/understand all the problems it fixes.
Btw, the email-patch-sending protocol still allows for putting all the
ugly history in for my (and the mailing lists) pleasure: that's what the
"---" marker after the explanation is for. So you can _both_ have a nice
clean commit message _and_ give more of a historical background for the
patch.
Anyway, in this case, the commit message ended up looking like this::
commit 581c1b14394aee60aff46ea67d05483261ed6527
Author: Jens Axboe <[email protected]>
Date: Mon Oct 31 09:23:54 2005 +0100
[PATCH] noop-iosched: avoid corrupted request merging
Tejun Heo notes:
"I'm currently debugging this. The problem is that we are using the
generic dispatch queue directly in the noop sched and merging is NOT
allowed on dispatch queues but generic handling of last_merge tries
to merge requests. I'm still trying to verify this, so I'll be back
with results soon."
In the meantime, disable merging for noop by setting REQ_NOMERGE in
elevator_noop_add_request().
Eventually, we should add a noop_list and do the dispatching like in the
other io schedulers. Merging is still beneficial for noop (and it has
always done it).
Signed-off-by: Jens Axboe <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
which is basically your email cleaned up and compressed into a readable
commit message.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
