From: Jason Wang <jasowang@redhat.com> To: Yongji Xie <xieyongji@bytedance.com> Cc: Parav Pandit <parav@nvidia.com>, virtualization@lists.linux-foundation.org, "Michael S. Tsirkin" <mst@redhat.com>, elic@nvidia.com, netdev@vger.kernel.org Subject: Re: [External] Re: [PATCH 0/7] Introduce vdpa management tool Date: Wed, 2 Dec 2020 13:48:30 +0800 [thread overview] Message-ID: <594b2086-802f-1053-2866-a25d4a327876@redhat.com> (raw) In-Reply-To: <CACycT3uw6KJgTo+dBzSj07p2P_PziD+WBfX4yWVX-nDNUD2M3A@mail.gmail.com> On 2020/12/1 下午5:55, Yongji Xie wrote: > On Tue, Dec 1, 2020 at 2:25 PM Jason Wang <jasowang@redhat.com> wrote: >> >> On 2020/11/30 下午3:07, Yongji Xie wrote: >>>>> Thanks for adding me, Jason! >>>>> >>>>> Now I'm working on a v2 patchset for VDUSE (vDPA Device in Userspace) >>>>> [1]. This tool is very useful for the vduse device. So I'm considering >>>>> integrating this into my v2 patchset. But there is one problem: >>>>> >>>>> In this tool, vdpa device config action and enable action are combined >>>>> into one netlink msg: VDPA_CMD_DEV_NEW. But in vduse case, it needs to >>>>> be splitted because a chardev should be created and opened by a >>>>> userspace process before we enable the vdpa device (call >>>>> vdpa_register_device()). >>>>> >>>>> So I'd like to know whether it's possible (or have some plans) to add >>>>> two new netlink msgs something like: VDPA_CMD_DEV_ENABLE and >>>>> VDPA_CMD_DEV_DISABLE to make the config path more flexible. >>>>> >>>> Actually, we've discussed such intermediate step in some early >>>> discussion. It looks to me VDUSE could be one of the users of this. >>>> >>>> Or I wonder whether we can switch to use anonymous inode(fd) for VDUSE >>>> then fetching it via an VDUSE_GET_DEVICE_FD ioctl? >>>> >>> Yes, we can. Actually the current implementation in VDUSE is like >>> this. But seems like this is still a intermediate step. The fd should >>> be binded to a name or something else which need to be configured >>> before. >> >> The name could be specified via the netlink. It looks to me the real >> issue is that until the device is connected with a userspace, it can't >> be used. So we also need to fail the enabling if it doesn't opened. >> > Yes, that's true. So you mean we can firstly try to fetch the fd > binded to a name/vduse_id via an VDUSE_GET_DEVICE_FD, then use the > name/vduse_id as a attribute to create vdpa device? It looks fine to > me. Yes, something like this. The anonymous fd will be created during dev_add() and the fd will be carried in the msg to userspace. Thanks > > Thanks, > Yongji >
WARNING: multiple messages have this Message-ID (diff)
From: Jason Wang <jasowang@redhat.com> To: Yongji Xie <xieyongji@bytedance.com> Cc: netdev@vger.kernel.org, elic@nvidia.com, "Michael S. Tsirkin" <mst@redhat.com>, virtualization@lists.linux-foundation.org Subject: Re: [External] Re: [PATCH 0/7] Introduce vdpa management tool Date: Wed, 2 Dec 2020 13:48:30 +0800 [thread overview] Message-ID: <594b2086-802f-1053-2866-a25d4a327876@redhat.com> (raw) In-Reply-To: <CACycT3uw6KJgTo+dBzSj07p2P_PziD+WBfX4yWVX-nDNUD2M3A@mail.gmail.com> On 2020/12/1 下午5:55, Yongji Xie wrote: > On Tue, Dec 1, 2020 at 2:25 PM Jason Wang <jasowang@redhat.com> wrote: >> >> On 2020/11/30 下午3:07, Yongji Xie wrote: >>>>> Thanks for adding me, Jason! >>>>> >>>>> Now I'm working on a v2 patchset for VDUSE (vDPA Device in Userspace) >>>>> [1]. This tool is very useful for the vduse device. So I'm considering >>>>> integrating this into my v2 patchset. But there is one problem: >>>>> >>>>> In this tool, vdpa device config action and enable action are combined >>>>> into one netlink msg: VDPA_CMD_DEV_NEW. But in vduse case, it needs to >>>>> be splitted because a chardev should be created and opened by a >>>>> userspace process before we enable the vdpa device (call >>>>> vdpa_register_device()). >>>>> >>>>> So I'd like to know whether it's possible (or have some plans) to add >>>>> two new netlink msgs something like: VDPA_CMD_DEV_ENABLE and >>>>> VDPA_CMD_DEV_DISABLE to make the config path more flexible. >>>>> >>>> Actually, we've discussed such intermediate step in some early >>>> discussion. It looks to me VDUSE could be one of the users of this. >>>> >>>> Or I wonder whether we can switch to use anonymous inode(fd) for VDUSE >>>> then fetching it via an VDUSE_GET_DEVICE_FD ioctl? >>>> >>> Yes, we can. Actually the current implementation in VDUSE is like >>> this. But seems like this is still a intermediate step. The fd should >>> be binded to a name or something else which need to be configured >>> before. >> >> The name could be specified via the netlink. It looks to me the real >> issue is that until the device is connected with a userspace, it can't >> be used. So we also need to fail the enabling if it doesn't opened. >> > Yes, that's true. So you mean we can firstly try to fetch the fd > binded to a name/vduse_id via an VDUSE_GET_DEVICE_FD, then use the > name/vduse_id as a attribute to create vdpa device? It looks fine to > me. Yes, something like this. The anonymous fd will be created during dev_add() and the fd will be carried in the msg to userspace. Thanks > > Thanks, > Yongji > _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
next prev parent reply other threads:[~2020-12-02 5:50 UTC|newest] Thread overview: 150+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-12 6:39 [PATCH 0/7] Introduce vdpa management tool Parav Pandit 2020-11-12 6:39 ` Parav Pandit 2020-11-12 6:39 ` [PATCH 1/7] vdpa: Add missing comment for virtqueue count Parav Pandit 2020-11-12 6:39 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 2/7] vdpa: Use simpler version of ida allocation Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 3/7] vdpa: Extend routine to accept vdpa device name Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 4/7] vdpa: Define vdpa parent device, ops and a netlink interface Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 5/7] vdpa: Enable a user to add and delete a vdpa device Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 6/7] vdpa: Enable user to query vdpa device info Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-12 6:40 ` [PATCH 7/7] vdpa/vdpa_sim: Enable user to create vdpasim net devices Parav Pandit 2020-11-12 6:40 ` Parav Pandit 2020-11-16 9:41 ` [PATCH 0/7] Introduce vdpa management tool Stefan Hajnoczi 2020-11-16 9:41 ` Stefan Hajnoczi 2020-11-17 19:41 ` Parav Pandit 2020-11-17 19:41 ` Parav Pandit 2020-11-16 22:23 ` Jakub Kicinski 2020-11-17 19:51 ` Parav Pandit 2020-11-17 19:51 ` Parav Pandit 2020-12-16 9:13 ` Michael S. Tsirkin 2020-12-16 9:13 ` Michael S. Tsirkin [not found] ` <20201216080610.08541f44@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> 2020-12-16 16:54 ` Parav Pandit 2020-12-16 16:54 ` Parav Pandit 2020-12-16 19:57 ` Michael S. Tsirkin 2020-12-16 19:57 ` Michael S. Tsirkin 2020-12-17 12:13 ` Parav Pandit 2020-12-17 12:13 ` Parav Pandit 2020-11-27 3:53 ` Jason Wang 2020-11-27 3:53 ` Jason Wang [not found] ` <CACycT3sYScObb9nN3g7L3cesjE7sCZWxZ5_5R1usGU9ePZEeqA@mail.gmail.com> 2020-11-30 3:36 ` [External] " Jason Wang 2020-11-30 3:36 ` Jason Wang 2020-11-30 7:07 ` Yongji Xie 2020-12-01 6:25 ` Jason Wang 2020-12-01 6:25 ` Jason Wang 2020-12-01 9:55 ` Yongji Xie 2020-12-01 11:32 ` Parav Pandit 2020-12-01 11:32 ` Parav Pandit 2020-12-01 14:18 ` Yongji Xie 2020-12-01 15:58 ` Parav Pandit 2020-12-01 15:58 ` Parav Pandit 2020-12-02 3:29 ` Yongji Xie 2020-12-02 4:53 ` Parav Pandit 2020-12-02 4:53 ` Parav Pandit 2020-12-02 5:51 ` Jason Wang 2020-12-02 5:51 ` Jason Wang 2020-12-02 6:24 ` Parav Pandit 2020-12-02 6:24 ` Parav Pandit 2020-12-02 7:55 ` Jason Wang 2020-12-02 7:55 ` Jason Wang 2020-12-02 9:27 ` Yongji Xie 2020-12-02 9:21 ` Yongji Xie 2020-12-02 11:13 ` Parav Pandit 2020-12-02 11:13 ` Parav Pandit 2020-12-02 13:18 ` Yongji Xie 2020-12-02 5:48 ` Jason Wang [this message] 2020-12-02 5:48 ` Jason Wang 2020-12-08 22:47 ` David Ahern 2020-12-08 22:47 ` David Ahern 2021-01-19 4:21 ` Parav Pandit 2021-01-19 4:21 ` Parav Pandit 2020-12-16 9:16 ` Michael S. Tsirkin 2020-12-16 9:16 ` Michael S. Tsirkin 2021-01-04 3:31 ` [PATCH linux-next v2 " Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 3:31 ` [PATCH linux-next v2 1/7] vdpa_sim_net: Make mac address array static Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 7:00 ` Jason Wang 2021-01-04 7:00 ` Jason Wang 2021-01-04 3:31 ` [PATCH linux-next v2 2/7] vdpa_sim_net: Add module param to disable default vdpa net device Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 3:31 ` [PATCH linux-next v2 3/7] vdpa: Extend routine to accept vdpa device name Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 3:31 ` [PATCH linux-next v2 4/7] vdpa: Define vdpa mgmt device, ops and a netlink interface Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 7:03 ` Jason Wang 2021-01-04 7:03 ` Jason Wang 2021-01-04 7:24 ` Parav Pandit 2021-01-04 7:24 ` Parav Pandit 2021-01-05 4:10 ` Jason Wang 2021-01-05 4:10 ` Jason Wang 2021-01-05 6:33 ` Parav Pandit 2021-01-05 6:33 ` Parav Pandit 2021-01-05 8:36 ` Jason Wang 2021-01-05 8:36 ` Jason Wang 2021-01-04 3:31 ` [PATCH linux-next v2 5/7] vdpa: Enable a user to add and delete a vdpa device Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 3:31 ` [PATCH linux-next v2 6/7] vdpa: Enable user to query vdpa device info Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 3:31 ` [PATCH linux-next v2 7/7] vdpa_sim_net: Add support for user supported devices Parav Pandit 2021-01-04 3:31 ` Parav Pandit 2021-01-04 7:05 ` Jason Wang 2021-01-04 7:05 ` Jason Wang 2021-01-04 7:21 ` Parav Pandit 2021-01-04 7:21 ` Parav Pandit 2021-01-05 4:06 ` Jason Wang 2021-01-05 4:06 ` Jason Wang 2021-01-05 6:22 ` Parav Pandit 2021-01-05 6:22 ` Parav Pandit 2021-01-05 10:31 ` [PATCH linux-next v3 0/6] Introduce vdpa management tool Parav Pandit 2021-01-05 10:31 ` Parav Pandit 2021-01-05 10:31 ` [PATCH linux-next v3 1/6] vdpa_sim_net: Make mac address array static Parav Pandit 2021-01-05 10:31 ` Parav Pandit 2021-01-07 13:45 ` Stefano Garzarella 2021-01-07 13:45 ` Stefano Garzarella 2021-01-05 10:31 ` [PATCH linux-next v3 2/6] vdpa: Extend routine to accept vdpa device name Parav Pandit 2021-01-05 10:31 ` Parav Pandit 2021-01-05 10:32 ` [PATCH linux-next v3 3/6] vdpa: Define vdpa mgmt device, ops and a netlink interface Parav Pandit 2021-01-05 10:32 ` Parav Pandit 2021-01-05 10:32 ` [PATCH linux-next v3 4/6] vdpa: Enable a user to add and delete a vdpa device Parav Pandit 2021-01-05 10:32 ` Parav Pandit 2021-01-05 10:32 ` [PATCH linux-next v3 5/6] vdpa: Enable user to query vdpa device info Parav Pandit 2021-01-05 10:32 ` Parav Pandit 2021-01-05 10:32 ` [PATCH linux-next v3 6/6] vdpa_sim_net: Add support for user supported devices Parav Pandit 2021-01-05 10:32 ` Parav Pandit 2021-01-05 11:48 ` Michael S. Tsirkin 2021-01-05 11:48 ` Michael S. Tsirkin 2021-01-05 12:02 ` Parav Pandit 2021-01-05 12:02 ` Parav Pandit 2021-01-05 12:14 ` Michael S. Tsirkin 2021-01-05 12:14 ` Michael S. Tsirkin 2021-01-05 12:30 ` Parav Pandit 2021-01-05 12:30 ` Parav Pandit 2021-01-05 13:23 ` Michael S. Tsirkin 2021-01-05 13:23 ` Michael S. Tsirkin 2021-01-07 3:48 ` Parav Pandit 2021-01-07 3:48 ` Parav Pandit 2021-01-12 4:14 ` Parav Pandit 2021-01-12 4:14 ` Parav Pandit 2021-01-14 4:17 ` Jason Wang 2021-01-14 4:17 ` Jason Wang 2021-01-14 7:58 ` Parav Pandit 2021-01-14 7:58 ` Parav Pandit 2021-01-15 5:38 ` Jason Wang 2021-01-15 5:38 ` Jason Wang 2021-01-15 6:27 ` Parav Pandit 2021-01-15 6:27 ` Parav Pandit 2021-01-19 11:09 ` Jason Wang 2021-01-19 11:09 ` Jason Wang 2021-01-20 3:21 ` Parav Pandit 2021-01-20 3:21 ` Parav Pandit 2021-01-20 3:46 ` Parav Pandit 2021-01-20 3:46 ` Parav Pandit 2021-01-18 18:03 ` Parav Pandit 2021-01-18 18:03 ` Parav Pandit 2021-01-20 7:53 ` Michael S. Tsirkin 2021-01-20 7:53 ` Michael S. Tsirkin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=594b2086-802f-1053-2866-a25d4a327876@redhat.com \ --to=jasowang@redhat.com \ --cc=elic@nvidia.com \ --cc=mst@redhat.com \ --cc=netdev@vger.kernel.org \ --cc=parav@nvidia.com \ --cc=virtualization@lists.linux-foundation.org \ --cc=xieyongji@bytedance.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.