Merge "libhardware loads a module from APEX" into main am: 6ee9b599f8 am: 51f5a3fe8a am: c1eeea6260 am: 2533af1db1
Original change: https://android-review.googlesource.com/c/platform/hardware/libhardware/+/2763345
Change-Id: Ib452dfe06c9866987f94c8345a1553ae07e5cbcd
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/Android.bp b/Android.bp
index 210db85..ee61635 100644
--- a/Android.bp
+++ b/Android.bp
@@ -92,6 +92,7 @@
srcs: ["hardware.c"],
shared_libs: [
+ "libapexsupport",
"libcutils",
"liblog",
"libvndksupport",
@@ -114,10 +115,16 @@
},
target: {
host: {
- exclude_shared_libs: ["libvndksupport"],
+ exclude_shared_libs: [
+ "libapexsupport",
+ "libvndksupport",
+ ],
},
recovery: {
- exclude_shared_libs: ["libvndksupport"],
+ exclude_shared_libs: [
+ "libapexsupport",
+ "libvndksupport",
+ ],
},
},
min_sdk_version: "29",
diff --git a/hardware.c b/hardware.c
index d0e91e9..7907331 100644
--- a/hardware.c
+++ b/hardware.c
@@ -34,17 +34,21 @@
#include <vndksupport/linker.h>
#endif
+#ifdef __ANDROID_APEX__
+#include <android/apexsupport.h>
+#endif
+
/** Base path of the hal modules */
#if defined(__LP64__)
-#define HAL_LIBRARY_PATH1 "/system/lib64/hw"
-#define HAL_LIBRARY_PATH2 "/vendor/lib64/hw"
-#define HAL_LIBRARY_PATH3 "/odm/lib64/hw"
+#define HAL_LIBRARY_SUBDIR "lib64/hw"
#else
-#define HAL_LIBRARY_PATH1 "/system/lib/hw"
-#define HAL_LIBRARY_PATH2 "/vendor/lib/hw"
-#define HAL_LIBRARY_PATH3 "/odm/lib/hw"
+#define HAL_LIBRARY_SUBDIR "lib/hw"
#endif
+#define HAL_LIBRARY_PATH1 "/system/" HAL_LIBRARY_SUBDIR
+#define HAL_LIBRARY_PATH2 "/vendor/" HAL_LIBRARY_SUBDIR
+#define HAL_LIBRARY_PATH3 "/odm/" HAL_LIBRARY_SUBDIR
+
/**
* There are a set of variant filename for modules. The form of the filename
* is "<MODULE_ID>.variant.so" so for the led module the Dream variants
@@ -79,7 +83,7 @@
int status = -EINVAL;
void *handle = NULL;
struct hw_module_t *hmi = NULL;
-#ifdef __ANDROID_VNDK__
+#if defined(__ANDROID_VNDK__) || defined(__ANDROID_APEX__)
const bool try_system = false;
#else
const bool try_system = true;
@@ -97,7 +101,7 @@
*/
handle = dlopen(path, RTLD_NOW);
} else {
-#if defined(__ANDROID_RECOVERY__) || !defined(__ANDROID__)
+#if defined(__ANDROID_RECOVERY__) || !defined(__ANDROID__) || defined(__ANDROID_APEX__)
handle = dlopen(path, RTLD_NOW);
#else
handle = android_load_sphal_library(path, RTLD_NOW);
@@ -151,7 +155,7 @@
/*
* If path is in in_path.
*/
-static bool path_in_path(const char *path, const char *in_path) {
+static bool __attribute__ ((unused)) path_in_path(const char *path, const char *in_path) {
char real_path[PATH_MAX];
if (realpath(path, real_path) == NULL) return false;
@@ -174,6 +178,22 @@
static int hw_module_exists(char *path, size_t path_len, const char *name,
const char *subname)
{
+#ifdef __ANDROID_APEX__
+ // When used in APEX, it should look only into the same APEX because
+ // libhardware modules don't provide ABI stability.
+ if (__builtin_available(android AAPEXSUPPORT_API, *)) {
+ AApexInfo *apex_info;
+ if (AApexInfo_create(&apex_info) == AAPEXINFO_OK) {
+ snprintf(path, path_len, "/apex/%s/%s/%s.%s.so",
+ AApexInfo_getName(apex_info), HAL_LIBRARY_SUBDIR, name, subname);
+ AApexInfo_destroy(apex_info);
+ if (access(path, R_OK) == 0)
+ return 0;
+ }
+ } else {
+ ALOGE("hw_module_exists: libapexsupport is not supported in %d.", __ANDROID_API__);
+ }
+#else // __ANDROID_APEX__
snprintf(path, path_len, "%s/%s.%s.so",
HAL_LIBRARY_PATH3, name, subname);
if (path_in_path(path, HAL_LIBRARY_PATH3) && access(path, R_OK) == 0)
@@ -191,6 +211,8 @@
return 0;
#endif
+#endif // __ANDROID_APEX__
+
return -ENOENT;
}