[Public]
Reviewed-by: Graham Sider <Graham.Sider@xxxxxxx>
> -----Original Message-----
> From: Yang, Philip <Philip.Yang@xxxxxxx>
> Sent: Wednesday, June 15, 2022 5:57 PM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Sider, Graham <Graham.Sider@xxxxxxx>; Yang, Philip
> <Philip.Yang@xxxxxxx>
> Subject: [PATCH 1/2] drm/amdkfd: Add queue to MES if it becomes active
>
> We remove the user queue from MES scheduler to update queue
> properties.
> If the queue becomes active after updating, add the user queue to MES
> scheduler, to be able to handle command packet submission.
>
> Signed-off-by: Philip Yang <Philip.Yang@xxxxxxx>
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> index e1797657b04c..67ae5b6385a2 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c
> @@ -866,8 +866,10 @@ static int update_queue(struct
> device_queue_manager *dqm, struct queue *q,
> * dqm->active_queue_count to determine whether a new runlist
> must be
> * uploaded.
> */
> - if (q->properties.is_active && !prev_active) {
> - increment_queue_count(dqm, &pdd->qpd, q);
> + if (q->properties.is_active) {
> + add_queue = true;
> + if (!prev_active)
> + increment_queue_count(dqm, &pdd->qpd, q);
> } else if (!q->properties.is_active && prev_active) {
> decrement_queue_count(dqm, &pdd->qpd, q);
> } else if (q->gws && !q->properties.is_gws) {
> --
> 2.35.1
