On 10/17/2012 12:50 PM, Robert Hancock wrote: > On 10/16/2012 07:38 PM, Aaron Lu wrote: >> On 10/16/2012 11:18 PM, Borislav Petkov wrote: >>> On Tue, Oct 16, 2012 at 03:58:24PM +0100, Alan Cox wrote: >>>> Can you check whether 3.6 works on them. I know 3.6 is horribly broken >>>> on several brands of AHCI controller (Jmicron for example). Dunno >>>> where Jeff is on fixing the regressions ? >>> >>> If by "works" you mean I don't see the message there, then yes, it does. >>> Logs say the message started appearing on Oct 4th after me building >>> Linus master after the merge window started. >>> >>> Ok, let me test 3.6.2 just in case ...<tests>... yes, no error message >>> there. >> >> This is brought by commit: 65fe1f0f66a57380229a4ced844188103135f37b, >> ahci: implement aggressive SATA device sleep support. >> >> Shane, got time to take a look? This debug message made people >> uncomfortable :-) > > I don't have whatever version of ATA command set defines this command, > but surely there's some identify bit which lists whether this log page > is supported. Right now checking for it is only conditional on NCQ support. Agree. If NCQ does not imply support of this log page, we should definitely refine the check condition used here. I suppose Shane will take care of this, but if he doesn't, I'll do that at a later time. Thanks, Aaron -- 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
