Skip to content

Commit

Permalink
Start statsd before starting zygote
Browse files Browse the repository at this point in the history
- zygote needs statsd for logging boot time event.
- statsd starting later leads into stats logging failure which
  is reported by all child processes of zygote later.
  This brings lots of noise in statsd error.

Bug: 159664734
Test: reboot and check if error log does not show up
  E statsd  : Found dropped events: 1 error -19 last atom tag 240 from uid 10169
Change-Id: Ie585febb50a9668671c8fda41a872595baae8385

Former-commit-id: 6423ccd96be44d148466787f7419bc078b08c8b5
  • Loading branch information
Keun young Park committed Jun 24, 2020
1 parent 1766d97 commit 3fae4ff
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions rootdir/init.rc
Original file line number Diff line number Diff line change
Expand Up @@ -821,20 +821,23 @@ on zygote-start && property:persist.sys.fuse=""
on zygote-start && property:ro.crypto.state=unencrypted
# A/B update verifier that marks a successful boot.
exec_start update_verifier_nonencrypted
start statsd
start netd
start zygote
start zygote_secondary

on zygote-start && property:ro.crypto.state=unsupported
# A/B update verifier that marks a successful boot.
exec_start update_verifier_nonencrypted
start statsd
start netd
start zygote
start zygote_secondary

on zygote-start && property:ro.crypto.state=encrypted && property:ro.crypto.type=file
# A/B update verifier that marks a successful boot.
exec_start update_verifier_nonencrypted
start statsd
start netd
start zygote
start zygote_secondary
Expand Down

0 comments on commit 3fae4ff

Please sign in to comment.