[PATCH] drm/amdgpu: use ttm pool first

[christian.koenig@xxxxxxx (Christian König)]

Am 24.01.2018 um 10:48 schrieb Chunming Zhou:
>
>
> On 2018å¹´01æ??24æ?¥ 17:43, Christian König wrote:
>> Am 24.01.2018 um 10:17 schrieb Chunming Zhou:
>>> dma path is low than ttm pool.
>>
>> That won't work correctly in some cases.
>>
>>> Question:
>>> But why need two path? What spicific problem can be solved by dma path?
>>
>> That is mostly for ARM and some (rare) x86 platforms without IOMMU 
>> where some memory is above the 40bit limit in the PCIe address space.
>>
>> Our GPUs can unfortunately only deal with 40bits of address space, so 
>> on those systems we need to go the slow path and make sure that we 
>> don't try to touch that high memory.
>>
>> What we could really improve is the detection using swiotlb_nr_tbl(), 
>> cause that is actually completely wrong.
> OK, the patch is trying to use ttm pool first, if dma_map_page fails, 
> then fallback to swtlb path, is that ok?

Nope, as I wrote that won't work. You must improve the detection if we 
need coherent or not coherent allocation.

Regards,
Christian.

>
> Regards,
> David Zhou
>>
>> Regards,
>> Christian.
>>
>>>
>>> Change-Id: I4a9fc7103658e643f33e267616fc69743b20b09e
>>> Signed-off-by: Chunming Zhou <david1.zhou at amd.com>
>>> ---
>>> Â  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 10 ++++++----
>>> Â  1 file changed, 6 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c 
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> index d32f48259c40..8d8a616bf4f5 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> @@ -996,6 +996,7 @@ static int amdgpu_ttm_tt_populate(struct ttm_tt 
>>> *ttm,
>>> Â Â Â Â Â  struct amdgpu_device *adev = amdgpu_ttm_adev(ttm->bdev);
>>> Â Â Â Â Â  struct amdgpu_ttm_tt *gtt = (void *)ttm;
>>> Â Â Â Â Â  bool slave = !!(ttm->page_flags & TTM_PAGE_FLAG_SG);
>>> +Â Â Â  int r;
>>> Â  Â Â Â Â Â  if (ttm->state != tt_unpopulated)
>>> Â Â Â Â Â Â Â Â Â  return 0;
>>> @@ -1017,13 +1018,14 @@ static int amdgpu_ttm_tt_populate(struct 
>>> ttm_tt *ttm,
>>> Â Â Â Â Â Â Â Â Â  return 0;
>>> Â Â Â Â Â  }
>>> Â  +Â Â Â  r = ttm_populate_and_map_pages(adev->dev, &gtt->ttm, ctx);
>>> +
>>> Â  #ifdef CONFIG_SWIOTLB
>>> -Â Â Â  if (swiotlb_nr_tbl()) {
>>> -Â Â Â Â Â Â Â  return ttm_dma_populate(&gtt->ttm, adev->dev, ctx);
>>> +Â Â Â  if ((r == -EFAULT) && swiotlb_nr_tbl()) {
>>> +Â Â Â Â Â Â Â  r = ttm_dma_populate(&gtt->ttm, adev->dev, ctx);
>>> Â Â Â Â Â  }
>>> Â  #endif
>>> -
>>> -Â Â Â  return ttm_populate_and_map_pages(adev->dev, &gtt->ttm, ctx);
>>> +Â Â Â  return r;
>>> Â  }
>>> Â  Â  static void amdgpu_ttm_tt_unpopulate(struct ttm_tt *ttm)
>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>