Re: [dpdk-dev] [PATCH 1/2] net/vhost: fix Vhost setup error path

2020-02-18 Thread Tiwei Bie
On Tue, Feb 18, 2020 at 06:22:39PM +0100, Maxime Coquelin wrote: > If for some reason vhost_driver_setup() fails, the list > element for the device may be freed without being removed > from the internal list of devices. > > This patch fixes all the error paths, by unregistering the > device from V

[dpdk-dev] [PATCH 1/2] net/vhost: fix Vhost setup error path

2020-02-18 Thread Maxime Coquelin
If for some reason vhost_driver_setup() fails, the list element for the device may be freed without being removed from the internal list of devices. This patch fixes all the error paths, by unregistering the device from Vhost library it has been registered, remove the device from the list, reset d

Re: [dpdk-dev] [PATCH 1/2] net/vhost: fix Vhost setup error path

2020-02-18 Thread Maxime Coquelin
On 2/18/20 5:15 PM, David Marchand wrote: > On Tue, Feb 18, 2020 at 3:35 PM Maxime Coquelin > wrote: >> >> If for some reason vhost_driver_setup() fails, the list >> element for the device may be freed without being removed >> from the internal list of devices. >> >> This patch fixes all the er

Re: [dpdk-dev] [PATCH 1/2] net/vhost: fix Vhost setup error path

2020-02-18 Thread David Marchand
On Tue, Feb 18, 2020 at 3:35 PM Maxime Coquelin wrote: > > If for some reason vhost_driver_setup() fails, the list > element for the device may be freed without being removed > from the internal list of devices. > > This patch fixes all the error paths, by unregistering the > device from Vhost lib

[dpdk-dev] [PATCH 1/2] net/vhost: fix Vhost setup error path

2020-02-18 Thread Maxime Coquelin
If for some reason vhost_driver_setup() fails, the list element for the device may be freed without being removed from the internal list of devices. This patch fixes all the error paths, by unregistering the device from Vhost library it has been registered, remove the device from the list, reset d