On 6/8/2021 5:28 PM, Zeng, Oak wrote:
Hi Nirmoy,
Why keep a unused parameter: +static int amdgpu_amdkfd_validate_vm_bo(void
*_unused.
We pass this func to amdgpu_vm_validate_pt_bos() which requires two
args: int (*validate)(void *p, struct amdgpu_bo *bo)
When I looked the codes, the
Hi Nirmoy,
Why keep a unused parameter: +static int amdgpu_amdkfd_validate_vm_bo(void
*_unused.
When I looked the codes, the only logic change is the validate page table bo in
allowed_domain instead of vram domain. Can you explain why validate page table
bo in vram domain cause a problem?
On 6/8/2021 1:42 PM, Christian König wrote:
Am 08.06.21 um 13:27 schrieb Nirmoy Das:
Fixes handling when page tables are in system memory.
v2: remove unwanted variable.
change amdgpu_amdkfd_validate instead of amdgpu_amdkfd_bo_validate.
Signed-off-by: Nirmoy Das
---
Fixes handling when page tables are in system memory.
v2: remove unwanted variable.
change amdgpu_amdkfd_validate instead of amdgpu_amdkfd_bo_validate.
Signed-off-by: Nirmoy Das
---
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 15 ---
1 file changed, 4 insertions(+), 11
Am 08.06.21 um 13:27 schrieb Nirmoy Das:
Fixes handling when page tables are in system memory.
v2: remove unwanted variable.
change amdgpu_amdkfd_validate instead of amdgpu_amdkfd_bo_validate.
Signed-off-by: Nirmoy Das
---
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 15