netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: jiangyiwen <jiangyiwen@huawei.com>
Cc: Jason Wang <jasowang@redhat.com>,
	stefanha@redhat.com, stefanha@gmail.com, netdev@vger.kernel.org,
	kvm@vger.kernel.org, virtualization@lists.linux-foundation.org
Subject: Re: [RFC] Discuss about an new idea "Vsock over Virtio-net"
Date: Mon, 3 Dec 2018 20:31:45 -0500	[thread overview]
Message-ID: <20181203202441-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <5C049EC2.3080002@huawei.com>

On Mon, Dec 03, 2018 at 11:10:58AM +0800, jiangyiwen wrote:
> On 2018/11/30 21:40, Michael S. Tsirkin wrote:
> > On Fri, Nov 30, 2018 at 09:10:03PM +0800, Jason Wang wrote:
> >>
> >> On 2018/11/30 下午8:55, Jason Wang wrote:
> >>>
> >>> On 2018/11/30 下午8:52, Michael S. Tsirkin wrote:
> >>>>>>    If you want to compare it with
> >>>>>> something that would be TCP or QUIC.  The fundamental
> >>>>>> difference between
> >>>>>> virtio-vsock and e.g. TCP is that TCP operates in a packet
> >>>>>> loss environment.
> >>>>>> So they are using timers for reliability, and receiver is
> >>>>>> always free to
> >>>>>> discard any unacked data.
> >>>>> Virtio-net knows nothing above L2, so they are totally
> >>>>> transparent to device
> >>>>> itself. I still don't get why not using virtio-net instead.
> >>>>>
> >>>>>
> >>>>> Thanks
> >>>> Is your question why is virtio-vsock used instead of TCP on top of IP
> >>>> on top of virtio-net?
> >>>>
> >>>>
> >>>
> >>> No, my question is why not do vsock through virtio-net.
> >>>
> >>> Thanks
> >>>
> >>
> >> Just to clarify, it's not about vosck over ethernet, and it's not about
> >> inventing new features or APIs. It's probably something like:
> >>
> >> - Let virtio-net driver probe vsock device and do vosck specific things if
> >> needed to share as much codes.
> >>
> >> - A new kind of sockfd (which is vsock based) for vhost-net for it to do
> >> vsock specific things (hopefully it can be transparent).
> >>
> >> The change should be totally transparent to userspace applications.
> >>
> >> Thanks
> > 
> > Which code is duplicated between virtio vsock and virtio net right now?
> > 
> 
> Hi Michael,
> 
> AFAIK, there is almost no duplicate code between virtio vsock and virtio net now.
> 
> But, if virtio vsock wants to support mergeable rx buffer and multiqueue feature,
> it has some duplicate codes from virtio net. Based on it, we both think vsock
> may use virtio net as a transport channel, in this way, vsock can use some of
> virtio net great features.
> 
> Thanks,
> Yiwen.

What I would do is just copy some code and show a performance
benefit. If that works out it will be clearer which code
should be shared.

-- 
MST

  reply	other threads:[~2018-12-04  1:31 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-11-15  3:56 [RFC] Discuss about an new idea "Vsock over Virtio-net" jiangyiwen
2018-11-15  4:19 ` Jason Wang
2018-11-15  6:46   ` jiangyiwen
2018-11-15  6:49     ` jiangyiwen
2018-11-15  8:19     ` Jason Wang
2018-11-15  9:02       ` jiangyiwen
2018-11-15  9:21         ` Jason Wang
2018-11-16  2:32           ` jiangyiwen
2018-11-16  6:35             ` Jason Wang
2018-11-15  7:04 ` Michael S. Tsirkin
2018-11-15  7:38   ` jiangyiwen
2018-11-15  8:10     ` Michael S. Tsirkin
2018-11-15  8:27       ` Jason Wang
2018-11-15  8:38       ` jiangyiwen
2018-11-15  8:24   ` Jason Wang
2018-11-29 14:00     ` Michael S. Tsirkin
2018-11-30 12:45       ` Jason Wang
2018-11-30 12:52         ` Michael S. Tsirkin
2018-11-30 12:55           ` Jason Wang
2018-11-30 13:10             ` Jason Wang
2018-11-30 13:40               ` Michael S. Tsirkin
2018-12-03  3:10                 ` jiangyiwen
2018-12-04  1:31                   ` Michael S. Tsirkin [this message]
2018-12-04  2:21                     ` jiangyiwen
2018-12-04  4:08                       ` Michael S. Tsirkin
2018-12-04  6:01                         ` jiangyiwen
2018-11-30 13:34             ` Michael S. Tsirkin
2018-11-15  9:36 ` Yan Vugenfirer
2018-11-16  2:37   ` jiangyiwen

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=20181203202441-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=jasowang@redhat.com \
    --cc=jiangyiwen@huawei.com \
    --cc=kvm@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=stefanha@gmail.com \
    --cc=stefanha@redhat.com \
    --cc=virtualization@lists.linux-foundation.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).