8e8bc5bf8c8f609f31818d278d6ac8b8e16a9345 - platform_frameworks_base

commit	8e8bc5bf8c8f609f31818d278d6ac8b8e16a9345	[log] [tgz]
author	Keun young Park <keunyoung@google.com>	Tue Jul 27 16:36:23 2021 -0700
committer	Keun-young Park <keunyoung@google.com>	Fri Aug 13 22:52:32 2021 +0000
tree	b5726e6c4c557cb29318921b8578e0a0177e86c9
parent	158824fe783058425f8ee4800c803398004fdbf7 [diff]

unite all locks under BroadcastRadio

- Each layer having separate a lock and making calls to other layers
  while holding lock leads into deadlock.
- Change all of them to use a single lock from the service to avoid
  deadlock.
- Performance impact should be minimal as there are not that many
  players for radio (one app, HAL, and system server) and the amount
  of calls are not that frequent.

Bug: 194818704
Test: atest BroadcastRadioTests
Change-Id: I5f51c0e0a04ae10428f721be508d5579dc1dd837
(cherry picked from commit 2cdb566acb733815063e3c2f4651112cadfc4189)
Merged-In: I5f51c0e0a04ae10428f721be508d5579dc1dd837

core/tests/BroadcastRadioTests/src/com/android/server/broadcastradio/hal2/StartProgramListUpdatesFanoutTest.java[diff]
services/core/java/com/android/server/broadcastradio/BroadcastRadioService.java[diff]
services/core/java/com/android/server/broadcastradio/hal1/BroadcastRadioService.java[diff]
services/core/java/com/android/server/broadcastradio/hal2/AnnouncementAggregator.java[diff]
services/core/java/com/android/server/broadcastradio/hal2/BroadcastRadioService.java[diff]
services/core/java/com/android/server/broadcastradio/hal2/RadioModule.java[diff]
services/core/java/com/android/server/broadcastradio/hal2/TunerSession.java[diff]

7 files changed