[Intel-gfx] [PATCH i-g-t] benchmarks/gem_busy: Fix compile error

/Android.mk, resulting in intel_chipset.h from drm being used instead. This patch adds the lib path to the LOCAL_C_INCLUDES Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- benchmarks/Android.mk | 2 ++ 1 file changed, 2 insertions(+) diff --git a/benchmarks/Android.mk b/benchmarks/Andr

[Intel-gfx] [PATCH i-g-t] gem_busy: Remove from android builds without cairo

The test has just developed cairo dependancies (or is at least now dependant on igt_kms.h) so add it to the list of tests that are only built if cairo is available. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/Android.mk | 1 + 1 file changed, 1 insertion(+) diff

[Intel-gfx] [PATCH i-g-t v2] gem_largeobject: Fix for clang compiler

] Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_largeobject.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/tests/gem_largeobject.c b/tests/gem_largeobject.c index c001c17..518396f 100644 --- a/tests/gem_largeobject.c +++ b/tests/gem_largeobje

[Intel-gfx] [PATCH i-g-t] gem_largeobject: Fix for clang compiler

The clang compiler generates a build error for binaries with >128MB data segments because "The gap between stack and binary is only guarenteed to be 128MB on x86_64" Simple solution is to allocate the required memory from the heap. Signed-off-by: Derek Morton <derek.j.

[Intel-gfx] [PATCH i-g-t] tests/Android.mk: Update cairo dependant test list

drm_read gem_exec_blt prime_mmap_kms Have cairo dependency through igt_kms.c so add to skip_tests_list to fix android build errors. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/Android.mk | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/Andr

[Intel-gfx] [PATCH i-g-t v4 4/5] tests/gem_scheduler: Add subtests to test batch priority behaviour

Add subtests to test each ring to check batch buffers of a higher priority will be executed before batch buffers of a lower priority. v2: Addressed review comments from Daniele Ceraolo Spurio v4: Changed priorities to +/-200 - Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.

[Intel-gfx] [PATCH i-g-t v4 5/5] gem_scheduler: Added subtests to test priority starving

: Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_scheduler.c | 134 ++ 1 file changed, 134 insertions(+) diff --git a/tests/gem_scheduler.c b/tests/gem_scheduler.c index 5

[Intel-gfx] [PATCH i-g-t v4 3/5] igt/gem_ctx_param_basic: Updated to support scheduler priority interface

From: John Harrison The GPU scheduler has added an execution priority level to the context object. There is an IOCTL interface to allow user apps/libraries to set this priority. This patch updates the context paramter IOCTL test to include the new interface. For:

[Intel-gfx] [PATCH i-g-t v4 2/5] tests/gem_scheduler: Add gem_scheduler test

Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/Makefile.sources | 1 + tests/gem_scheduler.c | 451 + 2 files changed, 452 insertions(+) create mode 100644 tests/gem_scheduler.c diff --git a/tests/Makefile.sources b

[Intel-gfx] [PATCH i-g-t v4 1/5] lib/intel_batchbuffer: Add functions to be used in the scheduler test

-by: Derek Morton <derek.j.mor...@intel.com> --- lib/intel_batchbuffer.c | 372 +++- lib/intel_batchbuffer.h | 14 ++ 2 files changed, 381 insertions(+), 5 deletions(-) diff --git a/lib/intel_batchbuffer.c b/lib/intel_batchbuffer.c index 6

[Intel-gfx] [PATCH i-g-t v4 0/5] Scheduler tests

ring() non static as another patch has already done so. Addressed more comments from Daniele Ceraolo Spurio Derek Morton (4): lib/intel_batchbuffer: Add functions to be used in the scheduler test tests/gem_scheduler: Add gem_scheduler test tests/gem_scheduler: Add subtests to test batch prio

[Intel-gfx] [PATCH i-g-t] benchmarks/gem_syslatency: Add extra android guard to attr_setaffinity_np

Android defines __USE_GNU but does not provide pthread_attr_setaffinity_np() so added an extra guard arround pthread_attr_setaffinity_np(). Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- benchmarks/gem_syslatency.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/benc

