From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35874) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bhNtB-0001QC-6G for qemu-devel@nongnu.org; Tue, 06 Sep 2016 17:26:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bhNt9-0004R0-6y for qemu-devel@nongnu.org; Tue, 06 Sep 2016 17:26:40 -0400 References: <1473133253-17598-1-git-send-email-david@gibson.dropbear.id.au> <3ea1b88e-d2e0-3fca-8ea3-74433c741365@redhat.com> <5e9f1ff0-5c2b-74b4-2e7c-be9e8daf8a7f@redhat.com> From: Thomas Huth Message-ID: Date: Tue, 6 Sep 2016 23:26:34 +0200 MIME-Version: 1.0 In-Reply-To: <5e9f1ff0-5c2b-74b4-2e7c-be9e8daf8a7f@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL 00/66] ppc-for-2.8 queue 20160906 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , Peter Maydell , David Gibson Cc: peter.maydell@linearo.org, "Michael S. Tsirkin" , "qemu-ppc@nongnu.org" , Alexander Graf , QEMU Developers On 06.09.2016 22:55, Thomas Huth wrote: > On 06.09.2016 22:23, Paolo Bonzini wrote: >> >> >> On 06/09/2016 21:35, Thomas Huth wrote: >>> D'oh! I'll have a look... >>> >>>>> I also see warnings on non-KVM hosts during 'make check' which seem >>>>> to be produced by this new test: >>>>> >>>>> TEST: tests/boot-serial-test... (pid=2836) >>>>> /i386/boot-serial/isapc: >>>>> warning: TCG doesn't support requested feature: CPUID.01H:EDX.vme [bit >>>>> 1] >>> That happens because the test is running QEMU with the isapc machine. I >>> also get that warning message if I simply run the following on the >>> command line: >>> >>> $ qemu-system-x86_64 -M isapc >>> warning: TCG doesn't support requested feature: CPUID.01H:EDX.vme [bit 1] >>> >>> So the warning is likely there since quite a while already, just nobody >>> is running with -M isapc nowadays anymore, so nobody noticed this ... >>> Does anybody have got a clue how to fix that warning? >> >> Implement vme... :) > > Hmmm, I was rather looking for an easier solution. I think I also found > one: By using "-cpu qemu32", the warning does not occur anymore... David, since you likely got to respin this pull request anyway, could you please squash the following patch into my "tests: Check serial output of firmware boot of some machines" patch? This silences the warning about the unsupported vme TCG feature: diff --git a/tests/boot-serial-test.c b/tests/boot-serial-test.c index b36c6bf..d98c564 100644 --- a/tests/boot-serial-test.c +++ b/tests/boot-serial-test.c @@ -29,10 +29,10 @@ static testdef_t tests[] = { { "ppc64", "ppce500", "", "U-Boot" }, { "ppc64", "prep", "", "Open Hack'Ware BIOS" }, { "ppc64", "pseries", "", "Open Firmware" }, - { "i386", "isapc", "-device sga", "SGABIOS" }, + { "i386", "isapc", "-cpu qemu32 -device sga", "SGABIOS" }, { "i386", "pc", "-device sga", "SGABIOS" }, { "i386", "q35", "-device sga", "SGABIOS" }, - { "x86_64", "isapc", "-device sga", "SGABIOS" }, + { "x86_64", "isapc", "-cpu qemu32 -device sga", "SGABIOS" }, { "x86_64", "q35", "-device sga", "SGABIOS" }, { "s390x", "s390-ccw-virtio", "-nodefaults -device sclpconsole,chardev=serial0", "virtio device" }, Thanks, Thomas