All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Renninger <trenn at suse.de>
To: devel@acpica.org
Subject: Re: [Devel] ACPI: Also allow ACPI table adding via initrd not only overriding
Date: Tue, 18 Feb 2014 19:52:46 +0100	[thread overview]
Message-ID: <139301482.3RWgg14omZ@skinner> (raw)
In-Reply-To: 94F2FBAB4432B54E8AACC7DFDE6C92E37C7D7846@ORSMSX103.amr.corp.intel.com

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

On Tuesday, February 18, 2014 06:38:45 PM Moore, Robert wrote:
> acpi_load_table won't work?

I guess not, there is:
        /* Must acquire the interpreter lock during this operation */
        status = acpi_ut_acquire_mutex(ACPI_MTX_INTERPRETER);

So the interpreter must be running already?
That would mean the adding can only happen much later.
This probably would eleminate most use cases for adding arbitrary
ACPI tables. Most tables (beside DSDT and SSDT) are needed really
early, otherwise the info could have been added as AML as well.

    Thomas 
 
> > -----Original Message-----
> > From: Devel [mailto:devel-bounces(a)acpica.org] On Behalf Of Thomas
> > Renninger
> > Sent: Tuesday, February 18, 2014 10:23 AM
> > Cc: x86(a)kernel.org; ck(a)conrad-kostecki.de; linux-kernel(a)vger.kernel.org;
> > mingo(a)redhat.com; hpa(a)zytor.com; tglx(a)linutronix.de; devel(a)acpica.org
> > Subject: [Devel] ACPI: Also allow ACPI table adding via initrd not only
> > overriding
> > 
> > The ACPICA patches have to go into separate acpica repository first.
> > It should also be checked in acpica whether the table signature the OS
> > likes to add already exists (from BIOS). In this case the table must not
> > be added.
> > 
> > Worked here by trying to override a DSDT and addind a BGRT table.
> > 
> >      Thomas
> > 
> > _______________________________________________
> > Devel mailing list
> > Devel(a)acpica.org
> > https://lists.acpica.org/mailman/listinfo/devel


WARNING: multiple messages have this Message-ID (diff)
From: Thomas Renninger <trenn@suse.de>
To: "Moore, Robert" <robert.moore@intel.com>
Cc: "x86@kernel.org" <x86@kernel.org>,
	"ck@conrad-kostecki.de" <ck@conrad-kostecki.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"mingo@redhat.com" <mingo@redhat.com>,
	"hpa@zytor.com" <hpa@zytor.com>,
	"tglx@linutronix.de" <tglx@linutronix.de>,
	"devel@acpica.org" <devel@acpica.org>
Subject: Re: [Devel] ACPI: Also allow ACPI table adding via initrd not only overriding
Date: Tue, 18 Feb 2014 19:52:46 +0100	[thread overview]
Message-ID: <139301482.3RWgg14omZ@skinner> (raw)
In-Reply-To: <94F2FBAB4432B54E8AACC7DFDE6C92E37C7D7846@ORSMSX103.amr.corp.intel.com>

On Tuesday, February 18, 2014 06:38:45 PM Moore, Robert wrote:
> acpi_load_table won't work?

I guess not, there is:
        /* Must acquire the interpreter lock during this operation */
        status = acpi_ut_acquire_mutex(ACPI_MTX_INTERPRETER);

So the interpreter must be running already?
That would mean the adding can only happen much later.
This probably would eleminate most use cases for adding arbitrary
ACPI tables. Most tables (beside DSDT and SSDT) are needed really
early, otherwise the info could have been added as AML as well.

    Thomas 
 
