Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Maxime Coquelin
On 04/12/2018 09:35 AM, Maxime Coquelin wrote: On 04/12/2018 09:34 AM, Chen, Junjie J wrote: On 04/12/2018 09:21 AM, Tan, Jianfeng wrote: On 4/12/2018 1:02 AM, Junjie Chen wrote: dev_start sets *dev_attached* after setup queues, this sets device to invalid state since no frontend is a

Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Tan, Jianfeng
On 4/12/2018 3:29 PM, Maxime Coquelin wrote: On 04/12/2018 09:21 AM, Tan, Jianfeng wrote: On 4/12/2018 1:02 AM, Junjie Chen wrote: dev_start sets *dev_attached* after setup queues, this sets device to invalid state since no frontend is attached. Also destroy_device set *started* to zero w

Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Maxime Coquelin
On 04/12/2018 09:34 AM, Chen, Junjie J wrote: On 04/12/2018 09:21 AM, Tan, Jianfeng wrote: On 4/12/2018 1:02 AM, Junjie Chen wrote: dev_start sets *dev_attached* after setup queues, this sets device to invalid state since no frontend is attached. Also destroy_device set *started* to zero

Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Chen, Junjie J
> > > > On 04/12/2018 09:21 AM, Tan, Jianfeng wrote: > > > > > > On 4/12/2018 1:02 AM, Junjie Chen wrote: > >> dev_start sets *dev_attached* after setup queues, this sets device to > >> invalid state since no frontend is attached. Also destroy_device set > >> *started* to zero which makes *allow

Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Maxime Coquelin
On 04/12/2018 09:21 AM, Tan, Jianfeng wrote: On 4/12/2018 1:02 AM, Junjie Chen wrote: dev_start sets *dev_attached* after setup queues, this sets device to invalid state since no frontend is attached. Also destroy_device set *started* to zero which makes *allow_queuing* always zero until dev

Re: [dpdk-dev] [PATCH v3] net/vhost: fix vhost invalid state

2018-04-12 Thread Tan, Jianfeng
On 4/12/2018 1:02 AM, Junjie Chen wrote: dev_start sets *dev_attached* after setup queues, this sets device to invalid state since no frontend is attached. Also destroy_device set *started* to zero which makes *allow_queuing* always zero until dev_start get called again. Actually, we should not