[PATCH 2/2] drm/amdkfd: Signal eviction fence on process destruction (v2)

[Felix Kuehling]

[moving to public mailing list]

Thank you. I'll also apply patch 2/2 to amd-staging-drm-next. It's not 
fixing a memory leak there, but it should make cleanup after process 
termination more efficient by avoiding delayed delete of BOs.

Regards,
   Felix

On 2020-03-04 10:46 p.m., Pan, Xinhui wrote:
> Series is Reviewed-by: xinhui pan <xinhui.pan at amd.com>
>
>> 2020年3月5日 05:50，Kuehling, Felix <Felix.Kuehling at amd.com> 写道：
>>
>> Otherwise BOs may wait for the fence indefinitely and never be destroyed.
>>
>> v2: Signal the fence right after destroying queues to avoid unnecessary
>>     delaye-delete in kfd_process_wq_release
>>
>> Signed-off-by: Felix Kuehling <Felix.Kuehling at amd.com>
>> ---
>> drivers/gpu/drm/amd/amdkfd/kfd_process.c | 5 +++++
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
>> index d5d4660221af..26f7f178b66d 100644
>> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
>> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
>> @@ -625,6 +625,11 @@ static void kfd_process_notifier_release(struct mmu_notifier *mn,
>>
>> 	/* Indicate to other users that MM is no longer valid */
>> 	p->mm = NULL;
>> +	/* Signal the eviction fence after user mode queues are
>> +	 * destroyed. This allows any BOs to be freed without
>> +	 * triggering pointless evictions or waiting for fences.
>> +	 */
>> +	dma_fence_signal(p->ef);
>>
>> 	mutex_unlock(&p->mutex);
>>
>> -- 
>> 2.25.1
>>