tree e2347a3e99f7540838a7276ae00b31defef8be60
parent 6a8a843fd6cee4b1569f4f560a005d41f55e7fc6
author Steven Moreland <smoreland@google.com> 1634079076 -0700
committer Steven Moreland <smoreland@google.com> 1634079775 +0000

Remove HidlInstrumentor logic entirely.

This userdebug-only feature is used for collecting profiling information
and getting coverage data. It was removed from user builds because it
isn't threadsafe, but also, it is causing a leak because it repeatedly
calls add_sysprop_change_callback.

For coverage information, we have no existing multi-process solution,
and we are focusing on in-process fuzzing.

For profiling information, perfetto/heapprofd/atrace/simpleperf and
others should be prefered.

HIDL is on the deprecation route, and unfortunately this feature has
bitrot, since the old VTS framework is deprecated, and we use a unified
tradefed now. If anyone needs this functionality still, please reach out
to me or file an issue. Thanks!

Note - we can't remove the HidlInstrumentor class itself due to
its ABI being frozen.

Fixes: 202852241
Test: build only
Change-Id: Ibce1f5762740570e6d9405a49c4d917ffb078152