drm/amdkfd: Simplify eviction state logic

Browse source

Always mark evicted queues with q->properties.is_evicted = true, even
queues that are inactive for other reason. This simplifies maintaining
the eviction state as it doesn't require updating is_evicted when other
queue activation conditions change.

On the other hand, we now need to check those other queue activation
conditions whenever an evicted queues is restored. To minimize code
duplication, move the queue activation check into a macro so it can be
maintained in one central place.

Signed-off-by: Felix Kuehling <Felix.Kuehling@amd.com>
Reviewed-by: Philip Cox <Philip.Cox@amd.com>
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

...

This commit is contained in:

Felix Kuehling 2019-05-01 18:20:13 -04:00 • committed by Alex Deucher

parent 052af915d8

commit bb2d2128a5

5 changed files with 56 additions and 68 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

84

drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c

View file

15

drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_cik.c

View file

10

drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_v9.c

View file

10

drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c

View file

5

drivers/gpu/drm/amd/amdkfd/kfd_priv.h

View file