diff options
author | dongseong.hwang <dongseong.hwang@intel.com> | 2016-01-22 02:14:42 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-01-22 10:16:15 +0000 |
commit | b482d9409f829c4c5c3c5030d251399b97cecea8 (patch) | |
tree | 1693ae34618cd6bd954dcacb0e2a108c008ed7c6 /cc/raster | |
parent | 489a998950c75693eaa5bd96eacd7e034fab3fec (diff) | |
download | chromium_src-b482d9409f829c4c5c3c5030d251399b97cecea8.zip chromium_src-b482d9409f829c4c5c3c5030d251399b97cecea8.tar.gz chromium_src-b482d9409f829c4c5c3c5030d251399b97cecea8.tar.bz2 |
cc: narrow ChromeOS perf workaround to ARM
GL_COMMANDS_COMPLETED_CHROMIUM makes EGL_SYNC_PRIOR_COMMANDS_IMPLICIT_EXTERNAL_ARM
fence slow down significantly. It's ARM GPU driver issue,
which chromium cannot workaround it.
BUG=522903, 580166
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
Review URL: https://codereview.chromium.org/1221433002
Cr-Commit-Position: refs/heads/master@{#370935}
Diffstat (limited to 'cc/raster')
-rw-r--r-- | cc/raster/one_copy_tile_task_worker_pool.cc | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/cc/raster/one_copy_tile_task_worker_pool.cc b/cc/raster/one_copy_tile_task_worker_pool.cc index ea1b947..9aa9750 100644 --- a/cc/raster/one_copy_tile_task_worker_pool.cc +++ b/cc/raster/one_copy_tile_task_worker_pool.cc @@ -420,10 +420,9 @@ void OneCopyTileTaskWorkerPool::PlaybackAndCopyOnWorkerThread( if (!staging_buffer->query_id) gl->GenQueriesEXT(1, &staging_buffer->query_id); -#if defined(OS_CHROMEOS) - // TODO(reveman): This avoids a performance problem on some ChromeOS - // devices. This needs to be removed to support native GpuMemoryBuffer - // implementations. crbug.com/436314 +#if defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) + // TODO(reveman): This avoids a performance problem on ARM ChromeOS + // devices. crbug.com/580166 gl->BeginQueryEXT(GL_COMMANDS_ISSUED_CHROMIUM, staging_buffer->query_id); #else gl->BeginQueryEXT(GL_COMMANDS_COMPLETED_CHROMIUM, @@ -460,7 +459,7 @@ void OneCopyTileTaskWorkerPool::PlaybackAndCopyOnWorkerThread( } if (resource_provider_->use_sync_query()) { -#if defined(OS_CHROMEOS) +#if defined(OS_CHROMEOS) && defined(ARCH_CPU_ARM_FAMILY) gl->EndQueryEXT(GL_COMMANDS_ISSUED_CHROMIUM); #else gl->EndQueryEXT(GL_COMMANDS_COMPLETED_CHROMIUM); |