On Tue, Oct 17, 2023 at 8:43 AM Hamza Mahfooz <[email protected]> wrote: > > On 10/17/23 08:10, Christian König wrote: > > Looks like RADV is actually hitting this. > > > > Signed-off-by: Christian König <[email protected]> > > Fixes: ca6c1e210aa7 ("drm/amdgpu: use the new drm_exec object for CS v3") > > Do you think this will fix the following issue?: > https://gitlab.freedesktop.org/drm/amd/-/issues/2893
No. That's a known issue with data migration back to VRAM after suspend. There are TTM changes required to fix that properly. Alex > > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > > index efdb1c48f431..d93a8961274c 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > > @@ -65,7 +65,8 @@ static int amdgpu_cs_parser_init(struct amdgpu_cs_parser > > *p, > > } > > > > amdgpu_sync_create(&p->sync); > > - drm_exec_init(&p->exec, DRM_EXEC_INTERRUPTIBLE_WAIT); > > + drm_exec_init(&p->exec, DRM_EXEC_INTERRUPTIBLE_WAIT | > > + DRM_EXEC_IGNORE_DUPLICATES); > > return 0; > > } > > > -- > Hamza >
