![]() | |||||||||||||
Messages in this thread |
On Thu, Jan 17 2002, Anton Altaparmakov wrote:
> Hi Jens,> > I think there is a bug here...> > At 13:46 17/01/02, Jens Axboe wrote:> >diff -urN -X exclude /ata/linux-2.5.3-pre1/drivers/ide/ide-taskfile.c
> >linux/drivers/ide/ide-taskfile.c> >--- /ata/linux-2.5.3-pre1/drivers/ide/ide-taskfile.c Thu Jan 17
> >06:32:54 2002> >+++ linux/drivers/ide/ide-taskfile.c Thu Jan 17 06:29:13 2002
> >@@ -994,10 +1032,11 @@> > if (!msect) {
> > nsect = 1;
> > while (rq->current_nr_sectors) {
> >- pBuf = rq->buffer + ((rq->nr_sectors -
> >rq->current_nr_sectors) * SECTOR_SIZE);
> >+ pBuf = ide_unmap_buffer(rq, &flags);> > That should be: pBuf = ide_map_buffer(rq, &flags);
> > But I think it actually ought to be:> > pBuf = ide_map_rq(rq, &flags)
>
> and the below unmap should consequently be:
>
> ide_unmap_rq(rq, pBuf, &flags)
>
> In either case it's wrong at the moment AFAICS...
Yeah you are right, typo... There's another mixup in there too, but only
a very minor race. I'll update the patch asap.
--
Jens Axboe
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
| ||||||||||||
| Last update: 2005-03-22 12:15 [from the cache] ©2003-2008 | |||||||||||||