Skip to content

Commit

Permalink
Revert "init: Run vendor-specific init hooks on post-fs"
Browse files Browse the repository at this point in the history
This reverts commit 6e152c2.

 * Breaks __system_property_update

Change-Id: I2466c03837b5755befb797343ed6fb771ae7b582
  • Loading branch information
moetayuko authored and luk1337 committed Nov 19, 2019
1 parent 62ef77c commit 03d0f55
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 12 deletions.
3 changes: 0 additions & 3 deletions init/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -496,9 +496,6 @@ Commands
> Loads persistent properties when /data has been decrypted.
This is included in the default init.rc.

`vendor_load_properties`
> Update with vendor-specific property runtime overrides
`loglevel <level>`
> Sets the kernel log level to level. Properties are expanded within _level_.
Expand Down
7 changes: 0 additions & 7 deletions init/builtins.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@
#include "service.h"
#include "subcontext.h"
#include "util.h"
#include "vendor_init.h"

using namespace std::literals::string_literals;

Expand Down Expand Up @@ -1059,11 +1058,6 @@ static Result<Success> do_load_system_props(const BuiltinArguments& args) {
return Success();
}

static Result<Success> do_vendor_load_properties(const BuiltinArguments& args) {
vendor_load_properties();
return Success();
}

static Result<Success> do_wait(const BuiltinArguments& args) {
auto timeout = kCommandRetryTimeout;
if (args.size() == 3) {
Expand Down Expand Up @@ -1227,7 +1221,6 @@ const BuiltinFunctionMap::Map& BuiltinFunctionMap::map() const {
{"interface_stop", {1, 1, {false, do_interface_stop}}},
{"load_persist_props", {0, 0, {false, do_load_persist_props}}},
{"load_system_props", {0, 0, {false, do_load_system_props}}},
{"vendor_load_properties", {0, 0, {false, do_vendor_load_properties}}},
{"loglevel", {1, 1, {false, do_loglevel}}},
{"mark_post_data", {0, 0, {false, do_mark_post_data}}},
{"mkdir", {1, 4, {true, do_mkdir}}},
Expand Down
4 changes: 4 additions & 0 deletions init/property_service.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
#include "selinux.h"
#include "subcontext.h"
#include "util.h"
#include "vendor_init.h"

using namespace std::literals;

Expand Down Expand Up @@ -912,6 +913,9 @@ void property_load_boot_defaults(bool load_debug_prop) {
}
}

// Update with vendor-specific property runtime overrides
vendor_load_properties();

property_initialize_ro_product_props();
property_derive_build_fingerprint();

Expand Down
2 changes: 0 additions & 2 deletions rootdir/init.rc
Original file line number Diff line number Diff line change
Expand Up @@ -350,8 +350,6 @@ on early-fs
start vold

on post-fs
vendor_load_properties

exec - system system -- /system/bin/vdc checkpoint markBootAttempt

# Once everything is setup, no need to modify /.
Expand Down

0 comments on commit 03d0f55

Please sign in to comment.