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

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

Issue 2727573003: gpu: Add sync token dependencies to flush metadata. (Closed)
Patch Set: use verified sync token Created 3 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/gpu_control.h
diff --git a/gpu/command_buffer/client/gpu_control.h b/gpu/command_buffer/client/gpu_control.h
index 6b7bdd52b4b46fa2755cf2a62c62dcf3bccc2a8b..859190dd9c8182c45adbb3c6e33d0644b6127e63 100644
--- a/gpu/command_buffer/client/gpu_control.h
+++ b/gpu/command_buffer/client/gpu_control.h
@@ -98,15 +98,18 @@ class GPU_EXPORT GpuControl {
// the lock provided by the client.
virtual bool IsFenceSyncReleased(uint64_t release) = 0;
- // Runs |callback| when sync token is signalled.
+ // Runs |callback| when sync token is signaled.
virtual void SignalSyncToken(const SyncToken& sync_token,
const base::Closure& callback) = 0;
+ // Returns true if the wait is valid.
jbauman 2017/03/09 02:00:09 Also comment that the wait command-buffer command
sunnyps 2017/03/10 03:26:20 Done.
+ virtual void WaitSyncToken(const SyncToken& sync_token) = 0;
+
// Under some circumstances a sync token may be used which has not been
- // verified to have been flushed. For example, fence syncs queued on the
- // same channel as the wait command guarantee that the fence sync will
- // be enqueued first so does not need to be flushed.
- virtual bool CanWaitUnverifiedSyncToken(const SyncToken* sync_token) = 0;
+ // verified to have been flushed. For example, fence syncs queued on the same
+ // channel as the wait command guarantee that the fence sync will be enqueued
+ // first so does not need to be flushed.
+ virtual bool CanWaitUnverifiedSyncToken(const SyncToken& sync_token) = 0;
private:
DISALLOW_COPY_AND_ASSIGN(GpuControl);

Powered by Google App Engine
This is Rietveld 408576698