From: "malattia@linux.it" <malattia@linux.it>
To: Lin Ming <ming.m.lin@intel.com>
Cc: "Moore, Robert" <robert.moore@intel.com>,
Len Brown <lenb@kernel.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>
Subject: Re: [patch 0/5] iASL patches in Debian
Date: Fri, 16 Jul 2010 06:29:57 +0900 [thread overview]
Message-ID: <20100715212956.GA28893@kamineko.org> (raw)
In-Reply-To: <1279071763.4404.5.camel@minggr.sh.intel.com>
On Wed, Jul 14, 2010 at 09:42:43AM +0800, Lin Ming wrote:
> On Thu, 2010-07-08 at 22:40 +0800, malattia@linux.it wrote:
> > On Thu, Jul 08, 2010 at 10:06:47PM +0000, Lin Ming wrote:
...
> > > > Compilation complete. 0 Errors, 83 Warnings, 18 Remarks, 0 Optimizations
> > > > ASL Input: MAIN.asl - 62 lines, 962044 bytes, 20265 keywords
> > > > -AML Output: oconst.aml - 255557 bytes, 862 named objects, 19403 executable opcodes
> > > > +AML Output: oconst.aml - 255575 bytes, 862 named objects, 19403 executable opcodes
> > >
> > > Why do the aml files have different size?
> > > Is it caused by big/small endian?
> >
> > They should be identical, possibly my patch is incomplete or not
> > correct. Out of the ~180 tests only the ones in the diff show
> > differences.
> > I'll have to check what is wrong on sparc64 for them.
>
> Hi,
>
> Did you find out why they are different?
>
> You can disassemble the aml files and see what the differences are.
haven't had the time to go after it yet. apologies.
But on a separate note, I assume there is interest in applying the
patches to the upstream tree (at least once they are complete).
Thanks!
--
mattia
:wq!
next prev parent reply other threads:[~2010-07-15 21:29 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-04 4:43 [patch 0/5] iASL patches in Debian malattia
2010-07-04 4:43 ` [patch 1/5] allow overriding only some of the CFLAGS malattia
2010-07-04 4:43 ` [patch 2/5] Small tweak to define uintptr_t and successfully build on debian/kfreebsd malattia
2010-07-04 4:43 ` [patch 3/5] Support compiling DSDT tables on big endian architectures malattia
2010-07-04 4:43 ` [patch 4/5] Add architectures that do not support unaligned access and fix aligment issues malattia
2010-07-04 4:43 ` [patch 5/5] Fix FTBFS on hurd-i386 malattia
2010-07-06 17:01 ` [patch 0/5] iASL patches in Debian Moore, Robert
2010-07-07 19:27 ` Moore, Robert
2010-07-07 22:07 ` malattia
2010-07-07 23:41 ` Moore, Robert
2010-07-08 13:41 ` malattia
2010-07-08 22:14 ` Lin Ming
2010-07-08 0:49 ` Lin Ming
2010-07-08 13:49 ` malattia
2010-07-08 22:06 ` Lin Ming
2010-07-08 14:40 ` malattia
2010-07-14 1:42 ` Lin Ming
2010-07-15 21:29 ` malattia [this message]
2010-07-08 14:44 ` Moore, Robert
-- strict thread matches above, loose matches on Subject: below --
2010-07-05 1:01 [Devel] " Lin Ming
2010-07-05 1:01 ` Lin Ming
2010-07-05 11:49 [Devel] " malattia
2010-07-05 11:49 ` malattia
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=20100715212956.GA28893@kamineko.org \
--to=malattia@linux.it \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=ming.m.lin@intel.com \
--cc=robert.moore@intel.com \
/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.