On Tue, 2008-09-02 at 00:59 +0400, Alexey Starikovskiy wrote: > Alexey Starikovskiy wrote: In this patch when timeout happens, EC will try the GPE interrupt mode after 5 seconds.Of course EC will be in polling mode before switching to GPE interrupt mode. It seems reasonable. In fact there also exists the mode switch from polling mode to interrupt mode after EC is initialized. Is it more appropriate that ec->gpe_retry is initialized? Thanks. > > Henrique de Moraes Holschuh wrote: > >> On Mon, 01 Sep 2008, Zhao Yakui wrote: > >>> Will the above two patches hit the upstream kernel? If the above > >>> two patches hits the upstream kernel, it seems that the boot option > >>> of "ec_intr=" comes back again and EC can't be switched from > >>> interrupt mode to polling mode if EC GPE interrupt is missing. In > >>> such case maybe the battery/AC/thermal driver can't work well if the > >>> info of > >>> battery/AC/thermal is related with EC. Maybe there exists the > >>> regression on some laptops. > >> > >> Right now, the fact that it gives up on interrupt mode too easily IS > >> causing > >> regressions on ThinkPads (like the T43 I own). Since polling mode does > >> work, it is just a performance regression, so you won't get many reports > >> about it since most people don't look for such stuff in their kernel > >> logs. > >> > >> Some ECs trigger the interrupt/poll-mode checks just on small windows > >> (typically during resume -- might even be a bug somewhere in ACPICA or > >> Linux, and not on the EC). We should not be giving up using interrupt > >> mode > >> on these so easily. Maybe retry enabling interrupt mode after some > >> seconds > >> a few times (like 3 or 5)? If it is a transient problem, that will avoid > >> the permanent performance regression of polled mode. > >> > > How about such patch? > Or even better (working?) patch ... > > > > Regards, > > Alex. > > > -- 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
