On Fri, Nov 2, 2018 at 3:36 PM Andrey Grodzovsky <[email protected]> wrote: > > Since only for those ASICs gpu reset is enabled by deafult. > Also update disable message and fix identation . > > Signed-off-by: Andrey Grodzovsky <[email protected]> > --- > tests/amdgpu/deadlock_tests.c | 12 +++++++----- > 1 file changed, 7 insertions(+), 5 deletions(-) > > diff --git a/tests/amdgpu/deadlock_tests.c b/tests/amdgpu/deadlock_tests.c > index 04600a7..00a17ca 100644 > --- a/tests/amdgpu/deadlock_tests.c > +++ b/tests/amdgpu/deadlock_tests.c > @@ -119,11 +119,13 @@ CU_BOOL suite_deadlock_tests_enable(void) > &minor_version, &device_handle)) > return CU_FALSE; > > - if (device_handle->info.family_id == AMDGPU_FAMILY_SI || > - device_handle->info.family_id == AMDGPU_FAMILY_CZ || > - device_handle->info.family_id == AMDGPU_FAMILY_RV || > - device_handle->info.family_id == AMDGPU_FAMILY_KV) { > - printf("\n\nCurrently hangs the CP on this ASIC, deadlock > suite disabled\n"); > + /* > + * Only enable for ASICs supporting GPU reset and for which it's > enabled > + * by default (currently GFX8/9 dGPUS
Missing ) in your comment. With that fixed: Reviewed-by: Alex Deucher <[email protected]> > + */ > + if (device_handle->info.family_id != AMDGPU_FAMILY_VI && > + device_handle->info.family_id != AMDGPU_FAMILY_AI) { > + printf("\n\nGPU reset is not enabled for the ASIC, deadlock > suite disabled\n"); > enable = CU_FALSE; > } > > -- > 2.7.4 > > _______________________________________________ > amd-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
