From: Jeff Garzik <jgarzik@pobox.com>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Linus Torvalds <torvalds@osdl.org>, Andrew Morton <akpm@osdl.org>,
linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
davej@redhat.com
Subject: Re: [PATCH] libata: fix broken Kconfig setup
Date: Mon, 17 Oct 2005 13:16:44 -0400 [thread overview]
Message-ID: <4353DC7C.6090705@pobox.com> (raw)
In-Reply-To: <200510171006.39206.jbarnes@virtuousgeek.org>
Jesse Barnes wrote:
> So sometimes the legacy IDE driver will lock up when it tries to drive
> both ports in a combined configuration? In that case, can't we just
-sometimes- When it tries to drive the SATA port, it locks up. My best
guess is that this is due to the fact that SATA emulates IDE shadow
registers in silicon, and the IDE driver does something weird that
confused the silicon's IDE emulation logic.
Under SATA, the IDE shadow registers are nothing but a buffer. Writing
to the Command or Control registers causes this buffer to be batched
into a SATA frame (a "FIS"), and sent to the device.
> disable the legacy IDE driver for these chips and force the use of the
> libata version?
More than a little ugly: The piix driver already excludes the SATA
device (unless CONFIG_BLK_DEV_IDE_SATA is defined), so the driver that
picks up the IDE is the non-PCI "generic IDE" legacy driver.
You would need to add code somewhere in a non-PCI driver to specifically
exclude a few PCI devices.
Removing the quirk means users/distros would simply have to know to
disable CONFIG_IDE completely. Doable, but also guaranteed to generate
bug reports.
Jeff
next prev parent reply other threads:[~2005-10-17 17:16 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-10-17 4:46 [PATCH] libata: fix broken Kconfig setup Jeff Garzik
2005-10-17 11:10 ` Matthias Urlichs
2005-10-17 11:20 ` Jeff Garzik
2005-10-17 15:14 ` Linus Torvalds
2005-10-17 15:32 ` Jeff Garzik
2005-10-17 15:58 ` Linus Torvalds
2005-10-17 16:21 ` Jeff Garzik
2005-10-17 16:38 ` Linus Torvalds
2005-10-17 16:53 ` Linus Torvalds
2005-10-17 17:11 ` Jeff Garzik
2005-10-17 17:25 ` Linus Torvalds
2005-10-17 17:38 ` Jeff Garzik
2005-10-19 11:49 ` Alistair John Strachan
2005-10-19 16:02 ` Randy.Dunlap
2005-10-17 17:01 ` Jeff Garzik
2005-10-18 11:15 ` Sergey Vlasov
2005-10-18 20:56 ` Jeff Garzik
2005-10-17 17:12 ` Linus Torvalds
2005-10-17 17:22 ` Jeff Garzik
2005-10-17 16:52 ` Jesse Barnes
2005-10-17 17:03 ` Jeff Garzik
2005-10-17 17:06 ` Jesse Barnes
2005-10-17 17:16 ` Jeff Garzik [this message]
2005-10-20 14:14 ` Alan Cox
2005-10-20 16:45 ` Jesse Barnes
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=4353DC7C.6090705@pobox.com \
--to=jgarzik@pobox.com \
--cc=akpm@osdl.org \
--cc=davej@redhat.com \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.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).