From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45003) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aMB2b-0003yC-78 for qemu-devel@nongnu.org; Thu, 21 Jan 2016 03:56:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aMB2X-0000ad-GR for qemu-devel@nongnu.org; Thu, 21 Jan 2016 03:56:29 -0500 Received: from mx1.redhat.com ([209.132.183.28]:36065) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aMB2X-0000aZ-Ab for qemu-devel@nongnu.org; Thu, 21 Jan 2016 03:56:25 -0500 Date: Thu, 21 Jan 2016 08:56:19 +0000 From: "Daniel P. Berrange" Message-ID: <20160121085619.GB19835@redhat.com> References: <1452873871-138914-1-git-send-email-pbonzini@redhat.com> <1452873871-138914-16-git-send-email-pbonzini@redhat.com> <56A077B9.3030209@reactos.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <56A077B9.3030209@reactos.org> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL 14/15] qemu-char: add logfile facility to all chardev backends Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?utf-8?B?SGVydsOp?= Poussineau Cc: Paolo Bonzini , qemu-devel@nongnu.org On Thu, Jan 21, 2016 at 07:16:25AM +0100, Herv=C3=A9 Poussineau wrote: > Hi, >=20 > This patch (commit d0d7708ba29cbcc343364a46bff981e0ff88366f) regresses = the following command line: > qemu-system-i386 -nodefaults -chardev vc,id=3Dmon0 -mon chardev=3Dmon0 >=20 > Before: > [nothing is print on console] >=20 > After: > QEMU 2.5.50 monitor - type 'help' for more information > (qemu) >=20 > In both cases, a working vc is created in the GTK+ UI. >=20 > Reverting the commit (and fixing the trivial conflict) makes things > work again. Thanks for the heads-up, I'll investigate and submit a fix Regards, Daniel --=20 |: http://berrange.com -o- http://www.flickr.com/photos/dberrange= / :| |: http://libvirt.org -o- http://virt-manager.or= g :| |: http://autobuild.org -o- http://search.cpan.org/~danberr= / :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vn= c :|