Re: [PATCH 2/2] drm/amdgpu: use GTT only as fallback for VRAM|GTT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[AMD Official Use Only - General]

-----Original Message-----
From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Hamza Mahfooz
Sent: Monday, November 27, 2023 10:53 AM
To: Christian König <ckoenig.leichtzumerken@xxxxxxxxx>; jani.nikula@xxxxxxxxxxxxxxx; kherbst@xxxxxxxxxx; dakr@xxxxxxxxxx; zackr@xxxxxxxxxx; Olsak, Marek <Marek.Olsak@xxxxxxx>; linux-graphics-maintainer@xxxxxxxxxx; amd-gfx@xxxxxxxxxxxxxxxxxxxxx; nouveau@xxxxxxxxxxxxxxxxxxxxx; intel-gfx@xxxxxxxxxxxxxxxxxxxxx; virtualization@xxxxxxxxxxxxxxx; spice-devel@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx
Subject: Re: [PATCH 2/2] drm/amdgpu: use GTT only as fallback for VRAM|GTT

On 11/27/23 09:54, Christian König wrote:
> Try to fill up VRAM as well by setting the busy flag on GTT allocations.
>
> This fixes the issue that when VRAM was evacuated for suspend it's
> never filled up again unless the application is restarted.

I found the subject description a bit misleading. Maybe use a Fixes tag describing it is a fix for suspend resume regression other than that, looks good to me.

Acked-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@xxxxxxx>

>

Link: https://gitlab.freedesktop.org/drm/amd/-/issues/2893

> Signed-off-by: Christian König <christian.koenig@xxxxxxx>
> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> index aa0dd6dad068..ddc8fb4db678 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
> @@ -173,6 +173,12 @@ void amdgpu_bo_placement_from_domain(struct amdgpu_bo *abo, u32 domain)
>                       abo->flags & AMDGPU_GEM_CREATE_PREEMPTIBLE ?
>                       AMDGPU_PL_PREEMPT : TTM_PL_TT;
>               places[c].flags = 0;
> +             /*
> +              * When GTT is just an alternative to VRAM make sure that we
> +              * only use it as fallback and still try to fill up VRAM first.
> +              */
> +             if (domain & AMDGPU_GEM_DOMAIN_VRAM)
> +                     places[c].flags |= TTM_PL_FLAG_BUSY;
>               c++;
>       }
>
--
Hamza





[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]