Re: [Qemu-arm] [Qemu-devel] [PATCH v9 0/5] add ACPI node for fw_cfg on pc and arm

["Michael S. Tsirkin" <mst@redhat.com>]

On Tue, Feb 23, 2016 at 04:52:16PM +0100, Igor Mammedov wrote:
> On Tue, 23 Feb 2016 11:39:05 +0100
> Gerd Hoffmann <kraxel@redhat.com> wrote:
> 
> > On Di, 2016-02-23 at 10:48 +0100, Igor Mammedov wrote:
> > > On Tue, 23 Feb 2016 09:51:52 +0100
> > > Gerd Hoffmann <kraxel@redhat.com> wrote:
> > >   
> > > > On Mo, 2016-02-22 at 13:30 +0100, Igor Mammedov wrote:  
> > > > > On Fri, 19 Feb 2016 13:20:24 -0500
> > > > > "Gabriel L. Somlo" <somlo@cmu.edu> wrote:
> > > > > 
> > > > > Gerd,
> > > > > 
> > > > > if you are going to apply it, you'll need to update
> > > > > expected tables for acpi test (rebuild-expected-aml.sh)
> > > > > as a commit on top of this series.    
> > > > 
> > > > Sure?
> > > > 
> > > > Applied the series, ran "make check", no errors.  
> > > it's a warning so far not a hard error but it's still there.  
> > 
> > Checked again, no warning printed here ...
> it works for me, I get something like:
> GTESTER check-qtest-x86_64
> acpi-test: Warning! DSDT mismatch. Actual [asl:/tmp/asl-66EZCY.dsl, aml:/tmp/aml-MBFZCY], Expected [asl:/tmp/asl-N1N0CY.dsl, aml:tests/acpi-test-data/q35/DSDT.bridge].

is iasl installed on the build system?
This test is skipped if there's no iasl.

> 
> > 
> > Also "git diff" doesn't list any changes after running the rebuild
> > script.
> it work for as well,
> just a wild guess, may be test for some reason doesn't use qemu with patches applied
> that would result in test not seeing any changes.
> 
> > But when booting a live iso I can see the new QEMU0002 device listed
> > in /proc/ioports.
> > 
> > Any clues what is going on here?
> > 
> > cheers,
> >   Gerd
> > 
> >