From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxx> Avoids a tiny bit of arithmetic and more importantly walking those empty arrays up to amdgpu_sched_jobs slots and amdgpu_ctx_num_entities, both which are pointless if there are no rings of the respective type. Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxx> --- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c index 5cb33ac99f70..b409395f6e0c 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c @@ -979,6 +979,9 @@ void amdgpu_ctx_mgr_usage(struct amdgpu_ctx_mgr *mgr, idr_for_each_entry(&mgr->ctx_handles, ctx, id) { for (hw_ip = 0; hw_ip < AMDGPU_HW_IP_NUM; ++hw_ip) { + if (!mgr->adev->num_ip_rings[hw_ip]) + continue; + for (i = 0; i < amdgpu_ctx_num_entities[hw_ip]; ++i) { struct amdgpu_ctx_entity *centity; ktime_t spend; -- 2.44.0