linux-kernel - Re: [PATCH] sata_nv: revert use of notifiers for now

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <45F1582D.2050005@garzik.org>
Date:	Fri, 09 Mar 2007 07:50:53 -0500
From:	Jeff Garzik <jeff@...zik.org>
To:	Robert Hancock <hancockr@...w.ca>
CC:	linux-kernel <linux-kernel@...r.kernel.org>,
	linux-ide@...r.kernel.org, s0348365@....ed.ac.uk
Subject: Re: [PATCH] sata_nv: revert use of notifiers for now

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@...w.ca>

applied to #upstream-fixes


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/