gpu: Thread-safe command buffer state lookup.

gpu/command_buffer/client/ring_buffer.cc

 // Copyright (c) 2011 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 implementation of the RingBuffer class.
 
 #include "gpu/command_buffer/client/ring_buffer.h"
 
 #include <stdint.h>
 
@@ skipping 128 matching lines @@
         in_use_offset_ = 0;
         free_offset_ = 0;
       }
       return;
     }
   }
   NOTREACHED() << "attempt to discard non-existant block";
 }
 
 unsigned int RingBuffer::GetLargestFreeSizeNoWaiting() {
-  unsigned int last_token_read = helper_->last_token_read();
   while (!blocks_.empty()) {
     Block& block = blocks_.front();
-    if (block.token > last_token_read || block.state == IN_USE) break;
+    if (!helper_->HasTokenPassed(block.token) || block.state == IN_USE) break;

[jbauman, 2016/12/09 03:16:03]

This will most likely hit the lock at least once per call to RingBuffer::Alloc,
as if any ringbuffer entry is being used on the service side it'll free blocks
until it hits that and has to check its token. That would probably be
acceptable, and I suspect the perf bots will let us know if not.

[sunnyps, 2016/12/09 04:09:37]

I didn't expose token from the helper directly because this wasn't handling
token wrap around and I wanted to avoid a similar mistake in the future.
HasTokenPassed has an early out if the cached token >= given token.

     FreeOldestBlock();
   }
   if (free_offset_ == in_use_offset_) {
     if (blocks_.empty()) {
       // The entire buffer is free.
       DCHECK_EQ(free_offset_, 0u);
       return size_;
     } else {
       // The entire buffer is in use.
       return 0;
@@ skipping 11 matching lines @@
   unsigned int largest_free_size = GetLargestFreeSizeNoWaiting();
   if (free_offset_ > in_use_offset_) {
     // It's free from free_offset_ to size_ and from 0 to in_use_offset_.
     return size_ - free_offset_ + in_use_offset_;
   } else {
     return largest_free_size;
   }
 }
 
 }  // namespace gpu