From 1dc4bfa7abbdab74ad7dd29d3c860583bfe0fd0c Mon Sep 17 00:00:00 2001 From: Alec Mouri Date: Fri, 1 Oct 2021 16:31:08 -0700 Subject: Remove buffer occupancy stats These statistics tracked double and triple buffering rates in SurfaceFlinger, both of which are not possible to track in BufferStateLayer, nor were these histograms particularly looked at when BufferQueueLayer was using this. Bug: 170662799 Bug: 199699421 Test: builds Change-Id: I6c65db1cb4de790e5bf46eaff5dee5f3491a6641 --- services/surfaceflinger/BufferLayer.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'services/surfaceflinger/BufferLayer.cpp') diff --git a/services/surfaceflinger/BufferLayer.cpp b/services/surfaceflinger/BufferLayer.cpp index d805294b07..ffb0a188ca 100644 --- a/services/surfaceflinger/BufferLayer.cpp +++ b/services/surfaceflinger/BufferLayer.cpp @@ -346,13 +346,13 @@ TimeStats::SetFrameRateVote frameRateToSetFrameRateVotePayload(Layer::FrameRate } } // namespace -bool BufferLayer::onPostComposition(const DisplayDevice* display, +void BufferLayer::onPostComposition(const DisplayDevice* display, const std::shared_ptr& glDoneFence, const std::shared_ptr& presentFence, const CompositorTiming& compositorTiming) { // mFrameLatencyNeeded is true when a new frame was latched for the // composition. - if (!mBufferInfo.mFrameLatencyNeeded) return false; + if (!mBufferInfo.mFrameLatencyNeeded) return; // Update mFrameEventHistory. { @@ -426,7 +426,6 @@ bool BufferLayer::onPostComposition(const DisplayDevice* display, mFrameTracker.advanceFrame(); mBufferInfo.mFrameLatencyNeeded = false; - return true; } void BufferLayer::gatherBufferInfo() { -- cgit v1.2.3-59-g8ed1b