> > -----Original Message-----
> > From: Devel [mailto:devel-bounces@acpica.org] On Behalf Of Thomas
> > Renninger
> > Sent: Tuesday, February 18, 2014 10:23 AM
> > Cc: x86@kernel.org; ck@conrad-kostecki.de; linux-kernel@vger.kernel.org;
> > mingo@redhat.com; hpa@zytor.com; tglx@linutronix.de; devel@acpica.org
> > Subject: [Devel] ACPI: Also allow ACPI table adding via initrd not only
> > overriding
> > 
> > The ACPICA patches have to go into separate acpica repository first.
> > It should also be checked in acpica whether the table signature the OS
> > likes to add already exists (from BIOS). In this case the table must not
> > be added.
> > 
> > Worked here by trying to override a DSDT and addind a BGRT table.
> > 
> >      Thomas
> > 
> > _______________________________________________
> > Devel mailing list
> > Devel@acpica.org
> > https://lists.acpica.org/mailman/listinfo/devel


         reply	other threads:[~2014-02-18 18:52 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-14 10:23 [PATCH] x86: HPET force enable for Soekris net6501 Conrad Kostecki
2014-02-14 17:46 ` H. Peter Anvin
2014-02-14 18:06   ` AW: " Conrad Kostecki
2014-02-14 18:08     ` H. Peter Anvin
2014-02-14 18:13       ` AW: " Conrad Kostecki
2014-02-14 18:16         ` H. Peter Anvin
2014-02-14 18:21           ` Thomas Gleixner
2014-02-14 18:22             ` H. Peter Anvin
2014-02-14 18:38               ` Thomas Gleixner
2014-02-14 18:39                 ` H. Peter Anvin
2014-02-14 19:15                   ` Thomas Gleixner
2014-02-14 19:26                     ` H. Peter Anvin
2014-02-14 19:59                       ` Thomas Gleixner
2014-02-14 20:06                         ` H. Peter Anvin
2014-02-14 21:16                           ` Thomas Gleixner
2014-02-14 21:18                             ` H. Peter Anvin
2014-02-14 21:47                               ` Thomas Gleixner
2014-02-14 21:48                                 ` H. Peter Anvin
2014-02-17 16:28                             ` Thomas Renninger
2014-02-17 17:19                               ` H. Peter Anvin
2014-02-17 18:23                                 ` [Devel] " Thomas Renninger
2014-02-17 18:23                                   ` AW: AW: " Thomas Renninger
2014-02-17 18:47                                   ` H. Peter Anvin
2014-02-17 19:25                                     ` [Devel] " Thomas Renninger
2014-02-17 19:25                                       ` AW: AW: " Thomas Renninger
2014-02-17 19:40                                       ` H. Peter Anvin
2014-02-18 18:22                                         ` [Devel] ACPI: Also allow ACPI table adding via initrd not only overriding Thomas Renninger
2014-02-18 18:22                                           ` Thomas Renninger
2014-02-18 18:22                                           ` [PATCH 1/4] ACPI: Provide support for ACPI table adding via OS Thomas Renninger
2014-02-18 18:22                                             ` [Devel] " Thomas Renninger
2014-02-18 18:27                                             ` H. Peter Anvin
2014-02-18 18:44                                               ` [Devel] " Thomas Renninger
2014-02-18 18:44                                                 ` Thomas Renninger
2014-02-18 20:51                                                 ` H. Peter Anvin
2014-02-18 18:22                                           ` [PATCH 2/4] ACPICA: Introduce new acpi_os_physical_table_add OS callback Thomas Renninger
2014-02-18 18:22                                             ` [Devel] " Thomas Renninger
2014-02-18 18:22                                           ` [PATCH 3/4] ACPICA: Add BGRT signature to known signatures Thomas Renninger
2014-02-18 18:22                                             ` [Devel] " Thomas Renninger
2014-02-18 18:22                                           ` [PATCH 4/4] ACPI: Add new table signatures that can be overridden/added Thomas Renninger
2014-02-18 18:22                                             ` [Devel] " Thomas Renninger
2014-02-18 18:52                                         ` Thomas Renninger [this message]
2014-02-18 18:52                                           ` [Devel] ACPI: Also allow ACPI table adding via initrd not only overriding Thomas Renninger
2014-02-18 19:59                                           ` Moore, Robert
2014-02-18 19:59                                             ` Moore, Robert
2014-02-19 11:14                                             ` Thomas Renninger
2014-02-19 11:14                                               ` Thomas Renninger
2014-02-19 13:03                                               ` Thomas Gleixner
2014-02-14 18:28           ` AW: AW: AW: [PATCH] x86: HPET force enable for Soekris net6501 Conrad Kostecki
2014-09-09 13:56             ` Eric Sesterhenn
2014-09-09 14:54               ` Thomas Gleixner
2014-09-09 15:26                 ` H. Peter Anvin
2014-09-09 15:41                   ` Thomas Gleixner
2014-09-12  9:41                   ` Eric Sesterhenn
2014-09-12 10:37                     ` Thomas Gleixner
2014-09-12 11:06                       ` [PATCH] x86: HPET force enable for e6xx based systems Eric Sesterhenn
2014-09-16  0:58                         ` [tip:x86/platform] " tip-bot for Peter Neubauer
  -- strict thread matches above, loose matches on Subject: below --
2014-02-18 18:38 [Devel] ACPI: Also allow ACPI table adding via initrd not only overriding Moore, Robert
2014-02-18 18:38 ` Moore, Robert
2014-02-19 11:22 [Devel] [PATCH 1/4] ACPI: Provide support for ACPI table adding via OS Thomas Renninger
2014-02-19 11:22 ` Thomas Renninger
2014-02-21  7:24 [Devel] " Zheng, Lv
2014-02-21  7:24 ` Zheng, Lv
2014-02-21  7:28 Zheng, Lv
2014-02-21  7:28 ` Zheng, Lv

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=139301482.3RWgg14omZ@skinner \
    --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.