qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Michael Roth <mdroth@linux.vnet.ibm.com>
To: "qemu-devel@nongnu.org Developers" <qemu-devel@nongnu.org>
Cc: Amit Shah <amit.shah@redhat.com>,
	Anthony Liguori <aliguori@linux.vnet.ibm.com>,
	Adam Litke <agl@us.ibm.com>
Subject: [Qemu-devel] virtio-serial semantics for binary data and guest agents
Date: Tue, 22 Feb 2011 16:40:55 -0600	[thread overview]
Message-ID: <4D643B77.60109@linux.vnet.ibm.com> (raw)

Hi everyone,

As some of you are aware we've been working for a few months now towards 
creating a qemu-specific guest agent to implement bi-directional RPCs 
between the host and the guest to support certain operations like 
copy/paste, guest-initiated shutdown, and basic file transfer.

Currently the protocol uses the xmlrpc-c library to marshall functions 
and arguments into a data format we can transport over 
virtio-serial/isa-serial via an HTTP-like protocol. Recently some 
concerns have been raised over pulling an external dependency such as 
xmlrpc-c into qemu, and we've been looking at some alternatives.

Some clear, well-defined approaches such as ASN.1/BER show much promise, 
but we currently lack a way to implement them cleanly due to the 
following drawbacks with virtio-serial which prevent us from being able 
to implement connection/session-oriented protocols:

If something in the guest is attempting to read/write from the 
virtio-serial device, and nothing is connected to virtio-serial's host 
character device (say, a socket)

1. writes will block until something connect()s, at which point the 
write will succeed

2. reads will always return 0 until something connect()s, at which point 
the reads will block until there's data

This makes it difficult (impossible?) to implement the notion of 
connect/disconnect or open/close over virtio-serial without layering 
another protocol on top using hackish things like length-encoded 
payloads or sentinel values to determine the end of one 
RPC/request/response/session and the start of the next.

For instance, if the host side disconnects, then reconnects before we 
read(), we may never get the read()=0, and our FD remains valid. Whereas 
with a tcp/unix socket our FD is no longer valid, and the read()=0 is an 
event we can check for at any point after the other end does a 
close/disconnect.

Or if the host side disconnects/closes before/while we write(), we 
block. If they reconnect, our write() succeeds, and they potentially end 
up with garbage meant for the previous process. With a tcp/unix socket, 
the write() will return an EPIPE indicating the FD is no longer valid.

Since virtio-serial is meant to be a general-purpose transport for raw 
binary data as well as a pv serial console, I wonder if a virtio-serial 
mode with semantics closer to tcp/unix sockets is necessary? Any thoughts?

Thanks,

Mike

             reply	other threads:[~2011-02-22 22:41 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-22 22:40 Michael Roth [this message]
2011-02-22 23:09 ` [Qemu-devel] Re: virtio-serial semantics for binary data and guest agents Anthony Liguori
2011-02-23  4:59 ` Amit Shah
2011-02-23 14:31   ` Michael Roth
2011-02-23 14:36     ` Michael Roth
2011-02-24 12:48     ` Amit Shah
2011-02-24 14:44       ` Anthony Liguori
2011-02-28 12:21         ` Amit Shah
2011-02-25 20:25       ` Michael Roth
2011-03-01  6:17         ` Amit Shah

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=4D643B77.60109@linux.vnet.ibm.com \
    --to=mdroth@linux.vnet.ibm.com \
    --cc=agl@us.ibm.com \
    --cc=aliguori@linux.vnet.ibm.com \
    --cc=amit.shah@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).