qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alon Levy <alevy@redhat.com>
To: qemu-devel@nongnu.org
Cc: amit.shah@redhat.com, hdegoede@redhat.com, aliguori@us.ibm.com,
	kraxel@redhat.com
Subject: [Qemu-devel] [PATCH v2 0/4] for spice post load char device hook
Date: Wed, 20 Mar 2013 11:55:10 +0200	[thread overview]
Message-ID: <1363773314-32332-1-git-send-email-alevy@redhat.com> (raw)

This reworks my former patch (http://patchwork.ozlabs.org/patch/227678/ - sorry
Hans, can't find the version you posted) per Gerd's suggestion. Specifically it
adds a new qemu_chr_fe_post_load api that is called by the front end and
implemented by the backend. virtio-console implements it, adding it's own hook
which is called by virtio-serial-bus upon post_load from the timer, so that
qemu_chr_fe_post_load is called when the vm is already in the running state.
This makes the spice-qemu-char usage very simple by not requiring yet another
timer.

Note about the added api: I decided to pass "connected" via
qemu_chr_fe_post_load in order not to introduce more api on qemu_chr_fe_.. for
state querying, like qemu_chr_fe_is_connected, since I'm not sure it would make
sense for other frontends.

v1 wasn't completely sent to the list, mistakenly sent before squashing one
patch.

Alon Levy (4):
  char: add a post_load callback
  virtio-serial: add a post_load callback implemented by port
  virtio-console: implement post_load to call to qemu_chr_fe_post_load
  spice-qemu-char: register interface on post load

 hw/virtio-console.c    | 11 +++++++++++
 hw/virtio-serial-bus.c |  5 +++++
 hw/virtio-serial.h     |  2 ++
 include/char/char.h    | 12 ++++++++++++
 qemu-char.c            |  7 +++++++
 spice-qemu-char.c      |  9 +++++++++
 6 files changed, 46 insertions(+)

-- 
1.8.1.4

             reply	other threads:[~2013-03-20  9:55 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-20  9:55 Alon Levy [this message]
2013-03-20  9:55 ` [Qemu-devel] [PATCH 1/4] char: add a post_load callback Alon Levy
2013-03-20 13:08   ` Anthony Liguori
2013-03-20 16:59     ` Alon Levy
2013-03-21  6:53       ` Gerd Hoffmann
2013-03-21  8:54         ` Alon Levy
2013-03-20 17:05     ` Alon Levy
2013-03-20 18:59       ` Anthony Liguori
2013-03-21  8:27         ` Hans de Goede
2013-03-21  8:36           ` Hans de Goede
2013-03-21 16:35         ` [Qemu-devel] [PATCH v3 0/2] spice-qemu-char fix agent mouse after migration Alon Levy
2013-03-21 16:35           ` [Qemu-devel] [PATCH 1/2] char: add qemu_chr_be_is_fe_connected Alon Levy
2013-03-21 18:18             ` Anthony Liguori
2013-03-21 18:35               ` Alon Levy
2013-03-21 19:24                 ` Anthony Liguori
2013-03-21 21:55                   ` Alon Levy
2013-03-21 22:05                     ` Alon Levy
2013-03-22  7:56               ` Hans de Goede
2013-03-22 13:50                 ` Anthony Liguori
2013-03-22 15:53                   ` Gerd Hoffmann
2013-03-22 16:50                   ` Hans de Goede
2013-03-22 17:11                     ` Anthony Liguori
2013-03-24 12:37                       ` Hans de Goede
2013-03-22  8:25               ` Gerd Hoffmann
2013-03-22  8:58                 ` Hans de Goede
2013-03-22 13:33                 ` Anthony Liguori
2013-03-21 16:35           ` [Qemu-devel] [PATCH 2/2] spice-qemu-char: register interface on post load Alon Levy
2013-03-22  8:07             ` Hans de Goede
2013-03-22  8:16               ` Alon Levy
2013-03-22  8:55                 ` Hans de Goede
2013-03-20  9:55 ` [Qemu-devel] [PATCH 2/4] virtio-serial: add a post_load callback implemented by port Alon Levy
2013-03-20  9:55 ` [Qemu-devel] [PATCH 3/4] virtio-console: implement post_load to call to qemu_chr_fe_post_load Alon Levy
2013-03-20  9:55 ` [Qemu-devel] [PATCH 4/4] spice-qemu-char: register interface on post load 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=1363773314-32332-1-git-send-email-alevy@redhat.com \
    --to=alevy@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=amit.shah@redhat.com \
    --cc=hdegoede@redhat.com \
    --cc=kraxel@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).