Re: [Qemu-devel] [PULL 14/15] qemu-char: add logfile facility to all chardev backends

[Markus Armbruster]

"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.