Re: [PATCH 2/3] drm/amdgpu: fix missing check in vm_flush()

Alex Deucher <[email protected]>
Newsgroups org.freedesktop.lists.amd-gfx
Message-ID <CADnq5_N8WgGpn15uBynApdkqJdnSaeOFXUyzpG8Q3_3iyHcK1w@mail.gmail.com>
+ David

Ping?

On Fri, Aug 7, 2026 at 9:04 AM Alex Deucher <[email protected]> wrote:
>
> Ping?
>
> Alex
>
> On Fri, Jun 5, 2026 at 6:44 PM Alex Deucher <[email protected]> wrote:
> >
> > We shouldn't return early if we need to emit spm update.
> >
> > Signed-off-by: Alex Deucher <[email protected]>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> > index 7e0e2281719b1..13031e4b6f1d5 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
> > @@ -809,7 +809,7 @@ void amdgpu_vm_flush(struct amdgpu_ring *ring, struct amdgpu_job *job,
> >                 &job->base.s_fence->scheduled == isolation->spearhead;
> >
> >         if (!vm_flush_needed && !gds_switch_needed && !need_pipe_sync &&
> > -           !cleaner_shader_needed)
> > +           !cleaner_shader_needed && !spm_update_needed)
> >                 return;
> >
> >         amdgpu_ring_ib_begin(ring);
> > --
> > 2.54.0
> >
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.