All of lore.kernel.org
 help / color / mirror / Atom feed
From: Prakash Punnoor <prakashp@arcor.de>
To: Jeff Garzik <jgarzik@pobox.com>
Cc: Doug McLain <nostar@comcast.net>,
	PALFFY Daniel <dpalffy-lists@rainstorm.org>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org
Subject: Re: sata_sil data corruption
Date: Mon, 21 Feb 2005 17:35:42 +0100	[thread overview]
Message-ID: <421A0DDE.7020904@arcor.de> (raw)
In-Reply-To: <20050221162923.GA29621@havoc.gtf.org>

[-- Attachment #1: Type: text/plain, Size: 1386 bytes --]

Jeff Garzik schrieb:
> On Mon, Feb 21, 2005 at 11:25:55AM +0000, Doug McLain wrote:
>
>>Jeff Garzik wrote:
>>
>>>Doug McLain wrote:
>>>
>>>
>>>>The sata_sil driver is without a doubt, totally hosed.  I, along with
>>>
>>>
>>>"without a doubt" being defined, of course, as "it works for a lot of
>>>people."
>>>
>>>   Jeff
>>>
>>>
>>>
>>>
>>
>>Thats like saying "turn up the radio" when your car makes a funny noise,
>>or "if a tree falls in the woods and nobody is there to hear it, does it
>>make a sound?"
>>
>>It's tempting and comforting to pick the good ones as an example, and
>>some bugs are hard enough to find, let alone fix.  In the end though, if
>>one is broke, it's still broke, isn't it?
>
>
> In this case, the bug _reports_ are hard to find.
>
> Each case with sata_sil is either solved with a BIOS update, a
> blacklist entry, or new cables.  Just read through bugzilla.kernel.org.

I personally think the driver is OK (works for me like a charm with my Samsung
drive), but as I reported a few times, SiI will corrupt data if ext-p2p
discard time setting in bios is set to too low values.

So, Jeff, don't you think adding a quirk to the kernel would prevent such
reports. (I am damn sure, this reports are cause of the setting I am talking
about.) Perhaps you could ask SiI how to do this in the kernel...

--
Prakash Punnoor

formerly known as Prakash K. Cheemplavam

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

      parent reply	other threads:[~2005-02-21 16:35 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-28 13:17 sata_sil data corruption PALFFY Daniel
2005-02-19 17:35 ` Jeff Garzik
2005-02-21 11:25   ` PALFFY Daniel
2005-02-21  9:02     ` Doug McLain
2005-02-21 16:17       ` Jeff Garzik
2005-02-21 11:25         ` Doug McLain
2005-02-21 16:29           ` Jeff Garzik
2005-02-21 11:47             ` Doug McLain
2005-02-21 16:35             ` Prakash Punnoor [this message]

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=421A0DDE.7020904@arcor.de \
    --to=prakashp@arcor.de \
    --cc=dpalffy-lists@rainstorm.org \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nostar@comcast.net \
    /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.