[Intel-gfx] [PATCH i-g-t v3 6/6] gem_scheduler: Added subtests to test priority bumping

When a higher priority batch buffer bumps a lower priority batch buffer all batch buffers in the scheduler queue get a small priority increase. Added a subtest to check this behaviour. Requested by Joonas Lahtinen during scheduler code review Signed-off-by: Derek Morton <derek.j.mor...@intel.

[Intel-gfx] [PATCH i-g-t v3 5/6] tests/gem_scheduler: Add subtests to test batch priority behaviour

Add subtests to test each ring to check batch buffers of a higher priority will be executed before batch buffers of a lower priority. v2: Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_scheduler.

[Intel-gfx] [PATCH i-g-t v3 4/6] igt/gem_ctx_param_basic: Updated to support scheduler priority interface

From: John Harrison The GPU scheduler has added an execution priority level to the context object. There is an IOCTL interface to allow user apps/libraries to set this priority. This patch updates the context paramter IOCTL test to include the new interface. For:

[Intel-gfx] [PATCH i-g-t v3 3/6] tests/gem_scheduler: Add gem_scheduler test

are executed in submission order but the batch buffers without interdependencies do not get held up. v2: Addressed review comments from Daniele Ceraolo Spurio v3: Added logic to use generic BSD ring if there is 1 and BSD1 / BSD2 if there are 2. Signed-off-by: Derek Morton <derek.j.mor...@intel.

[Intel-gfx] [PATCH i-g-t v3 2/6] lib/intel_batchbuffer: Add functions to be used in the scheduler test

a batch buffer which writes timestamps to a buffer object. Function to compare timestamps allowing for wrapping of the values. v2: Moved code to intel_batchbuffer (Daniel Vetter) Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --

[Intel-gfx] [PATCH i-g-t v3 1/6] ioctl_wrappers: make gem_has_ring non static

For tests that use multiple rings to test interactions it is useful to know if a ring exists without triggering the test to skip. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/ioctl_wrappers.c | 2 +- lib/ioctl_wrappers.h | 1 + 2 files changed, 2 insertions(+), 1 de

[Intel-gfx] [PATCH i-g-t v3 0/6] Scheduler tests

BSD if there is 1 ring and BSD1 / BSD2 if there are two. Derek Morton (5): ioctl_wrappers: make gem_has_ring non static lib/intel_batchbuffer: Add functions to be used in the scheduler test tests/gem_scheduler: Add gem_scheduler test tests/gem_scheduler: Add subtests to test batch prio

[Intel-gfx] [PATCH i-g-t v2 3/7] lib/intel_batchbuffer: Add functions to be used in the scheduler test

a batch buffer which writes timestamps to a buffer object. Function to compare timestamps allowing for wrapping of the values. v2: Moved code to intel_batchbuffer (Daniel Vetter) Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --

[Intel-gfx] [PATCH i-g-t v2 7/7] gem_scheduler: Added subtests to test priority bumping

When a higher priority batch buffer bumps a lower priority batch buffer all batch buffers in the scheduler queue get a small priority increase. Added a subtest to check this behaviour. Requested by Joonas Lahtinen during scheduler code review Signed-off-by: Derek Morton <derek.j.mor...@intel.

[Intel-gfx] [PATCH i-g-t v2 1/7] ioctl_wrappers: make gem_has_ring non static

For tests that use multiple rings to test interactions it is useful to know if a ring exists without triggering the test to skip. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/ioctl_wrappers.c | 2 +- lib/ioctl_wrappers.h | 1 + 2 files changed, 2 insertions(+), 1 de

[Intel-gfx] [PATCH i-g-t v2 6/7] tests/gem_scheduler: Add subtests to test batch priority behaviour

Add subtests to test each ring to check batch buffers of a higher priority will be executed before batch buffers of a lower priority. v2: Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_scheduler.

[Intel-gfx] [PATCH i-g-t v2 2/7] lib/ioctl_wrappers: Separate ring BSD1 from BSD2 checks

