Re: [PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-29 Thread Christian König
Am 28.08.2018 um 19:34 schrieb Oak Zeng: For compute vm acquired from amdgpu, vm.pasid is managed by kfd. Decouple pasid from such vm on process destroy to avoid duplicate pasid release. Signed-off-by: Oak Zeng One minor nit pick below, apart from that Acked-by: Christian König --- dri

[PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-28 Thread Oak Zeng
For compute vm acquired from amdgpu, vm.pasid is managed by kfd. Decouple pasid from such vm on process destroy to avoid duplicate pasid release. Signed-off-by: Oak Zeng --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h| 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 1 + driver

Re: [PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-28 Thread Oak
Good point. I also planned to do it the way you suggested. Sent from my iPad On 2018-08-28, at 7:21 AM, Christian König wrote: > Am 27.08.2018 um 22:45 schrieb Oak Zeng: >> For compute vm acquired from amdgpu, vm.pasid is managed >> by kfd. Decouple pasid from such vm on process destroy >> to

Re: [PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-28 Thread Christian König
Am 27.08.2018 um 22:45 schrieb Oak Zeng: For compute vm acquired from amdgpu, vm.pasid is managed by kfd. Decouple pasid from such vm on process destroy to avoid duplicate pasid release. Signed-off-by: Oak Zeng --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h| 1 + drivers/gpu/drm/amd

[PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-28 Thread Oak Zeng
For compute vm acquired from amdgpu, vm.pasid is managed by kfd. Decouple pasid from such vm on process destroy to avoid duplicate pasid release. Signed-off-by: Oak Zeng --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h| 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 1 + driver

RE: [PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-27 Thread Zeng, Oak
Thank you Felix. That is a great point. I will update patch. Thanks, Oak -Original Message- From: Kuehling, Felix Sent: Monday, August 27, 2018 4:05 PM To: Oak Zeng ; amd-gfx@lists.freedesktop.org Cc: Zeng, Oak Subject: Re: [PATCH 2/2] drm/amdkfd: Release an acquired process vm On

Re: [PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-27 Thread Felix Kuehling
On 2018-08-27 03:25 PM, Oak Zeng wrote: > For compute vm acquired from amdgpu, vm.pasid is managed > by kfd. Decouple pasid from such vm on process destroy > to avoid duplicate pasid release. > > Signed-off-by: Oak Zeng > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h| 1 + > drivers/g

[PATCH 2/2] drm/amdkfd: Release an acquired process vm

2018-08-27 Thread Oak Zeng
For compute vm acquired from amdgpu, vm.pasid is managed by kfd. Decouple pasid from such vm on process destroy to avoid duplicate pasid release. Signed-off-by: Oak Zeng --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h| 1 + drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v7.c | 1 + driver