LKML Archive on
help / color / mirror / Atom feed
From: Jeff Garzik <>
To: Robert Hancock <>
Cc: Alistair John Strachan <>,,
	IDE/ATA development list <>
Subject: Re: CK804 SATA Errors (still got them)
Date: Sun, 04 Mar 2007 18:46:26 -0500	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

Robert Hancock wrote:
> Alistair John Strachan wrote:
>>> Can you try reverting commit 721449bf0d51213fe3abf0ac3e3561ef9ea7827a
>>> (link below) and see what effect that has?
>>> =721449bf0d51213fe3abf0ac3e3561ef9ea7827a
>> Obviously, I'll let you know if it happens again, but I've reverted 
>> this commit and transferred 22.5GB over 45 minutes onto a RAID5 with 4 
>> HDs on an NVIDIA sata controller, and this error hasn't appeared.
>> So I'm inclined to (very unscientifically) say that this brings it 
>> back to 2.6.20's level of stability.
> Interesting. Can you try un-reverting that patch, and applying this one?
> The reading of the status register is something that was part of the 
> original
> NVidia code, which I'm not really sure why is there. Given that reading
> the status register clears the drive's interrupt status, that might be
> causing some wierd interaction with the ADMA controller. Also, I added in
> a printk for cases where notifiers are triggered but the command doesn't
> indicate completion - if you still get problems, let me know if you see
> that message.

AFAICS, when in ADMA mode, you absolutely should not touch the ATA 
shadow registers at all.

This is normal for all controllers with both a "legacy mode" and an 
"enhanced DMA mode" of some sort:  the internal silicon state machines 
"own" the ATA shadow registers while in enhanced DMA mode.  Reading or 
writing the ATA shadow registers while in enhanced DMA mode can lead to 
undefined results, running the gamut from no-op to data corruption and 
hardware lock-ups.

You may only access the ATA shadow registers when NV_ADMA_CTL_GO is 
cleared, and then NV_ADMA_STAT_LEGACY is set, indicating the NVIDIA chip 
is in register mode (aka legacy mode).


  parent reply	other threads:[~2007-03-04 23:46 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-01 13:39 CK804 SATA Errors (still got them) Alistair John Strachan
2007-03-01 14:45 ` Robert Hancock
2007-03-01 15:13   ` Alistair John Strachan
2007-03-02  1:20     ` Alistair John Strachan
2007-03-02  2:40       ` Robert Hancock
2007-03-02 15:47         ` Alistair John Strachan
2007-03-04 23:25           ` Robert Hancock
2007-03-04 23:41             ` Alistair John Strachan
2007-03-04 23:49               ` Robert Hancock
2007-03-04 23:50               ` Jeff Garzik
2007-03-04 23:46             ` Jeff Garzik [this message]
2007-03-05  3:52             ` Alistair John Strachan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).