On Mon, Aug 31, 2020 at 6:41 AM Nirmoy Das <nirmoy....@amd.com> wrote:
>
> UVD dependent jobs should run on the same udv instance.
> This patch disables gpu scheduler's load balancer for
> a context which binds jobs from same the context to a udv
> instance.

typos: udv -> uvd
With that fixed:
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com>

Does VCE need a similar fix?  What about UVD_ENC?

Alex


>
> Signed-off-by: Nirmoy Das <nirmoy....@amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> index 59032c26fc82..7cd398d25498 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c
> @@ -114,7 +114,9 @@ static int amdgpu_ctx_init_entity(struct amdgpu_ctx *ctx, 
> u32 hw_ip,
>         scheds = adev->gpu_sched[hw_ip][hw_prio].sched;
>         num_scheds = adev->gpu_sched[hw_ip][hw_prio].num_scheds;
>
> -       if (hw_ip == AMDGPU_HW_IP_VCN_ENC || hw_ip == AMDGPU_HW_IP_VCN_DEC) {
> +       if (hw_ip == AMDGPU_HW_IP_VCN_ENC ||
> +           hw_ip == AMDGPU_HW_IP_VCN_DEC ||
> +           hw_ip == AMDGPU_HW_IP_UVD) {
>                 sched = drm_sched_pick_best(scheds, num_scheds);
>                 scheds = &sched;
>                 num_scheds = 1;
> --
> 2.28.0
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to