http://bugzilla.kernel.org/show_bug.cgi?id=11896
------- Comment #20 from [EMAIL PROTECTED] 2008-11-04 02:14 ------- (In reply to comment #19) > I am not prepared to revert any patch from you. Then why your patch series, which reverts it appears in every bug report and in linux-acpi mail list? > What I cared is to make EC work more stably. You don't have a clue how to do it apparently. > In fact I point out what will be affected by your patch. You don't. > IMO when you disagree with other patch, please give some detailed reason or > say which box will be broken by the patch. With the "fast transaction" patch ASUS eeePC is reported to not trigger interrupt storm detection and does only 2-3 faux interrupts per transaction, compared to 20-30 without the fast transaction. Thus, by reverting fast transaction, you force EC driver to poll mode and degraded performance. Plus, to "limit" the interrupt count, you instead of disabling interrupt source try to make interrupt servicing longer, thus disabling other interrupts and prohibiting scheduling to other more important tasks. And please don't say that you read this for the first time -- all these messages are on the mailing list, you just don't read them. > > It is difficult to reproduce the issue about AE_TIME after the patch > mentioned in comment #16 is applied. But we can't say that it is impossible to > happen. > In fact I raise the issue of AE_TIME about your patch "fast transaction". > But you ignore it and told me to put the real data. Now the problem appears. msleep revert patch is on my queue since then (and I sent it to you more than a month ago), now then we have reason to apply it, thus it is sent to Len. > IMO the better patch is that we should get the root cause and the possible > potential fault should be considered. Why did you not come up with root cause yet? I asked you to bring one and you replied with "fast transaction" revert series. > It is not better that something is reverted when a problem happens. And it is > > reverted again when another problem happens. You are amazing... How the hell your 4-patch series did appear? > ( As I know, msleep is reverted at least twice in EC driver). Use of msleep looks to be more promising power-vise until proved to either hurt performance or break things. Now we know that it breaks things, so it will stay out of EC driver. > Thanks. No, thank you for wasting my time and keeping me busy. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. ------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
