qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: snabb-devel@googlegroups.com, qemu-devel@nongnu.org,
	Nikolay Nikolaev <n.nikolaev@virtualopensystems.com>,
	luke@snabb.co, a.motakis@virtualopensystems.com,
	tech@virtualopensystems.com
Subject: Re: [Qemu-devel] [PATCH v10 16/18] Add vhost-user protocol documentation
Date: Sun, 8 Jun 2014 18:05:41 +0300	[thread overview]
Message-ID: <20140608150541.GC5704@redhat.com> (raw)
In-Reply-To: <5390981F.6050405@redhat.com>

On Thu, Jun 05, 2014 at 10:17:35AM -0600, Eric Blake wrote:
> On 05/27/2014 06:06 AM, Nikolay Nikolaev wrote:
> > This document describes the basic message format used by vhost-user
> > for communication over a unix domain socket. The protocol is based
> > on the existing ioctl interface used for the kernel version of vhost.
> > 
> > Signed-off-by: Antonios Motakis <a.motakis@virtualopensystems.com>
> > Signed-off-by: Nikolay Nikolaev <n.nikolaev@virtualopensystems.com>
> > ---
> >  docs/specs/vhost-user.txt |  261 +++++++++++++++++++++++++++++++++++++++++++++
> >  1 file changed, 261 insertions(+)
> >  create mode 100644 docs/specs/vhost-user.txt
> > 
> > diff --git a/docs/specs/vhost-user.txt b/docs/specs/vhost-user.txt
> > new file mode 100644
> > index 0000000..e7f43f5
> > --- /dev/null
> > +++ b/docs/specs/vhost-user.txt
> > @@ -0,0 +1,261 @@
> > +Vhost-user Protocol
> 
> Pre-existing problem throughout this directory, but it might be nice to
> assert copyright and assign a license to this file.
> 
> 
> > +
> > +The protocol defines 2 sides of the communication, master and slave. Master is
> > +the application that shares it's virtqueues, in our case QEMU. Slave is the
> 
> s/it's/its/ (remember, "it's" only works if "it is" could be used in the
> same context)
> 
> > +consumer of the virtqueues.
> > +
> > +In the current implementation QEMU is the Master, and the Slave is intended to
> > +be a software ethernet switch running in user space, such as Snabbswitch.
> 
> s/ethernet/Ethernet/
> 
> > +Communication
> > +-------------
> > +
> > +The protocol for vhost-user is based on the existing implementation of vhost
> > +for the Linux Kernel. Most messages that can be send via the Unix domain socket
> 
> s/send/sent/

I made these changes after applying the patch, thanks!

> -- 
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
> 

  reply	other threads:[~2014-06-08 15:05 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20140527120050.15172.94908.stgit@3820>
2014-06-04 19:30 ` [Qemu-devel] [PATCH v10 00/18] Vhost and vhost-net support for userspace based backends Michael S. Tsirkin
2014-06-04 20:03   ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev
     [not found] ` <20140527120330.15172.91211.stgit@3820>
2014-06-05 14:00   ` [Qemu-devel] [PATCH v10 01/18] Add kvm_eventfds_enabled function Paolo Bonzini
     [not found] ` <20140527120638.15172.80806.stgit@3820>
2014-06-05 14:37   ` [Qemu-devel] [PATCH v10 15/18] Add the vhost-user netdev backend to the command line Luiz Capitulino
2014-06-09 13:28     ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev
2014-06-09 13:31       ` Michael S. Tsirkin
2014-06-09 13:43         ` Nikolay Nikolaev
2014-06-05 16:08   ` [Qemu-devel] " Eric Blake
2014-06-09 21:19     ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev
2014-06-09 22:22       ` Eric Blake
     [not found] ` <20140527120651.15172.72895.stgit@3820>
2014-06-05 16:17   ` [Qemu-devel] [PATCH v10 16/18] Add vhost-user protocol documentation Eric Blake
2014-06-08 15:05     ` Michael S. Tsirkin [this message]
2014-06-08 15:12 ` [Qemu-devel] [PATCH v10 00/18] Vhost and vhost-net support for userspace based backends Michael S. Tsirkin
2014-06-09 10:14   ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev
     [not found] ` <20140527120718.15172.9772.stgit@3820>
2014-07-09 14:24   ` [Qemu-devel] [PATCH v10 18/18] Add qtest for vhost-user Kevin Wolf
2014-07-09 15:09     ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev
2014-07-11 18:35     ` [Qemu-devel] " Michael S. Tsirkin
2014-07-11 18:35       ` [Qemu-devel] [snabb-devel] " Nikolay Nikolaev

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=20140608150541.GC5704@redhat.com \
    --to=mst@redhat.com \
    --cc=a.motakis@virtualopensystems.com \
    --cc=eblake@redhat.com \
    --cc=luke@snabb.co \
    --cc=n.nikolaev@virtualopensystems.com \
    --cc=qemu-devel@nongnu.org \
    --cc=snabb-devel@googlegroups.com \
    --cc=tech@virtualopensystems.com \
    /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).