ide-cd: move request prep from cdrom_start_rw_cont to rq issue path
... by factoring out the rq preparation code into a separate function called in the request routine. As a nice side effect, this minimizes the IRQ handler execution time. There should be no functionality change resulting from this patch. [bart: s/HWGROUP()/drive->hwif->hwgroup/ and remove extra newline] Signed-off-by: Borislav Petkov <petkovbb@gmail.com> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
This commit is contained in:
parent
e529c6087a
commit
90eb808e0f
|
@ -666,16 +666,9 @@ static int ide_cd_check_transfer_size(ide_drive_t *drive, int len)
|
||||||
|
|
||||||
static ide_startstop_t cdrom_newpc_intr(ide_drive_t *);
|
static ide_startstop_t cdrom_newpc_intr(ide_drive_t *);
|
||||||
|
|
||||||
/*
|
static ide_startstop_t ide_cd_prepare_rw_request(ide_drive_t *drive,
|
||||||
* Routine to send a read/write packet command to the drive. This is usually
|
struct request *rq)
|
||||||
* called directly from cdrom_start_{read,write}(). However, for drq_interrupt
|
|
||||||
* devices, it is called from an interrupt when the drive is ready to accept
|
|
||||||
* the command.
|
|
||||||
*/
|
|
||||||
static ide_startstop_t cdrom_start_rw_cont(ide_drive_t *drive)
|
|
||||||
{
|
{
|
||||||
struct request *rq = HWGROUP(drive)->rq;
|
|
||||||
|
|
||||||
if (rq_data_dir(rq) == READ) {
|
if (rq_data_dir(rq) == READ) {
|
||||||
unsigned short sectors_per_frame =
|
unsigned short sectors_per_frame =
|
||||||
queue_hardsect_size(drive->queue) >> SECTOR_BITS;
|
queue_hardsect_size(drive->queue) >> SECTOR_BITS;
|
||||||
|
@ -712,6 +705,19 @@ static ide_startstop_t cdrom_start_rw_cont(ide_drive_t *drive)
|
||||||
/* set up the command */
|
/* set up the command */
|
||||||
rq->timeout = ATAPI_WAIT_PC;
|
rq->timeout = ATAPI_WAIT_PC;
|
||||||
|
|
||||||
|
return ide_started;
|
||||||
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Routine to send a read/write packet command to the drive. This is usually
|
||||||
|
* called directly from cdrom_start_{read,write}(). However, for drq_interrupt
|
||||||
|
* devices, it is called from an interrupt when the drive is ready to accept
|
||||||
|
* the command.
|
||||||
|
*/
|
||||||
|
static ide_startstop_t cdrom_start_rw_cont(ide_drive_t *drive)
|
||||||
|
{
|
||||||
|
struct request *rq = drive->hwif->hwgroup->rq;
|
||||||
|
|
||||||
/* send the command to the drive and return */
|
/* send the command to the drive and return */
|
||||||
return cdrom_transfer_packet_command(drive, rq, cdrom_newpc_intr);
|
return cdrom_transfer_packet_command(drive, rq, cdrom_newpc_intr);
|
||||||
}
|
}
|
||||||
|
@ -1224,8 +1230,12 @@ static ide_startstop_t ide_cd_do_request(ide_drive_t *drive, struct request *rq,
|
||||||
} else {
|
} else {
|
||||||
xferlen = 32768;
|
xferlen = 32768;
|
||||||
fn = cdrom_start_rw_cont;
|
fn = cdrom_start_rw_cont;
|
||||||
|
|
||||||
if (cdrom_start_rw(drive, rq) == ide_stopped)
|
if (cdrom_start_rw(drive, rq) == ide_stopped)
|
||||||
return ide_stopped;
|
return ide_stopped;
|
||||||
|
|
||||||
|
if (ide_cd_prepare_rw_request(drive, rq) == ide_stopped)
|
||||||
|
return ide_stopped;
|
||||||
}
|
}
|
||||||
info->last_block = block;
|
info->last_block = block;
|
||||||
} else if (blk_sense_request(rq) || blk_pc_request(rq) ||
|
} else if (blk_sense_request(rq) || blk_pc_request(rq) ||
|
||||||
|
|
Loading…
Reference in New Issue