RE: [PATCH] drm/amdgpu: remove gart.ready flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[Public]

Then the most simple direction to drop the calltrace is to put off the creation of this bo after xx_gart_init is called?

Regards,
Guchun

-----Original Message-----
From: Kim, Jonathan <Jonathan.Kim@xxxxxxx> 
Sent: Thursday, January 20, 2022 11:05 PM
To: Koenig, Christian <Christian.Koenig@xxxxxxx>; Chen, Guchun <Guchun.Chen@xxxxxxx>; Christian König <ckoenig.leichtzumerken@xxxxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Subject: RE: [PATCH] drm/amdgpu: remove gart.ready flag

[Public]

Switching to a VRAM bounce buffer can drop performance around 4x~6x on Vega20 over larger access so it's not desired.

Jon

> -----Original Message-----
> From: Koenig, Christian <Christian.Koenig@xxxxxxx>
> Sent: January 20, 2022 9:10 AM
> To: Chen, Guchun <Guchun.Chen@xxxxxxx>; Christian König 
> <ckoenig.leichtzumerken@xxxxxxxxx>; Kim, Jonathan 
> <Jonathan.Kim@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH] drm/amdgpu: remove gart.ready flag
>
> I actually suggested to allocate the bounce buffer in VRAM, but that 
> add a bit more latency.
>
> Christian.
>
> Am 20.01.22 um 15:00 schrieb Chen, Guchun:
> > [Public]
> >
> > Hi Christian,
> >
> > Unfortunately, your patch brings another warning from the same
> sdma_access_bo's creation in amdgpu_ttm_init.
> >
> > In your patch, you introduce a new check of 
> > WARN_ON(!adev->gart.ptr)),
> however, sdma_access_bo targets to create a bo in GTT domain, but 
> adev-
> >gart.ptr is ready in gmc_v10_0_gart_init only.
> >
> > Hi Jonathan,
> >
> > Is it mandatory to create this sdma_access_bo in GTT domain? Can we
> change it to VRAM?
> >
> > Regards,
> > Guchun
> >
> > -----Original Message-----
> > From: Koenig, Christian <Christian.Koenig@xxxxxxx>
> > Sent: Wednesday, January 19, 2022 10:38 PM
> > To: Chen, Guchun <Guchun.Chen@xxxxxxx>; Christian König 
> > <ckoenig.leichtzumerken@xxxxxxxxx>; Kim, Jonathan 
> > <Jonathan.Kim@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> > Subject: Re: [PATCH] drm/amdgpu: remove gart.ready flag
> >
> > Hi Guchun,
> >
> > yes, just haven't found time to do this yet.
> >
> > Regards,
> > Christian.
> >
> > Am 19.01.22 um 15:24 schrieb Chen, Guchun:
> >> [Public]
> >>
> >> Hello Christian,
> >>
> >> Do you plan to submit your code to drm-next branch?
> >>
> >> Regards,
> >> Guchun
> >>
> >> -----Original Message-----
> >> From: Chen, Guchun
> >> Sent: Tuesday, January 18, 2022 10:22 PM
> >> To: 'Christian König' <ckoenig.leichtzumerken@xxxxxxxxx>; Kim, 
> >> Jonathan <Jonathan.Kim@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> >> Subject: RE: [PATCH] drm/amdgpu: remove gart.ready flag
> >>
> >> [Public]
> >>
> >> Thanks for the clarification. The patch is:
> >> Reviewed-by: Guchun Chen <guchun.chen@xxxxxxx>
> >>
> >> Regards,
> >> Guchun
> >>
> >> -----Original Message-----
> >> From: Christian König <ckoenig.leichtzumerken@xxxxxxxxx>
> >> Sent: Tuesday, January 18, 2022 10:10 PM
> >> To: Chen, Guchun <Guchun.Chen@xxxxxxx>; Kim, Jonathan 
> >> <Jonathan.Kim@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> >> Subject: Re: [PATCH] drm/amdgpu: remove gart.ready flag
> >>
> >> Am 18.01.22 um 14:28 schrieb Chen, Guchun:
> >>> [Public]
> >>>
> >>> - if (amdgpu_sriov_vf(adev) && amdgpu_in_reset(adev))
> >>> -         goto skip_pin_bo;
> >>> -
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> -
> >>> -skip_pin_bo:
> >>>
> >>> Does deleting skip_pin_bo path cause bo redundant pin in SRIOV case?
> >> Pinning/unpinning the BO was already removed as well.
> >>
> >> See Nirmoy's patches in the git log.
> >>
> >> Regards,
> >> Christian.
> >>
> >>> Regards,
> >>> Guchun
> >>>
> >>> -----Original Message-----
> >>> From: Christian König <ckoenig.leichtzumerken@xxxxxxxxx>
> >>> Sent: Tuesday, January 18, 2022 8:02 PM
> >>> To: Chen, Guchun <Guchun.Chen@xxxxxxx>; Kim, Jonathan 
> >>> <Jonathan.Kim@xxxxxxx>; amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> >>> Subject: [PATCH] drm/amdgpu: remove gart.ready flag
> >>>
> >>> That's just a leftover from old radeon days and was preventing CS 
> >>> and
> GART bindings before the hardware was initialized. But nowdays that is 
> perfectly valid.
> >>>
> >>> The only thing we need to warn about are GART binding before the
> table is even allocated.
> >>>
> >>> Signed-off-by: Christian König <christian.koenig@xxxxxxx>
> >>> ---
> >>>     drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c    | 35 +++-------
> >>>     drivers/gpu/drm/amd/amdgpu/amdgpu_gart.h    | 15 ++--
> >>>     drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c |  9 +--
> >>>     drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c     | 77 ++++++---------
> ------
> >>>     drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h     |  4 +-
> >>>     drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c      | 11 +--
> >>>     drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c       |  7 +-
> >>>     drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c       |  8 +--
> >>>     drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c       |  8 +--
> >>>     drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c       | 10 +--
> >>>     drivers/gpu/drm/amd/amdkfd/kfd_migrate.c    |  5 +-
> >>>     11 files changed, 52 insertions(+), 137 deletions(-)
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >>> index 645950a653a0..53cc844346f0 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
> >>> @@ -150,7 +150,7 @@ void amdgpu_gart_table_vram_free(struct
> amdgpu_device *adev)
> >>>      * replaces them with the dummy page (all asics).
> >>>      * Returns 0 for success, -EINVAL for failure.
> >>>      */
> >>> -int amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t
> offset,
> >>> +void amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t 
> >>> +offset,
> >>>                           int pages)
> >>>     {
> >>>           unsigned t;
> >>> @@ -161,13 +161,11 @@ int amdgpu_gart_unbind(struct
> amdgpu_device *adev, uint64_t offset,
> >>>           uint64_t flags = 0;
> >>>           int idx;
> >>>
> >>> - if (!adev->gart.ready) {
> >>> -         WARN(1, "trying to unbind memory from uninitialized GART
> !\n");
> >>> -         return -EINVAL;
> >>> - }
> >>> + if (WARN_ON(!adev->gart.ptr))
> >>> +         return;
> >>>
> >>>           if (!drm_dev_enter(adev_to_drm(adev), &idx))
> >>> -         return 0;
> >>> +         return;
> >>>
> >>>           t = offset / AMDGPU_GPU_PAGE_SIZE;
> >>>           p = t / AMDGPU_GPU_PAGES_IN_CPU_PAGE; @@ -188,7 +186,6
> @@ int
> >>> amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t offset,
> >>>                   amdgpu_gmc_flush_gpu_tlb(adev, 0, i, 0);
> >>>
> >>>           drm_dev_exit(idx);
> >>> - return 0;
> >>>     }
> >>>
> >>>     /**
> >>> @@ -204,7 +201,7 @@ int amdgpu_gart_unbind(struct
> amdgpu_device *adev, uint64_t offset,
> >>>      * Map the dma_addresses into GART entries (all asics).
> >>>      * Returns 0 for success, -EINVAL for failure.
> >>>      */
> >>> -int amdgpu_gart_map(struct amdgpu_device *adev, uint64_t offset,
> >>> +void amdgpu_gart_map(struct amdgpu_device *adev, uint64_t
> offset,
> >>>                       int pages, dma_addr_t *dma_addr, uint64_t flags,
> >>>                       void *dst)
> >>>     {
> >>> @@ -212,13 +209,8 @@ int amdgpu_gart_map(struct amdgpu_device
> *adev, uint64_t offset,
> >>>           unsigned i, j, t;
> >>>           int idx;
> >>>
> >>> - if (!adev->gart.ready) {
> >>> -         WARN(1, "trying to bind memory to uninitialized GART
> !\n");
> >>> -         return -EINVAL;
> >>> - }
> >>> -
> >>>           if (!drm_dev_enter(adev_to_drm(adev), &idx))
> >>> -         return 0;
> >>> +         return;
> >>>
> >>>           t = offset / AMDGPU_GPU_PAGE_SIZE;
> >>>
> >>> @@ -230,7 +222,6 @@ int amdgpu_gart_map(struct amdgpu_device
> *adev, uint64_t offset,
> >>>                   }
> >>>           }
> >>>           drm_dev_exit(idx);
> >>> - return 0;
> >>>     }
> >>>
> >>>     /**
> >>> @@ -246,20 +237,14 @@ int amdgpu_gart_map(struct
> amdgpu_device *adev, uint64_t offset,
> >>>      * (all asics).
> >>>      * Returns 0 for success, -EINVAL for failure.
> >>>      */
> >>> -int amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t offset,
> >>> +void amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t
> offset,
> >>>                        int pages, dma_addr_t *dma_addr,
> >>>                        uint64_t flags)
> >>>     {
> >>> - if (!adev->gart.ready) {
> >>> -         WARN(1, "trying to bind memory to uninitialized GART
> !\n");
> >>> -         return -EINVAL;
> >>> - }
> >>> -
> >>> - if (!adev->gart.ptr)
> >>> -         return 0;
> >>> + if (WARN_ON(!adev->gart.ptr))
> >>> +         return;
> >>>
> >>> - return amdgpu_gart_map(adev, offset, pages, dma_addr, flags,
> >>> -                        adev->gart.ptr);
> >>> + amdgpu_gart_map(adev, offset, pages, dma_addr, flags,
> >>> +adev->gart.ptr);
> >>>     }
> >>>
> >>>     /**
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.h
> >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.h
> >>> index 78895413cf9f..8fea3e04e411 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.h
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gart.h
> >>> @@ -46,7 +46,6 @@ struct amdgpu_gart {
> >>>           unsigned                        num_gpu_pages;
> >>>           unsigned                        num_cpu_pages;
> >>>           unsigned                        table_size;
> >>> - bool                            ready;
> >>>
> >>>           /* Asic default pte flags */
> >>>           uint64_t                        gart_pte_flags;
> >>> @@ -58,12 +57,12 @@ int amdgpu_gart_table_vram_pin(struct
> amdgpu_device *adev);  void amdgpu_gart_table_vram_unpin(struct
> amdgpu_device *adev);  int amdgpu_gart_init(struct amdgpu_device 
> *adev);  void amdgpu_gart_dummy_page_fini(struct amdgpu_device *adev); 
> -int amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t offset,
> >>> -                int pages);
> >>> -int amdgpu_gart_map(struct amdgpu_device *adev, uint64_t offset,
> >>> -             int pages, dma_addr_t *dma_addr, uint64_t flags,
> >>> -             void *dst);
> >>> -int amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t offset,
> >>> -              int pages, dma_addr_t *dma_addr, uint64_t flags);
> >>> +void amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t
> offset,
> >>> +                 int pages);
> >>> +void amdgpu_gart_map(struct amdgpu_device *adev, uint64_t
> offset,
> >>> +              int pages, dma_addr_t *dma_addr, uint64_t flags,
> >>> +              void *dst);
> >>> +void amdgpu_gart_bind(struct amdgpu_device *adev, uint64_t
> offset,
> >>> +               int pages, dma_addr_t *dma_addr, uint64_t flags);
> >>>     void amdgpu_gart_invalidate_tlb(struct amdgpu_device *adev); 
> >>> #endif diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> >>> index 9151950e0cc3..babdd915c4c3 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gtt_mgr.c
> >>> @@ -224,26 +224,21 @@ uint64_t amdgpu_gtt_mgr_usage(struct
> amdgpu_gtt_mgr *mgr)
> >>>      *
> >>>      * Re-init the gart for each known BO in the GTT.
> >>>      */
> >>> -int amdgpu_gtt_mgr_recover(struct amdgpu_gtt_mgr *mgr)
> >>> +void amdgpu_gtt_mgr_recover(struct amdgpu_gtt_mgr *mgr)
> >>>     {
> >>>           struct amdgpu_gtt_node *node;
> >>>           struct drm_mm_node *mm_node;
> >>>           struct amdgpu_device *adev;
> >>> - int r = 0;
> >>>
> >>>           adev = container_of(mgr, typeof(*adev), mman.gtt_mgr);
> >>>           spin_lock(&mgr->lock);
> >>>           drm_mm_for_each_node(mm_node, &mgr->mm) {
> >>>                   node = container_of(mm_node, typeof(*node),
> base.mm_nodes[0]);
> >>> -         r = amdgpu_ttm_recover_gart(node->tbo);
> >>> -         if (r)
> >>> -                 break;
> >>> +         amdgpu_ttm_recover_gart(node->tbo);
> >>>           }
> >>>           spin_unlock(&mgr->lock);
> >>>
> >>>           amdgpu_gart_invalidate_tlb(adev);
> >>> -
> >>> - return r;
> >>>     }
> >>>
> >>>     /**
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> >>> index b489cd8abe31..9cdb61ec669e 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
> >>> @@ -246,10 +246,7 @@ static int amdgpu_ttm_map_buffer(struct
> ttm_buffer_object *bo,
> >>>                   dma_addr_t *dma_addr;
> >>>
> >>>                   dma_addr = &bo->ttm->dma_address[mm_cur->start 
> >>> >>
> PAGE_SHIFT];
> >>> -         r = amdgpu_gart_map(adev, 0, num_pages, dma_addr,
> flags,
> >>> -                             cpu_addr);
> >>> -         if (r)
> >>> -                 goto error_free;
> >>> +         amdgpu_gart_map(adev, 0, num_pages, dma_addr, flags,
> cpu_addr);
> >>>           } else {
> >>>                   dma_addr_t dma_address;
> >>>
> >>> @@ -257,11 +254,8 @@ static int amdgpu_ttm_map_buffer(struct
> ttm_buffer_object *bo,
> >>>                   dma_address += 
> >>> adev->vm_manager.vram_base_offset;
> >>>
> >>>                   for (i = 0; i < num_pages; ++i) {
> >>> -                 r = amdgpu_gart_map(adev, i << PAGE_SHIFT, 1,
> >>> -                                     &dma_address, flags, cpu_addr);
> >>> -                 if (r)
> >>> -                         goto error_free;
> >>> -
> >>> +                 amdgpu_gart_map(adev, i << PAGE_SHIFT, 1,
> &dma_address,
> >>> +                                 flags, cpu_addr);
> >>>                           dma_address += PAGE_SIZE;
> >>>                   }
> >>>           }
> >>> @@ -826,14 +820,13 @@ static void
> amdgpu_ttm_tt_unpin_userptr(struct
> >>> ttm_device *bdev,  #endif  }
> >>>
> >>> -static int amdgpu_ttm_gart_bind(struct amdgpu_device *adev,
> >>> -                         struct ttm_buffer_object *tbo,
> >>> -                         uint64_t flags)
> >>> +static void amdgpu_ttm_gart_bind(struct amdgpu_device *adev,
> >>> +                          struct ttm_buffer_object *tbo,
> >>> +                          uint64_t flags)
> >>>     {
> >>>           struct amdgpu_bo *abo = ttm_to_amdgpu_bo(tbo);
> >>>           struct ttm_tt *ttm = tbo->ttm;
> >>>           struct amdgpu_ttm_tt *gtt = (void *)ttm;
> >>> - int r;
> >>>
> >>>           if (amdgpu_bo_encrypted(abo))
> >>>                   flags |= AMDGPU_PTE_TMZ; @@ -841,10 +834,8 @@ 
> >>> static int amdgpu_ttm_gart_bind(struct
> amdgpu_device *adev,
> >>>           if (abo->flags & AMDGPU_GEM_CREATE_CP_MQD_GFX9) {
> >>>                   uint64_t page_idx = 1;
> >>>
> >>> -         r = amdgpu_gart_bind(adev, gtt->offset, page_idx,
> >>> -                         gtt->ttm.dma_address, flags);
> >>> -         if (r)
> >>> -                 goto gart_bind_fail;
> >>> +         amdgpu_gart_bind(adev, gtt->offset, page_idx,
> >>> +                          gtt->ttm.dma_address, flags);
> >>>
> >>>                   /* The memory type of the first page defaults to UC. Now
> >>>                    * modify the memory type to NC from the second 
> >>> page of
> @@ -853,21 +844,13 @@ static int amdgpu_ttm_gart_bind(struct 
> amdgpu_device *adev,
> >>>                   flags &= ~AMDGPU_PTE_MTYPE_VG10_MASK;
> >>>                   flags |=
> AMDGPU_PTE_MTYPE_VG10(AMDGPU_MTYPE_NC);
> >>>
> >>> -         r = amdgpu_gart_bind(adev,
> >>> -                         gtt->offset + (page_idx << PAGE_SHIFT),
> >>> -                         ttm->num_pages - page_idx,
> >>> -                         &(gtt->ttm.dma_address[page_idx]), flags);
> >>> +         amdgpu_gart_bind(adev, gtt->offset + (page_idx <<
> PAGE_SHIFT),
> >>> +                          ttm->num_pages - page_idx,
> >>> +                          &(gtt->ttm.dma_address[page_idx]), 
> >>> + flags);
> >>>           } else {
> >>> -         r = amdgpu_gart_bind(adev, gtt->offset, ttm->num_pages,
> >>> -                              gtt->ttm.dma_address, flags);
> >>> +         amdgpu_gart_bind(adev, gtt->offset, ttm->num_pages,
> >>> +                          gtt->ttm.dma_address, flags);
> >>>           }
> >>> -
> >>> -gart_bind_fail:
> >>> - if (r)
> >>> -         DRM_ERROR("failed to bind %u pages at 0x%08llX\n",
> >>> -                   ttm->num_pages, gtt->offset);
> >>> -
> >>> - return r;
> >>>     }
> >>>
> >>>     /*
> >>> @@ -883,7 +866,7 @@ static int amdgpu_ttm_backend_bind(struct
> ttm_device *bdev,
> >>>           struct amdgpu_device *adev = amdgpu_ttm_adev(bdev);
> >>>           struct amdgpu_ttm_tt *gtt = (void*)ttm;
> >>>           uint64_t flags;
> >>> - int r = 0;
> >>> + int r;
> >>>
> >>>           if (!bo_mem)
> >>>                   return -EINVAL;
> >>> @@ -930,14 +913,10 @@ static int amdgpu_ttm_backend_bind(struct 
> >>> ttm_device *bdev,
> >>>
> >>>           /* bind pages into GART page tables */
> >>>           gtt->offset = (u64)bo_mem->start << PAGE_SHIFT;
> >>> - r = amdgpu_gart_bind(adev, gtt->offset, ttm->num_pages,
> >>> -         gtt->ttm.dma_address, flags);
> >>> -
> >>> - if (r)
> >>> -         DRM_ERROR("failed to bind %u pages at 0x%08llX\n",
> >>> -                   ttm->num_pages, gtt->offset);
> >>> + amdgpu_gart_bind(adev, gtt->offset, ttm->num_pages,
> >>> +                  gtt->ttm.dma_address, flags);
> >>>           gtt->bound = true;
> >>> - return r;
> >>> + return 0;
> >>>     }
> >>>
> >>>     /*
> >>> @@ -987,12 +966,7 @@ int amdgpu_ttm_alloc_gart(struct 
> >>> ttm_buffer_object *bo)
> >>>
> >>>           /* Bind pages */
> >>>           gtt->offset = (u64)tmp->start << PAGE_SHIFT;
> >>> - r = amdgpu_ttm_gart_bind(adev, bo, flags);
> >>> - if (unlikely(r)) {
> >>> -         ttm_resource_free(bo, &tmp);
> >>> -         return r;
> >>> - }
> >>> -
> >>> + amdgpu_ttm_gart_bind(adev, bo, flags);
> >>>           amdgpu_gart_invalidate_tlb(adev);
> >>>           ttm_resource_free(bo, &bo->resource);
> >>>           ttm_bo_assign_mem(bo, tmp); @@ -1006,19 +980,16 @@ int 
> >>> amdgpu_ttm_alloc_gart(struct
> ttm_buffer_object *bo)
> >>>      * Called by amdgpu_gtt_mgr_recover() from
> amdgpu_device_reset() to
> >>>      * rebind GTT pages during a GPU reset.
> >>>      */
> >>> -int amdgpu_ttm_recover_gart(struct ttm_buffer_object *tbo)
> >>> +void amdgpu_ttm_recover_gart(struct ttm_buffer_object *tbo)
> >>>     {
> >>>           struct amdgpu_device *adev = amdgpu_ttm_adev(tbo->bdev);
> >>>           uint64_t flags;
> >>> - int r;
> >>>
> >>>           if (!tbo->ttm)
> >>> -         return 0;
> >>> +         return;
> >>>
> >>>           flags = amdgpu_ttm_tt_pte_flags(adev, tbo->ttm, 
> >>> tbo->resource);
> >>> - r = amdgpu_ttm_gart_bind(adev, tbo, flags);
> >>> -
> >>> - return r;
> >>> + amdgpu_ttm_gart_bind(adev, tbo, flags);
> >>>     }
> >>>
> >>>     /*
> >>> @@ -1032,7 +1003,6 @@ static void
> amdgpu_ttm_backend_unbind(struct ttm_device *bdev,  {
> >>>           struct amdgpu_device *adev = amdgpu_ttm_adev(bdev);
> >>>           struct amdgpu_ttm_tt *gtt = (void *)ttm;
> >>> - int r;
> >>>
> >>>           /* if the pages have userptr pinning then clear that first */
> >>>           if (gtt->userptr) {
> >>> @@ -1052,10 +1022,7 @@ static void
> amdgpu_ttm_backend_unbind(struct ttm_device *bdev,
> >>>                   return;
> >>>
> >>>           /* unbind shouldn't be done for GDS/GWS/OA in
> ttm_bo_clean_mm */
> >>> - r = amdgpu_gart_unbind(adev, gtt->offset, ttm->num_pages);
> >>> - if (r)
> >>> -         DRM_ERROR("failed to unbind %u pages at 0x%08llX\n",
> >>> -                   gtt->ttm.num_pages, gtt->offset);
> >>> + amdgpu_gart_unbind(adev, gtt->offset, ttm->num_pages);
> >>>           gtt->bound = false;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h
> >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h
> >>> index 084aa49e627d..d9691f262f16 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.h
> >>> @@ -119,7 +119,7 @@ void amdgpu_vram_mgr_fini(struct
> amdgpu_device
> >>> *adev);
> >>>
> >>>     bool amdgpu_gtt_mgr_has_gart_addr(struct ttm_resource *mem); 
> >>> uint64_t amdgpu_gtt_mgr_usage(struct amdgpu_gtt_mgr *mgr); -int 
> >>> amdgpu_gtt_mgr_recover(struct amdgpu_gtt_mgr *mgr);
> >>> +void amdgpu_gtt_mgr_recover(struct amdgpu_gtt_mgr *mgr);
> >>>
> >>>     uint64_t amdgpu_preempt_mgr_usage(struct
> ttm_resource_manager
> >>> *man);
> >>>
> >>> @@ -162,7 +162,7 @@ int amdgpu_fill_buffer(struct amdgpu_bo *bo,
> >>>                           struct dma_fence **fence);
> >>>
> >>>     int amdgpu_ttm_alloc_gart(struct ttm_buffer_object *bo); -int 
> >>> amdgpu_ttm_recover_gart(struct ttm_buffer_object *tbo);
> >>> +void amdgpu_ttm_recover_gart(struct ttm_buffer_object *tbo);
> >>>     uint64_t amdgpu_ttm_domain_start(struct amdgpu_device *adev, 
> >>> uint32_t type);
> >>>
> >>>     #if IS_ENABLED(CONFIG_DRM_AMDGPU_USERPTR)
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
> >>> b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
> >>> index 4f8d356f8432..867c94a1a81a 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v10_0.c
> >>> @@ -1000,14 +1000,7 @@ static int gmc_v10_0_gart_enable(struct
> amdgpu_device *adev)
> >>>                   return -EINVAL;
> >>>           }
> >>>
> >>> - if (amdgpu_sriov_vf(adev) && amdgpu_in_reset(adev))
> >>> -         goto skip_pin_bo;
> >>> -
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> -
> >>> -skip_pin_bo:
> >>> + amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>>           r = adev->gfxhub.funcs->gart_enable(adev);
> >>>           if (r)
> >>>                   return r;
> >>> @@ -1033,8 +1026,6 @@ static int gmc_v10_0_gart_enable(struct
> amdgpu_device *adev)
> >>>                    (unsigned)(adev->gmc.gart_size >> 20),
> >>>                    (unsigned long long)amdgpu_bo_gpu_offset(adev-
> >gart.bo));
> >>>
> >>> - adev->gart.ready = true;
> >>> -
> >>>           return 0;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> >>> b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> >>> index cd6c38e083d0..3baece11d74e 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
> >>> @@ -469,16 +469,14 @@ static void gmc_v6_0_set_prt(struct
> amdgpu_device *adev, bool enable)  static int 
> gmc_v6_0_gart_enable(struct amdgpu_device *adev)  {
> >>>           uint64_t table_addr;
> >>> - int r, i;
> >>>           u32 field;
> >>> + int i;
> >>>
> >>>           if (adev->gart.bo == NULL) {
> >>>                   dev_err(adev->dev, "No VRAM object for PCIE GART.\n");
> >>>                   return -EINVAL;
> >>>           }
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> + amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>>
> >>>           table_addr = amdgpu_bo_gpu_offset(adev->gart.bo);
> >>>
> >>> @@ -558,7 +556,6 @@ static int gmc_v6_0_gart_enable(struct
> amdgpu_device *adev)
> >>>           dev_info(adev->dev, "PCIE GART of %uM enabled (table at
> 0x%016llX).\n",
> >>>                    (unsigned)(adev->gmc.gart_size >> 20),
> >>>                    (unsigned long long)table_addr);
> >>> - adev->gart.ready = true;
> >>>           return 0;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> >>> b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> >>> index ab8adbff9e2d..cf83f8c258f4 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> >>> @@ -613,17 +613,14 @@ static void gmc_v7_0_set_prt(struct
> amdgpu_device *adev, bool enable)  static int 
> gmc_v7_0_gart_enable(struct amdgpu_device *adev)  {
> >>>           uint64_t table_addr;
> >>> - int r, i;
> >>>           u32 tmp, field;
> >>> + int i;
> >>>
> >>>           if (adev->gart.bo == NULL) {
> >>>                   dev_err(adev->dev, "No VRAM object for PCIE GART.\n");
> >>>                   return -EINVAL;
> >>>           }
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> -
> >>> + amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>>           table_addr = amdgpu_bo_gpu_offset(adev->gart.bo);
> >>>
> >>>           /* Setup TLB control */
> >>> @@ -712,7 +709,6 @@ static int gmc_v7_0_gart_enable(struct
> amdgpu_device *adev)
> >>>           DRM_INFO("PCIE GART of %uM enabled (table at 0x%016llX).\n",
> >>>                    (unsigned)(adev->gmc.gart_size >> 20),
> >>>                    (unsigned long long)table_addr);
> >>> - adev->gart.ready = true;
> >>>           return 0;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> >>> b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> >>> index 054733838292..7cc7c8001734 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> >>> @@ -837,17 +837,14 @@ static void gmc_v8_0_set_prt(struct
> amdgpu_device *adev, bool enable)  static int 
> gmc_v8_0_gart_enable(struct amdgpu_device *adev)  {
> >>>           uint64_t table_addr;
> >>> - int r, i;
> >>>           u32 tmp, field;
> >>> + int i;
> >>>
> >>>           if (adev->gart.bo == NULL) {
> >>>                   dev_err(adev->dev, "No VRAM object for PCIE GART.\n");
> >>>                   return -EINVAL;
> >>>           }
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> -
> >>> + amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>>           table_addr = amdgpu_bo_gpu_offset(adev->gart.bo);
> >>>
> >>>           /* Setup TLB control */
> >>> @@ -953,7 +950,6 @@ static int gmc_v8_0_gart_enable(struct
> amdgpu_device *adev)
> >>>           DRM_INFO("PCIE GART of %uM enabled (table at 0x%016llX).\n",
> >>>                    (unsigned)(adev->gmc.gart_size >> 20),
> >>>                    (unsigned long long)table_addr);
> >>> - adev->gart.ready = true;
> >>>           return 0;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> >>> b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> >>> index c76ffd1a70cd..9f71f810c2f8 100644
> >>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> >>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> >>> @@ -1783,14 +1783,7 @@ static int gmc_v9_0_gart_enable(struct
> amdgpu_device *adev)
> >>>                   return -EINVAL;
> >>>           }
> >>>
> >>> - if (amdgpu_sriov_vf(adev) && amdgpu_in_reset(adev))
> >>> -         goto skip_pin_bo;
> >>> -
> >>> - r = amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>> - if (r)
> >>> -         return r;
> >>> -
> >>> -skip_pin_bo:
> >>> + amdgpu_gtt_mgr_recover(&adev->mman.gtt_mgr);
> >>>           r = adev->gfxhub.funcs->gart_enable(adev);
> >>>           if (r)
> >>>                   return r;
> >>> @@ -1807,7 +1800,6 @@ static int gmc_v9_0_gart_enable(struct
> amdgpu_device *adev)
> >>>           DRM_INFO("PTB located at 0x%016llX\n",
> >>>                           (unsigned long 
> >>> long)amdgpu_bo_gpu_offset(adev-
> >gart.bo));
> >>>
> >>> - adev->gart.ready = true;
> >>>           return 0;
> >>>     }
> >>>
> >>> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
> >>> b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
> >>> index d5d2cf2ee788..88db82b3d443 100644
> >>> --- a/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
> >>> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_migrate.c
> >>> @@ -86,10 +86,7 @@ svm_migrate_gart_map(struct amdgpu_ring
> *ring,
> >>> uint64_t npages,
> >>>
> >>>           cpu_addr = &job->ibs[0].ptr[num_dw];
> >>>
> >>> - r = amdgpu_gart_map(adev, 0, npages, addr, pte_flags, cpu_addr);
> >>> - if (r)
> >>> -         goto error_free;
> >>> -
> >>> + amdgpu_gart_map(adev, 0, npages, addr, pte_flags, cpu_addr);
> >>>           r = amdgpu_job_submit(job, &adev->mman.entity,
> >>>                                 AMDGPU_FENCE_OWNER_UNDEFINED, &fence);
> >>>           if (r)
> >>> --
> >>> 2.25.1





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux