diff options
author | Borislav Petkov <petkovbb@gmail.com> | 2009-05-01 20:29:53 +0200 |
---|---|---|
committer | Borislav Petkov <petkovbb@gmail.com> | 2009-05-15 06:43:45 +0200 |
commit | 10c0b3437c4dc0d14ac254bbe71e54ea5f238d97 (patch) | |
tree | 8d40e72ad1c096bcb4d4ca0475f62e9bebbec647 /drivers/ide/ide-tape.c | |
parent | 77f6bf57ba9d2c50173536dbfdacdab27cb867ca (diff) | |
download | op-kernel-dev-10c0b3437c4dc0d14ac254bbe71e54ea5f238d97.zip op-kernel-dev-10c0b3437c4dc0d14ac254bbe71e54ea5f238d97.tar.gz |
ide-tape: fix potential fs requests bug
ide-tape had a potential bug for fs requests when preparing the command
packet: it was writing the transfer length as a number of fixed blocks.
However, the block layer implies 512 byte blocks and ide-tape can have
other block sizes so account for that too.
ide-floppy does this calculation properly with the block size factor
(floppy->bs_factor).
Signed-off-by: Borislav Petkov <petkovbb@gmail.com>
Diffstat (limited to 'drivers/ide/ide-tape.c')
-rw-r--r-- | drivers/ide/ide-tape.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ide/ide-tape.c b/drivers/ide/ide-tape.c index e166045..fc79cf4 100644 --- a/drivers/ide/ide-tape.c +++ b/drivers/ide/ide-tape.c @@ -586,7 +586,7 @@ static void ide_tape_create_rw_cmd(idetape_tape_t *tape, struct ide_atapi_pc *pc, struct request *rq, u8 opcode) { - unsigned int length = blk_rq_sectors(rq); + unsigned int length = blk_rq_sectors(rq) / (tape->blk_size >> 9); ide_init_pc(pc); put_unaligned(cpu_to_be32(length), (unsigned int *) &pc->c[1]); |