Re: [PATCH 1/2] drm/amdgpu: Set pasid for compute vm

2018-08-28 Thread Felix Kuehling
See one more comment inline. With that fixed, the series is Reviewed-by: Felix Kuehling Regards,   Felix On 2018-08-28 01:34 PM, Oak Zeng wrote: > To make a amdgpu vm to a compute vm, the old pasid will be freed and > replaced with a pasid managed by kfd. Kfd can't reuse original pasid > allocat

[PATCH 1/2] drm/amdgpu: Set pasid for compute vm

2018-08-28 Thread Oak Zeng
To make a amdgpu vm to a compute vm, the old pasid will be freed and replaced with a pasid managed by kfd. Kfd can't reuse original pasid allocated by amdgpu because kfd uses different pasid policy with amdgpu. For example, all graphic devices share one same pasid in a process. Signed-off-by: Oak

[PATCH 1/2] drm/amdgpu: Set pasid for compute vm

2018-08-28 Thread Oak Zeng
To make a amdgpu vm to a compute vm, the old pasid will be freed and replaced with a pasid managed by kfd. Kfd can't reuse original pasid allocated by amdgpu because kfd uses different pasid policy with amdgpu. For example, all graphic devices share one same pasid in a process. Signed-off-by: Oak

Re: [PATCH 1/2] drm/amdgpu: Set pasid for compute vm

2018-08-27 Thread Felix Kuehling
On 2018-08-27 03:25 PM, Oak Zeng wrote: > To make a amdgpu vm to a compute vm, the old pasid will be freed and > replaced with a pasid managed by kfd. Kfd can't reuse original pasid > allocated by amdgpu because kfd uses different pasid policy with amdgpu. > For example, all graphic devices share o

[PATCH 1/2] drm/amdgpu: Set pasid for compute vm

2018-08-27 Thread Oak Zeng
To make a amdgpu vm to a compute vm, the old pasid will be freed and replaced with a pasid managed by kfd. Kfd can't reuse original pasid allocated by amdgpu because kfd uses different pasid policy with amdgpu. For example, all graphic devices share one same pasid in a process. Signed-off-by: Oak