mbox series

[RFC,0/7] vhost-vdpa: add support for iommufd

Message ID 20230503091337.2130631-1-lulu@redhat.com (mailing list archive)
Headers show
Series vhost-vdpa: add support for iommufd | expand

Message

Cindy Lu May 3, 2023, 9:13 a.m. UTC 
Hi All
There is the RFC to support the IOMMUFD in vdpa device
any comments are welcome
Thanks
Cindy

Cindy Lu (7):
  vhost: introduce new UAPI to support IOMMUFD
  qapi: support iommufd in vdpa
  virtio : add a ptr for vdpa_iommufd in VirtIODevice
  net/vhost-vdpa: Add the check for iommufd
  vhost-vdpa: Add the iommufd support in the map/unmap function
  vhost-vdpa: init iommufd function in vhost_vdpa start
  vhost-vdpa-iommufd: Add iommufd support for vdpa

 hw/virtio/meson.build          |   2 +-
 hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
 hw/virtio/vhost-vdpa.c         |  74 +++++++++-
 include/hw/virtio/vhost-vdpa.h |  47 +++++++
 include/hw/virtio/virtio.h     |   5 +
 linux-headers/linux/vhost.h    |  72 ++++++++++
 net/vhost-vdpa.c               |  31 +++--
 qapi/net.json                  |   1 +
 8 files changed, 451 insertions(+), 21 deletions(-)
 create mode 100644 hw/virtio/vhost-vdpa-iommufd.c

Comments

Jason Wang May 5, 2023, 3:29 a.m. UTC | #1 
Hi Cindy

On Wed, May 3, 2023 at 5:13 PM Cindy Lu <lulu@redhat.com> wrote:
>
> Hi All
> There is the RFC to support the IOMMUFD in vdpa device
> any comments are welcome
> Thanks
> Cindy

Please post the kernel patch as well as a reference.

Thanks

>
> Cindy Lu (7):
>   vhost: introduce new UAPI to support IOMMUFD
>   qapi: support iommufd in vdpa
>   virtio : add a ptr for vdpa_iommufd in VirtIODevice
>   net/vhost-vdpa: Add the check for iommufd
>   vhost-vdpa: Add the iommufd support in the map/unmap function
>   vhost-vdpa: init iommufd function in vhost_vdpa start
>   vhost-vdpa-iommufd: Add iommufd support for vdpa
>
>  hw/virtio/meson.build          |   2 +-
>  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
>  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
>  include/hw/virtio/vhost-vdpa.h |  47 +++++++
>  include/hw/virtio/virtio.h     |   5 +
>  linux-headers/linux/vhost.h    |  72 ++++++++++
>  net/vhost-vdpa.c               |  31 +++--
>  qapi/net.json                  |   1 +
>  8 files changed, 451 insertions(+), 21 deletions(-)
>  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
>
> --
> 2.34.3
>
Cindy Lu May 5, 2023, 6:29 a.m. UTC | #2 
On Fri, May 5, 2023 at 11:29 AM Jason Wang <jasowang@redhat.com> wrote:
>
> Hi Cindy
>
> On Wed, May 3, 2023 at 5:13 PM Cindy Lu <lulu@redhat.com> wrote:
> >
> > Hi All
> > There is the RFC to support the IOMMUFD in vdpa device
> > any comments are welcome
> > Thanks
> > Cindy
>
> Please post the kernel patch as well as a reference.
>
> Thanks
>
sure，will do
Thanks
cindy
> >
> > Cindy Lu (7):
> >   vhost: introduce new UAPI to support IOMMUFD
> >   qapi: support iommufd in vdpa
> >   virtio : add a ptr for vdpa_iommufd in VirtIODevice
> >   net/vhost-vdpa: Add the check for iommufd
> >   vhost-vdpa: Add the iommufd support in the map/unmap function
> >   vhost-vdpa: init iommufd function in vhost_vdpa start
> >   vhost-vdpa-iommufd: Add iommufd support for vdpa
> >
> >  hw/virtio/meson.build          |   2 +-
> >  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
> >  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
> >  include/hw/virtio/vhost-vdpa.h |  47 +++++++
> >  include/hw/virtio/virtio.h     |   5 +
> >  linux-headers/linux/vhost.h    |  72 ++++++++++
> >  net/vhost-vdpa.c               |  31 +++--
> >  qapi/net.json                  |   1 +
> >  8 files changed, 451 insertions(+), 21 deletions(-)
> >  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
> >
> > --
> > 2.34.3
> >
>
Michael S. Tsirkin June 5, 2023, 5:41 a.m. UTC | #3 
On Fri, May 05, 2023 at 02:29:23PM +0800, Cindy Lu wrote:
> On Fri, May 5, 2023 at 11:29 AM Jason Wang <jasowang@redhat.com> wrote:
> >
> > Hi Cindy
> >
> > On Wed, May 3, 2023 at 5:13 PM Cindy Lu <lulu@redhat.com> wrote:
> > >
> > > Hi All
> > > There is the RFC to support the IOMMUFD in vdpa device
> > > any comments are welcome
> > > Thanks
> > > Cindy
> >
> > Please post the kernel patch as well as a reference.
> >
> > Thanks
> >
> sure，will do
> Thanks
> cindy

