From: Alon Levy <alevy@redhat.com>
To: qemu-devel@nongnu.org, lcapitulino@redhat.com, kraxel@redhat.com,
armbru@redhat.com
Subject: [Qemu-devel] [PATCH 3/4] monitor, console: add QEVENT_SCREEN_DUMP_COMPLETE
Date: Fri, 24 Feb 2012 23:22:04 +0200 [thread overview]
Message-ID: <1330118525-14522-3-git-send-email-alevy@redhat.com> (raw)
In-Reply-To: <1330118525-14522-1-git-send-email-alevy@redhat.com>
Signed-off-by: Alon Levy <alevy@redhat.com>
---
QMP/qmp-events.txt | 14 ++++++++++++++
console.c | 18 ++++++++++++++++++
console.h | 1 +
monitor.c | 2 ++
monitor.h | 1 +
5 files changed, 36 insertions(+), 0 deletions(-)
diff --git a/QMP/qmp-events.txt b/QMP/qmp-events.txt
index 9286af5..0d1c9f4 100644
--- a/QMP/qmp-events.txt
+++ b/QMP/qmp-events.txt
@@ -335,3 +335,17 @@ Example:
"len": 10737418240, "offset": 134217728,
"speed": 0 },
"timestamp": { "seconds": 1267061043, "microseconds": 959568 } }
+
+SCREEN_DUMP_COMPLETE
+--------------------
+
+Emitted when screen-dump-async completes.
+
+Data:
+
+- "filename": Name of file containing screen dump (json-string)
+
+Example:
+
+{ "event": "SCREEN_DUMP_COMPLETE",
+ "data": { "filename": "/tmp/a.ppm" } }
diff --git a/console.c b/console.c
index 6a463f5..6750538 100644
--- a/console.c
+++ b/console.c
@@ -24,6 +24,8 @@
#include "qemu-common.h"
#include "console.h"
#include "qemu-timer.h"
+#include "qjson.h"
+#include "monitor.h"
//#define DEBUG_CONSOLE
#define DEFAULT_BACKSCROLL 512
@@ -1707,3 +1709,19 @@ PixelFormat qemu_default_pixelformat(int bpp)
}
return pf;
}
+
+void monitor_protocol_screen_dump_complete_event(const char *filename)
+{
+ QObject *event_data;
+ /*
+ * TODO: Only good for a single user, or a very aware user
+ * that changes the filename to distinguish between different
+ * screendumps.
+ * Should include support for different monitors on the same device,
+ * and for different devices. Got stuck trying to figure a canonical string
+ * represnetation of a device.
+ */
+ event_data = qobject_from_jsonf("{ 'filename': %s }", filename);
+ monitor_protocol_event(QEVENT_SCREEN_DUMP_COMPLETE, event_data);
+ qobject_decref(event_data);
+}
diff --git a/console.h b/console.h
index a95b581..c22803c 100644
--- a/console.h
+++ b/console.h
@@ -353,6 +353,7 @@ void vga_hw_update(void);
void vga_hw_invalidate(void);
void vga_hw_screen_dump(const char *filename);
void vga_hw_text_update(console_ch_t *chardata);
+void monitor_protocol_screen_dump_complete_event(const char *filename);
int is_graphic_console(void);
int is_fixedsize_console(void);
diff --git a/monitor.c b/monitor.c
index 953e748..1a65c41 100644
--- a/monitor.c
+++ b/monitor.c
@@ -493,6 +493,8 @@ void monitor_protocol_event(MonitorEvent event, QObject *data)
break;
case QEVENT_WAKEUP:
event_name = "WAKEUP";
+ case QEVENT_SCREEN_DUMP_COMPLETE:
+ event_name = "SCREEN_DUMP_COMPLETE";
break;
default:
abort();
diff --git a/monitor.h b/monitor.h
index 0d49800..227ebf2 100644
--- a/monitor.h
+++ b/monitor.h
@@ -41,6 +41,7 @@ typedef enum MonitorEvent {
QEVENT_DEVICE_TRAY_MOVED,
QEVENT_SUSPEND,
QEVENT_WAKEUP,
+ QEVENT_SCREEN_DUMP_COMPLETE,
QEVENT_MAX,
} MonitorEvent;
--
1.7.9.1
next prev parent reply other threads:[~2012-02-24 21:22 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-24 21:22 [Qemu-devel] [PATCH 1/4] qapi-schema: fix typos and explain 'spice' auth Alon Levy
2012-02-24 21:22 ` [Qemu-devel] [PATCH 2/4] qjson.h: include compiler.h for GCC_FMT_ATTR Alon Levy
2012-02-28 19:58 ` Luiz Capitulino
2012-02-28 20:49 ` Alon Levy
2012-02-24 21:22 ` Alon Levy [this message]
2012-02-28 20:01 ` [Qemu-devel] [PATCH 3/4] monitor, console: add QEVENT_SCREEN_DUMP_COMPLETE Luiz Capitulino
2012-02-28 20:51 ` Alon Levy
2012-02-29 14:38 ` Luiz Capitulino
2012-02-24 21:22 ` [Qemu-devel] [PATCH 4/4] add qmp screendump-async Alon Levy
2012-02-24 22:40 ` Anthony Liguori
2012-02-25 8:46 ` Alon Levy
2012-02-28 20:10 ` Luiz Capitulino
2012-02-29 7:49 ` Gerd Hoffmann
2012-02-29 14:52 ` Luiz Capitulino
2012-02-29 8:15 ` Alon Levy
2012-02-29 14:58 ` Luiz Capitulino
2012-02-29 15:37 ` Alon Levy
2012-02-28 20:05 ` Luiz Capitulino
2012-02-29 8:39 ` Alon Levy
2012-02-29 14:59 ` Luiz Capitulino
2012-03-05 13:40 ` Alon Levy
2012-03-05 13:45 ` Alon Levy
2012-02-28 8:05 ` [Qemu-devel] [PATCH 1/4] qapi-schema: fix typos and explain 'spice' auth Alon Levy
2012-02-28 19:57 ` Luiz Capitulino
2012-02-29 8:40 ` Alon Levy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1330118525-14522-3-git-send-email-alevy@redhat.com \
--to=alevy@redhat.com \
--cc=armbru@redhat.com \
--cc=kraxel@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).