From e23f59aa5ad5f25b697905c9b956bfee47c79ed3 Mon Sep 17 00:00:00 2001 From: Peiyong Lin Date: Wed, 7 Nov 2018 21:10:07 -0800 Subject: [PATCH] Add global property to store opt-in application package name. We provide a way in developer option to opt-in an application to use updated graphics driver. To make sure we set up the graphics environment correctly, we need to access the package name of the selected application. This patch introduces a global property to store the package name. BUG: 119221883 Test: Build, flash and boot, verify with prototype Change-Id: I49dfcccf387169c072fb9345f7a50c00fcdb0737 Merged-In: I49dfcccf387169c072fb9345f7a50c00fcdb0737 --- core/java/android/app/ActivityThread.java | 2 +- core/java/android/os/GraphicsEnvironment.java | 51 ++++++++++++---------- core/java/android/provider/Settings.java | 7 +++ core/proto/android/providers/settings/global.proto | 3 ++ .../src/android/provider/SettingsBackupTest.java | 1 + packages/SettingsLib/res/values/strings.xml | 3 ++ .../providers/settings/SettingsProtoDumpUtil.java | 3 ++ .../android/server/am/CoreSettingsObserver.java | 2 + 8 files changed, 48 insertions(+), 24 deletions(-) diff --git a/core/java/android/app/ActivityThread.java b/core/java/android/app/ActivityThread.java index a6f19f4c8f27..f453289b4bc5 100644 --- a/core/java/android/app/ActivityThread.java +++ b/core/java/android/app/ActivityThread.java @@ -5594,7 +5594,7 @@ public final class ActivityThread extends ClientTransactionHandler { } } - GraphicsEnvironment.getInstance().setup(context); + GraphicsEnvironment.getInstance().setup(context, mCoreSettings); Trace.traceEnd(Trace.TRACE_TAG_ACTIVITY_MANAGER); } diff --git a/core/java/android/os/GraphicsEnvironment.java b/core/java/android/os/GraphicsEnvironment.java index b1408f46635e..60003d3aa7d7 100644 --- a/core/java/android/os/GraphicsEnvironment.java +++ b/core/java/android/os/GraphicsEnvironment.java @@ -33,8 +33,6 @@ import java.io.File; import java.io.IOException; import java.io.InputStream; import java.io.InputStreamReader; -import java.util.HashSet; -import java.util.Set; /** @hide */ public class GraphicsEnvironment { @@ -60,9 +58,9 @@ public class GraphicsEnvironment { /** * Set up GraphicsEnvironment */ - public void setup(Context context) { + public void setup(Context context, Bundle coreSettings) { setupGpuLayers(context); - chooseDriver(context); + chooseDriver(context, coreSettings); } /** @@ -141,11 +139,12 @@ public class GraphicsEnvironment { /** * Choose whether the current process should use the builtin or an updated driver. */ - private static void chooseDriver(Context context) { + private static void chooseDriver(Context context, Bundle coreSettings) { String driverPackageName = SystemProperties.get(PROPERTY_GFX_DRIVER); if (driverPackageName == null || driverPackageName.isEmpty()) { return; } + // To minimize risk of driver updates crippling the device beyond user repair, never use an // updated driver for privileged or non-updated system apps. Presumably pre-installed apps // were tested thoroughly with the pre-installed driver. @@ -154,12 +153,16 @@ public class GraphicsEnvironment { if (DEBUG) Log.v(TAG, "ignoring driver package for privileged/non-updated system app"); return; } - Set whitelist = loadWhitelist(context, driverPackageName); - // Empty whitelist implies no updatable graphics driver. Typically, the pre-installed - // updatable graphics driver is supposed to be a place holder and contains no graphics - // driver and whitelist. - if (whitelist == null || whitelist.isEmpty()) { + String applicationPackageName = context.getPackageName(); + String devOptInApplicationName = coreSettings.getString( + Settings.Global.UPDATED_GFX_DRIVER_DEV_OPT_IN_APP); + boolean devOptIn = applicationPackageName.equals(devOptInApplicationName); + boolean whitelisted = onWhitelist(context, driverPackageName, ai.packageName); + if (!devOptIn && !whitelisted) { + if (DEBUG) { + Log.w(TAG, applicationPackageName + " is not on the whitelist."); + } return; } @@ -171,12 +174,6 @@ public class GraphicsEnvironment { Log.w(TAG, "driver package '" + driverPackageName + "' not installed"); return; } - if (!whitelist.contains(context.getPackageName())) { - if (DEBUG) { - Log.w(TAG, context.getPackageName() + " is not on the whitelist."); - } - return; - } // O drivers are restricted to the sphal linker namespace, so don't try to use // packages unless they declare they're compatible with that restriction. @@ -242,10 +239,18 @@ public class GraphicsEnvironment { return null; } - private static Set loadWhitelist(Context context, String driverPackageName) { + private static boolean onWhitelist(Context context, String driverPackageName, + String applicationPackageName) { String whitelistName = SystemProperties.get(PROPERTY_GFX_DRIVER_WHITELIST); + + // Empty whitelist implies no updatable graphics driver. Typically, the pre-installed + // updatable graphics driver is supposed to be a place holder and contains no graphics + // driver and whitelist. if (whitelistName == null || whitelistName.isEmpty()) { - return null; + if (DEBUG) { + Log.w(TAG, "No whitelist found."); + } + return false; } try { Context driverContext = context.createPackageContext(driverPackageName, @@ -253,11 +258,11 @@ public class GraphicsEnvironment { AssetManager assets = driverContext.getAssets(); InputStream stream = assets.open(whitelistName); BufferedReader reader = new BufferedReader(new InputStreamReader(stream)); - Set whitelist = new HashSet<>(); - for (String line; (line = reader.readLine()) != null; ) { - whitelist.add(line); + for (String packageName; (packageName = reader.readLine()) != null; ) { + if (packageName.equals(applicationPackageName)) { + return true; + } } - return whitelist; } catch (PackageManager.NameNotFoundException e) { if (DEBUG) { Log.w(TAG, "driver package '" + driverPackageName + "' not installed"); @@ -267,7 +272,7 @@ public class GraphicsEnvironment { Log.w(TAG, "Failed to load whitelist driver package, abort."); } } - return null; + return false; } private static native void setLayerPaths(ClassLoader classLoader, String layerPaths); diff --git a/core/java/android/provider/Settings.java b/core/java/android/provider/Settings.java index 34e9eb30721f..734df0dd36a4 100644 --- a/core/java/android/provider/Settings.java +++ b/core/java/android/provider/Settings.java @@ -11446,6 +11446,13 @@ public final class Settings { public static final String GPU_DEBUG_APP = "gpu_debug_app"; /** + * App that is selected to use updated graphics driver. + * @hide + */ + public static final String UPDATED_GFX_DRIVER_DEV_OPT_IN_APP = + "updated_gfx_driver_dev_opt_in_app"; + + /** * Ordered GPU debug layer list * i.e. ::...: * @hide diff --git a/core/proto/android/providers/settings/global.proto b/core/proto/android/providers/settings/global.proto index 99f303431055..c8cf8c306c15 100644 --- a/core/proto/android/providers/settings/global.proto +++ b/core/proto/android/providers/settings/global.proto @@ -384,6 +384,9 @@ message GlobalSettingsProto { // App allowed to load GPU debug layers. optional SettingProto debug_app = 1; optional SettingProto debug_layers = 2 [ (android.privacy).dest = DEST_AUTOMATIC ]; + // App opt in to load updated graphics driver instead of + // native graphcis driver through developer options. + optional SettingProto updated_gfx_driver_dev_opt_in_app = 6; } optional Gpu gpu = 59; diff --git a/core/tests/coretests/src/android/provider/SettingsBackupTest.java b/core/tests/coretests/src/android/provider/SettingsBackupTest.java index 8e383a56297d..8976f45fc4e3 100644 --- a/core/tests/coretests/src/android/provider/SettingsBackupTest.java +++ b/core/tests/coretests/src/android/provider/SettingsBackupTest.java @@ -444,6 +444,7 @@ public class SettingsBackupTest { Settings.Global.ENABLE_GPU_DEBUG_LAYERS, Settings.Global.GPU_DEBUG_APP, Settings.Global.GPU_DEBUG_LAYERS, + Settings.Global.UPDATED_GFX_DRIVER_DEV_OPT_IN_APP, Settings.Global.ENABLE_GNSS_RAW_MEAS_FULL_TRACKING, Settings.Global.INSTALL_CARRIER_APP_NOTIFICATION_PERSISTENT, Settings.Global.INSTALL_CARRIER_APP_NOTIFICATION_SLEEP_MILLIS, diff --git a/packages/SettingsLib/res/values/strings.xml b/packages/SettingsLib/res/values/strings.xml index d5b121768a66..4684ab99586c 100644 --- a/packages/SettingsLib/res/values/strings.xml +++ b/packages/SettingsLib/res/values/strings.xml @@ -1131,4 +1131,7 @@ + + + Opt in app to use updated graphcis driver in developement diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java index d5efcb549e64..34448f567ad8 100644 --- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java +++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProtoDumpUtil.java @@ -647,6 +647,9 @@ class SettingsProtoDumpUtil { dumpSetting(s, p, Settings.Global.GPU_DEBUG_LAYERS, GlobalSettingsProto.Gpu.DEBUG_LAYERS); + dumpSetting(s, p, + Settings.Global.UPDATED_GFX_DRIVER_DEV_OPT_IN_APP, + GlobalSettingsProto.Gpu.UPDATED_GFX_DRIVER_DEV_OPT_IN_APP); p.end(gpuToken); final long hdmiToken = p.start(GlobalSettingsProto.HDMI); diff --git a/services/core/java/com/android/server/am/CoreSettingsObserver.java b/services/core/java/com/android/server/am/CoreSettingsObserver.java index 160c753d9540..ee3d3a1ddee6 100644 --- a/services/core/java/com/android/server/am/CoreSettingsObserver.java +++ b/services/core/java/com/android/server/am/CoreSettingsObserver.java @@ -55,6 +55,8 @@ final class CoreSettingsObserver extends ContentObserver { // add other system settings here... sGlobalSettingToTypeMap.put(Settings.Global.DEBUG_VIEW_ATTRIBUTES, int.class); + sGlobalSettingToTypeMap.put(Settings.Global.UPDATED_GFX_DRIVER_DEV_OPT_IN_APP, + String.class); // add other global settings here... } -- 2.11.0