Commit 137e839b authored Apr 22, 2024 by Nolan Scobie

Remove bail on VK_PHYSICAL_DEVICE_TYPE_CPU from VulkanInterface::init

This was added as a temporary measure to prevent one type of emulator
where RE-GaneshVk lead to a black screen on boot from being affected by
flag-based rollouts of RE's Vulkan backends. That emulator now
explicitly sets debug.renderengine.backend=skiaglthreaded, so removing
this check will allow tests to be run against RE's VK backends on other
emulators.

Test: manual + librenderengine_test
Flag: EXEMPT actually gated by sysprop and 2 flags
Bug: b/327230380
Change-Id: I2bb31b0ff76344082c6f5b6396a52b093edb99a1

parent d10195fc

libs/renderengine/skia/VulkanInterface.cpp

+0 −5

Original line number	Diff line number	Diff line
		@@ -327,11 +327,6 @@ void VulkanInterface::init(bool protectedContent) {
		BAIL("Could not find a Vulkan 1.1+ physical device");
		}

		if (physDevProps.properties.deviceType == VK_PHYSICAL_DEVICE_TYPE_CPU) {
		// TODO: b/326633110 - SkiaVK is not working correctly on swiftshader path.
		BAIL("CPU implementations of Vulkan is not supported");
		}

		// Check for syncfd support. Bail if we cannot both import and export them.
		VkPhysicalDeviceExternalSemaphoreInfo semInfo = {
		VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_EXTERNAL_SEMAPHORE_INFO,