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

[Igor Mammedov]

On Tue, 23 Feb 2016 17:54:33 +0200
"Michael S. Tsirkin" <address@hidden> wrote:

> On Tue, Feb 23, 2016 at 04:52:16PM +0100, Igor Mammedov wrote:
> > 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].  
> 
> is iasl installed on the build system?
> This test is skipped if there's no iasl.
I've tried without iasl as well and it makes test fail
with error, so it's not Gerd's case.

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