Re: [igt-dev] [PATCH i-g-t 0/2] Modify amd_link_settings test

["Vudum, Lakshminarayana" <lakshminarayana.vudum@intel.com>]

Re-reported.

-----Original Message-----
From: Rodrigo Siqueira Jordao <rjordrigo@amd.com> 
Sent: Tuesday, March 8, 2022 12:34 PM
To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>
Cc: nicholas.choi@amd.com; harry.wentland@amd.com; nicholas.kazlauskas@amd.com; sunpeng.li@amd.com; rodrigo.siqueira@amd.com; igt-dev@lists.freedesktop.org; Wayne Lin <Wayne.Lin@amd.com>
Subject: Re: [PATCH i-g-t 0/2] Modify amd_link_settings test



On 2022-03-02 03:46, Wayne Lin wrote:
> "7f484824 lib/igt_amd: correct the link_settings bug"
> After applying above patch to fix a bug within igt_amd.c, it starts to 
> fail the test amd_link_settings@link-training-configs. The main 
> readson for the failure is due to inappropriate implementation of this test.
> 
> This patch series is trying to adjust some steps in the test.
> 
> Wayne Lin (2):
>    tests/amdgpu/amd_link_settings: Ignore CRC checking
>    tests/amdgpu/amd_link_settings: Change the flow when exit test
> 
>   tests/amdgpu/amd_link_settings.c | 67 ++++++--------------------------
>   1 file changed, 11 insertions(+), 56 deletions(-)
> 

Hi Lakshmi,

I think this is a false-positive since it failed in a test that does not associate with this series. Follows the link to this patch:

  https://patchwork.freedesktop.org/series/100936/

What do you think?

Thanks