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
1 file changed