[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <485F5D4C.90609@kernel.org>
Date: Mon, 23 Jun 2008 17:22:36 +0900
From: Tejun Heo <tj@...nel.org>
To: Pavel Machek <pavel@...e.cz>
CC: kernel list <linux-kernel@...r.kernel.org>,
benh@...nel.crashing.org, jgarzik@...ox.com,
IDE/ATA development list <linux-ide@...r.kernel.org>,
Trivial patch monkey <trivial@...nel.org>
Subject: Re: sata_svw data corruption, strange problems
Pavel Machek wrote:
> Hi!
>
>>> I see strange problems on machine with sata_svw. The machine seems to
>>> corrupt data every few days (ext3 error, dir index corrupted), and has
>>> some other very strange problems (keyboard misbehaves, pulling out
>>> SATA disk cures it, see
>>> https://bugzilla.novell.com/show_bug.cgi?id=400772 ).
>>>
>>> Then I got to the comment
>>>
>>> writeb(dmactl | ATA_DMA_START, mmio + ATA_DMA_CMD);
>>> /* There is a race condition in certain SATA controllers
>>> that can be seen when the r/w command is given to the controller
>>> before the host DMA is started. On a Read command, the controller
> ...
>>> ...and that would certainly explain what we are seeing. Are
>>> serverworks controllers broken by design?
>> The comment looks like a warning to me as the DMA engine is started
>> before the command is issued to the drive as explained in the next
>> comment.
>
> Ok, what about this?
>
> ---
>
> Clarify data corruption comment.
>
> Signed-off-by: Pavel Machek <pavel@...e.cz>
Acked-by: Tejun Heo <tj@...nel.org>
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists