Re: [PATCH 1/2] drm/amdgpu: free pasid early before converting a vm

2021-06-29 Thread Das, Nirmoy
On 6/23/2021 9:05 PM, Felix Kuehling wrote: On 2021-06-23 8:25 a.m., Nirmoy Das wrote: VM code should not be responsible for freeing pasid as pasid gets allocated outside of VM code, before initializing a vm. Signed-off-by: Nirmoy Das ---   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6

Re: [PATCH 1/2] drm/amdgpu: free pasid early before converting a vm

2021-06-23 Thread Felix Kuehling
On 2021-06-23 8:25 a.m., Nirmoy Das wrote: VM code should not be responsible for freeing pasid as pasid gets allocated outside of VM code, before initializing a vm. Signed-off-by: Nirmoy Das --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6 ++ drivers/gpu/drm/amd/amdgpu/amdgpu_vm

[PATCH 1/2] drm/amdgpu: free pasid early before converting a vm

2021-06-23 Thread Nirmoy Das
VM code should not be responsible for freeing pasid as pasid gets allocated outside of VM code, before initializing a vm. Signed-off-by: Nirmoy Das --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6 ++ drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 5 - 2 files changed, 6 inse

Re: [PATCH 1/2] drm/amdgpu: free pasid early before converting a vm

2021-06-23 Thread Christian König
Am 23.06.21 um 14:25 schrieb Nirmoy Das: VM code should not be responsible for freeing pasid as pasid gets allocated outside of VM code, before initializing a vm. Signed-off-by: Nirmoy Das Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 6 ++ driv