[PATCH 1/1] drm/amdkfd: use allowed domain for vmbo validation

[Christian König]

Am 08.06.21 um 11:58 schrieb Das, Nirmoy:
>
> On 6/8/2021 11:37 AM, Christian König wrote:
>> Am 08.06.21 um 11:32 schrieb Nirmoy Das:
>>> Remove fixed domain and use BO's allowed domain for PT/PD
>>> BO validation.
>>
>> "Fixes handling when page tables are in system memory".
>>
>> And I think you can now remove the validate structure as well.
>
>
> Do you mean " struct amdgpu_vm_parser param; " ?

Yes, exactly that one.

Christian.

>
>
> Thanks,
>
> Nirmoy
>
>>
>> Christian.
>>
>>>
>>> Signed-off-by: Nirmoy Das <nirmoy.das at amd.com>
>>> ---
>>>   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 3 +--
>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c 
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
>>> index d6cb7cf76623..a45eadfbf417 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
>>> @@ -336,7 +336,7 @@ static int amdgpu_amdkfd_bo_validate(struct 
>>> amdgpu_bo *bo, uint32_t domain,
>>>            "Called with userptr BO"))
>>>           return -EINVAL;
>>>   -    amdgpu_bo_placement_from_domain(bo, domain);
>>> +    amdgpu_bo_placement_from_domain(bo, bo->allowed_domains);
>>>         ret = ttm_bo_validate(&bo->tbo, &bo->placement, &ctx);
>>>       if (ret)
>>> @@ -369,7 +369,6 @@ static int vm_validate_pt_pd_bos(struct 
>>> amdgpu_vm *vm)
>>>       struct amdgpu_vm_parser param;
>>>       int ret;
>>>   -    param.domain = AMDGPU_GEM_DOMAIN_VRAM;
>>>       param.wait = false;
>>>         ret = amdgpu_vm_validate_pt_bos(adev, vm, 
>>> amdgpu_amdkfd_validate,
>>