Re: [PATCH] block: factor out bio_check_eod()
From: Tejun Heo <hidden>
Date: 2007-07-18 11:41:21
Also in:
dm-devel, linux-fsdevel, lkml
Jens Axboe wrote:
On Wed, Jul 18 2007, Tejun Heo wrote:quoted
Jens Axboe wrote:quoted
On Wed, Jul 18 2007, Tejun Heo wrote:quoted
Jens Axboe wrote:quoted
On Wed, Jul 18 2007, Tejun Heo wrote:quoted
End of device check is done twice in __generic_make_request() and it's fully inlined each time. Factor out bio_check_eod().Tejun, yeah I should seperate the cleanups and put them in the upstream branch. Will do so and add your signed-off to both of them.Would they be different from the one I just posted? No big deal either way. I'm just basing the zero-length barrier on top of these patches. Oh well, the changes are trivial anyway.This one ended up being the same, but in the first one you missed some of the cleanups. I ended up splitting the patch some more though, see the series: http://git.kernel.dk/?p=linux-2.6-block.git;a=shortlog;h=barrierAlright, will base on 662d5c5e6afb79d05db5563205b809c0de530286. Thanks.1781c6a39fb6e31836557618c4505f5f7bc61605, no? Unless you want to rewrite it completely :-)
I think I'll start from 662d5c5e and steal most parts from 1781c6a3. I like stealing, you know. :-) I think 1781c6a3 also can use splitting - zero length barrier implementation and issue_flush conversion. Anyways, how do I pull from git.kernel.dk? git://git.kernel.dk/linux-2.6-block.git gives me connection reset by server. Thanks. -- tejun