[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 14/15] qemu-char: add logfile facility to all cha
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 14/15] qemu-char: add logfile facility to all chardev backends |
Date: |
Fri, 12 Feb 2016 18:12:46 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
"Daniel P. Berrange" <address@hidden> writes:
> On Fri, Feb 12, 2016 at 05:49:38PM +0100, Markus Armbruster wrote:
>> "Daniel P. Berrange" <address@hidden> writes:
>>
>> > On Thu, Jan 21, 2016 at 07:16:25AM +0100, Hervé Poussineau wrote:
>> >> Hi,
>> >>
>> >> This patch (commit d0d7708ba29cbcc343364a46bff981e0ff88366f) regresses
>> >> the following command line:
>> >> qemu-system-i386 -nodefaults -chardev vc,id=mon0 -mon chardev=mon0
>> >>
>> >> Before:
>> >> [nothing is print on console]
>> >>
>> >> After:
>> >> QEMU 2.5.50 monitor - type 'help' for more information
>> >> (qemu)
>> >>
>> >> In both cases, a working vc is created in the GTK+ UI.
>> >>
>> >> Reverting the commit (and fixing the trivial conflict) makes things
>> >> work again.
>> >
>> > Thanks for the heads-up, I'll investigate and submit a fix
>>
>> For the record, it also breaks ivshmem-test when the slow tests are
>> included. I'd be willing to test your fix; got a pointer for me?
>
> https://lists.gnu.org/archive/html/qemu-devel/2016-02/msg02738.html
No luck. Please try the following reproducer:
$ make tests/ivshmem-test
$ QTEST_QEMU_BINARY='x86_64-softmmu/qemu-system-x86_64'
QTEST_QEMU_IMG=qemu-img MALLOC_PERTURB_=${MALLOC_PERTURB_:-$((RANDOM % 255 +
1))} gtester -k --verbose -m slow tests/ivshmem-test
You can instead run make check SPEED=slow, but that runs many more
tests.