[PATCH] drm/amdgpu/gfx: use a single pipe for vega10 for now

Alex Deucher alexdeucher at gmail.com
Tue Jun 13 03:03:42 UTC 2017


On Mon, Jun 12, 2017 at 3:17 PM, Alex Deucher <alexdeucher at gmail.com> wrote:
> Spreading queues across pipes regresses performance right now
> disable until we root cause the issue.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>

Ignore this patch.  Evan found the root cause.

Alex

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> index dfbf027..6540cd8 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.c
> @@ -125,7 +125,11 @@ void amdgpu_gfx_compute_queue_acquire(struct amdgpu_device *adev)
>                 if (mec >= adev->gfx.mec.num_mec)
>                         break;
>
> -               if (adev->gfx.mec.num_mec > 1) {
> +               if (adev->asic_type >= CHIP_VEGA10) {
> +                       /* policy: amdgpu owns all queues in the first pipe */
> +                       if (mec == 0 && pipe == 0)
> +                               set_bit(i, adev->gfx.mec.queue_bitmap);
> +               } else if (adev->gfx.mec.num_mec > 1) {
>                         /* policy: amdgpu owns the first two queues of the first MEC */
>                         if (mec == 0 && queue < 2)
>                                 set_bit(i, adev->gfx.mec.queue_bitmap);
> --
> 2.5.5
>


More information about the amd-gfx mailing list