X-Git-Url: http://git.mutantstargoat.com?a=blobdiff_plain;f=src%2Fvulkan%2Fvkutil.cc;h=35a85dd0e3a378d52ae8cdf43a910bcecc0e54f6;hb=22347bb45e427ea763295d77da0de0d2ef9c2395;hp=698e6f8d17e2bfa2f964ca0e320cc3679506198f;hpb=72995482b98ff2a014ddd737131a0935ead89977;p=demo

diff --git a/src/vulkan/vkutil.cc b/src/vulkan/vkutil.cc
index 698e6f8..35a85dd 100644
--- a/src/vulkan/vkutil.cc
+++ b/src/vulkan/vkutil.cc
@@ -1,3 +1,4 @@
+#include <gmath/gmath.h>
 #include <vulkan/vulkan.h>
 #include <stdio.h>
 #include <stdint.h>
@@ -6,27 +7,31 @@
 #include <string>
 #include <vector>
 
+#include "allocator.h"
 #include "vkutil.h"
 
 /* global variables */
-
-VkPipeline *vkgraphics_pipeline;
+VkPhysicalDevice vk_physical;
+VkDevice vk_device;
+VkCommandPool vk_pool;
+VkQueue vk_queue;
+VkSurfaceKHR vk_surface;
+VkSwapchainKHR vk_swapchain;
+
+VkPipeline *vkgparent_pipeline;
 VkFramebuffer *vkfbufs;
 VkRenderPass vkrpass;
-VkSwapchainKHR vkswapchain;
-VkImage *vkswapchain_images;
-VkImageView *vkswapchain_views;
-int vknum_swapchain_images;
-int vknext_swapchain_image;
-VkSurfaceKHR vksurface;
 VkInstance vkinst;
-VkPhysicalDevice vkpdev;
-VkDevice vkdev;
-VkQueue vkq;
-VkCommandPool vkcmdpool;
 VkCommandBuffer vkcmdbuf;	/* primary command buffer */
 int vkqfamily;
 
+VkSemaphore vk_img_avail_sema;
+VkSemaphore vk_rend_done_sema;
+VkImage *vkswapchain_images;
+VkImageView *vkswapchain_views;
+int vknum_swapchain_images;
+int vk_curr_swapchain_image;
+
 /* static functions */
 static const char *get_device_name_str(VkPhysicalDeviceType type);
 static const char *get_memtype_flags_str(VkMemoryPropertyFlags flags);
@@ -196,7 +201,7 @@ bool vku_create_device()
 
 	if(sel_dev < 0 || sel_qfamily < 0) {
 		fprintf(stderr, "failed to find any device with a graphics-capable command queue\n");
-		vkDestroyDevice(vkdev, 0);
+		vkDestroyDevice(vk_device, 0);
 		return false;
 	}
 
@@ -222,16 +227,16 @@ bool vku_create_device()
 	dev_info.enabledExtensionCount = sizeof devext_names / sizeof *devext_names;
 	dev_info.ppEnabledExtensionNames = devext_names;
 