Feceoru to patchwork-hook-test branch. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/ioctl_wrappers.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/ioctl_wrappers.c b/lib/ioctl_wrappers.c index f42e2c9..a4c6aa4 100644 --- a/lib/ioctl_wrappers.c +++

[Intel-gfx] [PATCH i-g-t v2 5/7] igt/gem_ctx_param_basic: Updated to support scheduler priority interface

From: John Harrison The GPU scheduler has added an execution priority level to the context object. There is an IOCTL interface to allow user apps/libraries to set this priority. This patch updates the context paramter IOCTL test to include the new interface. For:

[Intel-gfx] [PATCH i-g-t v2 4/7] tests/gem_scheduler: Add gem_scheduler test

are executed in submission order but the batch buffers without interdependencies do not get held up. v2: Addressed review comments from Daniele Ceraolo Spurio Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/Makefile.sources | 1 + tests/gem_scheduler.c

[Intel-gfx] [PATCH i-g-t v2 0/7] Scheduler tests

ates for comments from Daniele Ceraolo Spurio and Daniel Vetter Added tests requested by Joonas Lahtinen Derek Morton (6): ioctl_wrappers: make gem_has_ring non static lib/ioctl_wrappers: Separate ring BSD1 from BSD2 checks lib/intel_batchbuffer: Add functions to be used in the scheduler

[Intel-gfx] [PATCH i-g-t] lib/igt_core.c: Add partial match functionality to run-subtests

Add the ability to specify a substring of the subtest using --run-subtests. This allows 'string' to be used as an abbreviation of the wildcard expression '*string*' when defining which subtests should run. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/igt_core.c | 5 ++

[Intel-gfx] [PATCH i-g-t 0/4] Scheduler tests

by John Harrison to update gem_ctx_param_basic with ioctls to set context priorities. It is included as part of this patch set as Patch 4 is dependant on it. Patch 4 adds tests to check sheduler behaviour for batch buffers submitted at differing priorities. Derek Morton (3): lib/igt_bb_factory

[Intel-gfx] [PATCH i-g-t 2/4] tests/gem_scheduler: Add gem_scheduler test

are executed in submission order but the batch buffers without interdependencies do not get held up. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/Makefile.sources | 1 + tests/gem_scheduler.c | 409 + 2 files changed, 410 inse

[Intel-gfx] [PATCH i-g-t 1/4] lib/igt_bb_factory: Add igt_bb_factory library

a batch buffer which writes timestamps to a buffer object. Function to compare timestamps allowing for wrapping of the values. Intended for use by the gem_scheduler test initially but will be used by other tests in development. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --

[Intel-gfx] [PATCH i-g-t 3/4] igt/gem_ctx_param_basic: Updated to support scheduler priority interface

From: John Harrison The GPU scheduler has added an execution priority level to the context object. There is an IOCTL interface to allow user apps/libraries to set this priority. This patch updates the context paramter IOCTL test to include the new interface. For:

[Intel-gfx] [PATCH i-g-t 4/4] tests/gem_scheduler: Add subtests to test batch priority behaviour

Add subtests to test each ring to check batch buffers of a higher priority will be executed before batch buffers of a lower priority. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_scheduler.c | 34 -- 1 file changed, 28 insertions

[Intel-gfx] [PATCH i-g-t v3] lib/igt_core.c: Expand --run-subtest functionality.

(Ville Syrjala) support both ^ and ! as not operators (Dave Gordon) v3: Updated to use uwildmat() (Dave Gordon) Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- COPYING | 21 +++ lib/Makefile.sources| 2 + lib/igt_core.c | 17 +- lib/uwildmat/uwil

[Intel-gfx] [PATCH i-g-t v2] lib/igt_core.c: Expand --run-subtest functionality.

comma as list separator (Ville Syrjala) support both ^ and ! as not operators (Dave Gordon) Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/igt_core.c | 48 ++-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/lib/igt_

[Intel-gfx] [PATCH i-g-t] lib/igt_core.c: Expand --run-subtest functionality.

as a normal wildcard expression. This is required mainly on android to exclude subtests that test features that do not exist in the android driver while still being able to run other subtests in the binary when a wildcard expression is insufficient. Signed-off-by: Derek Morton <derek.j.

