qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functi


From: Marcel Apfelbaum
Subject: Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality
Date: Thu, 26 Dec 2013 19:15:00 +0200

On Thu, 2013-12-26 at 18:51 +0200, Michael S. Tsirkin wrote:
> On Thu, Dec 26, 2013 at 06:12:25PM +0200, Michael S. Tsirkin wrote:
> > On Thu, Dec 26, 2013 at 05:58:09PM +0200, Marcel Apfelbaum wrote:
> > > On Thu, 2013-12-26 at 17:41 +0200, Michael S. Tsirkin wrote:
> > > > On Thu, Dec 26, 2013 at 04:09:58PM +0200, Marcel Apfelbaum wrote:
> > > > > On Thu, 2013-12-26 at 15:53 +0200, Michael S. Tsirkin wrote:
> > > > > > On Thu, Dec 26, 2013 at 03:24:28PM +0200, Marcel Apfelbaum wrote:
> > > > > > > On Thu, 2013-12-26 at 14:46 +0200, Michael S. Tsirkin wrote:
> > > > > > > > On Thu, Dec 26, 2013 at 02:19:48PM +0200, Marcel Apfelbaum 
> > > > > > > > wrote:
> > > > > > > > > Acpi unit-test will fail every time the acpi tables change.
> > > > > > > > > 
> > > > > > > > > The series adds a script that rebuilds the expected aml 
> > > > > > > > > files, so the test
> > > > > > > > > will pass. It also validates the modifications.
> > > > > > > > > 
> > > > > > > > > The acpi unit test will rebuild the aml tables if 
> > > > > > > > > TEST_ACPI_REBUILD_AML
> > > > > > > > > environment variable is set.
> > > > > > > > 
> > > > > > > > OK I had to fix some trailing whitespace, otherwise
> > > > > > > > looks good.
> > > > > > > > I applied it all, pls check the pci branch.
> > > > > > > Thanks!
> > > > > > > I pulled the pci branch, but I did not see the patches yet. 
> > > > > > > I used git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git,
> > > > > > > pci branch.
> > > > > > > 
> > > > > > > Thanks,
> > > > > > > Marcel
> > > > > > 
> > > > > > 
> > > > > > Actually had to revert. Build fails with recent gcc:
> > > > > Strange, I update the gcc to the Fedora's latest 4.8.2 and
> > > > > it still doesn't happen.
> > > > > Anyway, I use qemu_write_full as advised and resent.
> > > > > 
> > > > > Thanks,
> > > > > Marcel
> > > > 
> > > > OK I applied this for now but I see an issue: iasl crashes below.
> > > Thanks!
> > > By the way, I resend the whole series, I hope you used it.
> > 
> > yes pushed, take a look
> > 
> > > > I think this happens when we run it on a file
> > > > it does not understand.
> > > > It's an iasl bug but would be nice to work around
> > > > it in some way (and it would be nice to report to
> > > > robert moore)
> > > 
> > > I cannot reproduce it, I'll try anyway. 
> > > By the way, the test checks if iasl exits with error,
> > > so it should handle this gracefully. (I hope!)
> > > 
> > > Regarding the IASL project, I'll see if I can open a bug,
> > 
> > no, just send mail.
> > 
> > > (if I can reproduce it, otherwise I risk they'll ask some
> > > questions I cannot answer...)
> > > 
> > > Thanks,
> > > Marcel
> > 
> > This is on fedora 19, 32 bit.
> 
> I see that the issue is with HPET table.
> 
> iasl -e DSDT -e SSDT -d HPET
> 
> crashes for me
> 
> iasl -d HPET
> 
> does not crash
> 
> I think that only DSDT and SSDT need -e.
> 
> Just skip -e for anything that is not an SSDT
> and not a DSDT.
Thanks for the tip. I'll send a fix soon.
Marcel

> 






reply via email to

[Prev in Thread] Current Thread [Next in Thread]