Re: [PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread sathyanarayanan kuppuswamy
On 2/7/19 1:15 PM, David Woodhouse wrote: On Thu, 2019-02-07 at 13:09 -0800, Raj, Ashok wrote: You are right.. they are completely orthogonal. We just don't have a way to handle the page-requests for request without PASID's. There are some of the vIOMMU work to pass the PRI to who owns the de

Re: [PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread Raj, Ashok
On Thu, Feb 07, 2019 at 09:15:24PM +, David Woodhouse wrote: > On Thu, 2019-02-07 at 13:09 -0800, Raj, Ashok wrote: > > You are right.. they are completely orthogonal. We just don't have > > a way to handle the page-requests for request without PASID's. > > > > There are some of the vIOMMU wor

Re: [PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread David Woodhouse
On Thu, 2019-02-07 at 13:09 -0800, Raj, Ashok wrote: > You are right.. they are completely orthogonal. We just don't have > a way to handle the page-requests for request without PASID's. > > There are some of the vIOMMU work to pass the PRI to who owns > the device, and we can certainly relax it t

Re: [PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread David Woodhouse
On Thu, 2019-02-07 at 10:44 -0800, sathyanarayanan.kuppusw...@linux.intel.com wrote: > From: Kuppuswamy Sathyanarayanan > > Intel IOMMU Page Request Services (PRS) only works with devices which > supports/uses PASID. So enable PRI only if the device also enables > PASID support. For more details

Re: [PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread Raj, Ashok
On Thu, Feb 07, 2019 at 08:08:06PM +, David Woodhouse wrote: > On Thu, 2019-02-07 at 10:44 -0800, sathyanarayanan.kuppusw...@linux.intel.com > wrote: > > From: Kuppuswamy Sathyanarayanan > > > > > > Intel IOMMU Page Request Services (PRS) only works with devices which > > supports/uses PASI

[PATCH v1 1/1] iommu/vt-d: Enable PRI only if the device enables PASID.

2019-02-07 Thread sathyanarayanan . kuppuswamy
From: Kuppuswamy Sathyanarayanan Intel IOMMU Page Request Services (PRS) only works with devices which supports/uses PASID. So enable PRI only if the device also enables PASID support. For more details, Please check the implementation of PRQ handler(prq_event_thread()) in intel-svm driver. Cc: J