Re: [GIT PATCH] block: cleanup patches, take#2

From: Bartlomiej Zolnierkiewicz
Date: Mon Mar 16 2009 - 14:08:22 EST


On Monday 16 March 2009, Tejun Heo wrote:
> Hello,
>
> This patchset is available in the following git tree.
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git block-cleanup
>
> This patchset contains the following 17 cleanup patches.
>
> 0001-ide-use-blk_run_queue-instead-of-blk_start_queuei.patch
> 0002-ide-don-t-set-REQ_SOFTBARRIER.patch
> 0003-ide-use-blk_update_request-instead-of-blk_end_req.patch
> 0004-block-merge-blk_invoke_request_fn-into-__blk_run_.patch
> 0005-block-kill-blk_start_queueing.patch
> 0006-block-don-t-set-REQ_NOMERGE-unnecessarily.patch
> 0007-block-cleanup-REQ_SOFTBARRIER-usages.patch
> 0008-block-clean-up-misc-stuff-after-block-layer-timeout.patch
> 0009-block-reorder-request-completion-functions.patch
> 0010-block-reorganize-request-fetching-functions.patch
> 0011-block-kill-blk_end_request_callback.patch
> 0012-block-clean-up-request-completion-API.patch
> 0013-block-move-rq-start_time-initialization-to-blk_rq_.patch
> 0014-block-implement-and-use-__-blk_end_request_all.patch
> 0015-block-kill-end_request.patch
> 0016-ubd-simplify-block-request-completion.patch
> 0017-block-clean-up-unnecessary-stuff-from-block-drivers.patch
>
> It's on top of the current linux-2.6-block/for-2.6.30[1]. Changes
> from the last take[2] are.
>
> * IDE changes separated out to 0001-0003
> * IDE end_all conversion dropped
>
> Bartlomiej, 0001-0003 are mostly trivial and shouldn't cause too much
> merge headaches later. Can these go through block tree? I'll base

Patches look fine but 0002-0003 will cause pata/block merge conflicts
for linux-next once they go into block tree so no ACK from me for this
approach.

$ patch -p1 --dry-run < 0002.patch
patching file drivers/ide/ide-disk.c
Hunk #1 FAILED at 405.
1 out of 1 hunk FAILED -- saving rejects to file drivers/ide/ide-disk.c.rej
patching file drivers/ide/ide-ioctls.c

$ patch -p1 --dry-run < 0003.patch
patching file drivers/ide/ide-cd.c
Reversed (or previously applied) patch detected! Assume -R? [n]

Thanks,
Bart
--
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/