On 2022-04-04 15:46, Tomasz Moń wrote:
On Mon, 2022-04-04 at 15:18 -0400, Alex Deucher wrote:On Mon, Apr 4, 2022 at 3:03 PM Tomasz Moń <desowin@xxxxxxxxx> wrote:On top of what commit does this work? It does not apply at top of v5.18-rc1.This is stuff for drm-next (https://nam11.safelinks.protection.outlook.com/?url="">). E.g., 5.19.This does not compile on current drm-next, just like I noted in the next paragraph of my email.It does apply, but fails to compile, on top of "drm/amdkfd: Create file descriptor after client is added to smi_clients list" that is commit: * cbe879c87245ce6272afe6456dbc8ce2c8f38d64 in amd-staging-drm- next * e45422695c196dbc665a95526c85ff4b8752aff2 in drm-next fetched from https://nam11.safelinks.protection.outlook.com/?url=""> The compile error is due to flush_tlb being undeclared.drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c: In function ‘amdgpu_vm_bo_update_mapping’: drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c:845:9: error: ‘flush_tlb’ undeclared (first use in this function); did you mean ‘kfd_flush_tlb’? 845 | flush_tlb |= (adev->gmc.xgmi.num_physical_nodes && | ^~~~~~~~~ | kfd_flush_tlb drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c:845:9: note: each undeclared identifier is reported only once for each function it appears in
This must be applied after patch "drm/amdgpu: fix TLB flushing during eviction", which adds local variable flush_tlb.
Regards,
Philip
Best Regards, Tomasz Mon