* [Qemu-devel] [PATCH] hmp: Print \n after [not inserted]
@ 2013-10-03 1:08 Lucas Meneghel Rodrigues
2013-10-03 1:30 ` Lucas Meneghel Rodrigues
0 siblings, 1 reply; 2+ messages in thread
From: Lucas Meneghel Rodrigues @ 2013-10-03 1:08 UTC (permalink / raw)
To: qemu-devel; +Cc: Lucas Meneghel Rodrigues, Stefan Hajnoczi
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.
CC: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
---
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
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [PATCH] hmp: Print \n after [not inserted]
2013-10-03 1:08 [Qemu-devel] [PATCH] hmp: Print \n after [not inserted] Lucas Meneghel Rodrigues
@ 2013-10-03 1:30 ` Lucas Meneghel Rodrigues
0 siblings, 0 replies; 2+ messages in thread
From: Lucas Meneghel Rodrigues @ 2013-10-03 1:30 UTC (permalink / raw)
To: Lucas Meneghel Rodrigues, qemu-devel; +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 <stefanha@redhat.com>
> Signed-off-by: Lucas Meneghel Rodrigues <lmr@redhat.com>
> ---
> 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
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-10-03 1:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-03 1:08 [Qemu-devel] [PATCH] hmp: Print \n after [not inserted] Lucas Meneghel Rodrigues
2013-10-03 1:30 ` Lucas Meneghel Rodrigues
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).