lkml.org 
[lkml]   [2005]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: SATA status report updated
Hi Rainer

Rainer Koenig wrote:
> Jeff Garzik <jgarzik@pobox.com> writes:
>>8) The DMA pad code is very buggy. It uses the dma_map_single() to
>>map a buffer, but never synchronizes nor flushes the buffer. This can
>>and will lead to data corruption, particularly on x86-64 platform.
>
>
> That's very bad since the target platform for that chipset is able
> to support AMD64. :-(

that was my conclusion as well!

> From your comments I've learned that my patch (just the device ID) is
> too tiny and the SiS provided patch is doing too much things that it
> shouldn't do. How can we find a solution for that?
>
> Would it make sense that I try to find the "goods" in the SiS patch and
> merge them somehow in the actual kernel? But: What kernel shall I take
> to do that work? The latest development kernel, the kernel of my
> distribution (whatever this will be, sooner or later it has to work
> with all distributions) or just a kernel that is "close" to the patch
> from SiS, e.g. 2.6.10?
>
> As I mentioned before, getting hardware to try out patches wouldn't be
> that big deal since I'm located in a PC factory and I can get test
> machines if needed. What would be good tests to e.g. detect the problems
> that you mentioned above? Are there hardware specific tests for SATA
> hard disks around? I would be very interested in that since testing
> also under Linux will become daily work for me and my colleauges from
> the system test department.

I'll be happy to test patches that come up. I'm currently running
2.6.13-rc6-mm1, because it also has the sis190 ethernet driver in it,
which actually does work :-)

Unfortunately I'm not able to check the logic of the driver, because
although I can read C, I'm totally unfamiliar with the disk controler
logic in the kernel...

Cheers

Simon

-
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-08-23 00:30    [W:0.061 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site