Re: [PATCH v2 0/5] Extend IGT to support Android

["Dixit, Ashutosh" <ashutosh.dixit@intel.com>]

On Tue, 29 Apr 2025 13:39:55 -0700, Jeevaka Prabu Badrappan wrote:
>
> The primary objective of this proposal is to extend the functionality of
> IGT to Android environments.
>
> Enabling IGT on Android will allow developers to perform comprehensive
> testing of graphics drivers, ensuring robustness and reliability across
> different platforms.

Could you give some more context here about why this is needed? Is it
because Google is moving Chromebooks to Android?

We also need to think about if this needs addition of an Android build to
CI and who will support that.

Also, it seems Android was added previously but removed in 2017 in
430946d0b881 ("igt: Remove Android support"). So this seems to be a new
attempt to add Android to IGT.

Thanks.

>
> By providing stub implementations for libraries that are not natively
> supported on Android, we will bridge the gap between Android and Linux
> environments, promoting code reuse and reducing development time.
>
> Changes done:
> - Create stub implementations of GLib, libunwind, libprocps, and
> other necessary libraries to satisfy IGT dependencies on Android.
> - Replace glib hash table with generic hash table implementation
>
> Note:
> - Incase of linux systems, system libraries will be used whereas for
> Android stubs will be used with reduced test coverage.
> - Support will be provided to maintain Android support.
>
> Jeevaka Prabu Badrappan (4):
>   Replace program_invocation_short_name with prog_name from command line
>   Avoid use of pthread_cancel by introducing an exit flag
>   Replace glib hash table with c specific implementation
>   igt-gpu-tools: Changes to compile for Android
>
> Sapna (1):
>   Add stub for libunwind, procps and glib
>
>  include/stub/glib.h          |  43 +++++++++++++
>  include/stub/libproc2/pids.h |  41 ++++++++++++
>  include/stub/libunwind.h     |  20 ++++++
>  lib/igt_core.c               |   4 ++
>  lib/igt_core.h               |   1 +
>  lib/igt_device_scan.c        | 118 ++++++++++++++++++++++++++---------
>  lib/igt_dummyload.c          |   6 +-
>  lib/igt_dummyload.h          |   2 +
>  lib/igt_kmod.c               |  10 +++
>  lib/xe/xe_spin.c             |   2 +-
>  tests/intel/xe_create.c      |   1 +
>  tools/gputop.c               |   6 +-
>  tools/intel_gpu_top.c        |   2 +-
>  13 files changed, 220 insertions(+), 36 deletions(-)
>  create mode 100644 include/stub/glib.h
>  create mode 100644 include/stub/libproc2/pids.h
>  create mode 100644 include/stub/libunwind.h
>
> --
> 2.49.0
>