RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-10 Thread Liu, Yi L
> From: Alex Williamson > Sent: Friday, July 10, 2020 8:55 PM > > On Fri, 10 Jul 2020 05:39:57 + > "Liu, Yi L" wrote: > > > Hi Alex, > > > > > From: Alex Williamson > > > Sent: Thursday, July 9, 2020 10:28 PM > > > > > > On Thu, 9 Jul 2020 07:16:31 + > > > "Liu, Yi L" wrote: > > > >

Re: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-10 Thread Alex Williamson
On Fri, 10 Jul 2020 05:39:57 + "Liu, Yi L" wrote: > Hi Alex, > > > From: Alex Williamson > > Sent: Thursday, July 9, 2020 10:28 PM > > > > On Thu, 9 Jul 2020 07:16:31 + > > "Liu, Yi L" wrote: > > > > > Hi Alex, > > > > > > After more thinking, looks like adding a r-b tree is

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-09 Thread Liu, Yi L
Hi Alex, > From: Alex Williamson > Sent: Thursday, July 9, 2020 10:28 PM > > On Thu, 9 Jul 2020 07:16:31 + > "Liu, Yi L" wrote: > > > Hi Alex, > > > > After more thinking, looks like adding a r-b tree is still not enough to > > solve the potential problem for free a range of PASID in one

Re: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-09 Thread Jacob Pan
On Thu, 9 Jul 2020 08:27:51 -0600 Alex Williamson wrote: > > So I'm wondering can we fall back to prior proposal which only free > > one PASID for a free request. how about your opinion? > > Doesn't it still seem like it would be a useful user interface to have > a mechanism to free all

Re: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-09 Thread Alex Williamson
On Thu, 9 Jul 2020 07:16:31 + "Liu, Yi L" wrote: > Hi Alex, > > After more thinking, looks like adding a r-b tree is still not enough to > solve the potential problem for free a range of PASID in one ioctl. If > caller gives [0, MAX_UNIT] in the free request, kernel anyhow should > loop all

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-09 Thread Liu, Yi L
Hi Alex, After more thinking, looks like adding a r-b tree is still not enough to solve the potential problem for free a range of PASID in one ioctl. If caller gives [0, MAX_UNIT] in the free request, kernel anyhow should loop all the PASIDs and search in the r-b tree. Even VFIO can track the

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Liu, Yi L
Hi Kevin, > From: Tian, Kevin > Sent: Thursday, July 9, 2020 10:18 AM > > > From: Liu, Yi L > > Sent: Thursday, July 9, 2020 10:08 AM > > > > Hi Kevin, > > > > > From: Tian, Kevin > > > Sent: Thursday, July 9, 2020 9:57 AM > > > > > > > From: Liu, Yi L > > > > Sent: Thursday, July 9, 2020

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Tian, Kevin
> From: Liu, Yi L > Sent: Thursday, July 9, 2020 10:08 AM > > Hi Kevin, > > > From: Tian, Kevin > > Sent: Thursday, July 9, 2020 9:57 AM > > > > > From: Liu, Yi L > > > Sent: Thursday, July 9, 2020 8:32 AM > > > > > > Hi Alex, > > > > > > > Alex Williamson > > > > Sent: Thursday, July 9,

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Liu, Yi L
Hi Kevin, > From: Tian, Kevin > Sent: Thursday, July 9, 2020 9:57 AM > > > From: Liu, Yi L > > Sent: Thursday, July 9, 2020 8:32 AM > > > > Hi Alex, > > > > > Alex Williamson > > > Sent: Thursday, July 9, 2020 3:55 AM > > > > > > On Wed, 8 Jul 2020 08:16:16 + > > > "Liu, Yi L" wrote: > >

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Tian, Kevin
> From: Liu, Yi L > Sent: Thursday, July 9, 2020 8:32 AM > > Hi Alex, > > > Alex Williamson > > Sent: Thursday, July 9, 2020 3:55 AM > > > > On Wed, 8 Jul 2020 08:16:16 + > > "Liu, Yi L" wrote: > > > > > Hi Alex, > > > > > > > From: Liu, Yi L < yi.l@intel.com> > > > > Sent: Friday,

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Liu, Yi L
Hi Alex, > Alex Williamson > Sent: Thursday, July 9, 2020 3:55 AM > > On Wed, 8 Jul 2020 08:16:16 + > "Liu, Yi L" wrote: > > > Hi Alex, > > > > > From: Liu, Yi L < yi.l@intel.com> > > > Sent: Friday, July 3, 2020 2:28 PM > > > > > > Hi Alex, > > > > > > > From: Alex Williamson > > >

Re: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Alex Williamson
On Wed, 8 Jul 2020 08:16:16 + "Liu, Yi L" wrote: > Hi Alex, > > > From: Liu, Yi L < yi.l@intel.com> > > Sent: Friday, July 3, 2020 2:28 PM > > > > Hi Alex, > > > > > From: Alex Williamson > > > Sent: Friday, July 3, 2020 5:19 AM > > > > > > On Wed, 24 Jun 2020 01:55:19 -0700 > > >

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-08 Thread Liu, Yi L
Hi Alex, > From: Liu, Yi L < yi.l@intel.com> > Sent: Friday, July 3, 2020 2:28 PM > > Hi Alex, > > > From: Alex Williamson > > Sent: Friday, July 3, 2020 5:19 AM > > > > On Wed, 24 Jun 2020 01:55:19 -0700 > > Liu Yi L wrote: > > > > > This patch allows user space to request PASID

RE: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-03 Thread Liu, Yi L
Hi Alex, > From: Alex Williamson > Sent: Friday, July 3, 2020 5:19 AM > > On Wed, 24 Jun 2020 01:55:19 -0700 > Liu Yi L wrote: > > > This patch allows user space to request PASID allocation/free, e.g. > > when serving the request from the guest. > > > > PASIDs that are not freed by userspace

Re: [PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-07-02 Thread Alex Williamson
On Wed, 24 Jun 2020 01:55:19 -0700 Liu Yi L wrote: > This patch allows user space to request PASID allocation/free, e.g. when > serving the request from the guest. > > PASIDs that are not freed by userspace are automatically freed when the > IOASID set is destroyed when process exits. > > Cc:

[PATCH v3 06/14] vfio/type1: Add VFIO_IOMMU_PASID_REQUEST (alloc/free)

2020-06-24 Thread Liu Yi L
This patch allows user space to request PASID allocation/free, e.g. when serving the request from the guest. PASIDs that are not freed by userspace are automatically freed when the IOASID set is destroyed when process exits. Cc: Kevin Tian CC: Jacob Pan Cc: Alex Williamson Cc: Eric Auger Cc: