Re: [PATCH 09/18] vhost-vdpa: remove the unncessary queue_index assignment

[Stefano Garzarella]

On Mon, Jun 21, 2021 at 12:16:41PM +0800, Jason Wang wrote:
> The queue_index of NetClientState should be assigned in set_netdev()
> afterwards, so trying to net_vhost_vdpa_init() is meaningless. This
> patch removes this.
>
> Signed-off-by: Jason Wang <jasowang@redhat.com>
> ---
> net/vhost-vdpa.c | 1 -
> 1 file changed, 1 deletion(-)

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>

> diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
> index 87b181a74e..572aed4ca2 100644
> --- a/net/vhost-vdpa.c
> +++ b/net/vhost-vdpa.c
> @@ -166,7 +166,6 @@ static int net_vhost_vdpa_init(NetClientState 
> *peer, const char *device,
>     assert(name);
>     nc = qemu_new_net_client(&net_vhost_vdpa_info, peer, device, 
>     name);
>     snprintf(nc->info_str, sizeof(nc->info_str), TYPE_VHOST_VDPA);
> -    nc->queue_index = 0;
>     s = DO_UPCAST(VhostVDPAState, nc, nc);
>     vdpa_device_fd = qemu_open_old(vhostdev, O_RDWR);
>     if (vdpa_device_fd == -1) {
> -- 2.25.1