[Intel-gfx] [PATCH i-g-t v6 20/21] tests/sw_sync: Add subtest test_sync_busy_fork

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 39 +++ 1 file changed, 39 inserti

[Intel-gfx] [PATCH i-g-t v6 19/21] tests/sw_sync: Add subtest test_sync_merge_invalid

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge_invalid that tests merging invalid fences. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 41 + 1 file changed, 41 insertions(+) diff -

[Intel-gfx] [PATCH i-g-t v6 08/21] tests/sw_sync: Add subtest test_sync_merge_same

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies merging a fence with itself does not fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 27 +++

[Intel-gfx] [PATCH i-g-t v6 13/21] tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest runs a single consumer thread and multiple producer thread that are synchronized using multiple timelines. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v6 11/21] tests/sw_sync: Add subtest test_sync_random_merge

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that creating many timelines and merging random fences from each timeline with eachother results in merged fences that are fully functional. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric

[Intel-gfx] [PATCH i-g-t v6 16/21] tests/sw_sync: Add subtest test_timeline_closed_signaled

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_timeline_closed_signaled that verifies that a signaled fence stays signaled after its timeline has been closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 18 ++ 1 file

[Intel-gfx] [PATCH i-g-t v6 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-11-17 Thread robert . foss
fence, it might get freed while still in use. Usually a kernel panick will follow. Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 28 1 file changed, 28 inserti

[Intel-gfx] [PATCH i-g-t v6 17/21] lib/sw_sync: Add igt_require_sw_sync to enable skipping on no sw_sync support

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't available on the host machine. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 22 ++ lib/sw_sync.h | 1 + 2 files

[Intel-gfx] [PATCH i-g-t v6 21/21] tests/sw_sync: Add subtest test_sync_busy_unixsocket

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process, where the timeline fd has been sent through a UNIX socket. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_

[Intel-gfx] [PATCH i-g-t v6 03/21] tests/sw_sync: Add subtest test_alloc_fence

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest alloc_fence that verifies that it's possible to allocate a fence on a timeline. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 16 ++

[Intel-gfx] [PATCH i-g-t v6 10/21] tests/sw_sync: Add subtest test_sync_multi_consumer_producer

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This test verifies that stressing the kernel by creating multiple consumer/producer threads that wait on a single timeline to be incremented by another conumer/producer thread does not fail. And that the order amongst the threads is main

[Intel-gfx] [PATCH i-g-t v6 05/21] tests/sw_sync: Add subtest test_alloc_merge_fence

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that merging two fences works in the simples possible case. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 23 +

[Intel-gfx] [PATCH i-g-t v6 15/21] tests/sw_sync: Add subtest test_timeline_closed

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that the fences of a timeline are not signalled when a timelne is closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 17 + 1 file changed, 17 insertions(+) diff -

[Intel-gfx] [PATCH i-g-t v6 12/21] tests/sw_sync: Add subtest test_sync_multi_timeline_wait

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting, timing out on a wait and that counting fences in various states works. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

[Intel-gfx] [PATCH i-g-t v6 07/21] tests/sw_sync: Add subtest test_sync_merge

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge that tests merging fences and the validity of the resulting merged fence. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

[Intel-gfx] [PATCH i-g-t v6 09/21] tests/sw_sync: Add subtest test_sync_multi_consumer

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies the access ordering of multiple consumer threads. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v6 18/21] tests/sw_sync: Add igt_require check for sw_sync feature

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Make sure that this test is skipped if the sw_sync feature is missing from the host system. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 1 + tests/sw_sync.c | 3 +++ 2 files changed, 4 insertions(+) diff

[Intel-gfx] [PATCH i-g-t v6 06/21] tests/sw_sync: Add subtest test_sync_busy

2016-11-17 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting on fences works properly. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 37 +

Re: [Intel-gfx] [PATCH 02/12] tests/kms_atomic_transition: don't assume max pipes

2016-11-15 Thread Robert Foss
On Tue, 2016-11-15 at 14:25 +0100, Tomeu Vizoso wrote: > On 15 November 2016 at 09:01, Daniel Vetter wrote: > > > > On Mon, Nov 14, 2016 at 06:59:16PM +0900, Gustavo Padovan wrote: > > > > > > From: Gustavo Padovan > > > > > > Signed-off-by:

