[igt-dev] [PATCH 1/7] lib/amdgpu: increase number of resources

[Rodrigo Siqueira Jordao]

Thanks Lakshmi for re-report.

Vitaly,
I merged your changes in the upstream, see:
https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/commits/master

Thanks
Siqueira

On 2022-07-26 23:14, Prosyak, Vitaly wrote:
> [AMD Official Use Only - General]
> 
> 
> Hi guys
> 
> What is wrong with the following patches:
> https://patchwork.freedesktop.org/series/106558/ 
> <https://patchwork.freedesktop.org/series/106558/>
> 
> Thanks in advance
> 
> Vitaly
> ------------------------------------------------------------------------
> *From:* Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> *Sent:* Monday, July 25, 2022 12:26 AM
> *To:* Siqueira, Rodrigo <Rodrigo.Siqueira at amd.com>
> *Cc:* Olsak, Marek <Marek.Olsak at amd.com>; Koenig, Christian 
> <Christian.Koenig at amd.com>; Deucher, Alexander 
> <Alexander.Deucher at amd.com>; igt-dev at lists.freedesktop.org 
> <igt-dev at lists.freedesktop.org>; Prosyak, Vitaly <Vitaly.Prosyak at amd.com>
> *Subject:* RE: [PATCH 1/7] lib/amdgpu: increase number of resources
> Re-reported.
> 
> -----Original Message-----
> From: Rodrigo Siqueira Jordao <Rodrigo.Siqueira at amd.com>
> Sent: Friday, July 22, 2022 1:26 PM
> To: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> Cc: marek.olsak at amd.com; christian.koenig at amd.com; 
> alexander.deucher at amd.com; igt-dev at lists.freedesktop.org; 
> vitaly.prosyak at amd.com
> Subject: Re: [PATCH 1/7] lib/amdgpu: increase number of resources
> 
> Hi Lakshmi,
> 
> I think we have a false positive result for this patchset. Could you 
> trigger the CI again?
> 
> Btw, this is the patchwork link:
> 
> https://patchwork.freedesktop.org/series/106558/ 
> <https://patchwork.freedesktop.org/series/106558/>> 
> Thanks
> Siqueira
> 
> On 2022-07-20 20:39, vitaly.prosyak at amd.com wrote:
>> From: Vitaly Prosyak <vitaly.prosyak at amd.com>
>> 
>> Add arrays of bo's for VRAM and GTT.
>> Eviction test uses 4 bo resources.
>> 
>> Signed-off-by: Vitaly Prosyak <vitaly.prosyak at amd.com>
>> ---
>>   lib/amdgpu/amd_ip_blocks.h | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>> 
>> diff --git a/lib/amdgpu/amd_ip_blocks.h b/lib/amdgpu/amd_ip_blocks.h 
>> index cb7d14747..c16ad1794 100644
>> --- a/lib/amdgpu/amd_ip_blocks.h
>> +++ b/lib/amdgpu/amd_ip_blocks.h
>> @@ -58,11 +58,13 @@ struct amdgpu_ring_context {
>>   
>>        amdgpu_bo_handle bo;
>>        amdgpu_bo_handle bo2;
>> +     amdgpu_bo_handle boa_vram[2];
>> +     amdgpu_bo_handle boa_gtt[2];
>>   
>>        amdgpu_context_handle context_handle;
>>        struct drm_amdgpu_info_hw_ip hw_ip_info;  /* result of 
>> amdgpu_query_hw_ip_info */
>>   
>> -     amdgpu_bo_handle resources[2]; /* amdgpu_bo_alloc_and_map */
>> +     amdgpu_bo_handle resources[4]; /* amdgpu_bo_alloc_and_map */
>>        amdgpu_va_handle va_handle;    /* amdgpu_bo_alloc_and_map */
>>        amdgpu_va_handle va_handle2;   /* amdgpu_bo_alloc_and_map */
>>   
>