From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH 1/2] net/vhost: set the vq consistently Date: Wed, 11 Apr 2018 17:02:11 +0200 Message-ID: References: <20180329160507.25081-1-3chas3@gmail.com> <6ec8aef7-4eac-811b-b520-47ca50206665@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Cc: mtetsuyah@gmail.com, yliu@fridaylinux.org, Robert Shearman , stable@dpdk.org, Chas Williams To: Chas Williams <3chas3@gmail.com>, dev@dpdk.org Return-path: In-Reply-To: <6ec8aef7-4eac-811b-b520-47ca50206665@redhat.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 04/11/2018 04:46 PM, Maxime Coquelin wrote: > > > On 03/29/2018 06:05 PM, Chas Williams wrote: >> From: Robert Shearman >> >> The vid and port of the vq were only set on queues created during initial >> device setup. >> >> Fixes: ee584e9710b9 ("vhost: add driver on top of the library") >> Cc: stable@dpdk.org >> >> Signed-off-by: Robert Shearman >> Signed-off-by: Chas Williams >> --- >>   drivers/net/vhost/rte_eth_vhost.c | 15 +++++++++++++++ >>   1 file changed, 15 insertions(+) >> > > Reviewed-by: Maxime Coquelin It does not apply because a patch from Junjie seems to address the same problem: net/vhost: fix crash when creating vdev dynamically Can you check dpdk-next-virtio/master and confirm this patch is no more required? Thanks, Maxime