[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qtest: Use -display none by default
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH] qtest: Use -display none by default |
Date: |
Mon, 18 Nov 2013 17:54:33 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 |
Am 18.11.2013 17:48, schrieb Stefan Hajnoczi:
> On Mon, Nov 18, 2013 at 05:36:34PM +0100, Andreas Färber wrote:
>> This avoids each test needing to add it to suppress windows popping up.
>>
>> Signed-off-by: Andreas Färber <address@hidden>
>> ---
>> tests/boot-order-test.c | 2 +-
>> tests/endianness-test.c | 6 +++---
>> tests/fw_cfg-test.c | 3 +--
>> tests/hd-geo-test.c | 2 +-
>> tests/i440fx-test.c | 2 +-
>> tests/libqtest.c | 1 +
>> tests/m48t59-test.c | 2 +-
>> tests/qom-test.c | 2 +-
>> tests/rtc-test.c | 2 +-
>> tests/tmp105-test.c | 2 +-
>> 10 files changed, 12 insertions(+), 12 deletions(-)
>
> I avoided making -display none common because tests may wish to have a
> display. But at the moment no test needs a display so I guess this is
> fine.
I was under the assumption that -display none -display sdl would work
just like mst's overriding the default -machine accel=qtest with
-machine accel=tcg?
Anyway, we could choose to go with your less invasive version for 1.7
and apply an adjusted version of this one for 1.8.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg