From: Robert Hancock <hancockrwd@gmail.com>
To: Juergen Beisert <jbe@pengutronix.de>
Cc: "linux-ide@vger.kernel.org" <linux-ide@vger.kernel.org>
Subject: Re: How does libata handles an 'ATA_ABORTED' error?
Date: Wed, 14 Dec 2011 23:51:48 -0600 [thread overview]
Message-ID: <4EE98AF4.7090509@gmail.com> (raw)
In-Reply-To: <201112140948.03859.jbe@pengutronix.de>
On 12/14/2011 02:48 AM, Juergen Beisert wrote:
> Hi list,
>
> I have a CF card running in true-ide mode connected to regular PC. This CF
> card does wear leveling of its flash memory internally like every other CF
> card. With one exception: When the CF's firmware detects a broken NAND page
> while writing a sector, it moves around the remaining (good) data to other
> pages. To do this job it must discard the already transmitted sector data in
> its SRAM, because it needs this SRAM to move around the other flash memory
> data.
>
> After the movement the firmware signals an 'ATA_ERR' in the status register
> and an 'ATA_ABORTED' in the error register to force the host to repeat to
> write the same data again (next time it will be successfull due to internal
> wear leveling is already done).
>
> As we see data lost when the systems are running in production, I'm now trying
> to find out if the libata/SCSI layer really repeats the sector write for this
> case and does the expected (or required) things. But I'm lost in these
> software layers and their error path.
>
> I found (in Documentation/DocBook/libata.tmpl):
>
> "This is indicated by UNC bit in the ERROR register. ATA
> devices reports UNC error only after certain number of
> retries cannot recover the data, so there's nothing much
> else to do other than notifying upper layer."
>
> which sounds to me as no repeat will happen for write errors, but
> the 'ATA_UNC' bit is not used to signal the "wear leveling case" shown above.
That seems like incorrect behavior by the device, ABRT is normally used
to indicate an invalid or unsupported command. UNC would likely be more
appropriate. But I don't think it ultimately makes a difference in this
case.
>
> As far as I understand the ATA errors are transformed to SCSI errors and then
> handled in the SCSI layer. But the documentation tells me it is not easy to
> always find an adequate SCSI error for an ATA error. So, I'm not sure if for
> the "wear leveling case" the SCSI layer receives a "valuable" error message.
From what I can see the SCSI error that gets returned in this case is
just an "aborted command" error.
>
> Does anybody can give me a hint, what really happens when the attached drive
> signals an 'ATA_ABORTED'? Does the libata/SCSI give up in this case, or will
> it repeat the command?
I don't know that the SCSI or block layers really pay much attention to
the error code in this case - I think it would always attempt some retries.
Certainly any of these errors would result in error messages showing up
in dmesg. Are you seeing any of this?
next prev parent reply other threads:[~2011-12-15 5:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-14 8:48 How does libata handles an 'ATA_ABORTED' error? Juergen Beisert
2011-12-15 5:51 ` Robert Hancock [this message]
2011-12-15 11:01 ` Juergen Beisert
2011-12-15 18:38 ` Robert Hancock
2011-12-16 4:26 ` Mark Lord
2011-12-17 2:41 ` Robert Hancock
2012-01-20 15:00 ` Juergen Beisert
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:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4EE98AF4.7090509@gmail.com \
--to=hancockrwd@gmail.com \
--cc=jbe@pengutronix.de \
--cc=linux-ide@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* 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).