Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-11-01 Thread Hyungwon Hwang
On Fri, 30 Oct 2015 12:16:47 +0100 Tobias Jakobi wrote: > Hello Hyungwon, > > first of all thanks for reviewing the series! > > > > Hyungwon Hwang wrote: > > On Tue, 22 Sep 2015 17:54:55 +0200 > > Tobias Jakobi wrote: > > > >>

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Hyungwon Hwang
On Tue, 22 Sep 2015 17:54:55 +0200 Tobias Jakobi wrote: > This tests async processing of G2D jobs. A separate thread is spawned > to monitor the DRM fd for events and check whether a G2D job was > completed. > > v2: Add GPLv2 header, argument handling and

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Tobias Jakobi
Hello Hyungwon, first of all thanks for reviewing the series! Hyungwon Hwang wrote: > On Tue, 22 Sep 2015 17:54:55 +0200 > Tobias Jakobi wrote: > >> This tests async processing of G2D jobs. A separate thread is spawned >> to monitor the DRM fd for events and

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Emil Velikov
On 30 October 2015 at 11:16, Tobias Jakobi wrote: > Hello Hyungwon, > > first of all thanks for reviewing the series! > > > > Hyungwon Hwang wrote: >> On Tue, 22 Sep 2015 17:54:55 +0200 >> Tobias Jakobi wrote: >> >>> +

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Emil Velikov
On 30 October 2015 at 11:28, Tobias Jakobi wrote: > Hello Emil, > > > Emil Velikov wrote: >> On 30 October 2015 at 11:16, Tobias Jakobi >> wrote: >>> Hello Hyungwon, >>> >>> first of all thanks for reviewing the series! >>> >>> >>>

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Tobias Jakobi
Hey Emil, Emil Velikov wrote: > On 30 October 2015 at 11:28, Tobias Jakobi > wrote: >> Hello Emil, >> >> >> Emil Velikov wrote: >>> On 30 October 2015 at 11:16, Tobias Jakobi >>> wrote: Hello Hyungwon, first of all

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Emil Velikov
On 30 October 2015 at 14:28, Tobias Jakobi wrote: > OK, I see what you mean. However this shouldn't happen as long as the > user properly zero initializes the event context structures, right? > It pains me to say it but in this day and age, not everyone zero

Re: [PATCH 06/13] tests/exynos: add fimg2d event test

2015-10-30 Thread Tobias Jakobi
Hello Emil, Emil Velikov wrote: > On 30 October 2015 at 11:16, Tobias Jakobi > wrote: >> Hello Hyungwon, >> >> first of all thanks for reviewing the series! >> >> >> >> Hyungwon Hwang wrote: >>> On Tue, 22 Sep 2015 17:54:55 +0200 >>> Tobias Jakobi

[PATCH 06/13] tests/exynos: add fimg2d event test

2015-09-22 Thread Tobias Jakobi
This tests async processing of G2D jobs. A separate thread is spawned to monitor the DRM fd for events and check whether a G2D job was completed. v2: Add GPLv2 header, argument handling and documentation. Test is only installed when requested. v3: Allocate G2D jobs with calloc which fixes