| commit | d8459b3d3c8f9b1bfb3b6871b9c7d6e235cca6fa | [log] [tgz] |
|---|---|---|
| author | Yabin Cui <yabinc@google.com> | Wed Aug 26 11:44:16 2015 -0700 |
| committer | Yabin Cui <yabinc@google.com> | Wed Aug 26 15:22:56 2015 -0700 |
| tree | af710eceb49a7f81734ad92dc06225f6c4d83468 | |
| parent | 53f11b620d2be294d3f04acd801e6a5c2f2f4a69 [diff] |
adb: fix a fake data race on transport:kick reported by tsan. It is reported by tsan as a double checked locking. But I think it is not a real data race. Because I think the old code is able to make sure t->kick() is only called once, and the caller of kick_transport is not relying on the side-effect of calling t->kick(). But as it is not perf critical, I don't mind breaking the double checked locking pattern. Bug: 23385662 Change-Id: Ie3597dd56bb514117c3865d2afcfd7c115731a78