From 3ae48f2e666dc560104fb0b341737d026ab75d2d Mon Sep 17 00:00:00 2001 From: Devin Moore Date: Mon, 6 Nov 2023 23:57:46 +0000 Subject: [PATCH] Stop including android.hidl.memory ashmem impl on V+ devices We no longer support HIDL in devices targeting Android V+ vendor. Test: m && launch_cvd && adb shell ls /vendor/lib64/hw Bug: 205764958 Change-Id: I6c528be1ae2bf34c790ab2efc83ce7a8db8c9770 --- target/product/base_system.mk | 8 +++++++- target/product/base_vendor.mk | 5 ++++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/target/product/base_system.mk b/target/product/base_system.mk index fd4318f99e..1510ab111b 100644 --- a/target/product/base_system.mk +++ b/target/product/base_system.mk @@ -21,7 +21,6 @@ PRODUCT_PACKAGES += \ am \ android.hidl.base-V1.0-java \ android.hidl.manager-V1.0-java \ - android.hidl.memory@1.0-impl \ android.system.suspend-service \ android.test.base \ android.test.mock \ @@ -315,6 +314,13 @@ PRODUCT_PACKAGES += \ HIDL_SUPPORT_SERVICES := \ hwservicemanager \ android.hidl.allocator@1.0-service \ + android.hidl.memory@1.0-impl \ + +# TODO(b/299166571) Remove this after the artifact path requirements checker picks up +# this library correctly with the *SHIPPING_API_LEVEL_34 variable +PRODUCT_ARTIFACT_PATH_REQUIREMENT_ALLOWED_LIST += \ + $(TARGET_COPY_OUT_SYSTEM)/lib/hw/android.hidl.memory@1.0-impl.so \ + $(TARGET_COPY_OUT_SYSTEM)/lib64/hw/android.hidl.memory@1.0-impl.so \ # Base modules when shipping api level is less than or equal to 34 PRODUCT_PACKAGES_SHIPPING_API_LEVEL_34 += \ diff --git a/target/product/base_vendor.mk b/target/product/base_vendor.mk index 8d5b7bfbdb..a0c5929d85 100644 --- a/target/product/base_vendor.mk +++ b/target/product/base_vendor.mk @@ -46,7 +46,6 @@ PRODUCT_HOST_PACKAGES += \ # Base modules and settings for the vendor partition. PRODUCT_PACKAGES += \ - android.hidl.memory@1.0-impl.vendor \ com.android.hardware.cas \ boringssl_self_test_vendor \ dumpsys_vendor \ @@ -75,6 +74,10 @@ PRODUCT_PACKAGES += \ selinux_policy_nonsystem \ shell_and_utilities_vendor \ +# Base modules when shipping api level is less than or equal to 34 +PRODUCT_PACKAGES_SHIPPING_API_LEVEL_34 += \ + android.hidl.memory@1.0-impl.vendor \ + # OMX not supported for 64bit_only builds # Only supported when SHIPPING_API_LEVEL is less than or equal to 33 ifneq ($(TARGET_SUPPORTS_OMX_SERVICE),false)