Re: [issue2003] acpica upgrade to 20110211 (again)

[Magliano Andrea <masterblaster@xxxxxxxxxx>]

Hi Sepherosa,

got some additional information (hope it helps).

After this hunk (from fbsd) the overrall system reaction is ok (say, like no loading of acpi.ko)

diff --git a/sys/dev/acpica5/acpi_ec.c b/sys/dev/acpica5/acpi_ec.c
index 432cb5a..7d4f02e 100644
--- a/sys/dev/acpica5/acpi_ec.c
+++ b/sys/dev/acpica5/acpi_ec.c
@@ -658,6 +658,14 @@ EcGpeQueryHandler(void *Context)
    device_printf(sc->ec_dev, "evaluation of query method %s failed: %s\n",
        qxx, AcpiFormatException(Status));
     }
+
+    /* Reenable runtime GPE if its execution was deferred. */
+    if (sc->ec_sci_pend) {
+        Status = AcpiFinishGpe(sc->ec_gpehandle, sc->ec_gpebit);
+        if (ACPI_FAILURE(Status))
+            device_printf(sc->ec_dev, "reenabling runtime GPE failed: %s\n",
+                          AcpiFormatException(Status));
+    }
 }

 /*


I tried booting a kernel with same debug level (events+function), but without the acpi code upgrade and i have an interrupt storm as well...

kernel bootlog with no acpi upgrade (i.e. acpi 20090521):

   evsci-0154 [136] EvSciXruptHandler     : ----Entry
 evevent-0309 [136] EvFixedEventDetect    : Fixed Event Block: Enable 00000120 Status 00000011
   evgpe-0525 [136] EvGpeDetect           : Read GPE Register at GPE08: Status=00, Enable=02
   evsci-0175 [136] EvSciXruptHandler     : ----Exit-        0       0

kernel bootlog with acpi upgrade:

   evsci-0154 [136] EvSciXruptHandler     : ----Entry
 evevent-0297 [137] EvFixedEventDetect    : ----Entry
 evevent-0309 [137] EvFixedEventDetect    : Fixed Event Block: Enable 00000120 Status 00000011
 evevent-0336 [137] EvFixedEventDetect    : ----Exit-        0       0
   evgpe-0525 [136] EvGpeDetect           : Read GPE Register at GPE00: Status=00, Enable=01
   evgpe-0525 [136] EvGpeDetect           : Read GPE Register at GPE08: Status=00, Enable=0E
   evsci-0175 [136] EvSciXruptHandler     : ----Exit-        0       0

the GPE registers may be interpreted in a different way in the new acpi release, but GPE00 register is now seen...

On Thu, 24 Feb 2011 02:39:55 +0000
"Sepherosa Ziehau \(via DragonFly issue tracker\)" <bugs@crater.dragonflybsd.org> wrote:

> 
> Sepherosa Ziehau <sepherosa@gmail.com> added the comment:
> 
> On Wed, Feb 23, 2011 at 7:18 PM, Magliano Andre'
> <masterblaster@tiscali.it> wrote:
> > Hi all,
> >
> > i just discovered it's not possible with github to view a full diff
> > patch just clicking the commit
> > (http://support.github.com/discussions/api/227-no-diff-of-addedremoved-files-in-commit-show-api),
> > but with the suggested url:
> >
> > https://github.com/{user}/{repo}/compare/{revision}^...{revision}.diff
> >
> > so the patch gain as a bz2 download (far faster than my previous github
> > link):
> > https://github.com/downloads/randy1/DragonFlyBSD/ea7ae20415c0af1882c4.diff.bz2
> 
> I will take a look at it.  Thanks for submission
> 
> Best Regards,
> sephe
> 
> ----------
> status: unread -> chatting
> 
> _____________________________________________________
> DragonFly issue tracker <bugs@lists.dragonflybsd.org>
> <http://bugs.dragonflybsd.org/issue2003>
> _____________________________________________________