diff options
| author | Mathias Agopian <mathias@google.com> | 2010-05-18 17:06:55 -0700 |
|---|---|---|
| committer | Mathias Agopian <mathias@google.com> | 2010-05-20 18:00:42 -0700 |
| commit | 898c4c91be8e11b6d5388c623ae80f12ac25fd27 (patch) | |
| tree | 8f59a103707c25a05bcf4fa074e944e766c15503 /libs/surfaceflinger/Barrier.h | |
| parent | 66c46a6bd15422fe898d533d1350d6df748dd95b (diff) | |
| download | frameworks_base-898c4c91be8e11b6d5388c623ae80f12ac25fd27.zip frameworks_base-898c4c91be8e11b6d5388c623ae80f12ac25fd27.tar.gz frameworks_base-898c4c91be8e11b6d5388c623ae80f12ac25fd27.tar.bz2 | |
fix the threading issue for setBuffercount()
this change introduces R/W locks in the right places.
on the server-side, it guarantees that setBufferCount()
is synchronized with "retire" and "resize".
on the client-side, it guarantees that setBufferCount()
is synchronized with "dequeue", "lockbuffer" and "queue"
Diffstat (limited to 'libs/surfaceflinger/Barrier.h')
| -rw-r--r-- | libs/surfaceflinger/Barrier.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/libs/surfaceflinger/Barrier.h b/libs/surfaceflinger/Barrier.h index e2bcf6a..6f8507e 100644 --- a/libs/surfaceflinger/Barrier.h +++ b/libs/surfaceflinger/Barrier.h @@ -29,10 +29,6 @@ public: inline Barrier() : state(CLOSED) { } inline ~Barrier() { } void open() { - // gcc memory barrier, this makes sure all memory writes - // have been issued by gcc. On an SMP system we'd need a real - // h/w barrier. - asm volatile ("":::"memory"); Mutex::Autolock _l(lock); state = OPENED; cv.broadcast(); |