Implemented new fence syncs which replaces the old sync points.

gpu/command_buffer/client/cmd_buffer_helper.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file contains the command buffer helper class.
 
 #ifndef GPU_COMMAND_BUFFER_CLIENT_CMD_BUFFER_HELPER_H_
 #define GPU_COMMAND_BUFFER_CLIENT_CMD_BUFFER_HELPER_H_
 
 #include <string.h>
@@ skipping 288 matching lines @@
   bool usable () const {
     return usable_;
   }
 
   void ClearUsable() {
     usable_ = false;
     context_lost_ = true;
     CalcImmediateEntries(0);
   }
 
+  uint32_t GenerateFenceSyncRelease();
+
+  bool IsFenceSyncRelease(uint32_t release) {
+    return release < next_fence_sync_release_;
+  }
+
+  bool IsFenceSyncFlushed(uint32_t release) {
+    return release <= flushed_fence_sync_release_;
+  }
+
   // Overridden from base::trace_event::MemoryDumpProvider:
   bool OnMemoryDump(const base::trace_event::MemoryDumpArgs& args,
                     base::trace_event::ProcessMemoryDump* pmd) override;
 
  private:
   // Returns the number of available entries (they may not be contiguous).
   int32 AvailableEntries() {
     return (get_offset() - put_ - 1 + total_entry_count_) % total_entry_count_;
   }
 
@@ skipping 15 matching lines @@
   CommandBuffer* command_buffer_;
   int32 ring_buffer_id_;
   int32 ring_buffer_size_;
   scoped_refptr<gpu::Buffer> ring_buffer_;
   CommandBufferEntry* entries_;
   int32 total_entry_count_;  // the total number of entries
   int32 immediate_entry_count_;
   int32 token_;
   int32 put_;
   int32 last_put_sent_;
-  int32 last_barrier_put_sent_;
+
+  uint32_t next_fence_sync_release_;

[sunnyps, 2015/09/23 21:44:45]

I think this should be part of GpuChannelHost::StreamFlushInfo. That way the id
used for the sync token is { GPU_IO, channel_id << 32 | stream_id, release_count
} and on the service side SyncPointClientState is also a per-stream thing. WDYT?

[David Yen, 2015/09/25 20:13:33]

The command buffer ID should be per context (as that is the level of where
release counts are kept track). I discussed though this has been moved to
GpuControl.

+  uint32_t flushed_fence_sync_release_;
 
 #if defined(CMD_HELPER_PERIODIC_FLUSH_CHECK)
   int commands_issued_;
 #endif
 
   bool usable_;
   bool context_lost_;
   bool flush_automatically_;
 
   base::TimeTicks last_flush_time_;
 
   // Incremented every time the helper flushes the command buffer.
   // Can be used to track when prior commands have been flushed.
   uint32 flush_generation_;
 
   friend class CommandBufferHelperTest;
   DISALLOW_COPY_AND_ASSIGN(CommandBufferHelper);
 };
 
 }  // namespace gpu
 
 #endif  // GPU_COMMAND_BUFFER_CLIENT_CMD_BUFFER_HELPER_H_