Am 30.08.2018 um 05:29 schrieb Zhang, Jerry (Junwei):
> On 08/29/2018 10:08 PM, Christian König wrote:
>> Otherwise we won't be able to use the AGP aperture.
>
> do you mean we use AGP for GTT shadow only now?
No, on older hw generations the page tables are usually larger than one
PAGE.
So we need to work a bit more on this,
Christian.
>
> Jerry
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 5 +----
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c    | 5 +++++
>> Â 2 files changed, 6 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> index 0cbf651a88a6..de990bdcdd6c 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> @@ -163,10 +163,7 @@ void amdgpu_bo_placement_from_domain(struct
>> amdgpu_bo *abo, u32 domain)
>>
>> Â Â Â Â Â if (domain & AMDGPU_GEM_DOMAIN_GTT) {
>> Â Â Â Â Â Â Â Â Â places[c].fpfn = 0;
>> -Â Â Â Â Â Â Â if (flags & AMDGPU_GEM_CREATE_SHADOW)
>> -Â Â Â Â Â Â Â Â Â Â Â places[c].lpfn = adev->gmc.gart_size >> PAGE_SHIFT;
>> -Â Â Â Â Â Â Â else
>> -Â Â Â Â Â Â Â Â Â Â Â places[c].lpfn = 0;
>> +Â Â Â Â Â Â Â places[c].lpfn = 0;
>> Â Â Â Â Â Â Â Â Â places[c].flags = TTM_PL_FLAG_TT;
>> Â Â Â Â Â Â Â Â Â if (flags & AMDGPU_GEM_CREATE_CPU_GTT_USWC)
>> Â Â Â Â Â Â Â Â Â Â Â Â Â places[c].flags |= TTM_PL_FLAG_WC |
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>> index a3675c7b6190..abe1db4c63f9 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
>> @@ -346,6 +346,11 @@ int amdgpu_vm_validate_pt_bos(struct
>> amdgpu_device *adev, struct amdgpu_vm *vm,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â r = amdgpu_ttm_alloc_gart(&bo->tbo);
>> Â Â Â Â Â Â Â Â Â Â Â Â Â if (r)
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> +Â Â Â Â Â Â Â Â Â Â Â if (bo->shadow) {
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r = amdgpu_ttm_alloc_gart(&bo->shadow->tbo);
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (r)
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> +Â Â Â Â Â Â Â Â Â Â Â }
>> Â Â Â Â Â Â Â Â Â Â Â Â Â list_move(&bo_base->vm_status, &vm->relocated);
>> Â Â Â Â Â Â Â Â Â }
>> Â Â Â Â Â }
>>
