Reviewed-by: Chunming Zhou <david1.zhou at amd.com> as well.
On 2017å¹´12æ??14æ?¥ 16:20, Christian König wrote:
> Here again a better commit message is needed. Just something like the
> following:
>
> This enables eviction of other per VM BOs during allocation and allows
> reaping of deleted per VM BOs during CS again.
>
> With that fixed the patch is Reviewed-by: Christian König
> <christian.koenig at amd.com>.
>
> Regards,
> Christian.
>
> Am 14.12.2017 um 09:12 schrieb Roger He:
>> Change-Id: I0c6571c2a64e6c5bdad80ccbcccb40eba1c20b4e
>> Signed-off-by: Roger He <Hongbo.He at amd.com>
>> ---
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c    | 6 +++++-
>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 7 ++++++-
>> Â 2 files changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>> index 44523a8..ace6152 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>> @@ -343,7 +343,11 @@ static int amdgpu_cs_bo_validate(struct
>> amdgpu_cs_parser *p,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct amdgpu_bo *bo)
>> Â {
>> Â Â Â Â Â struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
>> -Â Â Â struct ttm_operation_ctx ctx = { true, false };
>> +Â Â Â struct ttm_operation_ctx ctx = {
>> +Â Â Â Â Â Â Â .interruptible = true,
>> +Â Â Â Â Â Â Â .no_wait_gpu = false,
>> +Â Â Â Â Â Â Â .resv = bo->tbo.resv
>> +Â Â Â };
>> Â Â Â Â Â uint32_t domain;
>> Â Â Â Â Â int r;
>> Â diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> index dc0a8be..86340cf 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>> @@ -327,7 +327,12 @@ static int amdgpu_bo_do_create(struct
>> amdgpu_device *adev,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint64_t init_value,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct amdgpu_bo **bo_ptr)
>> Â {
>> -Â Â Â struct ttm_operation_ctx ctx = { !kernel, false };
>> +Â Â Â struct ttm_operation_ctx ctx = {
>> +Â Â Â Â Â Â Â .interruptible = !kernel,
>> +Â Â Â Â Â Â Â .no_wait_gpu = false,
>> +Â Â Â Â Â Â Â .allow_reserved_eviction = true,
>> +Â Â Â Â Â Â Â .resv = resv
>> +Â Â Â };
>> Â Â Â Â Â struct amdgpu_bo *bo;
>> Â Â Â Â Â enum ttm_bo_type type;
>> Â Â Â Â Â unsigned long page_align;
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
