| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 1 | /* | 
|  | 2 | ** | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 3 | ** Copyright 2012 The Android Open Source Project | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 4 | ** | 
|  | 5 | ** Licensed under the Apache License Version 2.0(the "License"); | 
|  | 6 | ** you may not use this file except in compliance with the License. | 
|  | 7 | ** You may obtain a copy of the License at | 
|  | 8 | ** | 
|  | 9 | **     http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 10 | ** | 
|  | 11 | ** Unless required by applicable law or agreed to in writing software | 
|  | 12 | ** distributed under the License is distributed on an "AS IS" BASIS | 
|  | 13 | ** WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND either express or implied. | 
|  | 14 | ** See the License for the specific language governing permissions and | 
|  | 15 | ** limitations under the License. | 
|  | 16 | */ | 
|  | 17 |  | 
| Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 18 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 19 | #pragma clang diagnostic push | 
|  | 20 | #pragma clang diagnostic ignored "-Wconversion" | 
|  | 21 |  | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 22 | // #define LOG_NDEBUG 0 | 
|  | 23 | #undef LOG_TAG | 
|  | 24 | #define LOG_TAG "FramebufferSurface" | 
|  | 25 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 26 | #include <errno.h> | 
| Mark Salyzyn | a5e161b | 2016-09-29 08:08:05 -0700 | [diff] [blame] | 27 | #include <stdio.h> | 
|  | 28 | #include <stdlib.h> | 
|  | 29 | #include <string.h> | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 30 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 31 | #include <utils/String8.h> | 
| Mark Salyzyn | 7823e12 | 2016-09-29 08:08:05 -0700 | [diff] [blame] | 32 | #include <log/log.h> | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 33 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 34 | #include <hardware/hardware.h> | 
| Dan Stoza | 84493cd | 2015-03-12 15:12:44 -0700 | [diff] [blame] | 35 | #include <gui/BufferItem.h> | 
| Mathias Agopian | a934764 | 2017-02-13 16:42:28 -0800 | [diff] [blame] | 36 | #include <gui/BufferQueue.h> | 
| Dan Stoza | 84493cd | 2015-03-12 15:12:44 -0700 | [diff] [blame] | 37 | #include <gui/Surface.h> | 
| Mathias Agopian | a934764 | 2017-02-13 16:42:28 -0800 | [diff] [blame] | 38 |  | 
| Courtney Goeltzenleuchter | 19987a8 | 2017-07-14 12:16:56 -0600 | [diff] [blame] | 39 | #include <ui/DebugUtils.h> | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 40 | #include <ui/GraphicBuffer.h> | 
| Mathias Agopian | a934764 | 2017-02-13 16:42:28 -0800 | [diff] [blame] | 41 | #include <ui/Rect.h> | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 42 |  | 
| Mathias Agopian | 33ceeb3 | 2013-04-01 16:54:58 -0700 | [diff] [blame] | 43 | #include "FramebufferSurface.h" | 
|  | 44 | #include "HWComposer.h" | 
| Fabien Sanglard | 1971b63 | 2017-03-10 14:50:03 -0800 | [diff] [blame] | 45 | #include "../SurfaceFlinger.h" | 
| Jamie Gennis | cdbaecb | 2012-10-12 14:18:10 -0700 | [diff] [blame] | 46 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 47 | // ---------------------------------------------------------------------------- | 
|  | 48 | namespace android { | 
|  | 49 | // ---------------------------------------------------------------------------- | 
|  | 50 |  | 
| Peiyong Lin | 34beb7a | 2018-03-28 11:57:12 -0700 | [diff] [blame] | 51 | using ui::Dataspace; | 
|  | 52 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 53 | /* | 
|  | 54 | * This implements the (main) framebuffer management. This class is used | 
|  | 55 | * mostly by SurfaceFlinger, but also by command line GL application. | 
|  | 56 | * | 
|  | 57 | */ | 
|  | 58 |  | 
| Marin Shalamanov | 0f10d0d | 2020-08-06 20:04:06 +0200 | [diff] [blame] | 59 | FramebufferSurface::FramebufferSurface(HWComposer& hwc, PhysicalDisplayId displayId, | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 60 | const sp<IGraphicBufferConsumer>& consumer, | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 61 | const ui::Size& size, const ui::Size& maxSize) | 
| Dominik Laskowski | 075d317 | 2018-05-24 15:50:06 -0700 | [diff] [blame] | 62 | : ConsumerBase(consumer), | 
|  | 63 | mDisplayId(displayId), | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 64 | mMaxSize(maxSize), | 
| Dominik Laskowski | 075d317 | 2018-05-24 15:50:06 -0700 | [diff] [blame] | 65 | mCurrentBufferSlot(-1), | 
|  | 66 | mCurrentBuffer(), | 
|  | 67 | mCurrentFence(Fence::NO_FENCE), | 
|  | 68 | mHwc(hwc), | 
|  | 69 | mHasPendingRelease(false), | 
|  | 70 | mPreviousBufferSlot(BufferQueue::INVALID_BUFFER_SLOT), | 
|  | 71 | mPreviousBuffer() { | 
| Dominik Laskowski | 3415776 | 2018-10-31 13:07:19 -0700 | [diff] [blame] | 72 | ALOGV("Creating for display %s", to_string(displayId).c_str()); | 
| Fabien Sanglard | 9d96de4 | 2016-10-11 00:15:18 +0000 | [diff] [blame] | 73 |  | 
| Andy McFadden | b0d1dd3 | 2012-09-10 14:08:09 -0700 | [diff] [blame] | 74 | mName = "FramebufferSurface"; | 
| Mathias Agopian | db89edc | 2013-08-02 01:40:18 -0700 | [diff] [blame] | 75 | mConsumer->setConsumerName(mName); | 
|  | 76 | mConsumer->setConsumerUsageBits(GRALLOC_USAGE_HW_FB | | 
| Mathias Agopian | f5a3392 | 2012-09-19 18:16:22 -0700 | [diff] [blame] | 77 | GRALLOC_USAGE_HW_RENDER | | 
|  | 78 | GRALLOC_USAGE_HW_COMPOSER); | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 79 | const auto limitedSize = limitSize(size); | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 80 | mConsumer->setDefaultBufferSize(limitedSize.width, limitedSize.height); | 
| Fabien Sanglard | 1971b63 | 2017-03-10 14:50:03 -0800 | [diff] [blame] | 81 | mConsumer->setMaxAcquiredBufferCount( | 
|  | 82 | SurfaceFlinger::maxFrameBufferAcquiredBuffers - 1); | 
| Jesse Hall | 99c7dbb | 2013-03-14 14:29:29 -0700 | [diff] [blame] | 83 | } | 
|  | 84 |  | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 85 | void FramebufferSurface::resizeBuffers(const ui::Size& newSize) { | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 86 | const auto limitedSize = limitSize(newSize); | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 87 | mConsumer->setDefaultBufferSize(limitedSize.width, limitedSize.height); | 
| Lloyd Pique | 2ae2b3b | 2017-12-14 17:18:17 -0800 | [diff] [blame] | 88 | } | 
|  | 89 |  | 
| Jesse Hall | 7cd8597 | 2014-08-07 22:48:06 -0700 | [diff] [blame] | 90 | status_t FramebufferSurface::beginFrame(bool /*mustRecompose*/) { | 
| Jesse Hall | 028dc8f | 2013-08-20 16:35:32 -0700 | [diff] [blame] | 91 | return NO_ERROR; | 
|  | 92 | } | 
|  | 93 |  | 
| Mark Salyzyn | 92dc3fc | 2014-03-12 13:12:44 -0700 | [diff] [blame] | 94 | status_t FramebufferSurface::prepareFrame(CompositionType /*compositionType*/) { | 
| Jesse Hall | 38efe86 | 2013-04-06 23:12:29 -0700 | [diff] [blame] | 95 | return NO_ERROR; | 
|  | 96 | } | 
|  | 97 |  | 
| Jesse Hall | 99c7dbb | 2013-03-14 14:29:29 -0700 | [diff] [blame] | 98 | status_t FramebufferSurface::advanceFrame() { | 
| Chia-I Wu | 06d63de | 2017-01-04 14:58:51 +0800 | [diff] [blame] | 99 | uint32_t slot = 0; | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 100 | sp<GraphicBuffer> buf; | 
|  | 101 | sp<Fence> acquireFence(Fence::NO_FENCE); | 
| Peiyong Lin | 34beb7a | 2018-03-28 11:57:12 -0700 | [diff] [blame] | 102 | Dataspace dataspace = Dataspace::UNKNOWN; | 
| Chia-I Wu | 06d63de | 2017-01-04 14:58:51 +0800 | [diff] [blame] | 103 | status_t result = nextBuffer(slot, buf, acquireFence, dataspace); | 
| Courtney Goeltzenleuchter | 19987a8 | 2017-07-14 12:16:56 -0600 | [diff] [blame] | 104 | mDataSpace = dataspace; | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 105 | if (result != NO_ERROR) { | 
|  | 106 | ALOGE("error latching next FramebufferSurface buffer: %s (%d)", | 
|  | 107 | strerror(-result), result); | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 108 | } | 
|  | 109 | return result; | 
| Jesse Hall | 99c7dbb | 2013-03-14 14:29:29 -0700 | [diff] [blame] | 110 | } | 
|  | 111 |  | 
| Chia-I Wu | 06d63de | 2017-01-04 14:58:51 +0800 | [diff] [blame] | 112 | status_t FramebufferSurface::nextBuffer(uint32_t& outSlot, | 
|  | 113 | sp<GraphicBuffer>& outBuffer, sp<Fence>& outFence, | 
| Peiyong Lin | 34beb7a | 2018-03-28 11:57:12 -0700 | [diff] [blame] | 114 | Dataspace& outDataspace) { | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 115 | Mutex::Autolock lock(mMutex); | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 116 |  | 
| Dan Stoza | 84493cd | 2015-03-12 15:12:44 -0700 | [diff] [blame] | 117 | BufferItem item; | 
| Andy McFadden | 1585c4d | 2013-06-28 13:52:40 -0700 | [diff] [blame] | 118 | status_t err = acquireBufferLocked(&item, 0); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 119 | if (err == BufferQueue::NO_BUFFER_AVAILABLE) { | 
| Valerie Hau | 13f0d1a | 2019-03-22 10:35:42 -0700 | [diff] [blame] | 120 | mHwcBufferCache.getHwcBuffer(mCurrentBufferSlot, mCurrentBuffer, &outSlot, &outBuffer); | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 121 | return NO_ERROR; | 
|  | 122 | } else if (err != NO_ERROR) { | 
|  | 123 | ALOGE("error acquiring buffer: %s (%d)", strerror(-err), err); | 
|  | 124 | return err; | 
|  | 125 | } | 
|  | 126 |  | 
|  | 127 | // If the BufferQueue has freed and reallocated a buffer in mCurrentSlot | 
|  | 128 | // then we may have acquired the slot we already own.  If we had released | 
|  | 129 | // our current buffer before we call acquireBuffer then that release call | 
|  | 130 | // would have returned STALE_BUFFER_SLOT, and we would have called | 
|  | 131 | // freeBufferLocked on that slot.  Because the buffer slot has already | 
|  | 132 | // been overwritten with the new buffer all we have to do is skip the | 
|  | 133 | // releaseBuffer call and we should be in the same state we'd be in if we | 
|  | 134 | // had released the old buffer first. | 
|  | 135 | if (mCurrentBufferSlot != BufferQueue::INVALID_BUFFER_SLOT && | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 136 | item.mSlot != mCurrentBufferSlot) { | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 137 | mHasPendingRelease = true; | 
|  | 138 | mPreviousBufferSlot = mCurrentBufferSlot; | 
|  | 139 | mPreviousBuffer = mCurrentBuffer; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 140 | } | 
| Pablo Ceballos | 47650f4 | 2015-08-04 16:38:17 -0700 | [diff] [blame] | 141 | mCurrentBufferSlot = item.mSlot; | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 142 | mCurrentBuffer = mSlots[mCurrentBufferSlot].mGraphicBuffer; | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 143 | mCurrentFence = item.mFence; | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 144 |  | 
| Mathias Agopian | da27af9 | 2012-09-13 18:17:13 -0700 | [diff] [blame] | 145 | outFence = item.mFence; | 
| Valerie Hau | 13f0d1a | 2019-03-22 10:35:42 -0700 | [diff] [blame] | 146 | mHwcBufferCache.getHwcBuffer(mCurrentBufferSlot, mCurrentBuffer, &outSlot, &outBuffer); | 
| Peiyong Lin | 34beb7a | 2018-03-28 11:57:12 -0700 | [diff] [blame] | 147 | outDataspace = static_cast<Dataspace>(item.mDataSpace); | 
| Dominik Laskowski | 075d317 | 2018-05-24 15:50:06 -0700 | [diff] [blame] | 148 | status_t result = mHwc.setClientTarget(mDisplayId, outSlot, outFence, outBuffer, outDataspace); | 
| Courtney Goeltzenleuchter | 79d2724 | 2017-07-13 17:54:01 -0600 | [diff] [blame] | 149 | if (result != NO_ERROR) { | 
|  | 150 | ALOGE("error posting framebuffer: %d", result); | 
| Courtney Goeltzenleuchter | cd939aa | 2017-07-25 10:34:27 -0600 | [diff] [blame] | 151 | return result; | 
| Courtney Goeltzenleuchter | 79d2724 | 2017-07-13 17:54:01 -0600 | [diff] [blame] | 152 | } | 
| Courtney Goeltzenleuchter | 281e811 | 2017-07-13 17:54:01 -0600 | [diff] [blame] | 153 |  | 
| Courtney Goeltzenleuchter | 6461806 | 2017-08-02 15:55:06 -0600 | [diff] [blame] | 154 | return NO_ERROR; | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 155 | } | 
|  | 156 |  | 
| Jamie Gennis | 1a4d883 | 2012-08-02 20:11:05 -0700 | [diff] [blame] | 157 | void FramebufferSurface::freeBufferLocked(int slotIndex) { | 
|  | 158 | ConsumerBase::freeBufferLocked(slotIndex); | 
|  | 159 | if (slotIndex == mCurrentBufferSlot) { | 
|  | 160 | mCurrentBufferSlot = BufferQueue::INVALID_BUFFER_SLOT; | 
|  | 161 | } | 
|  | 162 | } | 
|  | 163 |  | 
| Jesse Hall | 851cfe8 | 2013-03-20 13:44:00 -0700 | [diff] [blame] | 164 | void FramebufferSurface::onFrameCommitted() { | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 165 | if (mHasPendingRelease) { | 
| Dominik Laskowski | 075d317 | 2018-05-24 15:50:06 -0700 | [diff] [blame] | 166 | sp<Fence> fence = mHwc.getPresentFence(mDisplayId); | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 167 | if (fence->isValid()) { | 
|  | 168 | status_t result = addReleaseFence(mPreviousBufferSlot, | 
|  | 169 | mPreviousBuffer, fence); | 
|  | 170 | ALOGE_IF(result != NO_ERROR, "onFrameCommitted: failed to add the" | 
|  | 171 | " fence: %s (%d)", strerror(-result), result); | 
|  | 172 | } | 
| Chia-I Wu | 1e24cce | 2017-11-10 10:36:01 -0800 | [diff] [blame] | 173 | status_t result = releaseBufferLocked(mPreviousBufferSlot, mPreviousBuffer); | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 174 | ALOGE_IF(result != NO_ERROR, "onFrameCommitted: error releasing buffer:" | 
|  | 175 | " %s (%d)", strerror(-result), result); | 
|  | 176 |  | 
|  | 177 | mPreviousBuffer.clear(); | 
|  | 178 | mHasPendingRelease = false; | 
|  | 179 | } | 
| Mathias Agopian | da27af9 | 2012-09-13 18:17:13 -0700 | [diff] [blame] | 180 | } | 
|  | 181 |  | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 182 | ui::Size FramebufferSurface::limitSize(const ui::Size& size) { | 
|  | 183 | return limitSizeInternal(size, mMaxSize); | 
|  | 184 | } | 
|  | 185 |  | 
|  | 186 | ui::Size FramebufferSurface::limitSizeInternal(const ui::Size& size, const ui::Size& maxSize) { | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 187 | ui::Size limitedSize = size; | 
|  | 188 | bool wasLimited = false; | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 189 | if (size.width > maxSize.width && maxSize.width != 0) { | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 190 | const float aspectRatio = static_cast<float>(size.width) / size.height; | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 191 | limitedSize.height = maxSize.width / aspectRatio; | 
|  | 192 | limitedSize.width = maxSize.width; | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 193 | wasLimited = true; | 
|  | 194 | } | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 195 | if (limitedSize.height > maxSize.height && maxSize.height != 0) { | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 196 | const float aspectRatio = static_cast<float>(size.width) / size.height; | 
| Marin Shalamanov | 4a92f55 | 2021-03-29 18:38:27 +0200 | [diff] [blame] | 197 | limitedSize.height = maxSize.height; | 
|  | 198 | limitedSize.width = maxSize.height * aspectRatio; | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 199 | wasLimited = true; | 
|  | 200 | } | 
|  | 201 | ALOGI_IF(wasLimited, "framebuffer size has been limited to [%dx%d] from [%dx%d]", | 
| Marin Shalamanov | 045b700 | 2021-01-07 16:56:24 +0100 | [diff] [blame] | 202 | limitedSize.width, limitedSize.height, size.width, size.height); | 
|  | 203 | return limitedSize; | 
| Brian Lindahl | a13f2d5 | 2020-03-05 11:54:17 +0100 | [diff] [blame] | 204 | } | 
|  | 205 |  | 
| Dan Stoza | f10c46e | 2014-11-11 10:32:31 -0800 | [diff] [blame] | 206 | void FramebufferSurface::dumpAsString(String8& result) const { | 
| Courtney Goeltzenleuchter | 19987a8 | 2017-07-14 12:16:56 -0600 | [diff] [blame] | 207 | Mutex::Autolock lock(mMutex); | 
| Chia-I Wu | 1e04361 | 2018-03-01 09:45:09 -0800 | [diff] [blame] | 208 | result.appendFormat("  FramebufferSurface: dataspace: %s(%d)\n", | 
| Peiyong Lin | 34beb7a | 2018-03-28 11:57:12 -0700 | [diff] [blame] | 209 | dataspaceDetails(static_cast<android_dataspace>(mDataSpace)).c_str(), | 
|  | 210 | mDataSpace); | 
| Chia-I Wu | 1e04361 | 2018-03-01 09:45:09 -0800 | [diff] [blame] | 211 | ConsumerBase::dumpLocked(result, "   "); | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 212 | } | 
|  | 213 |  | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 214 | void FramebufferSurface::dumpLocked(String8& result, const char* prefix) const | 
| Jesse Hall | 7adb0f8 | 2013-03-06 16:13:49 -0800 | [diff] [blame] | 215 | { | 
| Mathias Agopian | 74d211a | 2013-04-22 16:55:35 +0200 | [diff] [blame] | 216 | ConsumerBase::dumpLocked(result, prefix); | 
| Jesse Hall | 7adb0f8 | 2013-03-06 16:13:49 -0800 | [diff] [blame] | 217 | } | 
|  | 218 |  | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 219 | const sp<Fence>& FramebufferSurface::getClientTargetAcquireFence() const { | 
|  | 220 | return mCurrentFence; | 
|  | 221 | } | 
| Dan Stoza | 9e56aa0 | 2015-11-02 13:00:03 -0800 | [diff] [blame] | 222 |  | 
| Mathias Agopian | 3e87601 | 2012-06-07 17:52:54 -0700 | [diff] [blame] | 223 | // ---------------------------------------------------------------------------- | 
|  | 224 | }; // namespace android | 
|  | 225 | // ---------------------------------------------------------------------------- | 
| Ady Abraham | b0dbdaa | 2020-01-06 16:19:42 -0800 | [diff] [blame] | 226 |  | 
|  | 227 | // TODO(b/129481165): remove the #pragma below and fix conversion issues | 
|  | 228 | #pragma clang diagnostic pop // ignored "-Wconversion" |