Re: [PATCH i-g-t v2 00/13] Fix IGTs for Android

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux