Re: [PATCH 3/7] blk_end_request: changing "normal" drivers

From: Kiyoshi Ueda
Date: Wed Sep 05 2007 - 09:55:54 EST


Hi Benny,

Thank you for the comments.

On Wed, 05 Sep 2007 10:45:54 +0300, Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote:
> On Sat, Sep 01 2007 at 1:42 +0300, Kiyoshi Ueda <k-ueda@xxxxxxxxxxxxx> wrote:
> > arch/arm/plat-omap/mailbox.c | 9 ++++++---
> > arch/um/drivers/ubd_kern.c | 10 +---------
> > block/elevator.c | 4 ++--
> > block/ll_rw_blk.c | 15 +++++----------
> > drivers/block/DAC960.c | 6 ++----
> > drivers/block/floppy.c | 8 +++-----
> > drivers/block/lguest_blk.c | 5 +----
> > drivers/block/nbd.c | 4 +---
> > drivers/block/ps3disk.c | 6 +-----
> > drivers/block/sunvdc.c | 5 +----
> > drivers/block/sx8.c | 4 +---
> > drivers/block/ub.c | 4 ++--
> > drivers/block/viodasd.c | 5 +----
> > drivers/block/xen-blkfront.c | 5 ++---
> > drivers/cdrom/viocd.c | 5 +----
> > drivers/ide/ide-cd.c | 6 +++---
> > drivers/ide/ide-io.c | 22 +++++++---------------
> > drivers/message/i2o/i2o_block.c | 8 ++------
> > drivers/mmc/card/block.c | 24 +++++-------------------
> > drivers/mmc/card/queue.c | 4 ++--
> > drivers/s390/block/dasd.c | 4 +---
> > drivers/s390/char/tape_block.c | 3 +--
> > drivers/scsi/ide-scsi.c | 8 ++++----
> > drivers/scsi/scsi_lib.c | 13 ++-----------
> > 24 files changed, 57 insertions(+), 130 deletions(-)
>
> Please Separate this patch to at least three patches.
> 1. Block layer - block/elevator.c block/ll_rw_blk.c
> 2. SCSI-ML - drivers/scsi/scsi_lib.c
> 3. Normal drivers, can also be separated into logical
> groups like scsi/block etc..
>
> This is because these patches introduce conflicts
> to lots of queued work I have, and if you separate
> them it will help me with my giting. Also I think
> that this is logical. Block-layer and scsi_lib are
> not drivers, the Subject of the patch does not match.

I see.
OK, I will separate this patch for each driver so that
driver maintainers and developers can apply them easily.

Thanks,
Kiyoshi Ueda
-
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/