From: Simon Bridge <corwin@ihug.co.nz>
To: "Moore, Robert" <robert.moore@intel.com>
Cc: Thomas Renninger <trenn@suse.de>, linux-acpi@vger.kernel.org
Subject: RE: reality check: dsdt.dsl -> compile -> DSDT.aml -> decompile ->DSDT.dsl but dsdt.dsl neq DSDT.dsl
Date: Mon, 30 Jan 2006 15:53:37 +1300 [thread overview]
Message-ID: <1138589618.31634.21.camel@localhost.localdomain> (raw)
In-Reply-To: <971FCB6690CD0E4898387DBF7552B90E0421863E@orsmsx403.amr.corp.intel.com>
On Wed, 2006-01-25 at 09:55 -0800, Moore, Robert wrote:
> Well, yes, if the external cannot be resolved at runtime, it will
> generate an error -- Unless you have the update to ACPICA that allows
> unresolved references within packages, AND you enable the interpreter
> slack mode.
>
> > I got the DSDT.aml (the result of successfully compiling dsdt.dsl
> > remember) and decompiled it - the result is expected to be the same as
> > dsdt.dsl... but it wasn't! There were substantial differences. When
> this
> > new version was compiled, it produced errors.
>
> Any disassembler is not *guaranteed* to generate the exact source code.
> I would be interested in a bit more explanation, however.
>
Considering this reply and others, I've made some time to actually sit
down and *carefully* watch what I'm doing.
I started from scratch, so everything is clean, and ran iASL in
compatibility mode (-oa flag enabled). I kept a log of everything I did.
I figured it would help if folk didn't have to second-guess my thought
processes. Summary below - exact commands and output available.
I have discovered that decompiling seems to remove "external"
declarations.
All other changes involve substituting for "ones" and such like with the
appropriate, explicit, form needed (0x01 or 0x0001 or whatever). This
doesn't happen if iASL is run in compatibility mode.
The missing variables were:
\_PR.CPU0._PPC
Z007
If I just stick external statements for these at the top of the code, I
get the following from the compiler:
No back ptr to Op: type 8
No back ptr to Op: type 8
dsdt.dsl 150: Processor (CPU0, 0x00, 0x00008010, 0x06) {}
Error 1054 - ^ Name already exists in scope
(CPU0)
If I put the external statement after line 150 - then there are no
compiler errors. (To what extent does this matter? Is there a better way
of accommodating this?)
The remaining kernel errors are:
[4342789.367000] ACPI-0508: *** Error: Method execution failed
[\_SB_.BAT1._BST] (Node cbed7be0), AE_NOT_FOUND
[4342819.371000] ACPI-0362: *** Error: Looking up [Z007] in
namespace, AE_NOT_FOUND
This would seem to suggest that the external statements are removed by
decompiling (rather than compiling - whew), and that Z007 does not exist
anywhere.[1]
Perhaps the first error is the infamous acer smart-battery issue?
(Later: I see Acer 3002WLMi in the readme for the patch-set -
http://sourceforge.net/projects/sbs-linux/ - but not 3003LC)
Anyway - it would appear I have a dsdt suitable for including in the
archive...
[1] From previous discussions - there is a patch waiting, and I can try
the latest vanilla kernel with all the acpi patches.
next prev parent reply other threads:[~2006-01-30 2:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-01-25 17:55 reality check: dsdt.dsl -> compile -> DSDT.aml -> decompile ->DSDT.dsl but dsdt.dsl neq DSDT.dsl Moore, Robert
2006-01-30 2:53 ` Simon Bridge [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-01-26 10:22 Suietov, Fiodor F
2006-01-24 22:39 reality check: dsdt.dsl -> compile -> DSDT.aml -> decompile -> DSDT.dsl " Moore, Robert
2006-01-25 3:41 ` Simon Bridge
2006-01-22 1:52 Simon Bridge
2006-01-24 9:11 ` 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=1138589618.31634.21.camel@localhost.localdomain \
--to=corwin@ihug.co.nz \
--cc=linux-acpi@vger.kernel.org \
--cc=robert.moore@intel.com \
--cc=trenn@suse.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox