Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-10-12 Thread Steven Sistare
On 10/12/2022 10:40 AM, Jason Gunthorpe wrote: > On Wed, Oct 12, 2022 at 09:50:53AM -0400, Steven Sistare wrote: > >>> Anyhow, I think this conversation has convinced me there is no way to >>> fix VFIO_DMA_UNMAP_FLAG_VADDR. I'll send a patch reverting it due to >

Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-10-12 Thread Steven Sistare
On 10/12/2022 8:32 AM, Jason Gunthorpe wrote: > On Tue, Oct 11, 2022 at 04:30:58PM -0400, Steven Sistare wrote: >> On 10/11/2022 8:30 AM, Jason Gunthorpe wrote: >>> On Mon, Oct 10, 2022 at 04:54:50PM -0400, Steven Sistare wrote: >>>>> Do we have a solution to this

Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-10-11 Thread Steven Sistare
On 10/11/2022 8:30 AM, Jason Gunthorpe wrote: > On Mon, Oct 10, 2022 at 04:54:50PM -0400, Steven Sistare wrote: >>> Do we have a solution to this? >>> >>> If not I would like to make a patch removing VFIO_DMA_UNMAP_FLAG_VADDR >>> >>> Aside from the

Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-10-10 Thread Steven Sistare
On 10/6/2022 12:01 PM, Jason Gunthorpe wrote: > On Wed, Sep 21, 2022 at 08:09:54PM -0300, Jason Gunthorpe wrote: >> On Wed, Sep 21, 2022 at 03:30:55PM -0400, Steven Sistare wrote: >> >>>> If Steve wants to keep it then someone needs to fix the deadlock in >>>

Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-10-06 Thread Steven Sistare
On 10/6/2022 12:01 PM, Jason Gunthorpe wrote: > On Wed, Sep 21, 2022 at 08:09:54PM -0300, Jason Gunthorpe wrote: >> On Wed, Sep 21, 2022 at 03:30:55PM -0400, Steven Sistare wrote: >> >>>> If Steve wants to keep it then someone needs to fix the deadlock in >>>

Re: [PATCH RFC v2 00/13] IOMMUFD Generic interface

2022-09-22 Thread Steven Sistare
On 9/21/2022 2:44 PM, Jason Gunthorpe wrote: > On Wed, Sep 21, 2022 at 12:06:49PM -0600, Alex Williamson wrote: > >>> I still think the compat gaps are small. I've realized that >>> VFIO_DMA_UNMAP_FLAG_VADDR has no implementation in qemu, and since it >>> can deadlock the kernel I propose we