summaryrefslogtreecommitdiffstats
path: root/ui/ozone/platform/drm/gpu/drm_window.h
diff options
context:
space:
mode:
authorspang <spang@chromium.org>2015-04-17 09:12:20 -0700
committerCommit bot <commit-bot@chromium.org>2015-04-17 16:12:55 +0000
commit742b4e2e806240e87de922590bd4889bf6a4383f (patch)
tree956b7d0c275db37d299329bf491c9540cd9504f2 /ui/ozone/platform/drm/gpu/drm_window.h
parent767230290ce867242ef45b0b5a62a956014d0d3c (diff)
downloadchromium_src-742b4e2e806240e87de922590bd4889bf6a4383f.zip
chromium_src-742b4e2e806240e87de922590bd4889bf6a4383f.tar.gz
chromium_src-742b4e2e806240e87de922590bd4889bf6a4383f.tar.bz2
Revert of [ozone] Keep track of pending frames on drm_window when hardware display controller is disabled or … (patchset #8 id:120002 of https://codereview.chromium.org/1078183003/)
Reason for revert: May cause a GPU crash on boot. BUG=477761 Original issue's description: > [ozone] Keep track of pending frames on drm_window when hardware display controller is disabled or when in headless mode. > > When display controller is disabled, it used to stop tracking which buffer was the considered front. Surface would fast ack all incoming buffers without presenting them, which means that any one of them could be getting painted into while display controller is off. Upon wakeup, the controller would grab what it thought was the last front buffer, but in practice it was not because of the fast acks. Net result is that it may try to modeset with the buffer GL is painting into, making it hang on a fence in the driver. > > This patch removes all knowledge of front buffers from the display controller and explicitly sets it on the window, which tracks it while the controller is off. > > BUG=473831 > > Committed: https://crrev.com/b47c9f5fb6b669b27e055ab8b305309170849732 > Cr-Commit-Position: refs/heads/master@{#325326} TBR=dnicoara@chromium.org,alexst@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=473831 Review URL: https://codereview.chromium.org/1057413004 Cr-Commit-Position: refs/heads/master@{#325648}
Diffstat (limited to 'ui/ozone/platform/drm/gpu/drm_window.h')
-rw-r--r--ui/ozone/platform/drm/gpu/drm_window.h17
1 files changed, 0 insertions, 17 deletions
diff --git a/ui/ozone/platform/drm/gpu/drm_window.h b/ui/ozone/platform/drm/gpu/drm_window.h
index 00f2cc0..f655bbb 100644
--- a/ui/ozone/platform/drm/gpu/drm_window.h
+++ b/ui/ozone/platform/drm/gpu/drm_window.h
@@ -12,7 +12,6 @@
#include "ui/gfx/geometry/rect.h"
#include "ui/gfx/native_widget_types.h"
#include "ui/ozone/ozone_export.h"
-#include "ui/ozone/platform/drm/gpu/overlay_plane.h"
class SkBitmap;
@@ -74,16 +73,6 @@ class OZONE_EXPORT DrmWindow {
// Move the HW cursor to the specified location.
void MoveCursor(const gfx::Point& location);
- // Queue overlay planes and page flips.
- // If hardware display controller is available, forward the information
- // immediately, otherwise queue up on the window and forward when the hardware
- // is once again ready.
- void QueueOverlayPlane(const OverlayPlane& plane);
- bool SchedulePageFlip(bool is_sync, const base::Closure& callback);
-
- // About to enable hardware display controller, aquire pending frames.
- bool PrepareController(HardwareDisplayController* controller);
-
private:
// Draw the last set cursor & update the cursor plane.
void ResetCursor(bool bitmap_only);
@@ -117,12 +106,6 @@ class OZONE_EXPORT DrmWindow {
int cursor_frame_;
int cursor_frame_delay_ms_;
- // Planes and flips currently being queued in the absence of hardware display
- // controller.
- OverlayPlaneList pending_planes_;
- OverlayPlaneList last_submitted_planes_;
- bool last_swap_sync_;
-
DISALLOW_COPY_AND_ASSIGN(DrmWindow);
};