[Intel-gfx] [PATCH i-g-t v6 17/21] lib/sw_sync: Add igt_require_sw_sync to enable skipping on no sw_sync support

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't available on the host machine. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 22 ++ lib/sw_sync.h | 1 + 2 files

[Intel-gfx] [PATCH i-g-t v6 01/21] lib/sw_sync: Add helper functions for managing synchronization primitives

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Base functions to help testing the Sync File Framework (explicit fencing mechanism ported from Android). These functions allow you to create, use and destroy timelines and fences. Signed-off-by: Robert Foss <robert.f...@collabora.com>

[Intel-gfx] [PATCH i-g-t v6 00/21] Implement sw_sync test

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This series implements the sw_sync test and the lib/sw_sync helper functions for said test. The sw_sync subtests range from very basic tests of the sw_sync functionality, to stress testing and randomized tests. Changes since v1: Added &qu

[Intel-gfx] [PATCH i-g-t v6 12/21] tests/sw_sync: Add subtest test_sync_multi_timeline_wait

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting, timing out on a wait and that counting fences in various states works. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

[Intel-gfx] [PATCH i-g-t v6 08/21] tests/sw_sync: Add subtest test_sync_merge_same

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies merging a fence with itself does not fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 27 +++

[Intel-gfx] [PATCH i-g-t v6 18/21] tests/sw_sync: Add igt_require check for sw_sync feature

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Make sure that this test is skipped if the sw_sync feature is missing from the host system. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 1 + tests/sw_sync.c | 3 +++ 2 files changed, 4 insertions(+) diff

[Intel-gfx] [PATCH i-g-t v6 11/21] tests/sw_sync: Add subtest test_sync_random_merge

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that creating many timelines and merging random fences from each timeline with eachother results in merged fences that are fully functional. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric

[Intel-gfx] [PATCH i-g-t v6 09/21] tests/sw_sync: Add subtest test_sync_multi_consumer

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies the access ordering of multiple consumer threads. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v6 03/21] tests/sw_sync: Add subtest test_alloc_fence

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest alloc_fence that verifies that it's possible to allocate a fence on a timeline. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 16 ++

[Intel-gfx] [PATCH i-g-t v6 05/21] tests/sw_sync: Add subtest test_alloc_merge_fence

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that merging two fences works in the simples possible case. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 23 +

[Intel-gfx] [PATCH i-g-t v6 07/21] tests/sw_sync: Add subtest test_sync_merge

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge that tests merging fences and the validity of the resulting merged fence. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

Re: [Intel-gfx] [PATCH i-g-t v6 00/21] Implement sw_sync test

2016-11-18 Thread Robert Foss
On Fri, 2016-11-18 at 09:59 -0500, robert.f...@collabora.com wrote: > From: Robert Foss <robert.f...@collabora.com> > This is not a resubmission of v6, but rather a mislabeled v7. > > This series implements the sw_sync test and the lib/sw_sync helper > functions for said t

[Intel-gfx] [PATCH i-g-t v6 19/21] tests/sw_sync: Add subtest test_sync_merge_invalid

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge_invalid that tests merging invalid fences. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 41 + 1 file changed, 41 insertions(+) diff -

[Intel-gfx] [PATCH i-g-t v6 10/21] tests/sw_sync: Add subtest test_sync_multi_consumer_producer

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This test verifies that stressing the kernel by creating multiple consumer/producer threads that wait on a single timeline to be incremented by another conumer/producer thread does not fail. And that the order amongst the threads is main

[Intel-gfx] [PATCH i-g-t v6 04/21] tests/sw_sync: Add subtest test_alloc_fence_invalid_timeline

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtests tests that creating fences on negative timelines fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 7 +++ 1 file changed, 7 inserti

[Intel-gfx] [PATCH i-g-t v6 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-11-18 Thread robert . foss
fence, it might get freed while still in use. Usually a kernel panick will follow. Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 31 +++ 1 file changed, 31 inserti

[Intel-gfx] [PATCH i-g-t v6 21/21] tests/sw_sync: Add subtest test_sync_busy_unixsocket

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process, where the timeline fd has been sent through a UNIX socket. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_

[Intel-gfx] [PATCH i-g-t v6 16/21] tests/sw_sync: Add subtest test_timeline_closed_signaled

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_timeline_closed_signaled that verifies that a signaled fence stays signaled after its timeline has been closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 18 ++ 1 file

[Intel-gfx] [PATCH i-g-t v6 06/21] tests/sw_sync: Add subtest test_sync_busy

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting on fences works properly. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 51 +

[Intel-gfx] [PATCH i-g-t v6 20/21] tests/sw_sync: Add subtest test_sync_busy_fork

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 39 +++ 1 file changed, 39 inserti

[Intel-gfx] [PATCH i-g-t v6 13/21] tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest runs a single consumer thread and multiple producer thread that are synchronized using multiple timelines. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v6 15/21] tests/sw_sync: Add subtest test_timeline_closed

2016-11-18 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that the fences of a timeline are not signalled when a timelne is closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 17 + 1 file changed, 17 insertions(+) diff -

Re: [Intel-gfx] [PATCH i-g-t v6 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-11-17 Thread Robert Foss
y a kernel panick will follow. > > > > Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> > > Signed-off-by: Robert Foss <robert.f...@collabora.com> > > --- > >  tests/sw_sync.c | 28 > >  1 file changed, 28 insertion

[Intel-gfx] [PATCH i-g-t v8 09/21] tests/sw_sync: Add subtest test_sync_multi_consumer

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies the access ordering of multiple consumer threads. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v8 15/21] tests/sw_sync: Add subtest test_timeline_closed

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that the fences of a timeline are not signalled when a timelne is closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 17 + 1 file changed, 17 insertions(+) diff -

[Intel-gfx] [PATCH i-g-t v8 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-11-21 Thread robert . foss
fence, it might get freed while still in use. Usually a kernel panick will follow. Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 31 +++ 1 file changed, 31 inserti

[Intel-gfx] [PATCH i-g-t v8 12/21] tests/sw_sync: Add subtest test_sync_multi_timeline_wait

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting, timing out on a wait and that counting fences in various states works. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

[Intel-gfx] [PATCH i-g-t v8 05/21] tests/sw_sync: Add subtest test_alloc_merge_fence

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that merging two fences works in the simples possible case. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 23 +

[Intel-gfx] [PATCH i-g-t v8 04/21] tests/sw_sync: Add subtest test_alloc_fence_invalid_timeline

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtests tests that creating fences on negative timelines fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 7 +++ 1 file changed, 7 inserti

[Intel-gfx] [PATCH i-g-t v8 11/21] tests/sw_sync: Add subtest test_sync_random_merge

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that creating many timelines and merging random fences from each timeline with eachother results in merged fences that are fully functional. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric

[Intel-gfx] [PATCH i-g-t v8 13/21] tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest runs a single consumer thread and multiple producer thread that are synchronized using multiple timelines. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tes

[Intel-gfx] [PATCH i-g-t v8 10/21] tests/sw_sync: Add subtest test_sync_multi_consumer_producer

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This test verifies that stressing the kernel by creating multiple consumer/producer threads that wait on a single timeline to be incremented by another conumer/producer thread does not fail. And that the order amongst the threads is main

[Intel-gfx] [PATCH i-g-t v8 20/21] tests/sw_sync: Add subtest test_sync_busy_fork

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 39 +++ 1 file changed, 39 inserti

[Intel-gfx] [PATCH i-g-t v8 08/21] tests/sw_sync: Add subtest test_sync_merge_same

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies merging a fence with itself does not fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 27 +++

[Intel-gfx] [PATCH i-g-t v8 18/21] tests/sw_sync: Add igt_require check for sw_sync feature

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Make sure that this test is skipped if the sw_sync feature is missing from the host system. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 1 + tests/sw_sync.c | 3 +++ 2 files changed, 4 insertions(+) diff

[Intel-gfx] [PATCH i-g-t v8 00/21] Implement sw_sync test

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This series implements the sw_sync test and the lib/sw_sync helper functions for said test. The sw_sync subtests range from very basic tests of the sw_sync functionality, to stress testing and randomized tests. Changes since v1: Added &qu

[Intel-gfx] [PATCH i-g-t v8 01/21] lib/sw_sync: Add helper functions for managing synchronization primitives

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Base functions to help testing the Sync File Framework (explicit fencing mechanism ported from Android). These functions allow you to create, use and destroy timelines and fences. Signed-off-by: Robert Foss <robert.f...@collabora.com>

[Intel-gfx] [PATCH i-g-t v8 03/21] tests/sw_sync: Add subtest test_alloc_fence

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest alloc_fence that verifies that it's possible to allocate a fence on a timeline. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 16 ++

[Intel-gfx] [PATCH i-g-t v8 16/21] tests/sw_sync: Add subtest test_timeline_closed_signaled

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_timeline_closed_signaled that verifies that a signaled fence stays signaled after its timeline has been closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 18 ++ 1 file

[Intel-gfx] [PATCH i-g-t v8 06/21] tests/sw_sync: Add subtest test_sync_busy

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> This subtest verifies that waiting on fences works properly. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- tests/sw_sync.c | 51 +

[Intel-gfx] [PATCH i-g-t v8 07/21] tests/sw_sync: Add subtest test_sync_merge

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge that tests merging fences and the validity of the resulting merged fence. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> --- te

[Intel-gfx] [PATCH i-g-t v8 21/21] tests/sw_sync: Add subtest test_sync_busy_unixsocket

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_busy_fork which increments the timeline in a forked child process, where the timeline fd has been sent through a UNIX socket. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_

[Intel-gfx] [PATCH i-g-t v8 17/21] lib/sw_sync: Add igt_require_sw_sync to enable skipping on no sw_sync support

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't available on the host machine. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/sw_sync.c | 22 ++ lib/sw_sync.h | 1 + 2 files

[Intel-gfx] [PATCH i-g-t v8 19/21] tests/sw_sync: Add subtest test_sync_merge_invalid

2016-11-21 Thread robert . foss
From: Robert Foss <robert.f...@collabora.com> Add subtest test_sync_merge_invalid that tests merging invalid fences. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/sw_sync.c | 41 + 1 file changed, 41 insertions(+) diff -

Re: [Intel-gfx] [PATCH i-g-t v3] lib/debugfs: Support new generic ABI for CRC capture

2016-11-29 Thread Robert Foss
Hi tomeu, This patch does not seem to apply cleanly on upstream/master. Rob. > The kernel has now a new debugfs ABI that can also allow capturing > frame > CRCs for drivers other than i915. > > Add alternative codepaths so the new ABI is used if the kernel is > recent > enough, and fall back

Re: [Intel-gfx] [PATCH i-g-t 0/2] Improve error report when there are less overlays than expected

2016-11-29 Thread Robert Foss
Hi tomeu, This series looks good to me, feel free to add my r-b. Rob. > Hi, > > kms_plane_scaling would happily try to use the cursor plane as if it > was an > overlay, and the first signal of it would be the kernel refusing the > SetPlane > call because of the pixel format not matching. > >

[Intel-gfx] [PATCH i-g-t v11 01/21] lib/sw_sync: Add helper functions for managing synchronization primitives

2016-12-07 Thread Robert Foss
Base functions to help testing the Sync File Framework (explicit fencing mechanism ported from Android). These functions allow you to create, use and destroy timelines and fences. Signed-off-by: Robert Foss <robert.f...@collabora.com> Signed-off-by: Gustavo Padovan <gus

[Intel-gfx] [PATCH i-g-t v11 03/21] tests/sw_sync: Add subtest test_alloc_fence

2016-12-07 Thread Robert Foss
Add subtest alloc_fence that verifies that it's possible to allocate a fence on a timeline. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 16 +

[Intel-gfx] [PATCH i-g-t v11 04/21] tests/sw_sync: Add subtest test_alloc_fence_invalid_timeline

2016-12-07 Thread Robert Foss
This subtests tests that creating fences on negative timelines fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 8 1 file cha

[Intel-gfx] [PATCH i-g-t v11 05/21] tests/sw_sync: Add subtest test_alloc_merge_fence

2016-12-07 Thread Robert Foss
This subtest verifies that merging two fences works in the simples possible case. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 23 +

[Intel-gfx] [PATCH i-g-t v11 00/21] Implement sw_sync test

2016-12-07 Thread Robert Foss
ib/sw_sync: - Extracted sw_sync path search from kernel_has_sw_sync Rafael Antognolli (1): tests/sw_sync: Add subtest test_sync_expired_merge Robert Foss (20): lib/sw_sync: Add helper functions for managing synchronization primitives test/sw_sync: Add sw_sync test tests/sw_sync: Add

[Intel-gfx] [PATCH i-g-t v11 16/21] tests/sw_sync: Add subtest test_timeline_closed_signaled

2016-12-07 Thread Robert Foss
Add subtest test_timeline_closed_signaled that verifies that a signaled fence stays signaled after its timeline has been closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 18 ++

[Intel-gfx] [PATCH i-g-t v11 20/21] tests/sw_sync: Add subtest test_sync_busy_fork

2016-12-07 Thread Robert Foss
Add subtest test_sync_busy_fork which increments the timeline in a forked child process. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 39 +++ 1 file changed, 3

[Intel-gfx] [PATCH i-g-t v11 13/21] tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer

2016-12-07 Thread Robert Foss
This subtest runs a single consumer thread and multiple producer thread that are synchronized using multiple timelines. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.

[Intel-gfx] [PATCH i-g-t v11 06/21] tests/sw_sync: Add subtest test_sync_busy

2016-12-07 Thread Robert Foss
This subtest verifies that waiting on fences works properly. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- te

[Intel-gfx] [PATCH i-g-t v11 10/21] tests/sw_sync: Add subtest test_sync_multi_consumer_producer

2016-12-07 Thread Robert Foss
This test verifies that stressing the kernel by creating multiple consumer/producer threads that wait on a single timeline to be incremented by another conumer/producer thread does not fail. And that the order amongst the threads is maintained. Signed-off-by: Robert Foss <rober

[Intel-gfx] [PATCH i-g-t v11 21/21] tests/sw_sync: Add subtest test_sync_busy_unixsocket

2016-12-07 Thread Robert Foss
Add subtest test_sync_busy_fork which increments the timeline in a forked child process, where the timeline fd has been sent through a UNIX socket. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_

[Intel-gfx] [PATCH i-g-t v11 11/21] tests/sw_sync: Add subtest test_sync_random_merge

2016-12-07 Thread Robert Foss
This subtest verifies that creating many timelines and merging random fences from each timeline with eachother results in merged fences that are fully functional. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by:

[Intel-gfx] [PATCH i-g-t v11 17/21] lib/sw_sync: Add igt_require_sw_sync to enable skipping on no sw_sync support

2016-12-07 Thread Robert Foss
Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't available on the host machine. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- lib/sw_sync.c | 14 ++ lib/sw_sync.h | 1 + 2 files

[Intel-gfx] [PATCH i-g-t v11 19/21] tests/sw_sync: Add subtest test_sync_merge_invalid

2016-12-07 Thread Robert Foss
Add subtest test_sync_merge_invalid that tests merging invalid fences. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 41 + 1 file changed, 41 insertions(+)

[Intel-gfx] [PATCH i-g-t v11 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-12-07 Thread Robert Foss
fence, it might get freed while still in use. Usually a kernel panick will follow. Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> -

[Intel-gfx] [PATCH i-g-t v11 07/21] tests/sw_sync: Add subtest test_sync_merge

2016-12-07 Thread Robert Foss
Add subtest test_sync_merge that tests merging fences and the validity of the resulting merged fence. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- te

[Intel-gfx] [PATCH i-g-t v11 02/21] test/sw_sync: Add sw_sync test

2016-12-07 Thread Robert Foss
Add initial tests for sw_sync. Signed-off-by: Robert Foss <robert.f...@collabora.com> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/Makefi

[Intel-gfx] [PATCH i-g-t v11 15/21] tests/sw_sync: Add subtest test_timeline_closed

2016-12-07 Thread Robert Foss
This subtest verifies that the fences of a timeline are not signalled when a timelne is closed. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 17 + 1 file changed, 17 insertions(+)

[Intel-gfx] [PATCH i-g-t v11 09/21] tests/sw_sync: Add subtest test_sync_multi_consumer

2016-12-07 Thread Robert Foss
This subtest verifies the access ordering of multiple consumer threads. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- te

[Intel-gfx] [PATCH i-g-t v11 08/21] tests/sw_sync: Add subtest test_sync_merge_same

2016-12-07 Thread Robert Foss
This subtest verifies merging a fence with itself does not fail. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 27 +++

[Intel-gfx] [PATCH i-g-t v11 12/21] tests/sw_sync: Add subtest test_sync_multi_timeline_wait

2016-12-07 Thread Robert Foss
This subtest verifies that waiting, timing out on a wait and that counting fences in various states works. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Eric Engestrom <e...@engestrom.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- te

[Intel-gfx] [PATCH i-g-t v11 18/21] tests/sw_sync: Add igt_require check for sw_sync feature

2016-12-07 Thread Robert Foss
Make sure that this test is skipped if the sw_sync feature is missing from the host system. Signed-off-by: Robert Foss <robert.f...@collabora.com> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> --- tests/sw_sync.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests

Re: [Intel-gfx] [PATCH igt] igt/kms_atomic: Match CRTC harder for special planes

2016-12-13 Thread Robert Foss
pick a PRIMARY plane which was not the kernel's idea of crtc->primary, causing plane_primary_legacy to fail; ditto for cursor. Make find_plane try harder, by preferring to return planes which are already on the requested CRTC. Signed-off-by: Daniel Stone <dani...@collabora.com> Reported-b

Re: [Intel-gfx] [PATCH igt] igt/kms_atomic: Match CRTC harder for special planes

2016-12-14 Thread Robert Foss
which was not the kernel's idea of crtc->primary, causing plane_primary_legacy to fail; ditto for cursor. Make find_plane try harder, by preferring to return planes which are already on the requested CRTC. Signed-off-by: Daniel Stone <dani...@collabora.com> Reported-by: Robert Foss

[Intel-gfx] [PATCH i-g-t v2 11/12] tests/kms_atomic_transition: add in_fences tests

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic_transition.c | 95 ++- 1 file changed, 85

[Intel-gfx] [PATCH i-g-t v2 02/12] tests/kms_atomic_transition: don't assume max pipes

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic_transition.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[Intel-gfx] [PATCH i-g-t v2 04/12] lib/igt_kms: export properties names

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/igt_kms.c | 6 +++--- lib/igt_kms.h | 23 +++ 2 files changed, 2

[Intel-gfx] [PATCH i-g-t v2 01/12] tests/kms_atomic_transition: use igt timeout instead of blocking

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> If the event never arrives we can timeout and end the test. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic_transition.c | 2 ++

[Intel-gfx] [PATCH i-g-t v2 05/12] tests/kms_atomic: use global atomic properties definitions

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic.c | 123 - 1 file changed, 37

[Intel-gfx] [PATCH i-g-t v2 06/12] lib/igt_kms: Add support for the IN_FENCE_FD property

2016-12-14 Thread Robert Foss
Add support dor the IN_FENCE_FD property to enable setting in fences for atomic commits. Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/igt_kms.c | 20 lib/igt_kms.h | 5 + 2 files changed, 25 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt

[Intel-gfx] [PATCH i-g-t v2 03/12] lib/igt_kms: move igt_kms_get_alt_edid() to the right place

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- lib/igt_kms.c | 34 +- 1 file changed, 17 insertions(+), 17 deleti

[Intel-gfx] [PATCH i-g-t v2 09/12] tests/kms_atomic_transition: add fencing parameter to run_transition_tests

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic_transition.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-)

[Intel-gfx] [PATCH i-g-t v2 08/12] tests/kms_atomic: stress possible fence settings

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> --- tests/kms_atomic.c | 186 ++--- 1 file chang

[Intel-gfx] [PATCH i-g-t v2 00/12] tests/kms_atomic_transition add fence testing

2016-12-14 Thread Robert Foss
: stress possible fence settings tests/kms_atomic_transition: add fencing parameter to run_transition_tests tests/kms_atomic_transition: add out_fences tests tests/kms_atomic_transition: add in_fences tests tests/kms_atomic_transition: set out_fence for all crtcs Robert Foss (1): lib

[Intel-gfx] [PATCH i-g-t v2 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-12-14 Thread Robert Foss
From: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Add support for the OUT_FENCE_PTR property to enable setting out fences for atomic commits. Signed-off-by: Gustavo Padovan <gustavo.pado...@collabora.co.uk> Signed-off-by: Robert Foss <robert.f...@collabora.com> ---

<    1   2   3   4   5   6   7   8   >