[PATCH 3/8] drm/amdkfd: Fix SDMA oversubsription handling

[Oded Gabbay]

On Thu, Nov 2, 2017 at 1:21 AM, Felix Kuehling <Felix.Kuehling at amd.com> wrote:
> SDMA only supports a fixed number of queues. HWS cannot handle
> oversubscription.
>
> Signed-off-by: shaoyun liu <shaoyun.liu at amd.com>
> Signed-off-by: Felix Kuehling <Felix.Kuehling at amd.com>
> ---
>  drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c b/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> index 2bec902..a3f1e62 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process_queue_manager.c
> @@ -191,6 +191,24 @@ int pqm_create_queue(struct process_queue_manager *pqm,
>
>         switch (type) {
>         case KFD_QUEUE_TYPE_SDMA:
> +               if (dev->dqm->queue_count >=
> +                       CIK_SDMA_QUEUES_PER_ENGINE * CIK_SDMA_ENGINE_NUM) {
> +                       pr_err("Over-subscription is not allowed for SDMA.\n");
> +                       retval = -EPERM;
> +                       goto err_create_queue;
> +               }
> +
> +               retval = create_cp_queue(pqm, dev, &q, properties, f, *qid);
> +               if (retval != 0)
> +                       goto err_create_queue;
> +               pqn->q = q;
> +               pqn->kq = NULL;
> +               retval = dev->dqm->ops.create_queue(dev->dqm, q, &pdd->qpd,
> +                                               &q->properties.vmid);
> +               pr_debug("DQM returned %d for create_queue\n", retval);
> +               print_queue(q);
> +               break;
> +
>         case KFD_QUEUE_TYPE_COMPUTE:
>                 /* check if there is over subscription */
>                 if ((sched_policy == KFD_SCHED_POLICY_HWS_NO_OVERSUBSCRIPTION) &&
> --
> 2.7.4
>
This patch is:
Reviewed-by: Oded Gabbay <oded.gabbay at gmail.com>