<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, Apr 29, 2025 at 5:14 PM Dixit, Ashutosh <<a href="mailto:ashutosh.dixit@intel.com">ashutosh.dixit@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 29 Apr 2025 13:39:55 -0700, Jeevaka Prabu Badrappan wrote:<br>
><br>
> The primary objective of this proposal is to extend the functionality of<br>
> IGT to Android environments.<br>
><br>
> Enabling IGT on Android will allow developers to perform comprehensive<br>
> testing of graphics drivers, ensuring robustness and reliability across<br>
> different platforms.<br>
<br>
Could you give some more context here about why this is needed? Is it<br>
because Google is moving Chromebooks to Android? </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
We also need to think about if this needs addition of an Android build to<br>
CI and who will support that.<br>
<br>
Also, it seems Android was added previously but removed in 2017 in<br>
430946d0b881 ("igt: Remove Android support"). So this seems to be a new<br>
attempt to add Android to IGT.<br></blockquote><div><br></div><div>Hello, </div><div><br></div><div>Yes - IGT is already in Android but was running off a snapshot from 5 years ago.</div><div>To run it on Android, all required dependencies such as Android.bp were kept downstream.</div><div><br></div><div>On Android, we're trying to move back to upstream and stay close to upstream and constantly update our downstream repo.</div><div>The plan is to keep the Android specific build system such as Android.bp in the downstream branch as it's not the most relevant to the upstream repo.</div><div><br></div><div>Thanks,</div><div>-Mark</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks.<br>
<br>
><br>
> By providing stub implementations for libraries that are not natively<br>
> supported on Android, we will bridge the gap between Android and Linux<br>
> environments, promoting code reuse and reducing development time.<br>
><br>
> Changes done:<br>
> - Create stub implementations of GLib, libunwind, libprocps, and<br>
> other necessary libraries to satisfy IGT dependencies on Android.<br>
> - Replace glib hash table with generic hash table implementation<br>
><br>
> Note:<br>
> - Incase of linux systems, system libraries will be used whereas for<br>
> Android stubs will be used with reduced test coverage.<br>
> - Support will be provided to maintain Android support.<br>
><br>
> Jeevaka Prabu Badrappan (4):<br>
>   Replace program_invocation_short_name with prog_name from command line<br>
>   Avoid use of pthread_cancel by introducing an exit flag<br>
>   Replace glib hash table with c specific implementation<br>
>   igt-gpu-tools: Changes to compile for Android<br>
><br>
> Sapna (1):<br>
>   Add stub for libunwind, procps and glib<br>
><br>
>  include/stub/glib.h          |  43 +++++++++++++<br>
>  include/stub/libproc2/pids.h |  41 ++++++++++++<br>
>  include/stub/libunwind.h     |  20 ++++++<br>
>  lib/igt_core.c               |   4 ++<br>
>  lib/igt_core.h               |   1 +<br>
>  lib/igt_device_scan.c        | 118 ++++++++++++++++++++++++++---------<br>
>  lib/igt_dummyload.c          |   6 +-<br>
>  lib/igt_dummyload.h          |   2 +<br>
>  lib/igt_kmod.c               |  10 +++<br>
>  lib/xe/xe_spin.c             |   2 +-<br>
>  tests/intel/xe_create.c      |   1 +<br>
>  tools/gputop.c               |   6 +-<br>
>  tools/intel_gpu_top.c        |   2 +-<br>
>  13 files changed, 220 insertions(+), 36 deletions(-)<br>
>  create mode 100644 include/stub/glib.h<br>
>  create mode 100644 include/stub/libproc2/pids.h<br>
>  create mode 100644 include/stub/libunwind.h<br>
><br>
> --<br>
> 2.49.0<br>
><br>
</blockquote></div></div>