| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "gpu/command_buffer/service/sync_point_manager.h" | 5 #include "gpu/command_buffer/service/sync_point_manager.h" |
| 6 | 6 |
| 7 #include <climits> | 7 #include <climits> |
| 8 | 8 |
| 9 #include "base/logging.h" | 9 #include "base/logging.h" |
| 10 #include "base/rand_util.h" | 10 #include "base/rand_util.h" |
| 11 #include "base/sequence_checker.h" |
| 11 | 12 |
| 12 namespace gpu { | 13 namespace gpu { |
| 13 | 14 |
| 14 static const int kMaxSyncBase = INT_MAX; | 15 static const int kMaxSyncBase = INT_MAX; |
| 15 | 16 |
| 16 SyncPointManager::SyncPointManager() | 17 // static |
| 18 SyncPointManager* SyncPointManager::Create(bool allow_threaded_calls) { |
| 19 return new SyncPointManager(allow_threaded_calls); |
| 20 } |
| 21 |
| 22 SyncPointManager::SyncPointManager(bool allow_threaded_calls) |
| 17 : next_sync_point_(base::RandInt(1, kMaxSyncBase)) { | 23 : next_sync_point_(base::RandInt(1, kMaxSyncBase)) { |
| 18 // To reduce the risk that a sync point created in a previous GPU process | 24 // To reduce the risk that a sync point created in a previous GPU process |
| 19 // will be in flight in the next GPU process, randomize the starting sync | 25 // will be in flight in the next GPU process, randomize the starting sync |
| 20 // point number. http://crbug.com/373452 | 26 // point number. http://crbug.com/373452 |
| 27 |
| 28 if (!allow_threaded_calls) { |
| 29 sequence_checker_.reset(new base::SequenceChecker); |
| 30 } |
| 21 } | 31 } |
| 22 | 32 |
| 23 SyncPointManager::~SyncPointManager() { | 33 SyncPointManager::~SyncPointManager() { |
| 24 } | 34 } |
| 25 | 35 |
| 26 uint32 SyncPointManager::GenerateSyncPoint() { | 36 uint32 SyncPointManager::GenerateSyncPoint() { |
| 27 base::AutoLock lock(lock_); | 37 base::AutoLock lock(lock_); |
| 28 uint32 sync_point = next_sync_point_++; | 38 uint32 sync_point = next_sync_point_++; |
| 29 // When an integer overflow occurs, don't return 0. | 39 // When an integer overflow occurs, don't return 0. |
| 30 if (!sync_point) | 40 if (!sync_point) |
| 31 sync_point = next_sync_point_++; | 41 sync_point = next_sync_point_++; |
| 32 | 42 |
| 33 // Note: wrapping would take days for a buggy/compromized renderer that would | 43 // Note: wrapping would take days for a buggy/compromized renderer that would |
| 34 // insert sync points in a loop, but if that were to happen, better explicitly | 44 // insert sync points in a loop, but if that were to happen, better explicitly |
| 35 // crash the GPU process than risk worse. | 45 // crash the GPU process than risk worse. |
| 36 // For normal operation (at most a few per frame), it would take ~a year to | 46 // For normal operation (at most a few per frame), it would take ~a year to |
| 37 // wrap. | 47 // wrap. |
| 38 CHECK(sync_point_map_.find(sync_point) == sync_point_map_.end()); | 48 CHECK(sync_point_map_.find(sync_point) == sync_point_map_.end()); |
| 39 sync_point_map_.insert(std::make_pair(sync_point, ClosureList())); | 49 sync_point_map_.insert(std::make_pair(sync_point, ClosureList())); |
| 40 return sync_point; | 50 return sync_point; |
| 41 } | 51 } |
| 42 | 52 |
| 43 void SyncPointManager::RetireSyncPoint(uint32 sync_point) { | 53 void SyncPointManager::RetireSyncPoint(uint32 sync_point) { |
| 44 DCHECK(thread_checker_.CalledOnValidThread()); | 54 CheckSequencedThread(); |
| 45 ClosureList list; | 55 ClosureList list; |
| 46 { | 56 { |
| 47 base::AutoLock lock(lock_); | 57 base::AutoLock lock(lock_); |
| 48 SyncPointMap::iterator it = sync_point_map_.find(sync_point); | 58 SyncPointMap::iterator it = sync_point_map_.find(sync_point); |
| 49 if (it == sync_point_map_.end()) { | 59 if (it == sync_point_map_.end()) { |
| 50 LOG(ERROR) << "Attempted to retire sync point that" | 60 LOG(ERROR) << "Attempted to retire sync point that" |
| 51 " didn't exist or was already retired."; | 61 " didn't exist or was already retired."; |
| 52 return; | 62 return; |
| 53 } | 63 } |
| 54 list.swap(it->second); | 64 list.swap(it->second); |
| 55 sync_point_map_.erase(it); | 65 sync_point_map_.erase(it); |
| 56 } | 66 } |
| 57 for (ClosureList::iterator i = list.begin(); i != list.end(); ++i) | 67 for (ClosureList::iterator i = list.begin(); i != list.end(); ++i) |
| 58 i->Run(); | 68 i->Run(); |
| 59 } | 69 } |
| 60 | 70 |
| 61 void SyncPointManager::AddSyncPointCallback(uint32 sync_point, | 71 void SyncPointManager::AddSyncPointCallback(uint32 sync_point, |
| 62 const base::Closure& callback) { | 72 const base::Closure& callback) { |
| 63 DCHECK(thread_checker_.CalledOnValidThread()); | 73 CheckSequencedThread(); |
| 64 { | 74 { |
| 65 base::AutoLock lock(lock_); | 75 base::AutoLock lock(lock_); |
| 66 SyncPointMap::iterator it = sync_point_map_.find(sync_point); | 76 SyncPointMap::iterator it = sync_point_map_.find(sync_point); |
| 67 if (it != sync_point_map_.end()) { | 77 if (it != sync_point_map_.end()) { |
| 68 it->second.push_back(callback); | 78 it->second.push_back(callback); |
| 69 return; | 79 return; |
| 70 } | 80 } |
| 71 } | 81 } |
| 72 callback.Run(); | 82 callback.Run(); |
| 73 } | 83 } |
| 74 | 84 |
| 75 bool SyncPointManager::IsSyncPointRetired(uint32 sync_point) { | 85 bool SyncPointManager::IsSyncPointRetired(uint32 sync_point) { |
| 76 DCHECK(thread_checker_.CalledOnValidThread()); | 86 CheckSequencedThread(); |
| 77 { | 87 { |
| 78 base::AutoLock lock(lock_); | 88 base::AutoLock lock(lock_); |
| 79 SyncPointMap::iterator it = sync_point_map_.find(sync_point); | 89 SyncPointMap::iterator it = sync_point_map_.find(sync_point); |
| 80 return it == sync_point_map_.end(); | 90 return it == sync_point_map_.end(); |
| 81 } | 91 } |
| 82 } | 92 } |
| 83 | 93 |
| 94 void SyncPointManager::CheckSequencedThread() { |
| 95 DCHECK(!sequence_checker_ || |
| 96 sequence_checker_->CalledOnValidSequencedThread()); |
| 97 } |
| 98 |
| 84 } // namespace gpu | 99 } // namespace gpu |
| OLD | NEW |