Re: [Xen-devel] [Xen-users] Xen 4.3 RC1 Test Day on freenode channel #xentest is today

[Dietmar Hahn <dietmar.hahn@xxxxxxxxxxxxxx>]

Am Mittwoch 08 Mai 2013, 13:15:33 schrieb Ian Campbell:
> On Wed, 2013-05-08 at 13:07 +0100, Dietmar Hahn wrote:
> > Am Mittwoch 08 Mai 2013, 10:10:37 schrieb Ian Campbell:
> > > On Wed, 2013-05-08 at 09:30 +0100, Dietmar Hahn wrote:
> > > > Am Mittwoch 08 Mai 2013, 08:35:07 schrieb Lars Kurth:
> > > > > Hi everybody,
> > > > > 
> > > > > we have the first test day for Xen 4.3 today.
> > > > > 
> > > > > For more information on Test Days and how to join see 
> > > > > http://wiki.xen.org/wiki/Xen_Test_Days
> > > > > For test instructions see 
> > > > > http://wiki.xen.org/wiki/Xen_4.3_RC1_test_instructions
> > > > 
> > > > I tried to build xen-4.3.0-rc1 on openSUSE 12.3 (x86_64)
> > > > As described in the wiki 
> > > > http://wiki.xen.org/wiki/Compiling_Xen_From_Source
> > > > I did:
> > > > 
> > > > # git clone -b 4.3.0-rc1 git://xenbits.xen.org/xen.git
> > > > # cd xen
> > > > # ./configure --libdir=/usr/lib64
> > > > # make dist
> > > > 
> > > > The compiling breaks in building seabios and at the moment I have no 
> > > > glue
> > > > where to look.
> > > 
> > > All lists apart from xen-devel to bcc, adding release manager.
> > > 
> > > Looks like a conflict with your version of iasl, which are you using?
> > 
> > # iasl -v
> > 
> > Intel ACPI Component Architecture
> > ASL Optimizing Compiler version 20121220-64
> > Copyright (c) 2000 - 2012 Intel Corporation
> > 
> > > I would wager this relates to this upstream fix:
> > > 
> > > commit 298fce95db03e41d511d1890f3e667c105ac1363
> > > Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
> > > Date:   Sun Dec 2 01:15:03 2012 -0500
> > > 
> > >     ACPI: Remove unnecessary Return(One) calls from q35 DSDT.
> > >     
> > >     Recent iasl compilers warn about spurious return calls.  Remove extra
> > >     return calls from the GPE event methods.  This is the q35 dsdt
> > >     equivalent of patch 475ce3f5f6a986877d637ac39f6ee23484ecc992.
> > >     
> > > Can you see if cherry picking that from
> > > git://git.seabios.org/seabios.git fixes the issue for you please. I
> > > don't think we even use the q35 DSDT so this is just a build issue.
> > 
> > This patch updates the file src/q35-acpi-dsdt.dsl which doesn't exist in
> > our version of seabios. But while looking at the commit messages I found
> > 475ce3f5f6a986877d637ac39f6ee23484ecc992.
> > This updates src/acpi-dsdt.dsl and the warnings
> > 
> > out/acpi-dsdt.dsl.i    578:             Return(0x01)
> > Warning  1104 -                                  ^ Reserved method should 
> > not return a value (_L00)
> > 
> > went away. With that patch in mind I fixed src/ssdt-pcihp.dsl and the 
> > warnings
> > 
> > out/ssdt-pcihp.dsl.i     12:  Method (_EJ0, 1) { Return(PCEJ(0x01)) } Name 
> > (_SUN, 0x01) }
> > Warning  1104 - Reserved method should not return a value ^  (_EJ0)
> > 
> > went away to
> 
> Excellent, George I think we should cherry-pick
> 475ce3f5f6a986877d637ac39f6ee23484ecc992 from upstream seabios for 4.3.
> 
> >  but the errors remain:
> 
> Sorry, I must have missed those in the original mail.
> 
> > I played a little bit with the seabios-dir/Makefile and found that all the
> > *.off files are empty and *.hex too. This caused  the errors
> > 
> > out/../src/acpi.c:417:34: error: âssdt_proc_endâ undeclared (first use in 
> > this function)
> > out/../src/acpi.c:417:34: note: each undeclared identifier is reported only 
> > once for each function it appears in
> > out/../src/acpi.c:417:34: error: âssdt_proc_startâ undeclared (first use in 
> > this function)
> > out/../src/acpi.c:439:26: error: âssdp_proc_amlâ undeclared (first use in 
> > this function)
> > out/../src/acpi.c:440:18: error: âssdt_proc_nameâ undeclared (first use in 
> > this function)
> > out/../src/acpi.c:443:18: error: âssdt_proc_idâ undeclared (first use in 
> > this function)
> > In file included from out/ccode32flat.o.tmp.c:45:0:
> > 
> > So the question is why are these empty.
> > Maybe it has to do with my iasl as this creates the *.lst files as input for
> > generating the *.off files
> 
> Could be related to
> 
>         commit f396871e3df9da14c65ab24a5a881674e06150a9
>         Author: Kevin O'Connor <kevin@xxxxxxxxxxxx>
>         Date:   Sun Feb 10 21:06:14 2013 -0500
>         
>             Update tools/acpi_extract.py to handle iasl 20130117 release.
>             
>             The latest iasl release produces "lst" output with "number:  " 
> instead
>             of "number....".  Update the tool to support both formats.
>             
>             Reported-by: Bo Yang <boyang@xxxxxxxx>
>             Signed-off-by: Kevin O'Connor <kevin@xxxxxxxxxxxx>

Yes this patch did the job!

>         
> Except that quotes a newer version than what you have.
> 
> Could you try building the upstream 1.7.2.1 release in your environment?

I'll try it but this will take a little bit of time.
Thanks.

Dietmar.

-- 
Company details: http://ts.fujitsu.com/imprint.html

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel