diff options
| author | Hans Boehm <hboehm@google.com> | 2019-04-19 17:34:31 -0700 |
|---|---|---|
| committer | Hans Boehm <hboehm@google.com> | 2019-04-25 16:06:24 +0000 |
| commit | 81dc7ab1df59021463757e06a95b2abc937a73db (patch) | |
| tree | 3ae773ed2d58546cfd0bfd8c07f659985d75bdd6 /test/ProfileTestMultiDex/Second.java | |
| parent | fe9181db92a778c4ef3c4d25e0302553bf970e26 (diff) | |
Speed up and slightly simplify Mutex
Eliminate the separate seqentially consistent contention counter load
in Mutex::ExclusiveUnlock by putting the contenion counter in the
Mutex's state word.
Replace some CHECK_GE checks with CHECK_GT. We were checking quantities
intended to be non-negative against >= 0 just before decrementing them.
Remove a pointless volatile declaration.
Introduce constants for the first FUTEX_WAKE argument. Remove all uses
of -1 as that argument, everywhere in ART. It appears to work, but the
documentation says it's wrong.
This does not yet address the ReaderWriterMutex issue, which is handled
in a different way in a separate CL.
Benchmark runs with and without this CL weakly suggest a tiny, not
statistically significant, improvement in both time and space with this
CL.
Bug: 111835365
Test: Build and boot AOSP. TreeHugger.
Change-Id: Ie53c65f2ce774a8cb4d224e2c1b3a110eb880f0c
Diffstat (limited to 'test/ProfileTestMultiDex/Second.java')
0 files changed, 0 insertions, 0 deletions
