As far as I know we don't need #ifdefs cause there are dummy functions
when IOMMU is not compiled in.
But this patch is a NAK since it circumvents the /dev/mem restriction
when IOMMU is disabled and that is not something we can easily allow.
Regards,
Christian.
Am 19.09.2017 um 17:36 schrieb Tom St Denis:
Do we need to put #ifdefs of IOMMU_SUPPORT around this too?
The cavium driver doesn't seem to have any depends on IOMMU_*
anywhere... (for instance)
Tom
On 19/09/17 11:30 AM, Tom St Denis wrote:
Signed-off-by: Tom St Denis <tom.stde...@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 0e5f78f3a97e..1ee51a15a56c 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -1791,13 +1791,14 @@ static ssize_t
amdgpu_iova_to_phys_read(struct file *f, char __user *buf,
struct iommu_domain *dom;
dom = iommu_get_domain_for_dev(adev->dev);
- if (!dom)
- return -EFAULT;
result = 0;
while (size) {
// get physical address and map
- phys = iommu_iova_to_phys(dom, *pos);
+ if (dom)
+ phys = iommu_iova_to_phys(dom, *pos);
+ else
+ phys = *pos;
// copy upto one page
if (size > PAGE_SIZE)
@@ -1837,13 +1838,14 @@ static ssize_t
amdgpu_iova_to_phys_write(struct file *f, const char __user *buf,
struct iommu_domain *dom;
dom = iommu_get_domain_for_dev(adev->dev);
- if (!dom)
- return -EFAULT;
result = 0;
while (size) {
// get physical address and map
- phys = iommu_iova_to_phys(dom, *pos);
+ if (dom)
+ phys = iommu_iova_to_phys(dom, *pos);
+ else
+ phys = *pos;
// copy upto one page
if (size > PAGE_SIZE)
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx