All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jukka Ruohonen <jruohonen at iki.fi>
To: devel@acpica.org
Subject: [Devel] AcpiClearGpe() and recent changes
Date: Sun, 06 Jun 2010 12:17:26 +0300	[thread overview]
Message-ID: <20100606091726.GA27517@marx.bitnet> (raw)

[-- Attachment #1: Type: text/plain, Size: 1556 bytes --]

Hello.

I have a question about this recent change:

        31 March 2010. Summary of changes for version 20100331:

        1) ACPI CA Core Subsystem:

        Completed a major update for the GPE support in order to improve
        support for shared GPEs and to simplify both host OS and ACPICA
        code. Added a reference count mechanism to support shared GPEs that
        require multiple device drivers.  Several external interfaces have
        changed. One external interface has been removed. One new external
        interface was added. Most of the GPE external interfaces now use the
        GPE spinlock instead of the events mutex (and the Flags parameter
        for many GPE interfaces has been removed.) See the updated ACPICA
        Programmer Reference for details. Matthew Garrett, Bob Moore, Rafael
        Wysocki. ACPICA BZ 831.

Now the AcpiClearGpe() function operates with a spin mutex on AcpiGbl_GpeLock.
Previously it was possible to avoid getting a lock if the ACPI_NOT_ISR was
not specified, and thus it was possible to also call AcpiClearGpe() from a
GPE handler. If my reading is correct, this is no longer possible because
the GPE dispatch functions obtain the same AcpiGbl_GpeLock already before
calling the custom handler installed via AcpiInstallGpeHandler().

The documentation however claims that "this function [AcpiClearGpe()] may be
called from an interrupt service routine (typically a GPE handler) or a
device driver".

Am I missing or misinterpreting something?

- Jukka.

             reply	other threads:[~2010-06-06  9:17 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-06  9:17 Jukka Ruohonen [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-06-10 20:09 [Devel] AcpiClearGpe() and recent changes Moore, Robert
2010-06-10 20:23 Jukka Ruohonen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100606091726.GA27517@marx.bitnet \
    --to=devel@acpica.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.