Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-18 Thread Marc Zyngier
On 15/01/16 22:47, Sinan Kaya wrote: > On 1/15/2016 12:32 PM, Marc Zyngier wrote: Do you have a link to that? Seeing it would help to ease my concerns. The QEMU driver has not been posted yet. As far as I know, it just discovers the memory resources on the platform object

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Mark Rutland
Hi, [adding KVM people, given this is meant for virtualization] On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: > The Qualcomm Technologies HIDMA device has been designed to support > virtualization technology. The driver has been divided into two to follow > the hardware design. >

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Mark Rutland
On Fri, Jan 15, 2016 at 10:12:00AM -0500, Sinan Kaya wrote: > Hi Mark, > > On 1/15/2016 9:56 AM, Mark Rutland wrote: > > Hi, > > > > [adding KVM people, given this is meant for virtualization] > > > > On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: > >> The Qualcomm Technologies

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Mark Rutland
On Fri, Jan 15, 2016 at 03:14:28PM +, Marc Zyngier wrote: > On 15/01/16 14:56, Mark Rutland wrote: > > Hi, > > > > [adding KVM people, given this is meant for virtualization] > > > > On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: > >> The Qualcomm Technologies HIDMA device has

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Sinan Kaya
>>> This doesn't seem to tie into KVM or VFIO, and as far as I can tell >>> there's no mechanism for associating channels with a particular virtual >>> address space (i.e. no configuration of an external or internal IOMMU), >>> nor pinning of guest pages to allow for DMA to occur safely. >> >> I'm

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Marc Zyngier
On 15/01/16 15:40, Sinan Kaya wrote: > On 1/15/2016 10:14 AM, Marc Zyngier wrote: >> On 15/01/16 14:56, Mark Rutland wrote: >>> Hi, >>> >>> [adding KVM people, given this is meant for virtualization] >>> >>> On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: The Qualcomm Technologies

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Marc Zyngier
On 15/01/16 17:16, Sinan Kaya wrote: This doesn't seem to tie into KVM or VFIO, and as far as I can tell there's no mechanism for associating channels with a particular virtual address space (i.e. no configuration of an external or internal IOMMU), nor pinning of guest pages to

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Sinan Kaya
On 1/15/2016 10:14 AM, Marc Zyngier wrote: > On 15/01/16 14:56, Mark Rutland wrote: >> Hi, >> >> [adding KVM people, given this is meant for virtualization] >> >> On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: >>> The Qualcomm Technologies HIDMA device has been designed to support >>>

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Sinan Kaya
On 1/15/2016 10:36 AM, Mark Rutland wrote: > On Fri, Jan 15, 2016 at 03:14:28PM +, Marc Zyngier wrote: >> On 15/01/16 14:56, Mark Rutland wrote: >>> Hi, >>> >>> [adding KVM people, given this is meant for virtualization] >>> >>> On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote:

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Sinan Kaya
Hi Mark, On 1/15/2016 9:56 AM, Mark Rutland wrote: > Hi, > > [adding KVM people, given this is meant for virtualization] > > On Mon, Jan 11, 2016 at 09:45:43AM -0500, Sinan Kaya wrote: >> The Qualcomm Technologies HIDMA device has been designed to support >> virtualization technology. The

Re: [PATCH V12 3/7] dma: add Qualcomm Technologies HIDMA management driver

2016-01-15 Thread Sinan Kaya
On 1/15/2016 12:32 PM, Marc Zyngier wrote: >>> Do you have a link to that? Seeing it would help to ease my concerns. >> > >> > The QEMU driver has not been posted yet. As far as I know, it just >> > discovers the memory >> > resources on the platform object and creates mappings for the guest >>