Patches 1-4 and 6-13 are Reviewed-by: Petri Latvala <petri.latvala@xxxxxxxxx> Nitpicks on the commit message on patch 6 (replied to it). -- Petri Latvala On Tue, May 16, 2017 at 03:24:49PM +0200, Arkadiusz Hiler wrote: > IGTs are broken for Android since the introduction of dependency on procps. Over > time other incompatibilities built up. > > I took the liberty to fix some of the issues, workaround couple of others and > blacklist heavily incompatible tools/tests. > > It builds on (almost) vanilla AOSP now. > > Github: <https://github.com/ivyl/igt/commits/android> > Howto: <https://gist.github.com/ivyl/1e05af15ae37b575e03dc69e5e2488fc> > now including docker exmaple / script for your convenience > > DEP1: <https://github.com/ivyl/libkmod-android> - just builds Google's one > DEP2: <https://github.com/android-ia/external_libpciaccess> > > We should include a note on Android compatibility in the README and do > "continuous compilation" of the patches as they arrive on the mailing list, > otherwise this **will get broken again soon**. > > This is long as it is, but not complete yet. > > Here are some of more obvious TODOs: > * introduce something like IGT_HAS_CAIRO define for convenience > * revise igt_kms dependency on cairo and enable everything what is independent > * revise kms tests and do the above > * review all things that are disabled on Android and try to enable them > * do something less ugly with config.h generation on Android > > v2: couple of suggested fixed + everything that was not compatible since last > patch > > Cc: Petri Latvala <petri.latvala@xxxxxxxxx> > Cc: Antonio Argenziano <antonio.argenziano@xxxxxxxxx> > Cc: Vinay Belgaumkar <vinay.belgaumkar@xxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Robert Foss <robert.foss@xxxxxxxxxxxxx> > > > Arkadiusz Hiler (13): > tests/drm_import_export: Include {i915_,}drm.h properly > Make conditions on HAVE_UDEV consistent > lib/igt_aux: Include unistd.h for gettid() on Android > lib/igt_aux: Make procps optional > chamelium: Fix build issues on Android > tools/Android.mk: Add guc_logger and l3_parity skip list > tests/Android.mk: Add perf to skip list > Android.mk: Fix libkmod use > Android.mk: Filter out *.h from src files > Android.mk: Use drm stubs > tools/Android.mk: Fix zlib inclusion > tests/gem_exec_nop: Disable headless subtest on cairoless Android > tests/gem_exec_nop: Rename signal() to fence_signal() > > benchmarks/Android.mk | 5 ++++- > configure.ac | 6 +++++- > demos/Android.mk | 3 ++- > lib/Android.mk | 7 ++++--- > lib/Makefile.am | 7 +++++++ > lib/Makefile.sources | 7 ------- > lib/igt.h | 2 ++ > lib/igt_aux.c | 37 +++++++++++++++++++++++++++++++++---- > lib/igt_aux.h | 5 +++++ > lib/igt_chamelium.h | 3 +++ > lib/igt_kmod.h | 4 ++++ > lib/tests/Android.mk | 4 ++-- > tests/Android.mk | 9 +++++---- > tests/Makefile.am | 6 ++++++ > tests/Makefile.sources | 6 ------ > tests/drm_import_export.c | 4 ++-- > tests/gem_exec_nop.c | 14 +++++++++----- > tests/testdisplay_hotplug.c | 2 +- > tools/Android.mk | 14 +++++++++----- > tools/intel_l3_parity.c | 2 +- > tools/intel_l3_parity.h | 2 +- > tools/intel_l3_udev_listener.c | 2 +- > 22 files changed, 106 insertions(+), 45 deletions(-) > > -- > 2.9.3 > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx