Re: [BUG][5.20] refcount_t: underflow; use-after-free

2022-08-14 Thread Maíra Canal
Hi Mikhail Looks like this use-after-free problem was introduced on 90af0ca047f3049c4b46e902f432ad6ef1e2ded6. Checking this patch it seems like: if amdgpu_cs_vm_handling return r != 0, then it will unlock bo_list_mutex inside the function amdgpu_cs_vm_handling and again on amdgpu_cs_parser_fini.

[BUG][5.20] refcount_t: underflow; use-after-free

2022-08-14 Thread Mikhail Gavrilov
Hi folks. Joined testing 5.20 today (7ebfc85e2cd7). I encountered a frequently GPU freeze, after which a message appears in the kernel logs: [ 220.280990] [ cut here ] [ 220.281000] refcount_t: underflow; use-after-free. [ 220.281019] WARNING: CPU: 1 PID: 3746 at

Re: [PATCH] drm/amdgpu: Fix interrupt handling on ih_soft ring

2022-08-14 Thread Christian König
Am 12.08.22 um 22:56 schrieb Mukul Joshi: There are no backing hardware registers for ih_soft ring. As a result, don't try to access hardware registers for read and write pointers when processing interrupts on the IH soft ring. Mhm, the original plan was to have different ih function for the

Re: [PATCH] drm/amd/amdgpu: Replace kmap() with kmap_local_page()

2022-08-14 Thread Christian König
Am 12.08.22 um 19:57 schrieb Fabio M. De Francesco: kmap() is being deprecated in favor of kmap_local_page(). There are two main problems with kmap(): (1) It comes with an overhead as mapping space is restricted and protected by a global lock for synchronization and (2) it also requires global