Re: [PATCH 00/13] st: remove scsi_execute_async usage (the second half)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 16 Dec 2008 22:35:07 +0200 (EET)
Kai Makisara <Kai.Makisara@xxxxxxxxxxx> wrote:

> > Can you replace the first and second patches in the patchset with the
> > following patch (that is, scsi-misc and this patch + #3-13 patches in
> > the patchset)?
> > 
> 
> I did this and have done both the basic tests and tests with big block 
> sizes. No problems and data corruption seen so far.

Thanks for testing!


> However, one thing 
> puzzles me: today trying block size above 6144 kB returns EBUSY whereas 
> yesterday I could use 16000 kB (although with data corruption but this 
> should be fixed by your new patch that increments the offset between 
> bios)?

Can you check where the error happened?

If the error happened in ll_back_merge_fn via blk_rq_append_bio
(EINVAL), it might be due to the following fix (which I put in the
patch) :

http://marc.info/?l=linux-scsi&m=122927795113939&w=2

This patch should do the right thing but this leads to more
segments. But if your scsi host enables use_clustering, the error
should not happen there.


BTW, st driver in 2.6.27 can handle 16000 kB?


> I will do some more tests tomorrow but your code looks really good.

Thanks a lot!
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux