qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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

  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).