Sunil Khatri
71353c1a4f
drm/amdgpu: change DRM_DBG_DRIVER to drm_dbg_driver
...
update the functions in amdgpu_userqueues.c from
DRM_DBG_DRIVER to drm_dbg_driver so multi gpu instance
can be logged in.
Signed-off-by: Sunil Khatri <sunil.khatri@amd.com >
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com >
Reviewed-by: Christian König <christian.koenig@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-05-05 13:29:38 -04:00
Sunil Khatri
c46a37628a
drm/amdgpu: change DRM_ERROR to drm_file_err in amdgpu_userq.c
...
change the DRM_ERROR and drm_err to drm_file_err
to add process name and pid to the logging.
Signed-off-by: Sunil Khatri <sunil.khatri@amd.com >
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com >
Reviewed-by: Christian König <christian.koenig@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-05-05 13:29:33 -04:00
Sunil Khatri
8c97cdb1a6
drm/amdgpu: use drm_file_err in fence timeouts
...
use drm_file_err instead of DRM_ERROR which adds
process and pid information in the userqueue error
logging.
Sample log:
[ 19.802315] amdgpu 0000:0a:00.0: [drm] *ERROR* comm: ibus-x11 pid: 2055 client: Unset ... Couldn't unmap all the queues
[ 19.802319] amdgpu 0000:0a:00.0: [drm] *ERROR* comm: ibus-x11 pid: 2055 client: Unset ... Failed to evict userqueue
[ 19.838432] amdgpu 0000:0a:00.0: [drm] *ERROR* comm: systemd-logind pid: 1042 client: Unset ... Couldn't unmap all the queues
[ 19.838436] amdgpu 0000:0a:00.0: [drm] *ERROR* comm: systemd-logind pid: 1042 client: Unset ... Failed to evict userqueue
Signed-off-by: Sunil Khatri <sunil.khatri@amd.com >
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com >
Reviewed-by: Christian König <christian.koenig@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-05-05 13:29:25 -04:00
Sunil Khatri
30ff75809d
drm/amdgpu: add drm_file reference in userq_mgr
...
drm_file will be used in usermode queues code to
enable better process information in logging and hence
add drm_file part of the userq_mgr struct.
update the drm_file pointer in userq_mgr for each
amdgpu_driver_open_kms.
Signed-off-by: Sunil Khatri <sunil.khatri@amd.com >
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com >
Reviewed-by: Christian König <christian.koenig@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-05-05 13:29:18 -04:00
Dan Carpenter
97c39b4da6
drm/amdgpu/userq: remove unnecessary NULL check
...
The "ticket" pointer points to in the middle of the &exec struct so it
can't be NULL. Remove the check.
Reviewed-by: Christian König <christian.koenig@amd.com >
Acked-by: Shashank Sharma <shashank.sharma@amd.com >
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-30 18:18:52 -04:00
Alex Deucher
482d485332
drm/amdgpu/userq: take the userq_mgr lock in enforce isolation
...
Add the missing locking.
Fixes: 94976e7e5e ("drm/amdgpu/userq: add helpers to start/stop scheduling")
Reviewed-by: Arvind Yadav <Arvind.Yadav@amd.com >
Reviewed-by: Prike Liang <Prike.Liang@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-30 18:09:26 -04:00
Alex Deucher
c5e02d6588
drm/amdgpu/userq: take the userq_mgr lock in suspend/resume
...
Add the missing locking.
Fixes: 73e12e98ec ("drm/amdgpu/userq: add suspend and resume helpers")
Reviewed-by: Arvind Yadav <Arvind.Yadav@amd.com >
Reviewed-by: Prike Liang <Prike.Liang@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-30 18:08:24 -04:00
Arvind Yadav
56801cb83c
drm/amdgpu: remove DRM_AMDGPU_NAVI3X_USERQ config for UQ
...
DRM_AMDGPU_NAVI3X_USERQ config support is not required for
usermode queue.
v2: rebase.
Cc: Arunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com >
Reviewed-by: Sunil Khatri <sunil.khatri@amd.com >
Reviewed-by: Alex Deucher <alexander.deucher@amd.com >
Reviewed-by: Christian König <christian.koenig@amd.com >
Signed-off-by: Arvind Yadav <Arvind.Yadav@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-30 18:06:00 -04:00
Sunil Khatri
127e612bf1
drm/amdgpu: update fence ptr with context:seqno
...
log context:seqno of the fence during timeout rather
than logging fence pointer.
Reviewed-by: Arvind Yadav <Arvind.Yadav@amd.com >
Cc: Tvrtko Ursulin <tvrtko.ursulin@igalia.com >
Signed-off-by: Sunil Khatri <sunil.khatri@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-22 08:51:46 -04:00
Alex Deucher
42a6667780
drm/amdgpu/userq: use consistent function naming
...
s/userqueue/userq/
1. remove the mix of amdgpu_userqueue and amdgpu_userq
2. to be consistent with other amdgpu_userq_fence.c
3. it's shorter
Reviewed-by: Prike Liang <Prike.Liang@amd.com >
Signed-off-by: Alex Deucher <alexander.deucher@amd.com >
2025-04-22 08:51:46 -04:00