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

[Igor Mammedov]

On Tue, 23 Feb 2016 11:39:05 +0100
Gerd Hoffmann <address@hidden> wrote:

> On Di, 2016-02-23 at 10:48 +0100, Igor Mammedov wrote:
> > On Tue, 23 Feb 2016 09:51:52 +0100
> > Gerd Hoffmann <address@hidden> 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" <address@hidden> 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].


> 
> 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
> 
>