Is this effort going anywhere? It will soon be too late for
the next merge window.

> > >
> > > Cindy Lu (7):
> > >   vhost: introduce new UAPI to support IOMMUFD
> > >   qapi: support iommufd in vdpa
> > >   virtio : add a ptr for vdpa_iommufd in VirtIODevice
> > >   net/vhost-vdpa: Add the check for iommufd
> > >   vhost-vdpa: Add the iommufd support in the map/unmap function
> > >   vhost-vdpa: init iommufd function in vhost_vdpa start
> > >   vhost-vdpa-iommufd: Add iommufd support for vdpa
> > >
> > >  hw/virtio/meson.build          |   2 +-
> > >  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
> > >  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
> > >  include/hw/virtio/vhost-vdpa.h |  47 +++++++
> > >  include/hw/virtio/virtio.h     |   5 +
> > >  linux-headers/linux/vhost.h    |  72 ++++++++++
> > >  net/vhost-vdpa.c               |  31 +++--
> > >  qapi/net.json                  |   1 +
> > >  8 files changed, 451 insertions(+), 21 deletions(-)
> > >  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
> > >
> > > --
> > > 2.34.3
> > >
> >
Cindy Lu June 5, 2023, 8:04 a.m. UTC | #4 
On Mon, Jun 5, 2023 at 1:41 PM Michael S. Tsirkin <mst@redhat.com> wrote:
>
> On Fri, May 05, 2023 at 02:29:23PM +0800, Cindy Lu wrote:
> > On Fri, May 5, 2023 at 11:29 AM Jason Wang <jasowang@redhat.com> wrote:
> > >
> > > Hi Cindy
> > >
> > > On Wed, May 3, 2023 at 5:13 PM Cindy Lu <lulu@redhat.com> wrote:
> > > >
> > > > Hi All
> > > > There is the RFC to support the IOMMUFD in vdpa device
> > > > any comments are welcome
> > > > Thanks
> > > > Cindy
> > >
> > > Please post the kernel patch as well as a reference.
> > >
> > > Thanks
> > >
> > sure，will do
> > Thanks
> > cindy
>
> Is this effort going anywhere? It will soon be too late for
> the next merge window.
>
Hi Michael
I'm now working on some vduse issue, I will go back to work in IOMMUFD soon

