* [PATCH v2] ide: Fix IDE PIO size calculation
@ 2013-06-20 15:30 Steven J. Hill
2013-07-10 19:09 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Steven J. Hill @ 2013-06-20 15:30 UTC (permalink / raw)
To: David S . Miller; +Cc: Steven J. Hill, linux-ide, Leonid Yegoshin
From: Steven J. Hill <sjhill@mips.com>
IDE PIO calculates the size wrong when passing the parameter
"ide-core.nodma=0.0" to the kernel.
Signed-off-by: Leonid Yegoshin <Leonid.Yegoshin@imgtec.com>
Acked-by: Steven J. Hill <Steven.Hill@imgtec.com>
Signed-off-by: Steven J. Hill <sjhill@mips.com>
---
Changes in v2: Remove internal gerrit ID from commit message.
drivers/ide/ide-taskfile.c | 5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/ide/ide-taskfile.c b/drivers/ide/ide-taskfile.c
index 729428e..dabb88b 100644
--- a/drivers/ide/ide-taskfile.c
+++ b/drivers/ide/ide-taskfile.c
@@ -239,9 +239,6 @@ void ide_pio_bytes(ide_drive_t *drive, struct ide_cmd *cmd,
unsigned nr_bytes = min(len, cursg->length - cmd->cursg_ofs);
int page_is_high;
- if (nr_bytes > PAGE_SIZE)
- nr_bytes = PAGE_SIZE;
-
page = sg_page(cursg);
offset = cursg->offset + cmd->cursg_ofs;
@@ -249,6 +246,8 @@ void ide_pio_bytes(ide_drive_t *drive, struct ide_cmd *cmd,
page = nth_page(page, (offset >> PAGE_SHIFT));
offset %= PAGE_SIZE;
+ nr_bytes = min_t(unsigned, nr_bytes, (PAGE_SIZE - offset));
+
page_is_high = PageHighMem(page);
if (page_is_high)
local_irq_save(flags);
--
1.7.2.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] ide: Fix IDE PIO size calculation
2013-06-20 15:30 [PATCH v2] ide: Fix IDE PIO size calculation Steven J. Hill
@ 2013-07-10 19:09 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2013-07-10 19:09 UTC (permalink / raw)
To: Steven.Hill; +Cc: sjhill, linux-ide, Leonid.Yegoshin
From: "Steven J. Hill" <Steven.Hill@imgtec.com>
Date: Thu, 20 Jun 2013 10:30:37 -0500
> From: Steven J. Hill <sjhill@mips.com>
>
> IDE PIO calculates the size wrong when passing the parameter
> "ide-core.nodma=0.0" to the kernel.
>
> Signed-off-by: Leonid Yegoshin <Leonid.Yegoshin@imgtec.com>
> Acked-by: Steven J. Hill <Steven.Hill@imgtec.com>
>
> Signed-off-by: Steven J. Hill <sjhill@mips.com>
Applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-07-10 19:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-20 15:30 [PATCH v2] ide: Fix IDE PIO size calculation Steven J. Hill
2013-07-10 19:09 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).