diff options
| author | Jaikumar Ganesh <jaikumar@google.com> | 2011-06-06 11:02:30 +0900 |
|---|---|---|
| committer | Jaikumar Ganesh <jaikumar@google.com> | 2011-06-23 17:37:07 -0700 |
| commit | 5312af0c95d781a00298c6f919f95f6f6f442099 (patch) | |
| tree | 410821d783038a46f807af81936d7678ccc7e6d6 /tools/aidl/generate_java.cpp | |
| parent | 7656b21e6313671437c9fbc9bb16096072b1ead5 (diff) | |
Fix excessive locking synchronization leading to deadlocks.
We should never have to look on the entire instance. There
are many places in the code where it there and we need to fix it.
This is a start. In this particular case what we are protecting
is the properties map of the remote device and thus we just need to lock
on it. Ofcourse, it would be better to have a state machine but this
is good for now.
Deadlock: When a new device was found it will call addProperties which will hold
a lock. addProperties calls into BluetoothService. Now Settings app makes a call
into BluetoothService which will call into this file and we have a deadlock.
Change-Id: Ieb69d5ace222bf5d1e6677af151241153303099f
Diffstat (limited to 'tools/aidl/generate_java.cpp')
0 files changed, 0 insertions, 0 deletions