-	if(vkCreateDevice(phys_devices[sel_dev], &dev_info, 0, &vkdev) != 0) {
+	if(vkCreateDevice(phys_devices[sel_dev], &dev_info, 0, &vk_device) != 0) {
 		fprintf(stderr, "failed to create device %d\n", sel_dev);
 		return false;
 	}
 	printf("created device %d\n", sel_dev);
 
-	vkpdev = phys_devices[sel_dev];
+	vk_physical = phys_devices[sel_dev];
 	vkqfamily = sel_qfamily;
 
-	vkGetDeviceQueue(vkdev, sel_qfamily, 0, &vkq);
+	vkGetDeviceQueue(vk_device, sel_qfamily, 0, &vk_queue);
 
 	/* create command buffer pool */
 	memset(&cmdpool_info, 0, sizeof cmdpool_info);
@@ -239,15 +244,15 @@ bool vku_create_device()
 	cmdpool_info.flags = VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT;
 	cmdpool_info.queueFamilyIndex = sel_qfamily;
 
-	if(vkCreateCommandPool(vkdev, &cmdpool_info, 0, &vkcmdpool) != 0) {
-		fprintf(stderr, "failed to get command quque!\n");
+	if(vkCreateCommandPool(vk_device, &cmdpool_info, 0, &vk_pool) != 0) {
+		fprintf(stderr, "failed to get command queue!\n");
 		return false;
 	}
 
-	if(!(vkcmdbuf = vku_alloc_cmdbuf(vkcmdpool, VK_COMMAND_BUFFER_LEVEL_PRIMARY))) {
-		fprintf(stderr, "failed to create primary command buffer\n");
-		return false;
-	}
+	/*	if(!(vkcmdbuf = vku_alloc_cmdbuf(vk_pool, VK_COMMAND_BUFFER_LEVEL_PRIMARY))) {
+			fprintf(stderr, "failed to create primary command buffer\n");
+			return false;
+		} */
 
 	return true;
 }
@@ -255,9 +260,13 @@ bool vku_create_device()
 void vku_cleanup()
 {
 	if(vkinst) {
-		vkDeviceWaitIdle(vkdev);
-		vkDestroyCommandPool(vkdev, vkcmdpool, 0);
-		vkDestroyDevice(vkdev, 0);
+		vkDeviceWaitIdle(vk_device);
+		vkDestroyCommandPool(vk_device, vk_pool, 0);
+
+		vkDestroySemaphore(vk_device, vk_img_avail_sema, 0);
+		vkDestroySemaphore(vk_device, vk_rend_done_sema, 0);
+
+		vkDestroyDevice(vk_device, 0);
 		vkDestroyInstance(vkinst, 0);
 		vkinst = 0;
 	}
@@ -266,6 +275,25 @@ void vku_cleanup()
 	phys_devices = 0;
 }
 
+bool vku_create_semaphores()
+{
+	VkSemaphoreCreateInfo sinf;
+	memset(&sinf, 0, sizeof sinf);
+
+	sinf.sType = VK_STRUCTURE_TYPE_SEMAPHORE_CREATE_INFO;
+	if(vkCreateSemaphore(vk_device, &sinf, 0, &vk_img_avail_sema) != VK_SUCCESS) {
+		fprintf(stderr, "Failed to create semaphore\n");
+		return false;
+	}
+
+	if(vkCreateSemaphore(vk_device, &sinf, 0, &vk_rend_done_sema) != VK_SUCCESS) {
+		fprintf(stderr, "Failed to create semaphore\n");
+		return false;
+	}
+
+	return true;
+}
+
 VkCommandBuffer vku_alloc_cmdbuf(VkCommandPool pool, VkCommandBufferLevel level)
 {
 	VkCommandBuffer cmdbuf;
@@ -277,15 +305,33 @@ VkCommandBuffer vku_alloc_cmdbuf(VkCommandPool pool, VkCommandBufferLevel level)
 	inf.level = level;
 	inf.commandBufferCount = 1;
 
-	if(vkAllocateCommandBuffers(vkdev, &inf, &cmdbuf) != 0) {
+	if(vkAllocateCommandBuffers(vk_device, &inf, &cmdbuf) != 0) {
 		return 0;
 	}
 	return cmdbuf;
 }
 
+bool vku_alloc_cmdbufs(VkCommandPool pool, VkCommandBufferLevel level, unsigned int count, VkCommandBuffer *buf_array)
+{
+	VkCommandBufferAllocateInfo cinf;
+	memset(&cinf, 0, sizeof cinf);
+
+	cinf.sType = VK_STRUCTURE_TYPE_COMMAND_BUFFER_ALLOCATE_INFO;
+	cinf.commandPool = vk_pool;
+	cinf.level = level;
+	cinf.commandBufferCount = count;
+
+	if(vkAllocateCommandBuffers(vk_device, &cinf, buf_array) != VK_SUCCESS) {
+		fprintf(stderr, "Failed to allocate command buffer\n");
+		return false;
+	}
+
+	return true;
+}
+
 void vku_free_cmdbuf(VkCommandPool pool, VkCommandBuffer buf)
 {
-	vkFreeCommandBuffers(vkdev, pool, 1, &buf);
+	vkFreeCommandBuffers(vk_device, pool, 1, &buf);
 }
 
 void vku_begin_cmdbuf(VkCommandBuffer buf, unsigned int flags)
@@ -322,83 +368,44 @@ void vku_submit_cmdbuf(VkQueue q, VkCommandBuffer buf, VkFence done_fence)
 	vkQueueSubmit(q, 1, &info, done_fence);
 }
 
-VkSwapchainKHR vku_create_swapchain(VkSurfaceKHR surf, int xsz, int ysz, int n,
-                                    VkPresentModeKHR pmode, VkSwapchainKHR prev)
+bool vku_get_surface_format(VkPhysicalDevice gpu, VkSurfaceKHR surface, VkSurfaceFormatKHR *sformat)
 {
-	VkSwapchainKHR sc;
-	VkSwapchainCreateInfoKHR inf;
+	uint32_t fcount;
 
-	memset(&inf, 0, sizeof inf);
-	inf.sType = VK_STRUCTURE_TYPE_SWAPCHAIN_CREATE_INFO_KHR;
-	inf.surface = surf;
-	inf.minImageCount = n;
-	inf.imageFormat = VK_FORMAT_B8G8R8A8_UNORM;	/* TODO enumerate and choose */
-	inf.imageColorSpace = VK_COLORSPACE_SRGB_NONLINEAR_KHR;
-	inf.imageExtent.width = xsz;
-	inf.imageExtent.height = ysz;
-	inf.imageArrayLayers = 1;
-	inf.imageUsage = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
-	inf.imageSharingMode = VK_SHARING_MODE_EXCLUSIVE;	/* XXX make this an option? */
-	inf.preTransform = VK_SURFACE_TRANSFORM_INHERIT_BIT_KHR;
-	inf.compositeAlpha = VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR;
-	inf.presentMode = pmode;
-	inf.oldSwapchain = prev;
-
-	if(vkCreateSwapchainKHR(vkdev, &inf, 0, &sc) != 0) {
-		return 0;
+	if(vkGetPhysicalDeviceSurfaceFormatsKHR(vk_physical,
+	                                        vk_surface, &fcount, 0) != VK_SUCCESS) {
+		fprintf(stderr, "Failed to get format count for physical device.\n");
+		return false;
+	}
+	if(fcount == 0) {
+		fprintf(stderr, "No color formats were found.\n");
+		return false;
 	}
-	return sc;
-}
-
-VkImage *vku_get_swapchain_images(VkSwapchainKHR sc, int *count)
-{
-	uint32_t nimg;
-	VkImage *images;
 
-	if(vkGetSwapchainImagesKHR(vkdev, sc, &nimg, 0) != 0) {
-		return 0;
+	VkSurfaceFormatKHR *formats = new VkSurfaceFormatKHR[fcount];
+	if(vkGetPhysicalDeviceSurfaceFormatsKHR(vk_physical, vk_surface,
+	                                        &fcount, formats) != VK_SUCCESS) {
+		delete [] formats;
+		fprintf(stderr, "Failed to get surface formats.\n");
+		return false;
 	}
-	images = new VkImage[nimg];
-	vkGetSwapchainImagesKHR(vkdev, sc, &nimg, images);
 
-	if(count) *count = (int)nimg;
-	return images;
-}
+	*sformat = formats[0];
 
-VkImageView *vku_create_image_views(VkImage *images, int count)
-{
-	VkImageView *iviews;
-
-	iviews = new VkImageView[count];
-	for(int i=0; i<count; i++) {
-		VkImageViewCreateInfo inf;
-		memset(&inf, 0, sizeof inf);
-
-		inf.sType = VK_STRUCTURE_TYPE_IMAGE_VIEW_CREATE_INFO;
-		inf.image = images[i];
-		inf.viewType = VK_IMAGE_VIEW_TYPE_2D;
-		inf.format = VK_FORMAT_B8G8R8A8_UNORM; //TODO
-		inf.components.r = inf.components.g = inf.components.b =
-		        inf.components.a = VK_COMPONENT_SWIZZLE_IDENTITY;
-		inf.subresourceRange.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT;
-		inf.subresourceRange.levelCount = 1;
-		inf.subresourceRange.layerCount = 1;
-
-		if(vkCreateImageView(vkdev, &inf, 0, iviews) != 0) {
-			fprintf(stderr, "Failed to create image views.\n");
-			delete iviews;
-			return 0;
-		}
+	if((fcount == 1) && (formats[0].format == VK_FORMAT_UNDEFINED)) {
+		sformat->format = VK_FORMAT_B8G8R8_UNORM;
 	}
 
-	return iviews;
+	delete [] formats;
+	return true;
 }
 
+
 int vku_get_next_image(VkSwapchainKHR sc)
 {
 	uint32_t next;
 
-	if(vkAcquireNextImageKHR(vkdev, sc, UINT64_MAX, 0, 0, &next) != 0) {
+	if(vkAcquireNextImageKHR(vk_device, sc, UINT64_MAX, 0, 0, &next) != 0) {
 		return -1;
 	}
 	return (int)next;
@@ -417,90 +424,7 @@ void vku_present(VkSwapchainKHR sc, int img_idx)
 	inf.pImageIndices = &index;
 	inf.pResults = &res;
 
-	vkQueuePresentKHR(vkq, &inf);
-}
-
-bool vku_create_renderpass()
-{
-	VkAttachmentDescription attachments[2];
-	memset(&attachments, 0, 2 * sizeof *attachments);
-
-	/* color */
-	attachments[0].format = VK_FORMAT_B8G8R8A8_UNORM;
-	attachments[0].samples = VK_SAMPLE_COUNT_1_BIT;
-	attachments[0].loadOp = VK_ATTACHMENT_LOAD_OP_CLEAR;
-	attachments[0].storeOp = VK_ATTACHMENT_STORE_OP_STORE;
-	attachments[0].stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
-	attachments[0].stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
-	attachments[0].initialLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
-	attachments[0].finalLayout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
-
-	/* depth */
-	attachments[1].format = VK_FORMAT_D32_SFLOAT_S8_UINT; //TODO
-	attachments[1].samples = VK_SAMPLE_COUNT_1_BIT;
-	attachments[1].loadOp = VK_ATTACHMENT_LOAD_OP_CLEAR;
-	attachments[1].storeOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
-	attachments[1].stencilLoadOp = VK_ATTACHMENT_LOAD_OP_DONT_CARE;
-	attachments[1].stencilStoreOp = VK_ATTACHMENT_STORE_OP_DONT_CARE;
-	attachments[1].initialLayout = VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL;
-	attachments[1].finalLayout = VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL;
-
-	VkAttachmentReference color_ref;
-	color_ref.attachment = 0;
-	color_ref.layout = VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL;
-
-	VkAttachmentReference depth_ref;
-	depth_ref.attachment = 1;
-	depth_ref.layout = VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL;
-
-	VkSubpassDescription subpass_desc;
-	memset(&subpass_desc, 0, sizeof subpass_desc);
-
-	subpass_desc.pipelineBindPoint = VK_PIPELINE_BIND_POINT_GRAPHICS;
-	subpass_desc.colorAttachmentCount = 1;
-	subpass_desc.pColorAttachments = &color_ref;
-	subpass_desc.pDepthStencilAttachment = &depth_ref;
-
-	VkRenderPassCreateInfo inf;
-	memset(&inf, 0, sizeof inf);
-
-	inf.sType = VK_STRUCTURE_TYPE_RENDER_PASS_CREATE_INFO;
-	inf.attachmentCount = 2;
-	inf.pAttachments = attachments;
-	inf.subpassCount = 1;
-	inf.pSubpasses = &subpass_desc;
-
-	if(vkCreateRenderPass(vkdev, &inf, 0, &vkrpass) != VK_SUCCESS) {
-		return false;
-	}
-
-	return true;
-}
-
-bool vku_create_framebuffers(VkImageView *image_views, int count, int w, int h)
-{
-	delete vkfbufs;
-	vkfbufs = new VkFramebuffer[count];
-
-	VkFramebufferCreateInfo inf;
-	memset(&inf, 0, sizeof inf);
-
-	inf.sType = VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO;
-	inf.renderPass = vkrpass;
-	inf.attachmentCount = count;
-	inf.pAttachments = image_views;
-	inf.width = w;
-	inf.height = h;
-	inf.layers = 1;
-
-	for(int i=0; i<count; i++) {
-		if(vkCreateFramebuffer(vkdev, &inf, 0, &vkfbufs[i]) != VK_SUCCESS) {
-			fprintf(stderr, "Failed to create framebuffer for image view: %d\n", i);
-			delete vkfbufs;
-			return false;
-		}
-	}
-	return true;
+	vkQueuePresentKHR(vk_queue, &inf);
 }
 
 struct vku_buffer *vku_create_buffer(int sz, unsigned int usage)
@@ -516,18 +440,30 @@ struct vku_buffer *vku_create_buffer(int sz, unsigned int usage)
 	binfo.usage = usage;
 	binfo.sharingMode = VK_SHARING_MODE_EXCLUSIVE;
 
-	if(vkCreateBuffer(vkdev, &binfo, 0, &buf->buf) != 0) {
+	if(vkCreateBuffer(vk_device, &binfo, 0, &buf->buf) != 0) {
 		fprintf(stderr, "failed to create %d byte buffer (usage: %x)\n", sz, usage);
 		return 0;
 	}
-	// TODO back with memory
+
+	VkMemoryRequirements mr;
+	vkGetBufferMemoryRequirements(vk_device, buf->buf, &mr);
+
+	DevMemBlock block;
+	if(!vku_allocate(mr.size, &block))
+		return 0;
+
+	buf->mem_pool = block.dev_mem;
+
 	return buf;
 }
 
 void vku_destroy_buffer(struct vku_buffer *buf)
 {
 	if(buf) {
-		vkDestroyBuffer(vkdev, buf->buf, 0);
+		//TODO change when the allocator changes
+		vku_free(buf->mem_pool);
+
+		vkDestroyBuffer(vk_device, buf->buf, 0);
 		delete buf;
 	}
 }
@@ -543,257 +479,6 @@ void vku_cmd_copybuf(VkCommandBuffer cmdbuf, VkBuffer dest, int doffs,
 	vkCmdCopyBuffer(cmdbuf, src, dest, 1, &copy);
 }
 
-/* paste
-
-static bool create_instance()
-{
-	uint32_t layer_count = 0;
-	std::vector<const char *> enabled_layers;
-
-	if(vkEnumerateInstanceLayerProperties(&layer_count, 0) != VK_SUCCESS) {
-		fprintf(stderr, "Failed to query layer properties.\n");
-		return false;
-	}
-
-	if(layer_count > 0) {
-		VkLayerProperties *layers = (VkLayerProperties *)alloca(layer_count * sizeof *layers);
-		vkEnumerateInstanceLayerProperties(&layer_count, layers);
-		for(uint32_t i=0; i<layer_count; i++) {
-			if(strcmp(layers[i].layerName, "VK_LAYER_LUNARG_standard_validation")) {
-				enabled_layers.push_back(layers[i].layerName);
-			}
-		}
-	}
-
-	uint32_t extensions_count = 0;
-	std::vector<const char *> enabled_extensions;
-
-	if(vkEnumerateInstanceExtensionProperties(0, &extensions_count, 0) != VK_SUCCESS) {
-		fprintf(stderr, "Failed to enumerate instance extension properties\n");
-		return false;
-	}
-
-	if(extensions_count > 0) {
-		VkExtensionProperties *extensions = (VkExtensionProperties *)alloca(extensions_count * sizeof *extensions);
-		vkEnumerateInstanceExtensionProperties(0, &extensions_count, extensions);
-
-		for(uint32_t i=0; i<extensions_count; i++) {
-			printf("Extension %u: %s %u.\n", i, extensions[i].extensionName, extensions[i].specVersion);
-			//enable all the available extensions
-			enabled_extensions.push_back(extensions[i].extensionName);
-			enabled_extension_names.push_back(std::string(extensions[i].extensionName));
-		}
-	}
-
-	VkInstanceCreateInfo create_info;
-	memset(&create_info, 0, sizeof create_info);
-	create_info.sType = VK_STRUCTURE_TYPE_INSTANCE_CREATE_INFO;
-
-	if(!enabled_layers.empty()) {
-		create_info.enabledLayerCount = enabled_layers.size();
-		create_info.ppEnabledLayerNames = enabled_layers.data();
-	}
-
-	if(!enabled_extensions.empty()) {
-		create_info.enabledExtensionCount = enabled_extensions.size();
-		create_info.ppEnabledExtensionNames = enabled_extensions.data();
-	}
-
-	if(vkCreateInstance(&create_info, 0, &inst) != VK_SUCCESS) {
-		fprintf(stderr, "Failed to create instance.\n");
-		return false;
-	}
-
-	if(!create_device())
-		return false;
-
-	return true;
-}
-
-static bool create_device()
-{
-	int qfam_idx = -1;
-	int pdev_idx = -1;
-
-	uint32_t dev_count;
-	if(vkEnumeratePhysicalDevices(inst, &dev_count, 0) != VK_SUCCESS) {
-		fprintf(stderr, "Failed to enumerate physical devices.\n");
-		return false;
-	}
-	printf("%u devices found.\n", (unsigned int)dev_count);
-
-	VkPhysicalDevice *phys_dev = (VkPhysicalDevice *)alloca(dev_count * sizeof *phys_dev);
-	vkEnumeratePhysicalDevices(inst, &dev_count, phys_dev);
-	VkPhysicalDeviceMemoryProperties memprop;
-
-	for(uint32_t i=0; i<dev_count; i++) {
-		VkPhysicalDeviceProperties dev_props;
-		vkGetPhysicalDeviceProperties(phys_dev[i], &dev_props);
-
-		//memory heaps:
-		vkGetPhysicalDeviceMemoryProperties(phys_dev[i], &memprop);
-		printf("\tNumber of heaps: %u\n", memprop.memoryHeapCount);
-		for(uint32_t j=0; j<memprop.memoryHeapCount; j++) {
-			printf("\t\tHeap %u size: %lu\n", j, (unsigned long)memprop.memoryHeaps[j].size);
-			printf("\t\tHeap %u flags: %s\n", j, heap_flags_str(memprop.memoryHeaps[j].flags));
-		}
-		//memory types
-		printf("\tMemory types: %u\n", memprop.memoryTypeCount);
-		for(uint32_t j=0; j<memprop.memoryTypeCount; j++) {
-			printf("\t\tType %u heap index: %u\n", j, memprop.memoryTypes[j].heapIndex);
-			printf("\t\tType %u flags: %s\n", j, memtype_flags_str(memprop.memoryTypes[j].propertyFlags));
-		}
-
-		//supported features
-		VkPhysicalDeviceFeatures features;
-		vkGetPhysicalDeviceFeatures(phys_dev[i], &features);
-
-		//queue families
-		uint32_t qfam_count;
-		vkGetPhysicalDeviceQueueFamilyProperties(phys_dev[i], &qfam_count, 0);
-		printf("\tQueue Families: %u\n", qfam_count);
-		VkQueueFamilyProperties *qfam_props = new VkQueueFamilyProperties[qfam_count];
-		vkGetPhysicalDeviceQueueFamilyProperties(phys_dev[i], &qfam_count, qfam_props);
-		for(uint32_t j=0; j<qfam_count; j++) {
-			printf("\t\tFamily %u flags: %s\n", j, queue_flags_str(qfam_props[j].queueFlags));
-			printf("\t\tFamily %u number of queues: %u\n", j, qfam_props[j].queueCount);
-
-			if((qfam_props[j].queueFlags & VK_QUEUE_GRAPHICS_BIT) && (pdev_idx == -1)) {
-				pdev_idx = i;
-				qfam_idx = j;
-				num_queues = qfam_props[j].queueCount;
-			}
-		}
-		delete [] qfam_props;
-	}
-
-	if(pdev_idx == -1) {
-		fprintf(stderr, "No suitable devices found.\n");
-		return false;
-	}
-
-	pdev = *(phys_dev + pdev_idx);
-	qfamily_idx = qfam_idx;
-
-		uint32_t layer_count;
-		if(vkEnumerateDeviceLayerProperties(pdev, &layer_count, 0) != VK_SUCCESS) {
-			fprintf(stderr, "Failed to enumerate device layers.\n");
-			return false;
-		}
-		if(layer_count > 0) {
-			VkLayerProperties *layers = (VkLayerProperties*)alloca(layer_count * sizeof *layers);
-			vkEnumerateDeviceLayerProperties(pdev, &layer_count, layers);
-			printf("%u layers found.\n", layer_count);
-			for(uint32_t i=0; i<layer_count; i++) {
-				printf("Layer %u: %s (%u, %u)\n", i, layers[i].layerName,
-						layers[i].specVersion, layers[i].implementationVersion);
-				printf("\tDesc: %s\n", layers[i].description);
-			}
-		}
-
-	VkDeviceCreateInfo dev_info;
-	memset(&dev_info, 0, sizeof dev_info);
-	dev_info.sType = VK_STRUCTURE_TYPE_DEVICE_CREATE_INFO;
-
-	VkDeviceQueueCreateInfo dev_qinfo;
-	memset(&dev_qinfo, 0, sizeof dev_qinfo);
-	dev_qinfo.sType = VK_STRUCTURE_TYPE_DEVICE_QUEUE_CREATE_INFO;
-	dev_qinfo.queueFamilyIndex = qfam_idx;
-	dev_qinfo.queueCount = 1;
-
-	dev_info.queueCreateInfoCount = 1;
-	dev_info.pQueueCreateInfos = &dev_qinfo;
-
-	if(vkCreateDevice(pdev, &dev_info, 0, &device) != VK_SUCCESS) {
-		fprintf(stderr, "Failed to create logical device.\n");
-		return false;
-	}
-
-	vkGetPhysicalDeviceMemoryProperties(pdev, &memprop);
-	for(uint32_t j=0; j<memprop.memoryTypeCount; j++) {
-		if(memprop.memoryTypes[j].propertyFlags & VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT) {
-			device_mem_idx = j;
-			printf("Selected device memory index: %u\n", device_mem_idx);
-			break;
-		}
-	}
-
-	return true;
-}
-
-static const char *dev_type_str(VkPhysicalDeviceType type)
-{
-	switch(type) {
-	case VK_PHYSICAL_DEVICE_TYPE_OTHER:
-		return "other";
-	case VK_PHYSICAL_DEVICE_TYPE_INTEGRATED_GPU:
-		return "integrated GPU";
-	case VK_PHYSICAL_DEVICE_TYPE_DISCRETE_GPU:
-		return "discrete GPU";
-	case VK_PHYSICAL_DEVICE_TYPE_VIRTUAL_GPU:
-		return "virtual GPU";
-	case VK_PHYSICAL_DEVICE_TYPE_CPU:
-		return "CPU";
-	default:
-		break;
-	}
-	return "unknown";
-}
-
-static const char *heap_flags_str(VkMemoryHeapFlags flags)
-{
-	static std::string str;
-	str.clear();
-	if(flags & VK_MEMORY_HEAP_DEVICE_LOCAL_BIT) {
-		str += "device-local ";
-	}
-	if(!str.empty())
-		str.pop_back();
-	return str.c_str();
-}
-
-static const char *memtype_flags_str(VkMemoryPropertyFlags flags)
-{
-	static std::string str;
-	str.clear();
-	if(flags & VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT) {
-		str += "device-local ";
-	}
-	if(flags & VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT) {
-		str += "host-visible ";
-	}
-	if(flags & VK_MEMORY_PROPERTY_HOST_COHERENT_BIT) {
-		str += "host-coherent ";
-	}
-	if(flags & VK_MEMORY_PROPERTY_HOST_CACHED_BIT) {
-		str += "host-cached ";
-	}
-	if(flags & VK_MEMORY_PROPERTY_LAZILY_ALLOCATED_BIT) {
-		str += "lazily-allocated ";
-	}
-	if(!str.empty())
-		str.pop_back();
-	return str.c_str();
-}
-
-static const char *queue_flags_str(VkQueueFlags flags)
-{
-	static std::string str;
-	str.clear();
-	if(flags & VK_QUEUE_GRAPHICS_BIT)
-		str += "graphics ";
-	if(flags & VK_QUEUE_COMPUTE_BIT)
-		str += "compute ";
-	if(flags & VK_QUEUE_TRANSFER_BIT)
-		str += "transfer ";
-	if(flags & VK_QUEUE_SPARSE_BINDING_BIT)
-		str += "sparse-binding ";
-	if(!str.empty())
-		str.pop_back();
-	return str.c_str();
-}
-*/
-
 const char *vku_get_vulkan_error_str(VkResult error)
 {
 	std::string errmsg;
@@ -839,7 +524,6 @@ const char *vku_get_vulkan_error_str(VkResult error)
 		break;
 	case VK_ERROR_FEATURE_NOT_PRESENT:
 		errmsg = std::string("VK_ERROR_FEATURE_NOT_PRESENT");
-		break;
 	case VK_ERROR_INCOMPATIBLE_DRIVER:
 		errmsg = std::string("VK_ERROR_INCOMPATIBLE_DRIVER");
 		break;
@@ -860,79 +544,6 @@ const char *vku_get_vulkan_error_str(VkResult error)
 	return errmsg.c_str();
 }
 
-bool vku_create_graphics_pipeline(VkPipelineLayout *layout)
-{
-	VkGraphicsPipelineCreateInfo inf;
-	memset(&inf, 0, sizeof inf);
-
-	inf.sType = VK_STRUCTURE_TYPE_GRAPHICS_PIPELINE_CREATE_INFO;
-	inf.layout = *layout;
-	inf.renderPass = vkrpass;
-
-	/* states */
-
-	/* how primitives are assembled */
-	VkPipelineInputAssemblyStateCreateInfo ias;
-	memset(&ias, 0, sizeof ias);
-	ias.sType = VK_STRUCTURE_TYPE_PIPELINE_INPUT_ASSEMBLY_STATE_CREATE_INFO;
-	ias.topology = VK_PRIMITIVE_TOPOLOGY_TRIANGLE_LIST;
-
-	/* rasterisation */
-	VkPipelineRasterizationStateCreateInfo rsi;
-	memset(&rsi, 0, sizeof rsi);
-	rsi.sType = VK_STRUCTURE_TYPE_PIPELINE_RASTERIZATION_STATE_CREATE_INFO;
-	rsi.polygonMode = VK_POLYGON_MODE_FILL;
-	rsi.cullMode = VK_CULL_MODE_BACK_BIT;
-	rsi.frontFace = VK_FRONT_FACE_COUNTER_CLOCKWISE;
-	rsi.lineWidth = 1.0f;
-
-	/* blend factors */
-	VkPipelineColorBlendAttachmentState bas[1];
-	memset(&bas[0], 0, sizeof bas[0]);
-
-	VkPipelineColorBlendStateCreateInfo cbs;
-	memset(&cbs, 0, sizeof cbs);
-	cbs.sType = VK_STRUCTURE_TYPE_PIPELINE_COLOR_BLEND_STATE_CREATE_INFO;
-	cbs.attachmentCount = 1;
-	cbs.pAttachments = bas;
-
-	/* number of viewport and scissors in this pipeline */
-	VkPipelineViewportStateCreateInfo vs;
-	memset(&vs, 0, sizeof vs);
-	vs.sType = VK_STRUCTURE_TYPE_PIPELINE_VIEWPORT_STATE_CREATE_INFO;
-	vs.viewportCount = 1;
-	vs.scissorCount = 1;
-
-	/* dynamic states: that can be changed later */
-	std::vector<VkDynamicState> ds_enabled;
-	ds_enabled.push_back(VK_DYNAMIC_STATE_VIEWPORT);
-	//ds_enabled.push_back(VK_DYNAMIC_STATE_SCISSOR);
-	VkPipelineDynamicStateCreateInfo ds;
-	ds.sType = VK_STRUCTURE_TYPE_PIPELINE_DYNAMIC_STATE_CREATE_INFO;
-	ds.pDynamicStates = ds_enabled.data();
-	ds.dynamicStateCount = static_cast<uint32_t>(ds_enabled.size());
-
-	/* depth tests */
-	VkPipelineDepthStencilStateCreateInfo dsi;
-	memset(&dsi, 0, sizeof dsi);
-	dsi.sType = VK_STRUCTURE_TYPE_PIPELINE_DEPTH_STENCIL_STATE_CREATE_INFO;
-	dsi.depthTestEnable = VK_TRUE;
-	dsi.depthWriteEnable = VK_TRUE;
-	dsi.depthCompareOp = VK_COMPARE_OP_LESS_OR_EQUAL;
-	dsi.back.failOp = VK_STENCIL_OP_KEEP;
-	dsi.back.passOp = VK_STENCIL_OP_KEEP;
-	dsi.back.compareOp = VK_COMPARE_OP_ALWAYS;
-	dsi.front = dsi.back;
-
-	/* multisampling - must be set even if not used */
-	VkPipelineMultisampleStateCreateInfo msi;
-	memset(&msi, 0, sizeof msi);
-	msi.sType = VK_STRUCTURE_TYPE_PIPELINE_MULTISAMPLE_STATE_CREATE_INFO;
-	msi.rasterizationSamples = VK_SAMPLE_COUNT_1_BIT;
-
-	//TODO in progress
-	return true;
-}
 
 static const char *get_device_name_str(VkPhysicalDeviceType type)
 {