Commit: c00365909daf0b60dce345c9c6c46bbfffc73560 - device-generic-common (git)

device/generic/common

Commit MetaInfo

• Révision: c00365909daf0b60dce345c9c6c46bbfffc73560 (tree)
• l'heure: 2019-11-20 03:02:09
• Auteur: Michael Goffioul <michael.goffioul@linc...>
• Commiter: Chih-Wei Huang

Message de Log

Fix native bridge wrapper

- Be more specific about houdini files to delete (avoid deleting

bundled system/etc/houdini*.sfs)

- Use PRODUCT_PROPERTY_OVERRIDES for ro.dalvik.vm.native.bridge to

correctly override value from build/make/target/product/runtime_libart.mk

- Use losetup/mount to mount the houdini sfs image, toybox's mount does

not seem to be able to mount -o loop... properly

v2: resolve conflict in nativebridge/bin/enable_nativebridge (cwhuang)

Change Summary

• modified: nativebridge/Android.mk
• modified: nativebridge/bin/enable_nativebridge
• modified: nativebridge/nativebridge.mk

Modification

--- a/nativebridge/Android.mk
+++ b/nativebridge/Android.mk
@@ -20,7 +20,7 @@ LOCAL_SHARED_LIBRARIES := libcutils libdl liblog
 LOCAL_C_INCLUDES := system/core/libnativebridge/include
 LOCAL_MULTILIB := both
 LOCAL_POST_INSTALL_CMD := $(hide) \
-    rm -rf $(TARGET_OUT)/*/{arm*,*houdini*} {$(TARGET_OUT),$(PRODUCT_OUT)}/vendor/{*/arm*,*/*houdini*}; \
+    rm -rf $(TARGET_OUT)/{bin,lib}/{arm*,*houdini*} {$(TARGET_OUT),$(PRODUCT_OUT)}/vendor/{bin,lib}/{arm*,*houdini*}; \
     mkdir -p $(TARGET_OUT)/{lib/arm,$(if $(filter true,$(TARGET_IS_64_BIT)),lib64/arm64)}; \
     touch $(TARGET_OUT)/lib/libhoudini.so $(if $(filter true,$(TARGET_IS_64_BIT)),$(TARGET_OUT)/lib64/libhoudini.so)
 

--- a/nativebridge/bin/enable_nativebridge
+++ b/nativebridge/bin/enable_nativebridge
@@ -15,7 +15,8 @@ urls[5]=http://tinyurl.com/y6g6cx27
 
 mount_sfs()
 {
-	mount -o 'context="u:object_r:system_file:s0"' $1 $2
+	lodevice=`losetup -r -f -s $1`
+	[ -n "$lodevice" ] && mount -o 'context="u:object_r:system_file:s0"' $lodevice $2
 }
 
 if [ -z "$1" ]; then

--- a/nativebridge/nativebridge.mk
+++ b/nativebridge/nativebridge.mk
@@ -32,7 +32,7 @@ PRODUCT_PROPERTY_OVERRIDES += \
 endif
 
 ifneq ($(HOUDINI_PREINSTALL),intel)
-PRODUCT_DEFAULT_PROPERTY_OVERRIDES := ro.dalvik.vm.native.bridge=libnb.so
+PRODUCT_PROPERTY_OVERRIDES := ro.dalvik.vm.native.bridge=libnb.so
 
 PRODUCT_PACKAGES := libnb
 endif