Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-04 Thread Yang, Zhiyong
Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for > server mode > > > > > -Original Message- > > From: Bie, Tiwei > > Sent: Wednesday, April 4, 2018 1:37 PM > > To: Yang, Zhiyong > > Cc: Tan, Jianfeng ; dev@dpdk.org;

Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-04 Thread Yang, Zhiyong
> -Original Message- > From: Bie, Tiwei > Sent: Wednesday, April 4, 2018 1:37 PM > To: Yang, Zhiyong > Cc: Tan, Jianfeng ; dev@dpdk.org; > maxime.coque...@redhat.com; tho...@monjalon.net; Wang, Zhihong > > Subject: Re: [PATCH v4 1/1] net/virtio-user: add support for server mode > > On

Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-03 Thread Tiwei Bie
On Tue, Apr 03, 2018 at 11:16:26PM +0800, Tan, Jianfeng wrote: > On 4/3/2018 8:20 PM, zhiyong.y...@intel.com wrote: [...] > > @@ -267,21 +270,27 @@ virtio_user_dev_setup(struct virtio_user_dev *dev) > > dev->vhostfds = NULL; > > dev->tapfds = NULL; > > Add a check here: > if (dev->is_serve

Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-03 Thread Tan, Jianfeng
> -Original Message- > From: Yang, Zhiyong > Sent: Wednesday, April 4, 2018 11:32 AM > To: Tan, Jianfeng; dev@dpdk.org > Cc: maxime.coque...@redhat.com; tho...@monjalon.net; Wang, Zhihong; > Bie, Tiwei > Subject: RE: [PATCH v4 1/1] net/virtio-user: add support for server mode > > Hi Jiaf

Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-03 Thread Yang, Zhiyong
Hi Jiafeng, Thanks for your review and comments firstly. Reply inline. > -Original Message- > From: Tan, Jianfeng > Sent: Tuesday, April 3, 2018 11:16 PM > To: Yang, Zhiyong ; dev@dpdk.org > Cc: maxime.coque...@redhat.com; tho...@monjalon.net; Wang, Zhihong > ; Bie, Tiwei > Subject: Re:

Re: [dpdk-dev] [PATCH v4 1/1] net/virtio-user: add support for server mode

2018-04-03 Thread Tan, Jianfeng
On 4/3/2018 8:20 PM, zhiyong.y...@intel.com wrote: virtio-user adds support for server mode in this patch. Client mode vhost-user startup firstly, server mode virtio-user startups and creates the socket file to exchange vhost messages. If the connection is broken, client mode vhost-user can s