[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V2 12/21] vhost-vdpa: open device fd in net_init_vhost_vdpa()
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH V2 12/21] vhost-vdpa: open device fd in net_init_vhost_vdpa() |
Date: |
Sat, 4 Sep 2021 16:41:00 -0400 |
On Fri, Sep 03, 2021 at 05:10:22PM +0800, Jason Wang wrote:
> This path switches to open device fd in net_init_vhost_vpda(). This is
patch?
> used to prepare for the multiqueue support.
>
> Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>
> Signed-off-by: Jason Wang <jasowang@redhat.com>
> ---
> net/vhost-vdpa.c | 23 +++++++++++++++--------
> 1 file changed, 15 insertions(+), 8 deletions(-)
>
> diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c
> index 912686457c..73d29a74ef 100644
> --- a/net/vhost-vdpa.c
> +++ b/net/vhost-vdpa.c
> @@ -156,24 +156,19 @@ static NetClientInfo net_vhost_vdpa_info = {
> };
>
> static int net_vhost_vdpa_init(NetClientState *peer, const char *device,
> - const char *name, const char *vhostdev)
> + const char *name, int vdpa_device_fd)
> {
> NetClientState *nc = NULL;
> VhostVDPAState *s;
> - int vdpa_device_fd = -1;
> int ret = 0;
> 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);
> s = DO_UPCAST(VhostVDPAState, nc, nc);
> - vdpa_device_fd = qemu_open_old(vhostdev, O_RDWR);
> - if (vdpa_device_fd == -1) {
> - return -errno;
> - }
> +
> s->vhost_vdpa.device_fd = vdpa_device_fd;
> ret = vhost_vdpa_add(nc, (void *)&s->vhost_vdpa);
> if (ret) {
> - qemu_close(vdpa_device_fd);
> qemu_del_net_client(nc);
> }
> return ret;
> @@ -201,6 +196,7 @@ int net_init_vhost_vdpa(const Netdev *netdev, const char
> *name,
> NetClientState *peer, Error **errp)
> {
> const NetdevVhostVDPAOptions *opts;
> + int vdpa_device_fd, ret;
>
> assert(netdev->type == NET_CLIENT_DRIVER_VHOST_VDPA);
> opts = &netdev->u.vhost_vdpa;
> @@ -209,5 +205,16 @@ int net_init_vhost_vdpa(const Netdev *netdev, const char
> *name,
> (char *)name, errp)) {
> return -1;
> }
> - return net_vhost_vdpa_init(peer, TYPE_VHOST_VDPA, name, opts->vhostdev);
> +
> + vdpa_device_fd = qemu_open_old(opts->vhostdev, O_RDWR);
> + if (vdpa_device_fd == -1) {
> + return -errno;
> + }
> +
> + ret = net_vhost_vdpa_init(peer, TYPE_VHOST_VDPA, name, vdpa_device_fd);
> + if (ret) {
> + qemu_close(vdpa_device_fd);
> + }
> +
> + return ret;
> }
> --
> 2.25.1
- [PATCH V2 03/21] vhost_net: remove the meaningless assignment in vhost_net_start_one(), (continued)
- [PATCH V2 03/21] vhost_net: remove the meaningless assignment in vhost_net_start_one(), Jason Wang, 2021/09/03
- [PATCH V2 04/21] vhost: use unsigned int for nvqs, Jason Wang, 2021/09/03
- [PATCH V2 05/21] vhost_net: do not assume nvqs is always 2, Jason Wang, 2021/09/03
- [PATCH V2 06/21] vhost-vdpa: remove the unnecessary check in vhost_vdpa_add(), Jason Wang, 2021/09/03
- [PATCH V2 07/21] vhost-vdpa: don't cleanup twice in vhost_vdpa_add(), Jason Wang, 2021/09/03
- [PATCH V2 08/21] vhost-vdpa: fix leaking of vhost_net in vhost_vdpa_add(), Jason Wang, 2021/09/03
- [PATCH V2 09/21] vhost-vdpa: tweak the error label in vhost_vdpa_add(), Jason Wang, 2021/09/03
- [PATCH V2 10/21] vhost-vdpa: fix the wrong assertion in vhost_vdpa_init(), Jason Wang, 2021/09/03
- [PATCH V2 11/21] vhost-vdpa: remove the unncessary queue_index assignment, Jason Wang, 2021/09/03
- [PATCH V2 12/21] vhost-vdpa: open device fd in net_init_vhost_vdpa(), Jason Wang, 2021/09/03
- Re: [PATCH V2 12/21] vhost-vdpa: open device fd in net_init_vhost_vdpa(),
Michael S. Tsirkin <=
- [PATCH V2 13/21] vhost-vdpa: classify one time request, Jason Wang, 2021/09/03
- [PATCH V2 14/21] vhost-vdpa: prepare for the multiqueue support, Jason Wang, 2021/09/03
- [PATCH V2 15/21] vhost-vdpa: let net_vhost_vdpa_init() returns NetClientState *, Jason Wang, 2021/09/03
- [PATCH V2 16/21] net: introduce control client, Jason Wang, 2021/09/03
- [PATCH V2 17/21] vhost-net: control virtqueue support, Jason Wang, 2021/09/03
- [PATCH V2 18/21] virito-net: use "qps" instead of "queues" when possible, Jason Wang, 2021/09/03