From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42350) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aUDkx-0000dP-4U for qemu-devel@nongnu.org; Fri, 12 Feb 2016 08:27:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aUDkt-0000Qv-Vo for qemu-devel@nongnu.org; Fri, 12 Feb 2016 08:27:31 -0500 Received: from mx1.redhat.com ([209.132.183.28]:60728) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aUDkt-0000Qm-Q0 for qemu-devel@nongnu.org; Fri, 12 Feb 2016 08:27:27 -0500 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 640E08E258 for ; Fri, 12 Feb 2016 13:27:27 +0000 (UTC) Date: Fri, 12 Feb 2016 14:27:23 +0100 From: Igor Mammedov Message-ID: <20160212142723.53e31c6b@nial.brq.redhat.com> In-Reply-To: <56BDD950.5000109@redhat.com> References: <20160211163539.506a5058@nial.brq.redhat.com> <20160212121526.599e75a6@nial.brq.redhat.com> <20160212111706.GD24725@redhat.com> <56BDD950.5000109@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] broken HMP command: info mtree List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org, dgilbert@redhat.com On Fri, 12 Feb 2016 14:08:32 +0100 Paolo Bonzini wrote: > On 12/02/2016 12:17, Daniel P. Berrange wrote: > > On Fri, Feb 12, 2016 at 12:15:26PM +0100, Igor Mammedov wrote: > >> On Thu, 11 Feb 2016 16:35:39 +0100 > >> Igor Mammedov wrote: > >> > >>> executing 'info mtree' from monitor prompt causes infinite loop > >>> printing it over and over. > >>> > >>> to reproduce build current master adn run: > >>> > >>> qemu-system-x86_64 -monitor stdio > >>> > >>> and then execute 'info mtree' in monitor prompt > >> > >> it bisects to: > >> > >> commit 9894dc0cdcc397ee5b26370bc53da6d360a363c2 > >> Author: Daniel P. Berrange > >> Date: Tue Jan 19 11:14:29 2016 +0000 > >> > >> char: convert from GIOChannel to QIOChannel > >> > >> In preparation for introducing TLS support to the TCP chardev > >> backend, convert existing chardev code from using GIOChannel > >> to QIOChannel. This simplifies the chardev code by removing > >> most of the OS platform conditional code for dealing with > >> file descriptor passing. > >> > >> Signed-off-by: Daniel P. Berrange > >> Message-Id: <1453202071-10289-3-git-send-email-berrange@redhat.com> > >> Signed-off-by: Paolo Bonzini > >> > >> > >> build with: > >> ./configure --target-list=x86_64-softmmu --enable-debug > >> on RHEL72ish host > >> > >> monitor output has to be stdio > > > > Sigh, so much pain from the chardev code. I'll investigate and send a > > suitable patch asap. > > Hmm, I cannot reproduce this though. Perhaps I'm affected because my stdout goes via remote ssh session. It looks like monitor tries to flush buffer but succeeds only partially and returns with EAGAIN and on the next flush attempt it tries to flush the same buffer again from the first byte again > > Paolo >