On 2018å¹´05æ??11æ?¥ 09:21, Zhang, Jerry (Junwei) wrote:
> On 05/10/2018 10:40 PM, Christian König wrote:
>> Am 10.05.2018 um 07:01 schrieb Junwei Zhang:
>>> Expect to add an evitable bo who has reservation object
>>> to the correct lru[bo->priority] list
>>
>> Nice catch, but since this affects only a very small use case can we
>> just remove
>> and readd the BO to the LRU?
>>
>> A call to ttm_bo_move_to_lru_tail() after setting the priority should be
>> sufficient.
>
> Yeah, at that moment, I thought setting a priority for tbo at
> initialization is a general way, which may help all associated modules
> to get out of potential pain if more priorities are adopted.
>
> Agree.
> The most simple way is to move the bo back to expected lru after
> priority setting. Going to prepare a patch.
No, there is more simpler way. see inline...
> (in this case, include amd-gfx mail only)
>
> Regards,
> Jerry
>
>>
>> Thanks,
>> Christian.
>>
>>>
>>> Signed-off-by: Junwei Zhang <Jerry.Zhang at amd.com>
>>> ---
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 11 ++++++-----
>>>  drivers/gpu/drm/ast/ast_ttm.c              | 2 +-
>>>  drivers/gpu/drm/bochs/bochs_mm.c           | 2 +-
>>>  drivers/gpu/drm/cirrus/cirrus_ttm.c        | 2 +-
>>> Â drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c |Â 2 +-
>>>  drivers/gpu/drm/mgag200/mgag200_ttm.c      | 2 +-
>>>  drivers/gpu/drm/nouveau/nouveau_bo.c       | 2 +-
>>>  drivers/gpu/drm/qxl/qxl_object.c           | 2 +-
>>>  drivers/gpu/drm/radeon/radeon_object.c     | 2 +-
>>>  drivers/gpu/drm/ttm/ttm_bo.c               | 8 +++++---
>>>  drivers/gpu/drm/virtio/virtgpu_object.c    | 2 +-
>>>  drivers/gpu/drm/vmwgfx/vmwgfx_resource.c   | 2 +-
>>>  drivers/staging/vboxvideo/vbox_ttm.c       | 2 +-
>>>  include/drm/ttm/ttm_bo_api.h               | 5 ++++-
>>> Â 14 files changed, 26 insertions(+), 20 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> index e62153a..9a25ecb 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> @@ -360,6 +360,7 @@ static int amdgpu_bo_do_create(struct
>>> amdgpu_device *adev,
>>> Â Â Â Â Â };
>>> Â Â Â Â Â struct amdgpu_bo *bo;
>>> Â Â Â Â Â unsigned long page_align, size = bp->size;
>>> +Â Â Â uint32_t prio = 0;
>>> Â Â Â Â Â size_t acc_size;
>>> Â Â Â Â Â int r;
>>> @@ -419,10 +420,13 @@ static int amdgpu_bo_do_create(struct
>>> amdgpu_device *adev,
>>> Â Â Â Â Â bo->tbo.bdev = &adev->mman.bdev;
>>> Â Â Â Â Â amdgpu_ttm_placement_from_domain(bo, bp->domain);
>>> +Â Â Â if (bp->type == ttm_bo_type_kernel)
>>> +Â Â Â Â Â Â Â prio = 1;
Why not just bo->tbo.priority = 1? In that case, you don't need to add
more parameter to ttm_bo_init_reserved.
In fact, this change is already in my per-vm-lru patch set.
Regards,
David Zhou
>>> Â Â Â Â Â r = ttm_bo_init_reserved(&adev->mman.bdev, &bo->tbo, size,
>>> bp->type,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â &bo->placement, page_align, &ctx, acc_size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, bp->resv, &amdgpu_ttm_bo_destroy);
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â prio, &bo->placement, page_align, &ctx,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â acc_size, NULL, bp->resv,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â &amdgpu_ttm_bo_destroy);
>>> Â Â Â Â Â if (unlikely(r != 0))
>>> Â Â Â Â Â Â Â Â Â return r;
>>> @@ -434,9 +438,6 @@ static int amdgpu_bo_do_create(struct
>>> amdgpu_device *adev,
>>> Â Â Â Â Â else
>>> Â Â Â Â Â Â Â Â Â amdgpu_cs_report_moved_bytes(adev, ctx.bytes_moved, 0);
>>> -Â Â Â if (bp->type == ttm_bo_type_kernel)
>>> -Â Â Â Â Â Â Â bo->tbo.priority = 1;
>>> -
>>> Â Â Â Â Â if (bp->flags & AMDGPU_GEM_CREATE_VRAM_CLEARED &&
>>> Â Â Â Â Â Â Â Â Â bo->tbo.mem.placement & TTM_PL_FLAG_VRAM) {
>>> Â Â Â Â Â Â Â Â Â struct dma_fence *fence;
>>> diff --git a/drivers/gpu/drm/ast/ast_ttm.c
>>> b/drivers/gpu/drm/ast/ast_ttm.c
>>> index fe354eb..aabb96a 100644
>>> --- a/drivers/gpu/drm/ast/ast_ttm.c
>>> +++ b/drivers/gpu/drm/ast/ast_ttm.c
>>> @@ -320,7 +320,7 @@ int ast_bo_create(struct drm_device *dev, int
>>> size, int
>>> align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct ast_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&ast->ttm.bdev, &astbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &astbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &astbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, ast_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret)
>>> diff --git a/drivers/gpu/drm/bochs/bochs_mm.c
>>> b/drivers/gpu/drm/bochs/bochs_mm.c
>>> index 39cd084..9693109 100644
>>> --- a/drivers/gpu/drm/bochs/bochs_mm.c
>>> +++ b/drivers/gpu/drm/bochs/bochs_mm.c
>>> @@ -366,7 +366,7 @@ static int bochs_bo_create(struct drm_device
>>> *dev, int
>>> size, int align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct bochs_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&bochs->ttm.bdev, &bochsbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &bochsbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &bochsbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, bochs_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret)
>>> diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c
>>> b/drivers/gpu/drm/cirrus/cirrus_ttm.c
>>> index f219532..c1d85f8 100644
>>> --- a/drivers/gpu/drm/cirrus/cirrus_ttm.c
>>> +++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c
>>> @@ -327,7 +327,7 @@ int cirrus_bo_create(struct drm_device *dev, int
>>> size, int
>>> align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct cirrus_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&cirrus->ttm.bdev, &cirrusbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &cirrusbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &cirrusbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, cirrus_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret)
>>> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
>>> b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
>>> index 4871025..8c24731 100644
>>> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
>>> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
>>> @@ -315,7 +315,7 @@ int hibmc_bo_create(struct drm_device *dev, int
>>> size, int
>>> align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct hibmc_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&hibmc->bdev, &hibmcbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &hibmcbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &hibmcbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, hibmc_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret) {
>>> diff --git a/drivers/gpu/drm/mgag200/mgag200_ttm.c
>>> b/drivers/gpu/drm/mgag200/mgag200_ttm.c
>>> index 05570f0..e4db046 100644
>>> --- a/drivers/gpu/drm/mgag200/mgag200_ttm.c
>>> +++ b/drivers/gpu/drm/mgag200/mgag200_ttm.c
>>> @@ -323,7 +323,7 @@ int mgag200_bo_create(struct drm_device *dev,
>>> int size,
>>> int align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct mgag200_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&mdev->ttm.bdev, &mgabo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &mgabo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &mgabo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, mgag200_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret)
>>> diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c
>>> b/drivers/gpu/drm/nouveau/nouveau_bo.c
>>> index 6f402c4..8a6f949 100644
>>> --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
>>> +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
>>> @@ -297,7 +297,7 @@
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct nouveau_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&drm->ttm.bdev, &nvbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â type, &nvbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â type, 0, &nvbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size, sg,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â robj, nouveau_bo_del_ttm);
>>> Â Â Â Â Â if (ret) {
>>> diff --git a/drivers/gpu/drm/qxl/qxl_object.c
>>> b/drivers/gpu/drm/qxl/qxl_object.c
>>> index 6a30196..3e8c64b 100644
>>> --- a/drivers/gpu/drm/qxl/qxl_object.c
>>> +++ b/drivers/gpu/drm/qxl/qxl_object.c
>>> @@ -108,7 +108,7 @@ int qxl_bo_create(struct qxl_device *qdev,
>>> Â Â Â Â Â qxl_ttm_placement_from_domain(bo, domain, pinned);
>>> -Â Â Â r = ttm_bo_init(&qdev->mman.bdev, &bo->tbo, size, type,
>>> +Â Â Â r = ttm_bo_init(&qdev->mman.bdev, &bo->tbo, size, type, 0,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â &bo->placement, 0, !kernel, size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, &qxl_ttm_bo_destroy);
>>> Â Â Â Â Â if (unlikely(r != 0)) {
>>> diff --git a/drivers/gpu/drm/radeon/radeon_object.c
>>> b/drivers/gpu/drm/radeon/radeon_object.c
>>> index edbb4cd..b274e3f 100644
>>> --- a/drivers/gpu/drm/radeon/radeon_object.c
>>> +++ b/drivers/gpu/drm/radeon/radeon_object.c
>>> @@ -256,7 +256,7 @@ int radeon_bo_create(struct radeon_device *rdev,
>>> Â Â Â Â Â radeon_ttm_placement_from_domain(bo, domain);
>>> Â Â Â Â Â /* Kernel allocation are uninterruptible */
>>> Â Â Â Â Â down_read(&rdev->pm.mclk_lock);
>>> -Â Â Â r = ttm_bo_init(&rdev->mman.bdev, &bo->tbo, size, type,
>>> +Â Â Â r = ttm_bo_init(&rdev->mman.bdev, &bo->tbo, size, type, 0,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â &bo->placement, page_align, !kernel, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â sg, resv, &radeon_ttm_bo_destroy);
>>> Â Â Â Â Â up_read(&rdev->pm.mclk_lock);
>>> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c
>>> b/drivers/gpu/drm/ttm/ttm_bo.c
>>> index 98e06f8..d1f0585 100644
>>> --- a/drivers/gpu/drm/ttm/ttm_bo.c
>>> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
>>> @@ -1109,6 +1109,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device
>>> *bdev,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_buffer_object *bo,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â unsigned long size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â enum ttm_bo_type type,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â uint32_t priority,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_placement *placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t page_alignment,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_operation_ctx *ctx,
>>> @@ -1175,7 +1176,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device
>>> *bdev,
>>> Â Â Â Â Â reservation_object_init(&bo->ttm_resv);
>>> Â Â Â Â Â atomic_inc(&bo->bdev->glob->bo_count);
>>> Â Â Â Â Â drm_vma_node_reset(&bo->vma_node);
>>> -Â Â Â bo->priority = 0;
>>> +Â Â Â bo->priority = priority;
>>> Â Â Â Â Â /*
>>> Â Â Â Â Â Â * For ttm_bo_type_device buffers, allocate
>>> @@ -1219,6 +1220,7 @@ int ttm_bo_init(struct ttm_bo_device *bdev,
>>> Â Â Â Â Â Â Â Â Â struct ttm_buffer_object *bo,
>>> Â Â Â Â Â Â Â Â Â unsigned long size,
>>> Â Â Â Â Â Â Â Â Â enum ttm_bo_type type,
>>> +Â Â Â Â Â Â Â uint32_t priority,
>>> Â Â Â Â Â Â Â Â Â struct ttm_placement *placement,
>>> Â Â Â Â Â Â Â Â Â uint32_t page_alignment,
>>> Â Â Â Â Â Â Â Â Â bool interruptible,
>>> @@ -1230,7 +1232,7 @@ int ttm_bo_init(struct ttm_bo_device *bdev,
>>> Â Â Â Â Â struct ttm_operation_ctx ctx = { interruptible, false };
>>> Â Â Â Â Â int ret;
>>> -Â Â Â ret = ttm_bo_init_reserved(bdev, bo, size, type, placement,
>>> +Â Â Â ret = ttm_bo_init_reserved(bdev, bo, size, type, priority,
>>> placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â page_alignment, &ctx, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sg, resv, destroy);
>>> Â Â Â Â Â if (ret)
>>> @@ -1288,7 +1290,7 @@ int ttm_bo_create(struct ttm_bo_device *bdev,
>>> Â Â Â Â Â Â Â Â Â return -ENOMEM;
>>> Â Â Â Â Â acc_size = ttm_bo_acc_size(bdev, size, sizeof(struct
>>> ttm_buffer_object));
>>> -Â Â Â ret = ttm_bo_init(bdev, bo, size, type, placement, page_alignment,
>>> +Â Â Â ret = ttm_bo_init(bdev, bo, size, type, 0, placement,
>>> page_alignment,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â interruptible, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, NULL);
>>> Â Â Â Â Â if (likely(ret == 0))
>>> diff --git a/drivers/gpu/drm/virtio/virtgpu_object.c
>>> b/drivers/gpu/drm/virtio/virtgpu_object.c
>>> index 9f2f470..c11eba8 100644
>>> --- a/drivers/gpu/drm/virtio/virtgpu_object.c
>>> +++ b/drivers/gpu/drm/virtio/virtgpu_object.c
>>> @@ -88,7 +88,7 @@ int virtio_gpu_object_create(struct
>>> virtio_gpu_device *vgdev,
>>> Â Â Â Â Â bo->dumb = false;
>>> Â Â Â Â Â virtio_gpu_init_ttm_placement(bo, pinned);
>>> -Â Â Â ret = ttm_bo_init(&vgdev->mman.bdev, &bo->tbo, size, type,
>>> +Â Â Â ret = ttm_bo_init(&vgdev->mman.bdev, &bo->tbo, size, type, 0,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â &bo->placement, 0, !kernel, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, &virtio_gpu_ttm_bo_destroy);
>>> Â Â Â Â Â /* ttm_bo_init failure will call the destroy */
>>> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
>>> b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
>>> index 6b3a942..e7a4216 100644
>>> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
>>> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
>>> @@ -385,7 +385,7 @@ int vmw_dmabuf_init(struct vmw_private *dev_priv,
>>> Â Â Â Â Â INIT_LIST_HEAD(&vmw_bo->res_list);
>>> Â Â Â Â Â ret = ttm_bo_init(bdev, &vmw_bo->base, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â 0, interruptible, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, bo_free);
>>> Â Â Â Â Â return ret;
>>> diff --git a/drivers/staging/vboxvideo/vbox_ttm.c
>>> b/drivers/staging/vboxvideo/vbox_ttm.c
>>> index 548edb7..b93f8f0 100644
>>> --- a/drivers/staging/vboxvideo/vbox_ttm.c
>>> +++ b/drivers/staging/vboxvideo/vbox_ttm.c
>>> @@ -329,7 +329,7 @@ int vbox_bo_create(struct drm_device *dev, int
>>> size, int
>>> align,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sizeof(struct vbox_bo));
>>> Â Â Â Â Â ret = ttm_bo_init(&vbox->ttm.bdev, &vboxbo->bo, size,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, &vboxbo->placement,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â ttm_bo_type_device, 0, &vboxbo->placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â align >> PAGE_SHIFT, false, acc_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â NULL, NULL, vbox_bo_ttm_destroy);
>>> Â Â Â Â Â if (ret)
>>> diff --git a/include/drm/ttm/ttm_bo_api.h
>>> b/include/drm/ttm/ttm_bo_api.h
>>> index c67977a..f4142af 100644
>>> --- a/include/drm/ttm/ttm_bo_api.h
>>> +++ b/include/drm/ttm/ttm_bo_api.h
>>> @@ -464,6 +464,7 @@ size_t ttm_bo_dma_acc_size(struct ttm_bo_device
>>> *bdev,
>>> Â Â * @bo: Pointer to a ttm_buffer_object to be initialized.
>>> Â Â * @size: Requested size of buffer object.
>>> Â Â * @type: Requested type of buffer object.
>>> + * @priority: Requested priority of buffer object.
>>> Â Â * @flags: Initial placement flags.
>>> Â Â * @page_alignment: Data alignment in pages.
>>> Â Â * @ctx: TTM operation context for memory allocation.
>>> @@ -496,6 +497,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device
>>> *bdev,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_buffer_object *bo,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â unsigned long size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â enum ttm_bo_type type,
>>> +Â Â Â Â Â Â Â Â Â Â Â Â uint32_t priority,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_placement *placement,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t page_alignment,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct ttm_operation_ctx *ctx,
>>> @@ -511,6 +513,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device
>>> *bdev,
>>> Â Â * @bo: Pointer to a ttm_buffer_object to be initialized.
>>> Â Â * @size: Requested size of buffer object.
>>> Â Â * @type: Requested type of buffer object.
>>> + * @priority: Requested priority of buffer object.
>>> Â Â * @flags: Initial placement flags.
>>> Â Â * @page_alignment: Data alignment in pages.
>>> Â Â * @interruptible: If needing to sleep to wait for GPU resources,
>>> @@ -542,7 +545,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device
>>> *bdev,
>>> Â Â * -ERESTARTSYS: Interrupted by signal while sleeping waiting for
>>> resources.
>>> Â Â */
>>> Â int ttm_bo_init(struct ttm_bo_device *bdev, struct
>>> ttm_buffer_object *bo,
>>> -Â Â Â Â Â Â Â unsigned long size, enum ttm_bo_type type,
>>> +Â Â Â Â Â Â Â unsigned long size, enum ttm_bo_type type, uint32_t priority,
>>> Â Â Â Â Â Â Â Â Â struct ttm_placement *placement,
>>> Â Â Â Â Â Â Â Â Â uint32_t page_alignment, bool interrubtible, size_t acc_size,
>>> Â Â Â Â Â Â Â Â Â struct sg_table *sg, struct reservation_object *resv,
>>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
