All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	jeff@garzik.org, linux-ide@vger.kernel.org, alan@redhat.com,
	linux-kernel@vger.kernel.org
Subject: Re: [patch 3/3] pata: "I do not think it means, what you think it means."
Date: Wed, 5 Mar 2008 14:21:13 +0100	[thread overview]
Message-ID: <200803051421.14167.bzolnier@gmail.com> (raw)
In-Reply-To: <20080305115610.397b3ea3@the-village.bc.nu>

On Wednesday 05 March 2008, Alan Cox wrote:
> 
> > I didn't know any of that.  The changelog might have been kinda fun, but
> > given that it failed to tell us that the patch fixes data-corruption
> > errors, the changelog was excrutiatingly bad.
> 
> I have no reason/evidence to believe it fixes data corruption errors of
> any kind. For the specific combinations of device it should simply avoid
> a long pause, complaints and a switch to lower speeds.

http://www.mail-archive.com/linux-ide@vger.kernel.org/msg16599.html

There is strange coincidence with being on the blacklist and FIFO corruption.

https://bugzilla.redhat.com/show_bug.cgi?id=433557

Bugzilla Bug 433557: Data corrupion with Fedora8 on HPT370 disk controller (Abit BX133 mobo)

IBM-DTLA-307030 is on the blacklist....

[ there can be more libata problems involved, anyway FC6 w/IDE works fine ]

> The ATA disk case with serverworks (which is a potential corruptor) was
> always correctly handled.

for OSB4 yes but...

/* Seagate Barracuda ATA IV Family drives in UDMA mode 5
 * can overrun their FIFOs when used with the CSB5 */

static const char *csb_bad_ata100[] = {

Thanks,
Bart

  reply	other threads:[~2008-03-05 13:21 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-04 23:16 [patch 3/3] pata: "I do not think it means, what you think it means." akpm
2008-03-05  1:34 ` Bartlomiej Zolnierkiewicz
2008-03-05  1:30   ` Andrew Morton
2008-03-05  1:54     ` Bartlomiej Zolnierkiewicz
2008-03-05 11:56     ` Alan Cox
2008-03-05 13:21       ` Bartlomiej Zolnierkiewicz [this message]
2008-03-05 14:39         ` Alan Cox
2008-03-05 16:13           ` Bartlomiej Zolnierkiewicz
2008-03-05 16:34         ` Krzysztof Halasa
2008-03-05 16:46           ` Bartlomiej Zolnierkiewicz
2008-03-05 16:55             ` Krzysztof Halasa
2008-03-05 17:22               ` Bartlomiej Zolnierkiewicz
2008-03-05 18:33                 ` Zan Lynx
2008-03-06  1:33                   ` Krzysztof Halasa
2008-03-06 11:13                     ` Ville Syrjälä
2008-03-06 14:58                       ` Mark Lord
2008-03-06 14:58                         ` Mark Lord
2008-03-06 15:00                         ` Mark Lord
2008-03-06 15:00                         ` Mark Lord
2008-03-06 22:10                       ` Krzysztof Halasa

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=200803051421.14167.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=alan@redhat.com \
    --cc=jeff@garzik.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.