On Thu, 20 Mar 2008, Alexey Starikovskiy wrote: > Henrique de Moraes Holschuh wrote: >> On Wed, 19 Mar 2008, Pavel Machek wrote: >> >>> Also, for polling mode to work on Acers, we need >>> to insert a delay. >> >> Can we, in the future, always trigger any such performance damaging "fixups" >> based on DMI white/black lists? As a rule? >> > Poll mode is not supposed to be entered on any non-broken HW. We have seen just how much we can trust that, haven't we? The truth is that even secondary last-resort paths like this need to work well, and as fast as possible. As things stand, we DO excercise them a lot more than we'd expect... > The fact that it happens now with Thinkpads at suspend is a bug. Sure, and one we should fix. That doesn't change the fact that any broken HW (or one hit by a kernel bug) that is not as broken as some Acers don't need to take that 5ms hit on every EC transaction, in an EC mode that is already way too slow without extra help. > EC region should not be accessed with interrupts of GPEs disabled. Looks like something we should tack a WARN_ONCE to, at least when in a debug mode compile? -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
