Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(248)

Unified Diff: gpu/command_buffer/client/query_tracker.h

Issue 184223003: gpu: Use explicit atomics instead of assuming that 32bit read/writes are atomic. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: re-upload Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: gpu/command_buffer/client/query_tracker.h
diff --git a/gpu/command_buffer/client/query_tracker.h b/gpu/command_buffer/client/query_tracker.h
index 9e7f501768b74a5bbfa4dccfb2bee6e153622dfa..ce9cc6b9479d6e96cf590e375d775d370edf1803 100644
--- a/gpu/command_buffer/client/query_tracker.h
+++ b/gpu/command_buffer/client/query_tracker.h
@@ -105,6 +105,8 @@ class GLES2_IMPL_EXPORT QueryTracker {
void MarkAsActive() {
state_ = kActive;
++submit_count_;
+ if (submit_count_ == INT_MAX)
+ submit_count_ = 1;
}
void MarkAsPending(int32 token) {
@@ -113,7 +115,7 @@ class GLES2_IMPL_EXPORT QueryTracker {
flushed_ = false;
}
- uint32 submit_count() const {
+ int32 submit_count() const {
return submit_count_;
}
@@ -144,7 +146,7 @@ class GLES2_IMPL_EXPORT QueryTracker {
GLenum target_;
QuerySyncManager::QueryInfo info_;
State state_;
- uint32 submit_count_;
+ int32 submit_count_;
int32 token_;
bool flushed_;
uint64 client_begin_time_us_; // Only used for latency query target.
« no previous file with comments | « no previous file | gpu/command_buffer/client/query_tracker.cc » ('j') | gpu/command_buffer/common/gles2_cmd_format.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698