Re: [PATCH 5/7] drm/amdkfd: Remove duplicate pqm_uninit()

2016-09-12 Thread Felix Kuehling
Yong is currently working on a follow-up fix to handle process shutdown on APUs as well as S3 suspend correctly. Regards, Felix On 16-09-10 12:06 PM, Oded Gabbay wrote: > On Sat, Sep 10, 2016 at 4:31 AM, Edward O'Callaghan > wrote: >> pqm_uninit() will be called in kfd_process_notifier_releas

Re: [PATCH 5/7] drm/amdkfd: Remove duplicate pqm_uninit()

2016-09-10 Thread Oded Gabbay
On Sat, Sep 10, 2016 at 4:31 AM, Edward O'Callaghan wrote: > pqm_uninit() will be called in kfd_process_notifier_release(), which > is when the process exits. Calling it in kfd_unbind_process_from_device() > is duplicate and in fact incorrect, because pqm should not be > uninitalized as long as th