/Monk
*From:*Christian König [mailto:ckoenig.leichtzumer...@gmail.com]
*Sent:* 2018年4月18日16:36
*To:* Liu, Monk <monk@amd.com>; Koenig, Christian
<christian.koe...@amd.com>; Deng, Emily <emily.d...@amd.com>
*Cc:* amd-gfx@lists.freedesktop.org
*Subject:* Re:
;;
Deng, Emily <emily.d...@amd.com>
Cc: amd-gfx@lists.freedesktop.org
Subject: Re: force app kill patch
See that in “sched_entity_fini”, we only call
dma_fence_put(entity->last_scheduled” under the condition of “If
(entity->fini_status)”, so
This way there is memory leak for the case of “en
See that in “sched_entity_fini”, we only call
dma_fence_put(entity->last_scheduled”under the condition of “If
(entity->fini_status)”, so
This way there is memory leak for the case of “entity->fini_stats ==0”
Good catch, we indeed should fix that.
1.Drm_sched_entity_fini(): it exit right
*Correctio for the scenario *
After we move fence_put(entity->last_sched) out of the fini_status check:
A potential race issue for the scenario:
1.Drm_sched_entity_fini(): it exit right after entity->job_queue empty,
[ but that time scheduler is not fast enough to deal with this