X-Git-Url: https://git.sesse.net/?p=movit;a=blobdiff_plain;f=resource_pool.cpp;h=bf6d12c8982ada1d99ee5ee6289c6f5a4c31ea59;hp=944910bde3829320abcbdd859b861c3995b3da4f;hb=f680714c0378a80e1638824c70c5ec62d6ade8ee;hpb=825c90789c229f502520bf0b665596d473f2636d

diff --git a/resource_pool.cpp b/resource_pool.cpp
index 944910b..bf6d12c 100644
--- a/resource_pool.cpp
+++ b/resource_pool.cpp
@@ -18,10 +18,12 @@ namespace movit {
 
 ResourcePool::ResourcePool(size_t program_freelist_max_length,
                            size_t texture_freelist_max_bytes,
-                           size_t fbo_freelist_max_length)
+                           size_t fbo_freelist_max_length,
+                           size_t vao_freelist_max_length)
 	: program_freelist_max_length(program_freelist_max_length),
 	  texture_freelist_max_bytes(texture_freelist_max_bytes),
 	  fbo_freelist_max_length(fbo_freelist_max_length),
+	  vao_freelist_max_length(vao_freelist_max_length),
 	  texture_freelist_bytes(0)
 {
 	pthread_mutex_init(&lock, NULL);
@@ -89,14 +91,24 @@ void ResourcePool::delete_program(GLuint glsl_program_num)
 		}
 	}
 	assert(found_program);
-	glDeleteProgram(glsl_program_num);
 
-	map<GLuint, pair<GLuint, GLuint> >::iterator shader_it =
+	map<GLuint, stack<GLuint> >::iterator instance_list_it = program_instances.find(glsl_program_num);
+	assert(instance_list_it != program_instances.end());
+
+	while (!instance_list_it->second.empty()) {
+		GLuint instance_program_num = instance_list_it->second.top();
+		instance_list_it->second.pop();
+		glDeleteProgram(instance_program_num);
+		program_masters.erase(instance_program_num);
+	}
+	program_instances.erase(instance_list_it);
+
+	map<GLuint, ShaderSpec>::iterator shader_it =
 		program_shaders.find(glsl_program_num);
 	assert(shader_it != program_shaders.end());
 
-	glDeleteShader(shader_it->second.first);
-	glDeleteShader(shader_it->second.second);
+	glDeleteShader(shader_it->second.vs_obj);
+	glDeleteShader(shader_it->second.fs_obj);
 	program_shaders.erase(shader_it);
 }
 
