[PATCH] drm/amdgpu: Adjust IB test timeout for XGMI configuration

[Alex Deucher]

On Wed, Apr 3, 2019 at 11:42 AM Liu, Shaoyun <Shaoyun.Liu at amd.com> wrote:
>
> On XGMI configuration the ib test may tooks longer to finish
>
> Change-Id: If3afd8eac3c342d32c387804b51fc4a4bdd35d35
> Signed-off-by: shaoyunl <shaoyun.liu at amd.com>
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> index 0b8ef2d..6c508d7 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c
> @@ -35,6 +35,7 @@
>  #include "amdgpu_trace.h"
>
>  #define AMDGPU_IB_TEST_TIMEOUT msecs_to_jiffies(1000)
> +#define AMDGPU_IB_TEST_GFX_XGMI_TIMEOU msecs_to_jiffies(2000)

Typo:
TIMEOU -> TIMEOUT

>
>  /*
>   * IB
> @@ -344,7 +345,8 @@ int amdgpu_ib_ring_tests(struct amdgpu_device *adev)
>                  * cost waiting for it coming back under RUNTIME only
>                 */
>                 tmo_gfx = 8 * AMDGPU_IB_TEST_TIMEOUT;
> -       }
> +       } else if (adev->gmc.xgmi.hive_id)
> +               tmo_gfx = AMDGPU_IB_TEST_GFX_XGMI_TIMEOU;
>

Coding style, if any clause has parens, all should.  With these issues
fixed, patch is:
Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

>         for (i = 0; i < adev->num_rings; ++i) {
>                 struct amdgpu_ring *ring = adev->rings[i];
> --
> 2.7.4
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx