From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34812) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VRXkK-0004gz-Ab for qemu-devel@nongnu.org; Wed, 02 Oct 2013 21:30:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VRXkE-0003Wc-B6 for qemu-devel@nongnu.org; Wed, 02 Oct 2013 21:30:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:36946) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VRXkE-0003WT-3b for qemu-devel@nongnu.org; Wed, 02 Oct 2013 21:30:22 -0400 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 (8.14.4/8.14.4) with ESMTP id r931UJZH014631 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 2 Oct 2013 21:30:20 -0400 Message-ID: <524CC89B.40308@redhat.com> Date: Wed, 02 Oct 2013 22:30:03 -0300 From: Lucas Meneghel Rodrigues MIME-Version: 1.0 References: <1380762514-15096-1-git-send-email-lmr@redhat.com> In-Reply-To: <1380762514-15096-1-git-send-email-lmr@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] hmp: Print \n after [not inserted] List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Lucas Meneghel Rodrigues , qemu-devel@nongnu.org Cc: Stefan Hajnoczi On 10/02/2013 10:08 PM, Lucas Meneghel Rodrigues wrote: > I've noticed this when virt-test QEMU monitor protocol > code was getting all confused with output like: > > 'Removable device: not locked, tray closed\n [not inserted](qemu)' > > Since it was breaking some assumptions on that code. I've > fixed the prompt matching code to be more lenient, but here > the human monitor is supposed to print the newline anyway. Oh well, it seems that we're not supposed to have the newline there by the look of the resulting output. Please ignore this patch. > CC: Stefan Hajnoczi > Signed-off-by: Lucas Meneghel Rodrigues > --- > hmp.c | 2 +- > roms/seabios | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hmp.c b/hmp.c > index 5891507..2d2e5f8 100644 > --- a/hmp.c > +++ b/hmp.c > @@ -367,7 +367,7 @@ void hmp_info_block(Monitor *mon, const QDict *qdict) > info->value->inserted->iops_wr_max, > info->value->inserted->iops_size); > } else { > - monitor_printf(mon, " [not inserted]"); > + monitor_printf(mon, " [not inserted]\n"); > } > > if (verbose) { > diff --git a/roms/seabios b/roms/seabios > index ece025f..7093aa5 160000 > --- a/roms/seabios > +++ b/roms/seabios > @@ -1 +1 @@ > -Subproject commit ece025f5980bae88fa677bc9c0d24d2e580e205d > +Subproject commit 7093aa58046f8685025b0198708e7768733a017d >