Re: [PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Eugenio Perez Martin
On Tue, Aug 2, 2022 at 3:10 PM Michael S. Tsirkin wrote: > > On Tue, Aug 02, 2022 at 02:58:36PM +0200, Laurent Vivier wrote: > > On 02/08/2022 14:57, Michael S. Tsirkin wrote: > > > On Tue, Aug 02, 2022 at 02:10:43PM +0200, Laurent Vivier wrote: > > > > On 02/08/2022 13:24, Eugenio Pérez wrote: >

Re: [PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Michael S. Tsirkin
On Tue, Aug 02, 2022 at 02:58:36PM +0200, Laurent Vivier wrote: > On 02/08/2022 14:57, Michael S. Tsirkin wrote: > > On Tue, Aug 02, 2022 at 02:10:43PM +0200, Laurent Vivier wrote: > > > On 02/08/2022 13:24, Eugenio Pérez wrote: > > > > Few steps allocate new resources, and all of the allocated can

Re: [PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Laurent Vivier
On 02/08/2022 14:57, Michael S. Tsirkin wrote: On Tue, Aug 02, 2022 at 02:10:43PM +0200, Laurent Vivier wrote: On 02/08/2022 13:24, Eugenio Pérez wrote: Few steps allocate new resources, and all of the allocated can be checked to avoid trying to free an invalid one. Signed-off-by: Eugenio Pére

Re: [PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Michael S. Tsirkin
On Tue, Aug 02, 2022 at 02:10:43PM +0200, Laurent Vivier wrote: > On 02/08/2022 13:24, Eugenio Pérez wrote: > > Few steps allocate new resources, and all of the allocated can be > > checked to avoid trying to free an invalid one. > > > > Signed-off-by: Eugenio Pérez qemu-trivial too I guess the

Re: [PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Laurent Vivier
On 02/08/2022 13:24, Eugenio Pérez wrote: Few steps allocate new resources, and all of the allocated can be checked to avoid trying to free an invalid one. Signed-off-by: Eugenio Pérez --- net/vhost-vdpa.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/net/vhost-vd

[PATCH 2/2] vdpa: Merge all net_init_vhost_vdpa error goto

2022-08-02 Thread Eugenio Pérez
Few steps allocate new resources, and all of the allocated can be checked to avoid trying to free an invalid one. Signed-off-by: Eugenio Pérez --- net/vhost-vdpa.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c index 303447a68e..ac18