diff options
Diffstat (limited to 'services/surfaceflinger/BufferStateLayer.h')
| -rw-r--r-- | services/surfaceflinger/BufferStateLayer.h | 111 |
1 files changed, 61 insertions, 50 deletions
diff --git a/services/surfaceflinger/BufferStateLayer.h b/services/surfaceflinger/BufferStateLayer.h index 00fa7f7a2d..e5674785af 100644 --- a/services/surfaceflinger/BufferStateLayer.h +++ b/services/surfaceflinger/BufferStateLayer.h @@ -36,9 +36,7 @@ public: ~BufferStateLayer() override; - // ----------------------------------------------------------------------- - // Interface implementation for Layer - // ----------------------------------------------------------------------- + // Implements Layer. const char* getType() const override { return "BufferStateLayer"; } void onLayerDisplayed(const sp<Fence>& releaseFence) override; @@ -47,21 +45,8 @@ public: void finalizeFrameEventHistory(const std::shared_ptr<FenceTime>& glDoneFence, const CompositorTiming& compositorTiming) override; - bool shouldPresentNow(nsecs_t expectedPresentTime) const override; + bool isBufferDue(nsecs_t /*expectedPresentTime*/) const override { return true; } - uint32_t doTransactionResize(uint32_t flags, Layer::State* /*stateToCommit*/) override { - return flags; - } - /*TODO:vhau return to using BufferStateLayer override once WM - * has removed deferred transactions! - void pushPendingState() override;*/ - bool applyPendingStates(Layer::State* stateToCommit) override; - - uint32_t getActiveWidth(const Layer::State& s) const override { return s.active.w; } - uint32_t getActiveHeight(const Layer::State& s) const override { return s.active.h; } - ui::Transform getActiveTransform(const Layer::State& s) const override { - return s.active.transform; - } Region getActiveTransparentRegion(const Layer::State& s) const override { return s.transparentRegionHint; } @@ -70,9 +55,11 @@ public: bool setTransform(uint32_t transform) override; bool setTransformToDisplayInverse(bool transformToDisplayInverse) override; bool setCrop(const Rect& crop) override; - bool setFrame(const Rect& frame) override; - bool setBuffer(const sp<GraphicBuffer>& buffer, const sp<Fence>& acquireFence, nsecs_t postTime, - nsecs_t desiredPresentTime, const client_cache_t& clientCacheId) override; + bool setBuffer(const std::shared_ptr<renderengine::ExternalTexture>& buffer, + const sp<Fence>& acquireFence, nsecs_t postTime, nsecs_t desiredPresentTime, + bool isAutoTimestamp, const client_cache_t& clientCacheId, uint64_t frameNumber, + std::optional<nsecs_t> dequeueTime, const FrameTimelineInfo& info, + const sp<ITransactionCompletedListener>& transactionListener) override; bool setAcquireFence(const sp<Fence>& fence) override; bool setDataspace(ui::Dataspace dataspace) override; bool setHdrMetadata(const HdrMetadata& hdrMetadata) override; @@ -80,28 +67,23 @@ public: bool setApi(int32_t api) override; bool setSidebandStream(const sp<NativeHandle>& sidebandStream) override; bool setTransactionCompletedListeners(const std::vector<sp<CallbackHandle>>& handles) override; - void forceSendCallbacks() override; bool addFrameEvent(const sp<Fence>& acquireFence, nsecs_t postedTime, nsecs_t requestedPresentTime) override; + bool setPosition(float /*x*/, float /*y*/) override; + bool setMatrix(const layer_state_t::matrix22_t& /*matrix*/, + bool /*allowNonRectPreservingTransforms*/); // Override to ignore legacy layer state properties that are not used by BufferStateLayer bool setSize(uint32_t /*w*/, uint32_t /*h*/) override { return false; } - bool setPosition(float /*x*/, float /*y*/) override { return false; } bool setTransparentRegionHint(const Region& transparent) override; - bool setMatrix(const layer_state_t::matrix22_t& /*matrix*/, - bool /*allowNonRectPreservingTransforms*/) override { - return false; - } - bool setCrop_legacy(const Rect& /*crop*/) override { return false; } - bool setOverrideScalingMode(int32_t /*overrideScalingMode*/) override { return false; } - void deferTransactionUntil_legacy(const sp<IBinder>& /*barrierHandle*/, - uint64_t /*frameNumber*/) override {} - void deferTransactionUntil_legacy(const sp<Layer>& /*barrierLayer*/, - uint64_t /*frameNumber*/) override {} Rect getBufferSize(const State& s) const override; FloatRect computeSourceBounds(const FloatRect& parentBounds) const override; - Layer::RoundedCornerState getRoundedCornerState() const override; + void setAutoRefresh(bool autoRefresh) override; + + bool setBufferCrop(const Rect& bufferCrop) override; + bool setDestinationFrame(const Rect& destinationFrame) override; + bool updateGeometry() override; // ----------------------------------------------------------------------- @@ -111,25 +93,41 @@ public: bool fenceHasSignaled() const override; bool framePresentTimeIsCurrent(nsecs_t expectedPresentTime) const override; bool onPreComposition(nsecs_t refreshStartTime) override; + uint32_t getEffectiveScalingMode() const override; + + // See mPendingBufferTransactions + void decrementPendingBufferCount(); + void bufferMayChange(const sp<GraphicBuffer>& newBuffer) override; + std::atomic<int32_t>* getPendingBufferCounter() override { return &mPendingBufferTransactions; } + std::string getPendingBufferCounterName() override { return mBlastTransactionName; } + + bool shouldPresentNow(nsecs_t /*expectedPresentTime*/) const override { return true; } protected: void gatherBufferInfo() override; uint64_t getHeadFrameNumber(nsecs_t expectedPresentTime) const; + void onSurfaceFrameCreated(const std::shared_ptr<frametimeline::SurfaceFrame>& surfaceFrame); + ui::Transform getInputTransform() const override; + Rect getInputBounds() const override; private: + friend class SlotGenerationTest; + friend class TransactionFrameTracerTest; + friend class TransactionSurfaceFrameTest; + + inline void tracePendingBufferCount(int32_t pendingBuffers); + bool updateFrameEventHistory(const sp<Fence>& acquireFence, nsecs_t postedTime, nsecs_t requestedPresentTime); - uint64_t getFrameNumber(nsecs_t expectedPresentTime) const override; + status_t addReleaseFence(const sp<CallbackHandle>& ch, const sp<Fence>& releaseFence); - bool getAutoRefresh() const override; - bool getSidebandStreamChanged() const override; + uint64_t getFrameNumber(nsecs_t expectedPresentTime) const override; bool latchSidebandStream(bool& recomputeVisibleRegions) override; bool hasFrameUpdate() const override; - status_t bindTextureImage() override; status_t updateTexImage(bool& recomputeVisibleRegions, nsecs_t latchTime, nsecs_t expectedPresentTime) override; @@ -139,29 +137,42 @@ private: sp<Layer> createClone() override; // Crop that applies to the buffer - Rect computeCrop(const State& s); + Rect computeBufferCrop(const State& s); -private: - friend class SlotGenerationTest; bool willPresentCurrentTransaction() const; - static const std::array<float, 16> IDENTITY_MATRIX; - - std::unique_ptr<renderengine::Image> mTextureImage; - - std::atomic<bool> mSidebandStreamChanged{false}; - - mutable uint64_t mFrameNumber{0}; - uint64_t mFrameCounter{0}; + bool bufferNeedsFiltering() const override; sp<Fence> mPreviousReleaseFence; - uint64_t mPreviousBufferId = 0; + ReleaseCallbackId mPreviousReleaseCallbackId = ReleaseCallbackId::INVALID_ID; uint64_t mPreviousReleasedFrameNumber = 0; - mutable bool mCurrentStateModified = false; bool mReleasePreviousBuffer = false; + + // Stores the last set acquire fence signal time used to populate the callback handle's acquire + // time. nsecs_t mCallbackHandleAcquireTime = -1; + std::deque<std::shared_ptr<android::frametimeline::SurfaceFrame>> mPendingJankClassifications; + // An upper bound on the number of SurfaceFrames in the pending classifications deque. + static constexpr int kPendingClassificationMaxSurfaceFrames = 25; + + const std::string mBlastTransactionName{"BufferTX - " + mName}; + // This integer is incremented everytime a buffer arrives at the server for this layer, + // and decremented when a buffer is dropped or latched. When changed the integer is exported + // to systrace with ATRACE_INT and mBlastTransactionName. This way when debugging perf it is + // possible to see when a buffer arrived at the server, and in which frame it latched. + // + // You can understand the trace this way: + // - If the integer increases, a buffer arrived at the server. + // - If the integer decreases in latchBuffer, that buffer was latched + // - If the integer decreases in setBuffer or doTransaction, a buffer was dropped + std::atomic<int32_t> mPendingBufferTransactions{0}; + + // Contains requested position and matrix updates. This will be applied if the client does + // not specify a destination frame. + ui::Transform mRequestedTransform; + // TODO(marissaw): support sticky transform for LEGACY camera mode class HwcSlotGenerator : public ClientCache::ErasedRecipient { |