On 6/8/26 14:23, Michał Grzelak wrote:
> Checking `invalidated` field causes build to fail since it is absent in
> struct amdgpu_vm. Replace it with &vm->individual.moved identically
> as did commit 59720bfd8c6d ("drm/amdgpu: restart the CS if some parts of
> the VM are still invalidated").
>
> Cc: Vitaly Prosyak <[email protected]>
> Cc: Christian König <[email protected]>
> Cc: Alex Deucher <[email protected]>
> Fixes: 40396ffdf612 ("drm/amdgpu: restart the CS if some parts of the VM are
> still invalidated")
> Signed-off-by: Michał Grzelak <[email protected]>
The field is renamed by a different patch so looks like a rebase/merge issue to
me, could be that the Fixes tag needs double checking.
But either way Reviewed-by: Christian König <[email protected]>
Thanks,
Christian.
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> index 548a4f14a9f8..5d8f5848bc0e 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
> @@ -1323,7 +1323,7 @@ static int amdgpu_cs_submit(struct amdgpu_cs_parser *p,
> e->range = NULL;
> }
>
> - if (r || !list_empty(&vm->invalidated)) {
> + if (r || !list_empty(&vm->individual.moved)) {
> r = -EAGAIN;
> mutex_unlock(&p->adev->notifier_lock);
> return r;