@@ -133,36 +145,11 @@ GLuint ResourcePool::compile_glsl_program(const string& vertex_shader,
 		}
 	} else {
 		// Not in the cache. Compile the shaders.
-		glsl_program_num = glCreateProgram();
-		check_error();
 		GLuint vs_obj = compile_shader(vertex_shader, GL_VERTEX_SHADER);
 		check_error();
 		GLuint fs_obj = compile_shader(fragment_shader_processed, GL_FRAGMENT_SHADER);
 		check_error();
-		glAttachShader(glsl_program_num, vs_obj);
-		check_error();
-		glAttachShader(glsl_program_num, fs_obj);
-		check_error();
-
-		// Bind the outputs, if we have multiple ones.
-		if (fragment_shader_outputs.size() > 1) {
-			for (unsigned output_index = 0; output_index < fragment_shader_outputs.size(); ++output_index) {
-				glBindFragDataLocation(glsl_program_num, output_index,
-				                       fragment_shader_outputs[output_index].c_str());
-			}
-		}
-
-		glLinkProgram(glsl_program_num);
-		check_error();
-
-		GLint success;
-		glGetProgramiv(glsl_program_num, GL_LINK_STATUS, &success);
-		if (success == GL_FALSE) {
-			GLchar error_log[1024] = {0};
-			glGetProgramInfoLog(glsl_program_num, 1024, NULL, error_log);
-			fprintf(stderr, "Error linking program: %s\n", error_log);
-			exit(1);
-		}
+		glsl_program_num = link_program(vs_obj, fs_obj, fragment_shader_outputs);
 
 		if (movit_debug_level == MOVIT_DEBUG_ON) {
 			// Output shader to a temporary file, for easier debugging.
@@ -180,12 +167,54 @@ GLuint ResourcePool::compile_glsl_program(const string& vertex_shader,
 
 		programs.insert(make_pair(key, glsl_program_num));
 		program_refcount.insert(make_pair(glsl_program_num, 1));
-		program_shaders.insert(make_pair(glsl_program_num, make_pair(vs_obj, fs_obj)));
+		ShaderSpec spec;
+		spec.vs_obj = vs_obj;
+		spec.fs_obj = fs_obj;
+		spec.fragment_shader_outputs = fragment_shader_outputs;
+		program_shaders.insert(make_pair(glsl_program_num, spec));
+		stack<GLuint> instances;
+		instances.push(glsl_program_num);
+		program_instances.insert(make_pair(glsl_program_num, instances));
+		program_masters.insert(make_pair(glsl_program_num, glsl_program_num));
 	}
 	pthread_mutex_unlock(&lock);
 	return glsl_program_num;
 }
 
+GLuint ResourcePool::link_program(GLuint vs_obj,
+                                  GLuint fs_obj,
+                                  const vector<string>& fragment_shader_outputs)
+{
+	GLuint glsl_program_num = glCreateProgram();
+	check_error();
+	glAttachShader(glsl_program_num, vs_obj);
+	check_error();
+	glAttachShader(glsl_program_num, fs_obj);
+	check_error();
+
+	// Bind the outputs, if we have multiple ones.
+	if (fragment_shader_outputs.size() > 1) {
+		for (unsigned output_index = 0; output_index < fragment_shader_outputs.size(); ++output_index) {
+			glBindFragDataLocation(glsl_program_num, output_index,
+					       fragment_shader_outputs[output_index].c_str());
+		}
+	}
+
+	glLinkProgram(glsl_program_num);
+	check_error();
+
+	GLint success;
+	glGetProgramiv(glsl_program_num, GL_LINK_STATUS, &success);
+	if (success == GL_FALSE) {
+		GLchar error_log[1024] = {0};
+		glGetProgramInfoLog(glsl_program_num, 1024, NULL, error_log);
+		fprintf(stderr, "Error linking program: %s\n", error_log);
+		exit(1);
+	}
+
+	return glsl_program_num;
+}
+
 void ResourcePool::release_glsl_program(GLuint glsl_program_num)
 {
 	pthread_mutex_lock(&lock);
@@ -206,6 +235,51 @@ void ResourcePool::release_glsl_program(GLuint glsl_program_num)
 	pthread_mutex_unlock(&lock);
 }
 
+GLuint ResourcePool::use_glsl_program(GLuint glsl_program_num)
+{
+	pthread_mutex_lock(&lock);
+	assert(program_instances.count(glsl_program_num));
+	stack<GLuint> &instances = program_instances[glsl_program_num];
+
+	GLuint instance_program_num;
+	if (!instances.empty()) {
+		// There's an unused instance of this program; just return it.
+		instance_program_num = instances.top();
+		instances.pop();
+	} else {
+		// We need to clone this program. (unuse_glsl_program()
+		// will later put it onto the list.)
+		map<GLuint, ShaderSpec>::iterator shader_it =
+			program_shaders.find(glsl_program_num);
+		assert(shader_it != program_shaders.end());
+
+		instance_program_num = link_program(
+			shader_it->second.vs_obj,
+			shader_it->second.fs_obj,
+			shader_it->second.fragment_shader_outputs);
+		program_masters.insert(make_pair(instance_program_num, glsl_program_num));
+	}
+	pthread_mutex_unlock(&lock);
+
+	glUseProgram(instance_program_num);
+	return instance_program_num;
+}
+
+void ResourcePool::unuse_glsl_program(GLuint instance_program_num)
+{
+	pthread_mutex_lock(&lock);
+
+	map<GLuint, GLuint>::const_iterator master_it = program_masters.find(instance_program_num);
+	assert(master_it != program_masters.end());
+
+	assert(program_instances.count(master_it->second));
+	stack<GLuint> &instances = program_instances[master_it->second];
+
+	instances.push(instance_program_num);
+
+	pthread_mutex_unlock(&lock);
+}
+
 GLuint ResourcePool::create_2d_texture(GLint internal_format, GLsizei width, GLsizei height)
 {
 	assert(width > 0);
@@ -235,24 +309,32 @@ GLuint ResourcePool::create_2d_texture(GLint internal_format, GLsizei width, GLs
 	switch (internal_format) {
 	case GL_RGBA32F_ARB:
 	case GL_RGBA16F_ARB:
+	case GL_RGBA16:
 	case GL_RGBA8:
+	case GL_RGB10_A2:
 	case GL_SRGB8_ALPHA8:
 		format = GL_RGBA;
 		break;
 	case GL_RGB32F:
 	case GL_RGB16F:
+	case GL_RGB16:
+	case GL_R11F_G11F_B10F:
 	case GL_RGB8:
+	case GL_RGB10:
 	case GL_SRGB8:
 	case GL_RGB565:
+	case GL_RGB9_E5:
 		format = GL_RGB;
 		break;
 	case GL_RG32F:
 	case GL_RG16F:
+	case GL_RG16:
 	case GL_RG8:
 		format = GL_RG;
 		break;
 	case GL_R32F:
 	case GL_R16F:
+	case GL_R16:
 	case GL_R8:
 		format = GL_RED;
 		break;
@@ -268,16 +350,26 @@ GLuint ResourcePool::create_2d_texture(GLint internal_format, GLsizei width, GLs
 	case GL_RGBA16F_ARB:
 	case GL_RGB32F:
 	case GL_RGB16F:
+	case GL_R11F_G11F_B10F:
+	case GL_RGB9_E5:
 	case GL_RG32F:
 	case GL_RG16F:
 	case GL_R32F:
 	case GL_R16F:
 		type = GL_FLOAT;
 		break;
+	case GL_RGBA16:
+	case GL_RGB16:
+	case GL_RG16:
+	case GL_R16:
+		type = GL_UNSIGNED_SHORT;
+		break;
 	case GL_SRGB8_ALPHA8:
 	case GL_SRGB8:
 	case GL_RGBA8:
 	case GL_RGB8:
+	case GL_RGB10_A2:
+	case GL_RGB10:
 	case GL_RG8:
 	case GL_R8:
 		type = GL_UNSIGNED_BYTE;
@@ -437,14 +529,82 @@ void ResourcePool::release_fbo(GLuint fbo_num)
 	pthread_mutex_unlock(&lock);
 }
 
+GLuint ResourcePool::create_vec2_vao(const set<GLint> &attribute_indices, GLuint vbo_num)
+{
+	void *context = get_gl_context_identifier();
+
+	pthread_mutex_lock(&lock);
+	if (vao_freelist.count(context) != 0) {
+		// See if there's a VAO the freelist we can use.
+		list<VAOFormatIterator>::iterator end = vao_freelist[context].end();
+		for (list<VAOFormatIterator>::iterator freelist_it = vao_freelist[context].begin();
+		     freelist_it != end; ++freelist_it) {
+			VAOFormatIterator vao_it = *freelist_it;
+			if (vao_it->second.vbo_num == vbo_num &&
+			    vao_it->second.attribute_indices == attribute_indices) {
+				vao_freelist[context].erase(freelist_it);
+				pthread_mutex_unlock(&lock);
+				return vao_it->second.vao_num;
+			}
+		}
+	}
+
+	// Create a new one.
+	VAO vao_format;
+	vao_format.attribute_indices = attribute_indices;
+	vao_format.vbo_num = vbo_num;
+
+	glGenVertexArrays(1, &vao_format.vao_num);
+	check_error();
+	glBindVertexArray(vao_format.vao_num);
+	check_error();
+	glBindBuffer(GL_ARRAY_BUFFER, vbo_num);
+	check_error();
+
+	for (set<GLint>::const_iterator attr_it = attribute_indices.begin(); attr_it != attribute_indices.end(); ++attr_it) {
+		glEnableVertexAttribArray(*attr_it);
+		check_error();
+		glVertexAttribPointer(*attr_it, 2, GL_FLOAT, GL_FALSE, 0, BUFFER_OFFSET(0));
+		check_error();
+	}
+
+	glBindVertexArray(0);
+	check_error();
+	glBindBuffer(GL_ARRAY_BUFFER, 0);
+	check_error();
+
+	pair<void *, GLuint> key(context, vao_format.vao_num);
+	assert(vao_formats.count(key) == 0);
+	vao_formats.insert(make_pair(key, vao_format));
+
+	pthread_mutex_unlock(&lock);
+	return vao_format.vao_num;
+}
+
+void ResourcePool::release_vec2_vao(GLuint vao_num)
+{
+	void *context = get_gl_context_identifier();
+
+	pthread_mutex_lock(&lock);
+	VAOFormatIterator vao_it = vao_formats.find(make_pair(context, vao_num));
+	assert(vao_it != vao_formats.end());
+	vao_freelist[context].push_front(vao_it);
+
+	shrink_vao_freelist(context, vao_freelist_max_length);
+	pthread_mutex_unlock(&lock);
+}
+
 void ResourcePool::clean_context()
 {
 	void *context = get_gl_context_identifier();
 
-	// Currently, we only need to worry about FBOs, as they are the only
-	// non-shareable resource we hold.
+	// Currently, we only need to worry about FBOs and VAOs, as they are the only
+	// non-shareable resources we hold.
 	shrink_fbo_freelist(context, 0);
 	fbo_freelist.erase(context);
+
+	shrink_vao_freelist(context, 0);
+	vao_freelist.erase(context);
 }
 
 void ResourcePool::cleanup_unlinked_fbos(void *context)
@@ -484,6 +644,18 @@ void ResourcePool::shrink_fbo_freelist(void *context, size_t max_length)
 	}
 }
 
+void ResourcePool::shrink_vao_freelist(void *context, size_t max_length)
+{
+	list<VAOFormatIterator> &freelist = vao_freelist[context];
+	while (freelist.size() > max_length) {
+		VAOFormatIterator free_vao_it = freelist.back();
+		glDeleteVertexArrays(1, &free_vao_it->second.vao_num);
+		check_error();
+		vao_formats.erase(free_vao_it);
+		freelist.pop_back();
+	}
+}
+
 size_t ResourcePool::estimate_texture_size(const Texture2D &texture_format)
 {
 	size_t bytes_per_pixel;
@@ -501,8 +673,16 @@ size_t ResourcePool::estimate_texture_size(const Texture2D &texture_format)
 	case GL_RGB16F_ARB:
 		bytes_per_pixel = 6;
 		break;
+	case GL_R11F_G11F_B10F:
+		bytes_per_pixel = 4;
+		break;
+	case GL_RGB9_E5:
+		bytes_per_pixel = 4;
+		break;
 	case GL_RGBA8:
 	case GL_SRGB8_ALPHA8:
+	case GL_RGB10_A2:
+	case GL_RGB10:
 		bytes_per_pixel = 4;
 		break;
 	case GL_RGB8:
@@ -530,6 +710,18 @@ size_t ResourcePool::estimate_texture_size(const Texture2D &texture_format)
 	case GL_RGB565:
 		bytes_per_pixel = 2;
 		break;
+	case GL_RGBA16:
+		bytes_per_pixel = 8;
+		break;
+	case GL_RGB16:
+		bytes_per_pixel = 6;
+		break;
+	case GL_RG16:
+		bytes_per_pixel = 4;
+		break;
+	case GL_R16:
+		bytes_per_pixel = 2;
+		break;
 	default:
 		// TODO: Add more here as needed.
 		assert(false);