From: Jeff Garzik <jeff@garzik.org>
To: Robert Hancock <hancockr@shaw.ca>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
linux-ide@vger.kernel.org, s0348365@sms.ed.ac.uk
Subject: Re: [PATCH] sata_nv: revert use of notifiers for now
Date: Fri, 09 Mar 2007 07:50:53 -0500 [thread overview]
Message-ID: <45F1582D.2050005@garzik.org> (raw)
In-Reply-To: <45F0A40A.4050508@shaw.ca>
Robert Hancock wrote:
> Commit 721449bf0d51213fe3abf0ac3e3561ef9ea7827a added support for using the
> ADMA notifier bits to determine which commands to check for completion.
> However there have been reports that this causes command timeouts in
> certain
> cases. This is still being investigated. In addition, apparently the
> notifiers
> won't work if ADMA is disabled on the other port as a result of an ATAPI
> device
> being connected, and we don't handle this case properly.
>
> For now, just restore the previous behavior of checking all active commands
> to see if they are complete, without relying on the notifiers.
>
> Signed-off-by: Robert Hancock <hancockr@shaw.ca>
applied to #upstream-fixes
prev parent reply other threads:[~2007-03-09 12:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-03-09 0:02 [PATCH] sata_nv: revert use of notifiers for now Robert Hancock
2007-03-09 12:50 ` Jeff Garzik [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=45F1582D.2050005@garzik.org \
--to=jeff@garzik.org \
--cc=hancockr@shaw.ca \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=s0348365@sms.ed.ac.uk \
/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.