Thanks
Cindy
> > > >
> > > > Cindy Lu (7):
> > > >   vhost: introduce new UAPI to support IOMMUFD
> > > >   qapi: support iommufd in vdpa
> > > >   virtio : add a ptr for vdpa_iommufd in VirtIODevice
> > > >   net/vhost-vdpa: Add the check for iommufd
> > > >   vhost-vdpa: Add the iommufd support in the map/unmap function
> > > >   vhost-vdpa: init iommufd function in vhost_vdpa start
> > > >   vhost-vdpa-iommufd: Add iommufd support for vdpa
> > > >
> > > >  hw/virtio/meson.build          |   2 +-
> > > >  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
> > > >  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
> > > >  include/hw/virtio/vhost-vdpa.h |  47 +++++++
> > > >  include/hw/virtio/virtio.h     |   5 +
> > > >  linux-headers/linux/vhost.h    |  72 ++++++++++
> > > >  net/vhost-vdpa.c               |  31 +++--
> > > >  qapi/net.json                  |   1 +
> > > >  8 files changed, 451 insertions(+), 21 deletions(-)
> > > >  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
> > > >
> > > > --
> > > > 2.34.3
> > > >
> > >
>
Michael S. Tsirkin Sept. 13, 2023, 1:31 p.m. UTC | #5 
On Wed, May 03, 2023 at 05:13:30PM +0800, Cindy Lu wrote:
> Hi All
> There is the RFC to support the IOMMUFD in vdpa device
> any comments are welcome
> Thanks
> Cindy

Any plans to work on this or should I consider this abandoned?


> Cindy Lu (7):
>   vhost: introduce new UAPI to support IOMMUFD
>   qapi: support iommufd in vdpa
>   virtio : add a ptr for vdpa_iommufd in VirtIODevice
>   net/vhost-vdpa: Add the check for iommufd
>   vhost-vdpa: Add the iommufd support in the map/unmap function
>   vhost-vdpa: init iommufd function in vhost_vdpa start
>   vhost-vdpa-iommufd: Add iommufd support for vdpa
> 
>  hw/virtio/meson.build          |   2 +-
>  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
>  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
>  include/hw/virtio/vhost-vdpa.h |  47 +++++++
>  include/hw/virtio/virtio.h     |   5 +
>  linux-headers/linux/vhost.h    |  72 ++++++++++
>  net/vhost-vdpa.c               |  31 +++--
>  qapi/net.json                  |   1 +
>  8 files changed, 451 insertions(+), 21 deletions(-)
>  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
> 
> -- 
> 2.34.3
Cindy Lu Sept. 14, 2023, 5:44 a.m. UTC | #6 
Hi Michael，
Really sorry for the delay, I was sick-leave for almost 2 months,
which caused the delay in the development of this feature.  I will
continue working on this feature soon.
Thanks
Cindy

On Wed, Sep 13, 2023 at 9:31 PM Michael S. Tsirkin <mst@redhat.com> wrote:
>
> On Wed, May 03, 2023 at 05:13:30PM +0800, Cindy Lu wrote:
> > Hi All
> > There is the RFC to support the IOMMUFD in vdpa device
> > any comments are welcome
> > Thanks
> > Cindy
>
> Any plans to work on this or should I consider this abandoned?
>
>
> > Cindy Lu (7):
> >   vhost: introduce new UAPI to support IOMMUFD
> >   qapi: support iommufd in vdpa
> >   virtio : add a ptr for vdpa_iommufd in VirtIODevice
> >   net/vhost-vdpa: Add the check for iommufd
> >   vhost-vdpa: Add the iommufd support in the map/unmap function
> >   vhost-vdpa: init iommufd function in vhost_vdpa start
> >   vhost-vdpa-iommufd: Add iommufd support for vdpa
> >
> >  hw/virtio/meson.build          |   2 +-
> >  hw/virtio/vhost-vdpa-iommufd.c | 240 +++++++++++++++++++++++++++++++++
> >  hw/virtio/vhost-vdpa.c         |  74 +++++++++-
> >  include/hw/virtio/vhost-vdpa.h |  47 +++++++
> >  include/hw/virtio/virtio.h     |   5 +
> >  linux-headers/linux/vhost.h    |  72 ++++++++++
> >  net/vhost-vdpa.c               |  31 +++--
> >  qapi/net.json                  |   1 +
> >  8 files changed, 451 insertions(+), 21 deletions(-)
> >  create mode 100644 hw/virtio/vhost-vdpa-iommufd.c
> >
> > --
> > 2.34.3
>