[AMD Official Use Only - General]

Thanks a lot, Rodrigo!

I am preparing more and more patches.

Vitaly
________________________________
From: Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com>
Sent: Wednesday, July 27, 2022 12:47 PM
To: Prosyak, Vitaly <Vitaly.Prosyak@amd.com>; Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Cc: Olsak, Marek <Marek.Olsak@amd.com>; Koenig, Christian <Christian.Koenig@amd.com>; Deucher, Alexander <Alexander.Deucher@amd.com>; igt-dev@lists.freedesktop.org <igt-dev@lists.freedesktop.org>
Subject: Re: [PATCH 1/7] lib/amdgpu: increase number of resources

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@intel.com>
> *Sent:* Monday, July 25, 2022 12:26 AM
> *To:* Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com>
> *Cc:* Olsak, Marek <Marek.Olsak@amd.com>; Koenig, Christian
> <Christian.Koenig@amd.com>; Deucher, Alexander
> <Alexander.Deucher@amd.com>; igt-dev@lists.freedesktop.org
> <igt-dev@lists.freedesktop.org>; Prosyak, Vitaly <Vitaly.Prosyak@amd.com>
> *Subject:* RE: [PATCH 1/7] lib/amdgpu: increase number of resources
> Re-reported.
>
> -----Original Message-----
> From: Rodrigo Siqueira Jordao <Rodrigo.Siqueira@amd.com>
> Sent: Friday, July 22, 2022 1:26 PM
> To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
> Cc: marek.olsak@amd.com; christian.koenig@amd.com;
> alexander.deucher@amd.com; igt-dev@lists.freedesktop.org;
> vitaly.prosyak@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@amd.com wrote:
>> From: Vitaly Prosyak <vitaly.prosyak@amd.com>
>>
>> Add arrays of bo's for VRAM and GTT.
>> Eviction test uses 4 bo resources.
>>
>> Signed-off-by: Vitaly Prosyak <vitaly.prosyak@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 */
>>
>