From b104c502e1db80eda9751b0cba19e2e24fc0c3c0 Mon Sep 17 00:00:00 2001 From: caozhiyuan Date: Sat, 26 Nov 2016 21:14:07 +0800 Subject: [PATCH] init: Fix sync issue on property trigger. This issue reproduces in the following senario. 1. ("", "") is added to queue; 2. property_triggers_enabled is set to 1; 3. user defined property is triggered, like sys.usb.config=adb; 4. ("sys.usb.config", "adb") is added to queue; 5. main loop interpret ("", "") and queue all current triggers to execution, so ("sys.usb.config", "adb") is queued for execution for the first time. 6. main loop interpret ("sys.usb.config", "adb"), it is queued for execution for a second time. The second time makes ASIT fail. Bug: http://b/28218187 Change-Id: I230e175e0dca8989f1e5bd812398da90082d0ec1 Signed-off-by: caozhiyuan --- init/init.cpp | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/init/init.cpp b/init/init.cpp index 734f1293e..371186bf3 100644 --- a/init/init.cpp +++ b/init/init.cpp @@ -351,11 +351,17 @@ static void process_kernel_cmdline() { if (qemu[0]) import_kernel_cmdline(true, import_kernel_nv); } +static int property_enable_triggers_action(const std::vector& args) +{ + /* Enable property triggers. */ + property_triggers_enabled = 1; + return 0; +} + static int queue_property_triggers_action(const std::vector& args) { + ActionManager::GetInstance().QueueBuiltinAction(property_enable_triggers_action, "enable_property_trigger"); ActionManager::GetInstance().QueueAllPropertyTriggers(); - /* enable property triggers */ - property_triggers_enabled = 1; return 0; }