Skip to content

ide-atapi: remove ide_atapi_pc from the irq handler

Borislav Petkov linux-ide lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/9ide-atapi: remove PC_FLAG_WRITING atapi flagdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-floppy_ioctl.c +2 more+10 -140 replies
2/9ide-atapi: replace pc->flags with drive->pc_flagsdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-floppy.h +3 more+103 -880 replies
3/9ide-atapi: replace pc->buf with rq->data in the irq handlerdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-tape.c+33 -230 replies
4/9ide-atapi: replace pc->req_xfer with drive->pc_req_xferdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-tape.c +1 more+22 -210 replies
5/9ide-atapi: replace pc->buf_size with drive->pc_buf_sizedrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-floppy_ioctl.c +2 more+8 -70 replies
6/9ide-atapi: remove pc arg to drive->pc_io_buffersdrivers/ide/ide-atapi.c drivers/ide/ide-tape.c include/linux/ide.h+13 -140 replies
7/9ide-atapi: use rq pointer directly instead of pc->rq derefdrivers/ide/ide-atapi.c+2 -20 replies
8/9ide-atapi: replace pc->error with drive->pc_errordrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-tape.c +1 more+16 -140 replies
9/9ide-atapi: remove pc arg to drive->pc_update_buffersdrivers/ide/ide-atapi.c drivers/ide/ide-floppy.c drivers/ide/ide-tape.c +1 more+16 -70 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/9] ide-atapi: remove ide_atapi_pc from the irq handler

Hi Bart,

here's a first attempt at removing all references to ide_atapi_pc in the ATAPI IRQ handler. I've moved some of the members to the drive struct and will deal with them later :). The next step is to add an ide_atapi_queue_pc() routine similar to ide_cd_queue_pc() and then rewrite all functions in the drivers to use struct requests and local buffers instead of pc->buf and then, after that works reliably, finally get rid of ide_atapi_pc completely.

This has been tested with ide-floppy.

 drivers/ide/ide-atapi.c        |  123 ++++++++++++++++-------------
 drivers/ide/ide-floppy.c       |   56 +++++++-------
 drivers/ide/ide-floppy.h       |    4 +-
 drivers/ide/ide-floppy_ioctl.c |   19 +++--
 drivers/ide/ide-tape.c         |  173 ++++++++++++++++++++++------------------
 include/linux/ide.h            |   20 +++--
 6 files changed, 214 insertions(+), 181 deletions(-)