On Mon, Aug 03, 2015 at 11:16:21AM -0400, Tejun Heo wrote: > From 8c0fa3e7ca99b0d6d96cb2cf194a912f643da7c5 Mon Sep 17 00:00:00 2001 > From: Tejun Heo <tj@xxxxxxxxxx> > Date: Mon, 3 Aug 2015 11:10:45 -0400 > > fe7173c206de ("libata: Implement support for sense data reporting") > and subsequent patches enabled NCQ autosense reporting; unfortunately, > this breaks libata EH behavior for devices which support the feature > because it assumes that only ATAPI devices report sense data and that > sense data trumps explicit device error indication. So, this isn't enough and apparently you're bypassing the entire device side error analyzing when the device reports sense data. Sorry that I missed this on the first submission but I can't see how this would work. Even if you fix the immediate issue in this thread, how is it gonna handle link error reported by the device? Who's gonna tell libata EH that the link better be reset and may be sped down if errors keep occurring? This is a significant regression in EH behavior. I'm gonna revert the whole ACS-4 sense thing. ATM, I'm pretty doubtful that this buys antying substantial for us and even if so we need to figure out a better way to integrate this. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html