All of lore.kernel.org
 help / color / mirror / Atom feed
From: Moore, Robert <robert.moore at intel.com>
To: devel@acpica.org
Subject: Re: [Devel] Reserved method (_PLD) is a buffer instead of a package
Date: Thu, 08 Jul 2010 08:36:10 -0700	[thread overview]
Message-ID: <4911F71203A09E4D9981D27F9D830858A1164BA7@orsmsx503.amr.corp.intel.com> (raw)
In-Reply-To: 201007081111.59571.trenn@suse.de

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



>-----Original Message-----
>From: Thomas Renninger [mailto:trenn(a)suse.de]
>Sent: Thursday, July 08, 2010 2:12 AM
>To: Moore, Robert
>Cc: devel(a)acpica.org
>Subject: Re: [Devel] Reserved method (_PLD) is a buffer instead of a
>package
>
>On Wednesday 07 July 2010 21:52:02 Moore, Robert wrote:
>...
>> Newer versions of iASL will in fact complain about a named buffer that
>> is returned, I think.
>I checked with version 20100702 and it does not.


Maybe only in very limited cases, because a named object can be changed at runtime.


>Never mind, I just want to get a picture about this problem.
>It would be a feature/enhancement to get iasl behave equal, but probably
>with a rather low prio...
>
>> Yes, your change at the end is correct.
>Thanks!
>
>> The ACPICA auto-repair code should be able to repair this problem. If
>> a buffer is returned, it will wrap a package object around it.
>Oh dear. If the world would be perfect, then...
>
>> That doesn't change the fact that the original ASL is incorrect.
>So the policy is to workaround/"runtime fix" the buggy ASL code in the
>kernel/interpreter (silently?) and complain about it in the compiler.
>Sounds reasonable.

Yes, that is the behavior.

1) The goal of iASL is to catch as many errors at compile time as possible.

2) The goal of ACPICA is to detect and repair as many errors at runtime as possible, and to always provide the drivers with valid data (from the evaluation of a predefined ACPI name) as long as AE_OK has been returned.

>
>Thanks,
>
>    Thomas

             reply	other threads:[~2010-07-08 15:36 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-08 15:36 Moore, Robert [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-07-08  9:11 [Devel] Reserved method (_PLD) is a buffer instead of a package Thomas Renninger
2010-07-07 19:52 Moore, Robert
2010-07-05 13:10 Thomas Renninger

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=4911F71203A09E4D9981D27F9D830858A1164BA7@orsmsx503.amr.corp.intel.com \
    --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.