Re: [PATCH libdrm 14/17] tests/amdgpu: ensure tests work by using c99 initializers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Aug 24, 2015 at 1:08 PM, Alex Deucher <alexdeucher@xxxxxxxxx> wrote:
> On Mon, Aug 24, 2015 at 12:34 PM, Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote:
>> In the latest version of CUnit the fourth parameter of the CU_SuiteInfo
>> struct is pSetUpFunc rather than *pTests.
>>
>> Seems like the CUnit ABI broke at some point, so let's the the robust
>> thing and use c99 designated initializers to correctly populate the
>> struct(s).
>>
>> Cc: Alex Deucher <alexander.deucher@xxxxxxx>
>> Cc: Leo Liu <leo.liu@xxxxxxx>
>> Signed-off-by: Emil Velikov <emil.l.velikov@xxxxxxxxx>
>
> Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx>

Any plans to push this?

Thanks,

Alex

>
>> ---
>>  tests/amdgpu/amdgpu_test.c | 32 ++++++++++++++++++++++++--------
>>  1 file changed, 24 insertions(+), 8 deletions(-)
>>
>> diff --git a/tests/amdgpu/amdgpu_test.c b/tests/amdgpu/amdgpu_test.c
>> index 6568990..71f357c 100644
>> --- a/tests/amdgpu/amdgpu_test.c
>> +++ b/tests/amdgpu/amdgpu_test.c
>> @@ -58,14 +58,30 @@ int drm_amdgpu[MAX_CARDS_SUPPORTED];
>>
>>  /** The table of all known test suites to run */
>>  static CU_SuiteInfo suites[] = {
>> -       { "Basic Tests", suite_basic_tests_init,
>> -         suite_basic_tests_clean, basic_tests },
>> -       { "BO Tests", suite_bo_tests_init,
>> -         suite_bo_tests_clean, bo_tests },
>> -       { "CS Tests", suite_cs_tests_init,
>> -         suite_cs_tests_clean, cs_tests },
>> -       { "VCE Tests", suite_vce_tests_init,
>> -         suite_vce_tests_clean, vce_tests },
>> +       {
>> +               .pName = "Basic Tests",
>> +               .pInitFunc = suite_basic_tests_init,
>> +               .pCleanupFunc = suite_basic_tests_clean,
>> +               .pTests = basic_tests,
>> +       },
>> +       {
>> +               .pName = "BO Tests",
>> +               .pInitFunc = suite_bo_tests_init,
>> +               .pCleanupFunc = suite_bo_tests_clean,
>> +               .pTests = bo_tests,
>> +       },
>> +       {
>> +               .pName = "CS Tests",
>> +               .pInitFunc = suite_cs_tests_init,
>> +               .pCleanupFunc = suite_cs_tests_clean,
>> +               .pTests = cs_tests,
>> +       },
>> +       {
>> +               .pName = "VCE Tests",
>> +               .pInitFunc = suite_vce_tests_init,
>> +               .pCleanupFunc = suite_vce_tests_clean,
>> +               .pTests = vce_tests,
>> +       },
>>         CU_SUITE_INFO_NULL,
>>  };
>>
>> --
>> 2.5.0
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@xxxxxxxxxxxxxxxxxxxxx
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux