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

Issue 1835173003: gpu: Remove GPU service side GMB creation from handle.

Created:
4 years, 8 months ago by reveman
Modified:
4 years, 8 months ago
Reviewers:
CC:
chromium-reviews, darin-cc_chromium.org, jam, piman+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@ozone-gmb-gpu-process-refactor
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

gpu: Remove GPU service side GMB creation from handle. Buffer creation from handle does not need to happen on the GPU service side. This reduces complexity and improves performance by avoiding a round-trip to the GPU process when importing buffers. BUG=597932

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -187 lines) Patch
M content/browser/gpu/browser_gpu_memory_buffer_manager.cc View 2 chunks +21 lines, -21 lines 0 comments Download
M content/browser/gpu/gpu_process_host.h View 1 chunk +0 lines, -10 lines 0 comments Download
M content/browser/gpu/gpu_process_host.cc View 1 chunk +0 lines, -24 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory.h View 1 chunk +0 lines, -9 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_io_surface.h View 1 chunk +0 lines, -6 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_io_surface.cc View 1 chunk +0 lines, -11 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_ozone_native_pixmap.h View 1 chunk +0 lines, -6 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_ozone_native_pixmap.cc View 1 chunk +0 lines, -14 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_surface_texture.h View 1 chunk +0 lines, -6 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_surface_texture.cc View 1 chunk +0 lines, -11 lines 0 comments Download
M content/common/gpu/gpu_memory_buffer_factory_test_template.h View 1 chunk +0 lines, -42 lines 0 comments Download
M content/common/gpu_host_messages.h View 2 chunks +0 lines, -13 lines 0 comments Download
M content/gpu/gpu_child_thread.cc View 2 chunks +0 lines, -14 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 1 (1 generated)
reveman
4 years, 8 months ago (2016-03-27 18:44:03 UTC) #1
Description was changed from

==========
gpu: Remove GPU service side GMB creation from handle.

Buffer creation from handle does not need to happen on the
GPU service side. This reduces complexity and improves
performance by avoiding a round-trip to the GPU process
when importing buffers.

Note: A round-trip to the IO thread can also be removed
in a follow up change.

BUG=597932
==========

to

==========
gpu: Remove GPU service side GMB creation from handle.

Buffer creation from handle does not need to happen on the
GPU service side. This reduces complexity and improves
performance by avoiding a round-trip to the GPU process
when importing buffers.

BUG=597932
==========

Powered by Google App Engine
This is Rietveld 408576698