lkml.org 
[lkml]   [2003]   [Apr]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] 2.5.67-ac2 fix new PIO handlers

On Mon, 28 Apr 2003, Osamu Tomita wrote:

> Hi,
>
> Bartlomiej Zolnierkiewicz wrote:
> >
> > Hi,
> >
> > During rewrite of bio walking patch to use rq->cbio instead
> > of rq->hard_bio I've realized I had screwed orig 2.5.66 patches :\
> >
> > I somehow forgot to update task_map_rq(), it should use
> > __bio_kmap_irq(rq->bio, current_idx, flags) instead of
> > bio_kmap_irq(rq->bio, flags). The result is that you get
> > bio corruption (-> data corruption) when using PIO multiple
> > with sectors multiply > 8 (PAGE_SIZE == bio_vec size).

Hmmm, not only > 8, also 3, 5.

> > Attached patch fixes it and fixes TASKFILE ioctl.
> >
> > Please apply to next -ac.
> Could you please tell me how to apply your patch. Aginst 2.5.62-ac2?
> I have a ext2 fs corruption problem at PIO mode on 2.5.67-ac2.

Did you use multi PIO? You can check with 'hdparm -m /dev/hdx'.
If yes this patch should fix the corruption.

> There is no problem on vanilla 2.5.67.
> I tryed the all patches below.
> tf-ioctls-1.diff
> tf-ioctls-2a.diff
> tf-ioctls-2b.diff
> tf-ioctls-3.diff
> tf-ioctls-4.diff
> tf-dio-1.diff
> tf-dio-2.diff
> tf-dio-3.diff
> tf-dio-4.diff
> masked-irq.diff
> They didn't solve problem.

No, they don't.

> This pio-fixes.diff can't be applied after them?
>
> Regards,
> Osamu Tomita

Could you please apply only pio_fixes.diff on top of the vanilla
2.5.67-ac2 and tell me if it solves the problem? It should.

Thanks,
--
Bartlomiej

-
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 13:35    [W:0.036 / U:2.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site