Am 15.09.2018 um 09:52 schrieb Christian König:
> Am 14.09.2018 um 22:26 schrieb Alex Deucher:
>> On Fri, Sep 14, 2018 at 3:12 PM Christian König
>> <ckoenig.leichtzumerken at gmail.com> wrote:
>>> That only worked by pure coincident. Completely remove the shifting and
>>> always apply correct PAGE_SHIFT.
>>>
>>> Signed-off-by: Christian König <christian.koenig at amd.com>
>>> ---
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c    | 12 ++++++------
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_gds.h   | 7 -------
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c   | 12 +++---------
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c   | 14 +++++++-------
>>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_object.c |Â 6 +++++-
>>>  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c   | 15 +++------------
>>>  drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c     | 9 ---------
>>>  drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c     | 9 ---------
>>>  drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c     | 12 +-----------
>>> Â 9 files changed, 25 insertions(+), 71 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>>> index d762d78e5102..8836186eb5ef 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c
>>> @@ -721,16 +721,16 @@ static int amdgpu_cs_parser_bos(struct
>>> amdgpu_cs_parser *p,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â e->bo_va = amdgpu_vm_bo_find(vm,
>>> ttm_to_amdgpu_bo(e->tv.bo));
>>>
>>> Â Â Â Â Â Â Â Â if (gds) {
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gds_base = amdgpu_bo_gpu_offset(gds);
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gds_size = amdgpu_bo_size(gds);
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gds_base = amdgpu_bo_gpu_offset(gds) >>
>>> PAGE_SHIFT;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gds_size = amdgpu_bo_size(gds) >> PAGE_SHIFT;
>>> Â Â Â Â Â Â Â Â }
>>> Â Â Â Â Â Â Â Â if (gws) {
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gws_base = amdgpu_bo_gpu_offset(gws);
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gws_size = amdgpu_bo_size(gws);
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gws_base = amdgpu_bo_gpu_offset(gws) >>
>>> PAGE_SHIFT;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->gws_size = amdgpu_bo_size(gws) >> PAGE_SHIFT;
>>> Â Â Â Â Â Â Â Â }
>>> Â Â Â Â Â Â Â Â if (oa) {
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->oa_base = amdgpu_bo_gpu_offset(oa);
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->oa_size = amdgpu_bo_size(oa);
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->oa_base = amdgpu_bo_gpu_offset(oa) >>
>>> PAGE_SHIFT;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â p->job->oa_size = amdgpu_bo_size(oa) >> PAGE_SHIFT;
>>> Â Â Â Â Â Â Â Â }
>>>
>>> Â Â Â Â Â Â Â Â if (!r && p->uf_entry.tv.bo) {
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gds.h
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gds.h
>>> index e73728d90388..ecbcefe49a98 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gds.h
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gds.h
>>> @@ -24,13 +24,6 @@
>>> Â #ifndef __AMDGPU_GDS_H__
>>> Â #define __AMDGPU_GDS_H__
>>>
>>> -/* Because TTM request that alloacted buffer should be PAGE_SIZE
>>> aligned,
>>> - * we should report GDS/GWS/OA size as PAGE_SIZE aligned
>>> - * */
>>> -#define AMDGPU_GDS_SHIFTÂ Â Â Â Â Â 2
>>> -#define AMDGPU_GWS_SHIFTÂ Â Â Â Â Â PAGE_SHIFT
>>> -#define AMDGPU_OA_SHIFTÂ Â Â Â Â Â Â Â Â Â Â Â Â Â Â PAGE_SHIFT
>>> -
>>> Â struct amdgpu_ring;
>>> Â struct amdgpu_bo;
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
>>> index d30a0838851b..7b3d1ebda9df 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c
>>> @@ -244,16 +244,10 @@ int amdgpu_gem_create_ioctl(struct drm_device
>>> *dev, void *data,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return -EINVAL;
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â }
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â flags |= AMDGPU_GEM_CREATE_NO_CPU_ACCESS;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (args->in.domains == AMDGPU_GEM_DOMAIN_GDS)
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â size = size << AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â else if (args->in.domains == AMDGPU_GEM_DOMAIN_GWS)
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â size = size << AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â else if (args->in.domains == AMDGPU_GEM_DOMAIN_OA)
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â size = size << AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â else
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return -EINVAL;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â /* GDS allocations must be DW aligned */
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (args->in.domains & AMDGPU_GEM_DOMAIN_GDS)
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â size = ALIGN(size, 4);
>>> Â Â Â Â Â Â Â Â }
>>> -Â Â Â Â Â Â size = roundup(size, PAGE_SIZE);
>>>
>>> Â Â Â Â Â Â Â Â if (flags & AMDGPU_GEM_CREATE_VM_ALWAYS_VALID) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r = amdgpu_bo_reserve(vm->root.base.bo, false);
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>>> index b766270d86cb..64cc483db973 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>>> @@ -528,13 +528,13 @@ static int amdgpu_info_ioctl(struct drm_device
>>> *dev, void *data, struct drm_file
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct drm_amdgpu_info_gds gds_info;
>>>
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â memset(&gds_info, 0, sizeof(gds_info));
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gds_gfx_partition_size =
>>> adev->gds.mem.gfx_partition_size >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.compute_partition_size =
>>> adev->gds.mem.cs_partition_size >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gds_total_size = adev->gds.mem.total_size
>>> >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gws_per_gfx_partition =
>>> adev->gds.gws.gfx_partition_size >> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gws_per_compute_partition =
>>> adev->gds.gws.cs_partition_size >> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.oa_per_gfx_partition =
>>> adev->gds.oa.gfx_partition_size >> AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.oa_per_compute_partition =
>>> adev->gds.oa.cs_partition_size >> AMDGPU_OA_SHIFT;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gds_gfx_partition_size =
>>> adev->gds.mem.gfx_partition_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.compute_partition_size =
>>> adev->gds.mem.cs_partition_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gds_total_size = adev->gds.mem.total_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gws_per_gfx_partition =
>>> adev->gds.gws.gfx_partition_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.gws_per_compute_partition =
>>> adev->gds.gws.cs_partition_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.oa_per_gfx_partition =
>>> adev->gds.oa.gfx_partition_size;
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â gds_info.oa_per_compute_partition =
>>> adev->gds.oa.cs_partition_size;
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return copy_to_user(out, &gds_info,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â min((size_t)size,
>>> sizeof(gds_info))) ? -EFAULT : 0;
>>> Â Â Â Â Â Â Â Â }
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> index e6909252aefa..e1f32a196f6d 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
>>> @@ -433,7 +433,11 @@ static int amdgpu_bo_do_create(struct
>>> amdgpu_device *adev,
>>> Â Â Â Â Â Â Â Â int r;
>>>
>>> Â Â Â Â Â Â Â Â page_align = roundup(bp->byte_align, PAGE_SIZE) >> PAGE_SHIFT;
>>> -Â Â Â Â Â Â size = ALIGN(size, PAGE_SIZE);
>>> +Â Â Â Â Â Â if (bp->domain & (AMDGPU_GEM_DOMAIN_GDS |
>>> AMDGPU_GEM_DOMAIN_GWS |
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â AMDGPU_GEM_DOMAIN_OA))
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â size <<= PAGE_SHIFT;
>>> +Â Â Â Â Â Â else
>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â size = ALIGN(size, PAGE_SIZE);
>> Is this right for GDS? I think we need to be in 4K units regardless
>> of the page size. I have to admit, all the shifting is mixing me up.
>
> Yeah, that is indeed correct. GDS only needs DW alignment.
Can anybody give me an rb or ab? Only things left in to review in this
series.
Christian.
>
> The shifting is purely done because TTM wants to work with pages and
> not bytes.
>
> Andrey already said he wanted to clean that up when there is time.
>
> Christian.
>
>>
>> Alex
>>
>>> Â Â Â Â Â Â Â Â if (!amdgpu_bo_validate_size(adev, size, bp->domain))
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return -ENOMEM;
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> index 1565344cc139..c691275cd1f0 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
>>> @@ -1825,19 +1825,10 @@ int amdgpu_ttm_init(struct amdgpu_device *adev)
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â (unsigned)(gtt_size / (1024 * 1024)));
>>>
>>> Â Â Â Â Â Â Â Â /* Initialize various on-chip memory pools */
>>> -Â Â Â Â Â Â adev->gds.mem.total_size = adev->gds.mem.total_size <<
>>> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.mem.gfx_partition_size =
>>> adev->gds.mem.gfx_partition_size << AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.mem.cs_partition_size =
>>> adev->gds.mem.cs_partition_size << AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.gws.total_size = adev->gds.gws.total_size <<
>>> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.gws.gfx_partition_size =
>>> adev->gds.gws.gfx_partition_size << AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.gws.cs_partition_size =
>>> adev->gds.gws.cs_partition_size << AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.oa.total_size = adev->gds.oa.total_size <<
>>> AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.oa.gfx_partition_size =
>>> adev->gds.oa.gfx_partition_size << AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â adev->gds.oa.cs_partition_size =
>>> adev->gds.oa.cs_partition_size << AMDGPU_OA_SHIFT;
>>> Â Â Â Â Â Â Â Â /* GDS Memory */
>>> Â Â Â Â Â Â Â Â if (adev->gds.mem.total_size) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r = ttm_bo_init_mm(&adev->mman.bdev, AMDGPU_PL_GDS,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gds.mem.total_size >>
>>> PAGE_SHIFT);
>>> + adev->gds.mem.total_size);
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (r) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DRM_ERROR("Failed initializing GDS heap.\n");
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return r;
>>> @@ -1847,7 +1838,7 @@ int amdgpu_ttm_init(struct amdgpu_device *adev)
>>> Â Â Â Â Â Â Â Â /* GWS */
>>> Â Â Â Â Â Â Â Â if (adev->gds.gws.total_size) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r = ttm_bo_init_mm(&adev->mman.bdev, AMDGPU_PL_GWS,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gds.gws.total_size >>
>>> PAGE_SHIFT);
>>> + adev->gds.gws.total_size);
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (r) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DRM_ERROR("Failed initializing gws heap.\n");
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return r;
>>> @@ -1857,7 +1848,7 @@ int amdgpu_ttm_init(struct amdgpu_device *adev)
>>> Â Â Â Â Â Â Â Â /* OA */
>>> Â Â Â Â Â Â Â Â if (adev->gds.oa.total_size) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â r = ttm_bo_init_mm(&adev->mman.bdev, AMDGPU_PL_OA,
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->gds.oa.total_size >>
>>> PAGE_SHIFT);
>>> + adev->gds.oa.total_size);
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (r) {
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DRM_ERROR("Failed initializing oa heap.\n");
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return r;
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
>>> b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
>>> index a15d9c0f233b..c0f9732cbaf7 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
>>> @@ -4170,15 +4170,6 @@ static void
>>> gfx_v7_0_ring_emit_gds_switch(struct amdgpu_ring *ring,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t gws_base,
>>> uint32_t gws_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t oa_base,
>>> uint32_t oa_size)
>>> Â {
>>> -Â Â Â Â Â Â gds_base = gds_base >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â gds_size = gds_size >> AMDGPU_GDS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â gws_base = gws_base >> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â gws_size = gws_size >> AMDGPU_GWS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â oa_base = oa_base >> AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â oa_size = oa_size >> AMDGPU_OA_SHIFT;
>>> -
>>> Â Â Â Â Â Â Â Â /* GDS Base */
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, PACKET3(PACKET3_WRITE_DATA, 3));
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, (WRITE_DATA_ENGINE_SEL(0) |
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>>> b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>>> index 3882689b2d8f..57e4b14e3bd1 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
>>> @@ -5396,15 +5396,6 @@ static void
>>> gfx_v8_0_ring_emit_gds_switch(struct amdgpu_ring *ring,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t gws_base,
>>> uint32_t gws_size,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t oa_base,
>>> uint32_t oa_size)
>>> Â {
>>> -Â Â Â Â Â Â gds_base = gds_base >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â gds_size = gds_size >> AMDGPU_GDS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â gws_base = gws_base >> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â gws_size = gws_size >> AMDGPU_GWS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â oa_base = oa_base >> AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â oa_size = oa_size >> AMDGPU_OA_SHIFT;
>>> -
>>> Â Â Â Â Â Â Â Â /* GDS Base */
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, PACKET3(PACKET3_WRITE_DATA, 3));
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, (WRITE_DATA_ENGINE_SEL(0) |
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>>> b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>>> index 75a91663019f..d31a2bc00d61 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
>>> @@ -1527,8 +1527,7 @@ static int gfx_v9_0_ngg_en(struct
>>> amdgpu_device *adev)
>>> Â Â Â Â Â Â Â Â gfx_v9_0_write_data_to_reg(ring, 0, false,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â SOC15_REG_OFFSET(GC, 0,
>>> mmGDS_VMID0_SIZE),
>>> (adev->gds.mem.total_size +
>>> - adev->gfx.ngg.gds_reserve_size) >>
>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â AMDGPU_GDS_SHIFT);
>>> + adev->gfx.ngg.gds_reserve_size));
>>>
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, PACKET3(PACKET3_DMA_DATA, 5));
>>> Â Â Â Â Â Â Â Â amdgpu_ring_write(ring, (PACKET3_DMA_DATA_CP_SYNC |
>>> @@ -3472,15 +3471,6 @@ static void
>>> gfx_v9_0_ring_emit_gds_switch(struct amdgpu_ring *ring,
>>> Â {
>>> Â Â Â Â Â Â Â Â struct amdgpu_device *adev = ring->adev;
>>>
>>> -Â Â Â Â Â Â gds_base = gds_base >> AMDGPU_GDS_SHIFT;
>>> -Â Â Â Â Â Â gds_size = gds_size >> AMDGPU_GDS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â gws_base = gws_base >> AMDGPU_GWS_SHIFT;
>>> -Â Â Â Â Â Â gws_size = gws_size >> AMDGPU_GWS_SHIFT;
>>> -
>>> -Â Â Â Â Â Â oa_base = oa_base >> AMDGPU_OA_SHIFT;
>>> -Â Â Â Â Â Â oa_size = oa_size >> AMDGPU_OA_SHIFT;
>>> -
>>> Â Â Â Â Â Â Â Â /* GDS Base */
>>> Â Â Â Â Â Â Â Â gfx_v9_0_write_data_to_reg(ring, 0, false,
>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â SOC15_REG_OFFSET(GC, 0,
>>> mmGDS_VMID0_BASE) + 2 * vmid,
>>> --
>>> 2.14.1
>>>
>>> _______________________________________________
>>> amd-gfx mailing list
>>> amd-gfx at lists.freedesktop.org
>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
>
