From: Christian König <christian.koe...@amd.com>

Most VM BOs end up in the duplicates list, validate it
first make -ENOMEM less likely.

Signed-off-by: Christian König <christian.koenig at amd.com>
Reviewed-by: Chunming Zhou <David1.Zhou at amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
index f532f70..0868921 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
@@ -421,11 +421,11 @@ static int amdgpu_cs_parser_relocs(struct 
amdgpu_cs_parser *p)

        amdgpu_vm_get_pt_bos(p->adev, &fpriv->vm, &duplicates);

-       r = amdgpu_cs_list_validate(p->adev, &fpriv->vm, &p->validated);
+       r = amdgpu_cs_list_validate(p->adev, &fpriv->vm, &duplicates);
        if (r)
                goto error_validate;

-       r = amdgpu_cs_list_validate(p->adev, &fpriv->vm, &duplicates);
+       r = amdgpu_cs_list_validate(p->adev, &fpriv->vm, &p->validated);

 error_validate:
        if (r)
-- 
2.5.0

Reply via email to