[Intel-gfx] [PATCH] tests/Android.mk: Make intel_residency CAIRO dependant

intel_residency has a cairo dependency through igt_fb.c. Remove it if ANDROID_HAS_CAIRO is not defined. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tools/Android.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/Android.mk b/tools/Android.mk index da4f3c0..5

[Intel-gfx] [PATCH] tools/Android.mk: Add zlib support

IGT does not build for Android due to a zlib dependency being added to intel_error_decode.c in a recent patch. This patch fixes the error by updating the Android makefile to add the path to the zlib library and using any LDFLAGS specified in Makefile.sources. Signed-off-by: Derek Morton

[Intel-gfx] [PATCH i-g-t] igt_core: Fix logging to display extended line

-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/igt_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/igt_core.c b/lib/igt_core.c index 84cf8d2..221ed7e 100644 --- a/lib/igt_core.c +++ b/lib/igt_core.c @@ -1748,7 +1748,7 @@ void igt_vlog(const char *domain

[Intel-gfx] [PATCH i-g-t v3] gem_flink_race/prime_self_import: Improve test reliability

to check the number of gem objects several times to filter out any fluctuations. v2: Moved the common code to a library and made the loop android specific (Daniel Vetter) v3: Renamed get_stable_obj_count -> igt_get_stable_obj_count Signed-off-by: Derek Morton <derek.j.mor...@intel.com>

[Intel-gfx] [PATCH i-g-t v2] gem_flink_race/prime_self_import: Improve test reliability

to check the number of gem objects several times to filter out any fluctuations. v2: Moved the common code to a library and made the loop android specific (Daniel Vetter) Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- lib/igt_debugfs.c

[Intel-gfx] [PATCH i-g-t] gem_flink_race/prime_self_import: Improve test reliability

to check the number of gem objects several times to filter out any fluctuations. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_flink_race.c| 34 +- tests/prime_self_import.c | 43 +-- 2 files c

[Intel-gfx] [PATCH i-g-t v3] tests/gem_exec_nop: Improved test run time

Reduced the Sleep period to 200mS and reduced the repetition count to 7 to decrease the test run time significantly. v2: Changed uS to us v3: removed the output formatting change as the issue will be addressed in a seperate patch from Thomas Wood. Signed-off-by: Derek Morton <derek.j.

[Intel-gfx] [PATCH i-g-t v2] tests/gem_exec_nop: Improved test run time

Reduced the Sleep period to 200mS and reduced the repetition count to 7 to decrease the test run time significantly. Also fixed a non ascii character that messed up the results table formatting. v2: Changed uS to us Signed-off-by: Derek Morton <derek.j.mor...@intel.com> ---

[Intel-gfx] [PATCH i-g-t] tests/gem_exec_nop: Improved test run time

Reduced the Sleep period to 200mS and reduced the repetition count to 7 to decrease the test run time significantly. Also fixed a non ascii character that messed up the results table formatting. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/gem_exec_nop.c | 6 +++---

[Intel-gfx] [PATCH i-g-t] tests: Add gem_exec_nop_concurrent test

This test is based on gem_exec_nop but submits nop batch buffers concurrently from different threads to check for ring hangs and other issues during concurrent submissions. Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/.gitignore| 1 + tests/Makefile.s

[Intel-gfx] [PATCH i-g-t v2] tests/core_prop_blob: Fix core_prop_blob for android

to a seperate patch (Thomas Wood) Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tests/core_prop_blob.c | 71 -- 1 file changed, 52 insertions(+), 19 deletions(-) diff --git a/tests/core_prop_blob.c b/tests/core_prop_blob.c index d

[Intel-gfx] [PATCH i-g-t] benchmarks/gem_blt: Include igt.h in gem_blt.c

To fix a build error on android Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- benchmarks/gem_blt.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/benchmarks/gem_blt.c b/benchmarks/gem_blt.c index 181a5f1..8ab5302 100644 --- a/benchmarks/gem_blt.c

[Intel-gfx] [PATCH i-g-t] core_prop_blob ioctl_wrappers: Fix new tests/benchmarks for android

is unsupported. gem_blt - included igt.h Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- benchmarks/gem_blt.c | 4 +--- lib/ioctl_wrappers.c | 13 + lib/ioctl_wrappers.h | 22 ++ tests/core_prop_blob.c | 37 ++--- 4

[Intel-gfx] [PATCH i-g-t] benchmarks: Fix build errors on Android M-Dessert

Android M-Dessert treats implicit declaration of function warnings as errors resulting in igt failing to build. This patch fixes the errors by including missing header files as required. Mostly this involved including igt.h in the benchmarks. Signed-off-by: Derek Morton <derek.j.mor...@intel.

[Intel-gfx] [PATCH i-g-t] tools/Android.mk: Fix compile error in intel_reg.c

The patch "tools: install the register definition files" caused a build error on android as it added 'PKGDATADIR' which was not defined in the Android build environment. This patch adds that define to tools/Android.mk. It also copies the files it points to so they are actually in the target file

[Intel-gfx] [PATCH i-g-t v2] tools/Android.mk: Fix compile error in intel_reg.c

arget file system. v2: Added local variable for tool path Signed-off-by: Derek Morton <derek.j.mor...@intel.com> --- tools/Android.mk | 12 1 file changed, 12 insertions(+) diff --git a/tools/Android.mk b/tools/Android.mk index 0a196e4..934d3a1 100644 --- a/tools/Androi

[Intel-gfx] [PATCH i-g-t v2] benckmarks/Android.mk: Fix building benchmarks for Android

the built benchmarks should be put. v2: I discovered that the existing definitions of LOCAL_MODULE_PATH were creating what should have been an invalid path. Not sure how it was ever working previously, but fixed now. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- benchmarks/Android.mk | 3

[Intel-gfx] [PATCH i-g-t] benckmarks/Android.mk: Fix building benchmarks for Android

The commit benchmarks: Do not install to system-wide bin/ changed the benchmark file list from bin_PROGRAMS to benchmarks_PROGRAMS. However Android.mk was not updated, resulting in IGT failing to build for Android. This commit adds that change. It also adds LOCAL_MODULE_PATH to specify where the

[Intel-gfx] [PATCH i-g-t] benckmarks/Android.mk: Fix building benchmarks for Android

the built benchmarks should be put. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- benchmarks/Android.mk | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/benchmarks/Android.mk b/benchmarks/Android.mk index da11c44..ed9a903 100644 --- a/benchmarks/Android.mk +++ b

[Intel-gfx] [PATCH i-g-t v2] benchmarks/Android.mk, tools/Android.mk: Fix android build error

: Don't add test/benchmark to the module name if the source already has the suffix. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- benchmarks/Android.mk | 6 +- tools/Android.mk | 6 +- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/benchmarks/Android.mk b

[Intel-gfx] [PATCH i-g-t] tests/gem_render_linear_blits: Increase min swap required

leaks. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_render_linear_blits.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/tests/gem_render_linear_blits.c b/tests/gem_render_linear_blits.c index f83c6d4..5dd210d 100644 --- a/tests

[Intel-gfx] [PATCH i-g-t] benchmarks/Android.mk, tools/Android.mk: Fix android build error

Recently added tools / benckmarks have the same module name as existing tests. Android does not allow duplicate modules. This patch appends _benchmark and _tool to the module names used when building benckmarks and tools to prevent clashes with tests of the same name. Signed-off-by: Derek Morton

[Intel-gfx] [PATCH i-g-t v2] benchmark/: fix gem_exec_nop complie error on android

: Also rename gem_mmap to gem_mmap_benchmark. Another file which breaks android which was added after this patch was 1st submitted. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- benchmarks/Makefile.sources | 9 +- benchmarks/gem_exec_nop.c | 153

[Intel-gfx] [PATCH i-g-t] tools/Makefile.sources: fix igt_stats complie error on android

There are two versions of igt_stats.c in tools and lib\tests which causes the build system to have two build modules with the same name. This patch specifies a different target name for the tool so there is no clash. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tools/Makefile.sources

[Intel-gfx] [PATCH i-g-t] benchmark/: fix gem_exec_nop complie error on android

. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- benchmarks/Makefile.sources | 6 +- benchmarks/gem_exec_nop.c | 153 benchmarks/gem_exec_nop_benchmark.c | 153 3 files changed, 158 insertions(+), 154

[Intel-gfx] [PATCH i-g-t v3] Android.mk: Disable tools that do not build for android

to fix it for Android. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- Android.mk | 2 +- tools/Android.mk | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/Android.mk b/Android.mk index 1ab3e64..681d114 100644 --- a/Android.mk +++ b/Android.mk @@ -1,2 +1,2

[Intel-gfx] [PATCH i-g-t v3] Android.mk: Disable tools that do not build for android

to fix it for Android. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- Android.mk | 2 +- tools/Android.mk | 5 + 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/Android.mk b/Android.mk index 1ab3e64..681d114 100644 --- a/Android.mk +++ b/Android.mk @@ -1,2 +1,2

[Intel-gfx] [PATCH i-g-t] tests/gem_fenced_exec_thrash.c: Fix memory leaks

gem_fenced_exec_thrash was not freeing any resources between subtests. On 1Gb android systems this resulted in the test failing with an OOM error. Added cleanup code to free BOs at the end of each subtest. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_fenced_exec_thrash.c

[Intel-gfx] [PATCH i-g-t v3] tests/gem_fence_thrash.c: Reduce memory usage

it to run on 1Mb systems. Also fixed a leak of the 'threads' variable. v2: Simplified as per Chris Wilson's suggestion. v3: Added calls to munmap() from bo_copy() Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_fence_thrash.c | 8 +++- 1 file changed, 7 insertions(+), 1

[Intel-gfx] [PATCH i-g-t] tests/gem_fence_thrash.c: Reduce memory usage

allowing it to run on 1Mb systems. Also fixed a leak of the 'threads' variable. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_fence_thrash.c | 33 ++--- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/tests/gem_fence_thrash.c b/tests

[Intel-gfx] [PATCH i-g-t] Android.mk: Disable tools that do not build for android

Disable the tools / demo code that do not currently build for android until they can be fixed. Affected tools / demos intel_reg intel_display_crc intel_sprite_on Signed-off-by: Derek Morton derek.j.mor...@intel.com --- Android.mk | 2 +- tools/Android.mk | 2 ++ 2 files changed, 3

[Intel-gfx] [PATCH i-g-t v2] Android.mk: Disable tools that do not build for android

Disable the tools / demo code that do not currently build for android until they can be fixed. Affected tools / demos intel_reg intel_display_crc intel_sprite_on v2: intel_display_crc compiled conditionally on ANDROID_HAS_CAIRO flag. Signed-off-by: Derek Morton derek.j.mor...@intel.com

[Intel-gfx] [PATCH i-g-t v4] lib/tests/igt_segfault Add unit test to test segfault handling

Unit test to check a segfaulting subtest is handled correctly. v2: Added script to check subtest results v3: Removed script. Updated test to use fork to monitor return status. v4: Added igt_segfault to .gitignore Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/tests/.gitignore

[Intel-gfx] [PATCH i-g-t v5] libs/igt_core.c: Fix compile warnings in igt_core.c

Fixed variables incorrectly declared as int instead of size_t. v2: Addressed comments from Tim Gore v3: Removed 'unused parameter' changes v4: Changed to size_t v5: Moved declarations out of for loops Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/igt_core.c | 18

[Intel-gfx] [PATCH i-g-t v4] lib/tests/igt_segfault Add unit test to test segfault handling

Unit test to check a segfaulting subtest is handled correctly. v2: Added script to check subtest results v3: Removed script. Updated test to use fork to monitor return status. v4: Added igt_segfault to .gitignore Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/tests/.gitignore

[Intel-gfx] [PATCH i-g-t v2] lib/tests/igt_segfault.c Add unit test to test segfault handling

Unit test to check a segfaulting subtest is handled correctly. v2: Added script to check subtest results Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/tests/Makefile.sources | 3 ++ lib/tests/igt_segfault.c| 57 ++ lib/tests

[Intel-gfx] [PATCH i-g-t v3] lib/tests/igt_segfault.c Add unit test to test segfault handling

Unit test to check a segfaulting subtest is handled correctly. v2: Added script to check subtest results v3: Removed script. Updated test to use fork to monitor return status. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/tests/Makefile.sources | 1 + lib/tests/igt_segfault.c

[Intel-gfx] [PATCH i-g-t] lib: Enable building unit tests on android

Add a make file for android so the unit tests can be built. Enabled asserts for the library code so the unit test behaviour is correct. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/Android.mk | 4 +++- lib/tests/Android.mk | 41 + 2

[Intel-gfx] [PATCH i-g-t] lib/tests/igt_segfault.c Add unit test to test segfault handling

Unit test to check a segfaulting subtest is handled correctly. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/tests/Makefile.sources | 2 ++ lib/tests/igt_segfault.c | 57 ++ 2 files changed, 59 insertions(+) create mode 100644 lib

[Intel-gfx] [PATCH i-g-t v3] libs/igt_core.c: Fix compile warnings in igt_core.c

Fixed variables incorrectly declared as signed instead of unsigned. v2: Addressed comments from Tim Gore v3: Removed 'unused parameter' changes Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/igt_core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff

[Intel-gfx] [PATCH i-g-t v4] libs/igt_core.c: Fix compile warnings in igt_core.c

Fixed variables incorrectly declared as int instead of size_t. v2: Addressed comments from Tim Gore v3: Removed 'unused parameter' changes v4: Changed to size_t Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/igt_core.c | 14 +++--- 1 file changed, 7 insertions(+), 7

[Intel-gfx] [PATCH i-g-t] libs/igt_core.c: Fix compile warnings in igt_core.c

Fixed variables incorrectly declared as signed instead of unsigned. Fixed 'unused parameter' warning from signal handlers that were not using the signal parameter. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/igt_core.c | 24 +--- 1 file changed, 17

[Intel-gfx] [PATCH i-g-t] libs/igt_core.c: Fix compile warnings in igt_core.c

Fixed variables incorrectly declared as signed instead of unsigned. Fixed 'unused parameter' warning from signal handlers that were not using the signal parameter. v2: Addressed comments from Tim Gore Signed-off-by: Derek Morton derek.j.mor...@intel.com --- lib/igt_core.c | 20

[Intel-gfx] [PATCH i-g-t] lib/igt_core.c: Flag the test as failing after a segfault

fatal_signal_handler() was trapping fatal errors but not flagging the test as failing or setting an exit code. The result was that the test would return Ok or Skipped depending on what the other subtests did even though one of the subtests had segfaulted. Signed-off-by: Derek Morton derek.j.mor

[Intel-gfx] [PATCH i-g-t] tests/Android.mk: Treat all KMS tests as Cairo dependent

future maintainance. Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/Android.mk | 36 +++- 1 file changed, 11 insertions(+), 25 deletions(-) diff --git a/tests/Android.mk b/tests/Android.mk index fac9931..10ef3e2 100644 --- a/tests/Android.mk +++ b

[Intel-gfx] [PATCH i-g-t v2] tests/gem_cpu_reloc: Fix gem_cpu_reloc OOM failure

: Addresed comments from Thomas Wood Signed-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_cpu_reloc.c | 52 ++- 1 file changed, 39 insertions(+), 13 deletions(-) diff --git a/tests/gem_cpu_reloc.c b/tests/gem_cpu_reloc.c index 058089c

[Intel-gfx] [PATCH i-g-t] tests/gem_cpu_reloc: Fix gem_cpu_reloc OOM failure

-off-by: Derek Morton derek.j.mor...@intel.com --- tests/gem_cpu_reloc.c | 48 1 file changed, 36 insertions(+), 12 deletions(-) diff --git a/tests/gem_cpu_reloc.c b/tests/gem_cpu_reloc.c index 058089c..2f5fc60 100644 --- a/tests/gem_cpu_reloc.c