Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fb4aadf6 authored by Daniel Norman's avatar Daniel Norman
Browse files

Replaces 'oneshot' and 'interface' with 'onrestart restart audioserver'. 

This default service should still restart on audioserver restart because
audioserver uses 'onrestart restart vendor.audio-hal-2-0' for restarts.

Bug: 138114550
Bug: 80227481
Test: for i in {0..100}; do \
      adb shell 'ps|grep audio;killall audioserver;ps|grep audio' \
      && sleep 2; \
      done
Test: Ran steps mentioned in http://b/80227481#comment1, device did not
get stuck in boot animation. Killing either audioserver or audio-hal
caused both to restart (just once) and the device booted correctly after
the sleep finished.
Change-Id: Ice128b51f79d3a9de22a719e6d897c8dca915e28
parent 3835f619

audio/common/all-versions/default/service/android.hardware.audio@2.0-service.rc

+1 −6
Original line number Diff line number Diff line
@@ -6,9 +6,4 @@ service vendor.audio-hal-2-0 /vendor/bin/hw/android.hardware.audio@2.0-service 
    capabilities BLOCK_SUSPEND

    ioprio rt 4

    writepid /dev/cpuset/foreground/tasks /dev/stune/foreground/tasks

    # audioflinger restarts itself when it loses connection with the hal

    # and its .rc file has an "onrestart restart audio-hal" rule, thus

    # an additional auto-restart from the init process isn't needed.

    oneshot

    interface android.hardware.audio@4.0::IDevicesFactory default

    interface android.hardware.audio@2.0::IDevicesFactory default
 
    onrestart restart audioserver