Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Eduardo Habkost
On Tue, Oct 10, 2017 at 01:56:55PM +0100, Peter Maydell wrote: > On 10 October 2017 at 13:52, Eduardo Habkost wrote: > > This shouldn't happen if CONFIG_IASL is not defined. Did you run > > configure again after removing iasl? > > I've neither added nor removed iasl from the build systems > in q

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Dou Liyang
Hi Peter, At 10/10/2017 08:56 PM, Peter Maydell wrote: On 10 October 2017 at 13:52, Eduardo Habkost wrote: This shouldn't happen if CONFIG_IASL is not defined. Did you run configure again after removing iasl? I've neither added nor removed iasl from the build systems in question. Pull reque

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Dou Liyang
Hi Peter, At 10/10/2017 05:40 PM, Peter Maydell wrote: On 10 October 2017 at 08:41, Dou Liyang wrote: If we want do the "bios-tables-test", we need iasl support. otherwise, when I run "make V=1 check" without iasl, the test will report error: This works in master at the moment, I think (whe

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Dou Liyang
Hi Eduardo, At 10/10/2017 08:52 PM, Eduardo Habkost wrote: This shouldn't happen if CONFIG_IASL is not defined. Did you run configure again after removing iasl? Oops, Yes, the result above is in CONFIG_IASL = iasl. :-( I have been aware of this problem. I am waiting the result of the test w

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Peter Maydell
On 10 October 2017 at 13:52, Eduardo Habkost wrote: > This shouldn't happen if CONFIG_IASL is not defined. Did you run > configure again after removing iasl? I've neither added nor removed iasl from the build systems in question. Pull request tests just apply the pull and run make/make check. t

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Eduardo Habkost
On Tue, Oct 10, 2017 at 03:41:56PM +0800, Dou Liyang wrote: > Hi Eduardo, > > I couldn't reproduce the failure too. But, I have some concern below. > > At 10/06/2017 11:37 PM, Peter Maydell wrote: > > On 6 October 2017 at 16:27, Eduardo Habkost wrote: > > > On Fri, Oct 06, 2017 at 01:17:42PM +01

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Peter Maydell
On 10 October 2017 at 08:41, Dou Liyang wrote: > If we want do the "bios-tables-test", we need iasl support. > > otherwise, when I run "make V=1 check" without iasl, the test > will report error: This works in master at the moment, I think (whether by skipping the test or by using pre-compiled ta

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-10 Thread Dou Liyang
Hi Eduardo, I couldn't reproduce the failure too. But, I have some concern below. At 10/06/2017 11:37 PM, Peter Maydell wrote: On 6 October 2017 at 16:27, Eduardo Habkost wrote: On Fri, Oct 06, 2017 at 01:17:42PM +0100, Peter Maydell wrote: This fails tests, I'm afraid: [...] acpi-test: Wa

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-06 Thread Peter Maydell
On 6 October 2017 at 16:27, Eduardo Habkost wrote: > On Fri, Oct 06, 2017 at 01:17:42PM +0100, Peter Maydell wrote: >> This fails tests, I'm afraid: > [...] >> acpi-test: Warning! DSDT mismatch. Actual [asl:/tmp/asl-6QYK7Y.dsl, >> aml:/tmp/aml-E8YK7Y], Expected [asl:/tmp/asl-1H1I7Y.dsl, >> aml:tes

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-06 Thread Eduardo Habkost
On Fri, Oct 06, 2017 at 01:17:42PM +0100, Peter Maydell wrote: > On 5 October 2017 at 21:36, Eduardo Habkost wrote: > > The following changes since commit d8f932cc696250cb740240d668b39df5fbb2d5a0: > > > > Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' > > into staging

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-06 Thread Thomas Huth
On 06.10.2017 14:17, Peter Maydell wrote: [...] > > As an aside, these acpi tests are a bit noisy with > V=1, especially the extra blank lines. > FWIW, I suggested a patch here to fix that: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg05404.html ... but apparently Igor did not like

Re: [Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-06 Thread Peter Maydell
On 5 October 2017 at 21:36, Eduardo Habkost wrote: > The following changes since commit d8f932cc696250cb740240d668b39df5fbb2d5a0: > > Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' > into staging (2017-10-05 16:54:29 +0100) > > are available in the git repository at:

[Qemu-devel] [PULL 0/9] x86 and machine queue, 2017-10-05

2017-10-05 Thread Eduardo Habkost
The following changes since commit d8f932cc696250cb740240d668b39df5fbb2d5a0: Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' into staging (2017-10-05 16:54:29 +0100) are available in the git repository at: git://github.com/ehabkost/qemu.git tags/x86-and-machine-pul