Commit ffc50c68 authored Jun 08, 2021 by Greg Daniel

Reset buffer tracking state in VulkanSurface.

When we fail to create an SkSurface from a dequeued
buffer we then call cancelBuffer on that buffer. However,
we don't update our tracking to say the buffer is no
longer dequeued with and doesn't own the fence any more.

I don't think this is the main issue causing the crash in
the attached bug because I don't think we're seeing the
error in the logs for this chunk of code. However this
possibly related issue was found while tracking down our
use of the dequeue_fence fd.

Test: manual building and running of phone
Bug: 187240173

Change-Id: Icb4099eeea5be6aedd5376e07a6e3454f5d1d1e3

parent 209f21d2

libs/hwui/renderthread/VulkanSurface.cpp

+3 −1

Original line number	Diff line number	Diff line
		@@ -429,7 +429,9 @@ VulkanSurface::NativeBufferInfo* VulkanSurface::dequeueNativeBuffer() {
		kTopLeft_GrSurfaceOrigin, mWindowInfo.colorspace, nullptr);
		if (bufferInfo->skSurface.get() == nullptr) {
		ALOGE("SkSurface::MakeFromAHardwareBuffer failed");
		mNativeWindow->cancelBuffer(mNativeWindow.get(), buffer, fence_fd.release());
		mNativeWindow->cancelBuffer(mNativeWindow.get(), buffer,
		mNativeBuffers[idx].dequeue_fence.release());
		mNativeBuffers[idx].dequeued = false;
		return nullptr;
		}
		}