[PATCH] drm/amdgpu: use amdgpu_vm_get_pd_bo in the GEM code

Alex Deucher alexdeucher at gmail.com
Thu Sep 29 16:35:11 UTC 2016


On Thu, Sep 29, 2016 at 8:14 AM, Christian König
<deathsimple at vodafone.de> wrote:
> From: Christian König <christian.koenig at amd.com>
>
> Instead of messing with the PD directly.
>
> Signed-off-by: Christian König <christian.koenig at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> index 8ff90fe..fd0c13b 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
> @@ -555,7 +555,8 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
>         struct amdgpu_fpriv *fpriv = filp->driver_priv;
>         struct amdgpu_bo *abo;
>         struct amdgpu_bo_va *bo_va;
> -       struct ttm_validate_buffer tv, tv_pd;
> +       struct amdgpu_bo_list_entry vm_pd;
> +       struct ttm_validate_buffer tv;
>         struct ww_acquire_ctx ticket;
>         struct list_head list, duplicates;
>         uint32_t invalid_flags, va_flags = 0;
> @@ -600,9 +601,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
>         tv.shared = true;
>         list_add(&tv.head, &list);
>
> -       tv_pd.bo = &fpriv->vm.page_directory->tbo;
> -       tv_pd.shared = true;
> -       list_add(&tv_pd.head, &list);
> +       amdgpu_vm_get_pd_bo(&fpriv->vm, &list, &vm_pd);
>
>         r = ttm_eu_reserve_buffers(&ticket, &list, true, &duplicates);
>         if (r) {
> --
> 2.5.0
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the amd-gfx mailing list