[Mesa-dev] [PATCH v1 00/13] Implement sw_sync test

[Robert Foss]

On 2016-08-24 02:01 AM, Daniel Vetter wrote:
> On Tue, Aug 23, 2016 at 01:56:02PM -0400, robert.foss at collabora.com wrote:
>> From: Robert Foss <robert.foss at collabora.com>
>>
>> This series implements the sw_sync test and the lib/sw_sync helper functions
>> for said test.
>>
>> Gustavo Padovans sw_sync series was just de-staged in
>> gregkh-staging/staging-next [1], and this test is targeted at verifying the
>> functionality implemented in that series.
>>
>> The sw_sync subtests range from very basic tests of the sw_sync functionality,
>> to stress testing and randomized tests.
>>
>> [1] http://git.kernel.org/cgit/linux/kernel/git/gregkh/staging.git/
>
> This seems to be patches against igt (but the series is lacking the i-g-t-
> subject tag per CONTRIBUTING). I'm ok with that, but I thought the idea
> was to merge these testcases into the kernel's selftests? Did the plan
> change?
> -Daniel

Of course igt should've been added.

I think a some miscommunication must've happened :)

The kernel's selftests already does some of these tests, but apart from 
the sw_sync test, more subtests in kms_atomic and another test 
(kms_fence) are on scheduled for upstreaming. Since the kernel isn't 
really a great place for a testing framework, collecting all of this 
functionality in one place might make more sense.

>
>>
>> Robert Foss (13):
>>   lib/sw_sync: Add helper functions for managing synchronization
>>     primitives
>>   tests/sw_sync: Add sw_sync test
>>   tests/sw_sync: Add subtest test_alloc_fence
>>   tests/sw_sync: Add subtest test_alloc_fence_invalid_timeline
>>   tests/sw_sync: Add subtest test_alloc_merge_fence
>>   tests/sw_sync: Add subtest test_sync_wait
>>   tests/sw_sync: Add subtest test_sync_merge
>>   tests/sw_sync: Add subtest test_sync_merge_same
>>   tests/sw_sync: Add subtest test_sync_multi_consumer
>>   tests/sw_sync: Add subtest test_sync_multi_consumer_producer
>>   tests/sw_sync: Add subtest test_sync_random_merge
>>   tests/sw_sync: Add subtest test_sync_multi_timeline_wait
>>   tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer
>>
>>  lib/Makefile.sources   |   2 +
>>  lib/sw_sync.c          | 237 +++++++++++++++++
>>  lib/sw_sync.h          |  49 ++++
>>  tests/Makefile.sources |   1 +
>>  tests/sw_sync.c        | 693 +++++++++++++++++++++++++++++++++++++++++++++++++
>>  5 files changed, 982 insertions(+)
>>  create mode 100644 lib/sw_sync.c
>>  create mode 100644 lib/sw_sync.h
>>  create mode 100644 tests/sw_sync.c
>>
>> --
>> 2.7.4
>>
>