Am 18.07.2018 um 15:40 schrieb James Zhu:
>
> On 2018-07-18 08:55 AM, Christian König wrote:
>> Going to completely rework the context to ring mapping with Nayan's GSoC
>> work, but for now just stopping to expose the second UVD instance should
>> do it.
>>
>> Signed-off-by: Christian König <christian.koenig at amd.com>
>> ---
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c      | 13 +++++--------
>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_queue_mgr.c |Â 9 ++-------
>> Â 2 files changed, 7 insertions(+), 15 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> index e451a3f25beb..75da3c41f3b3 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> @@ -286,7 +286,7 @@ static int amdgpu_info_ioctl(struct drm_device
>> *dev, void *data, struct drm_file
>> Â Â Â Â Â struct drm_crtc *crtc;
>> Â Â Â Â Â uint32_t ui32 = 0;
>> Â Â Â Â Â uint64_t ui64 = 0;
>> -Â Â Â int i, j, found;
>> +Â Â Â int i, found;
>> Â Â Â Â Â int ui32_size = sizeof(ui32);
>> Â Â Â Â Â Â if (!info->return_size || !info->return_pointer)
>> @@ -348,8 +348,7 @@ static int amdgpu_info_ioctl(struct drm_device
>> *dev, void *data, struct drm_file
>> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â Â Â Â Â case AMDGPU_HW_IP_UVD:
>> Â Â Â Â Â Â Â Â Â Â Â Â Â type = AMD_IP_BLOCK_TYPE_UVD;
>> -Â Â Â Â Â Â Â Â Â Â Â for (i = 0; i < adev->uvd.num_uvd_inst; i++)
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ring_mask |= adev->uvd.inst[i].ring.ready << i;
>> +Â Â Â Â Â Â Â Â Â Â Â ring_mask |= adev->uvd.inst[0].ring.ready;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â ib_start_alignment = 64;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â ib_size_alignment = 64;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> @@ -362,11 +361,9 @@ static int amdgpu_info_ioctl(struct drm_device
>> *dev, void *data, struct drm_file
>> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â Â Â Â Â case AMDGPU_HW_IP_UVD_ENC:
>> Â Â Â Â Â Â Â Â Â Â Â Â Â type = AMD_IP_BLOCK_TYPE_UVD;
>> -Â Â Â Â Â Â Â Â Â Â Â for (i = 0; i < adev->uvd.num_uvd_inst; i++)
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â for (j = 0; j < adev->uvd.num_enc_rings; j++)
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ring_mask |=
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->uvd.inst[i].ring_enc[j].ready <<
>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â (j + i * adev->uvd.num_enc_rings);
>> +Â Â Â Â Â Â Â Â Â Â Â for (i = 0; i < adev->uvd.num_enc_rings; i++)
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ring_mask |=
>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â adev->uvd.inst[0].ring_enc[i].ready << i;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â ib_start_alignment = 64;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â ib_size_alignment = 64;
>> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_queue_mgr.c
>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_queue_mgr.c
>> index ea9850c9224d..d8357290ad09 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_queue_mgr.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_queue_mgr.c
>> @@ -66,8 +66,6 @@ static int amdgpu_identity_map(struct amdgpu_device
>> *adev,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u32 ring,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â struct amdgpu_ring **out_ring)
>> Â {
>> -Â Â Â u32 instance;
>> -
>> Â Â Â Â Â switch (mapper->hw_ip) {
>> Â Â Â Â Â case AMDGPU_HW_IP_GFX:
>> Â Â Â Â Â Â Â Â Â *out_ring = &adev->gfx.gfx_ring[ring];
>> @@ -79,16 +77,13 @@ static int amdgpu_identity_map(struct
>> amdgpu_device *adev,
>> Â Â Â Â Â Â Â Â Â *out_ring = &adev->sdma.instance[ring].ring;
>> Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â case AMDGPU_HW_IP_UVD:
>> -Â Â Â Â Â Â Â instance = ring;
>> -Â Â Â Â Â Â Â *out_ring = &adev->uvd.inst[instance].ring;
>> +Â Â Â Â Â Â Â *out_ring = &adev->uvd.inst[0].ring;
>> Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â case AMDGPU_HW_IP_VCE:
>> Â Â Â Â Â Â Â Â Â *out_ring = &adev->vce.ring[ring];
>> Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â case AMDGPU_HW_IP_UVD_ENC:
>> -Â Â Â Â Â Â Â instance = ring / adev->uvd.num_enc_rings;
>> -Â Â Â Â Â Â Â *out_ring =
>> - &adev->uvd.inst[instance].ring_enc[ring%adev->uvd.num_enc_rings];
>> +Â Â Â Â Â Â Â *out_ring = &adev->uvd.inst[0].ring_enc[ring];
>
> Hi Christian,
>
> If here uses amdgpu_identity_map, I think only first instance's ring
> is in use.
>
> In current Vega20 situation, should we change to use amdgpu_lru_map or
> something else to schedule which instance/ring for use ?
Yes and no. I'm going to completely remove the queue manager and replace
it with Nayan's work on the scheduler.
So no more lru map, but instead a function which picks the best ring
based on actual load during context creation.
Regards,
Christian.
>
> James
>
>> Â Â Â Â Â Â Â Â Â break;
>> Â Â Â Â Â case AMDGPU_HW_IP_VCN_DEC:
>> Â Â Â Â Â Â Â Â Â *out_ring = &adev->vcn.ring_dec;
>
