From: Paul Berry <stereotype...@gmail.com> --- src/mapi/glapi/gen/gl_API.xml | 2 +- src/mesa/Makefile.sources | 1 + src/mesa/main/marshal.c | 123 ++++++++++++++++++++++++++++++++++++++++++ src/mesa/main/marshal.h | 10 ++++ 4 files changed, 135 insertions(+), 1 deletion(-) create mode 100644 src/mesa/main/marshal.c
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml index e2e2719..817421c 100644 --- a/src/mapi/glapi/gen/gl_API.xml +++ b/src/mapi/glapi/gen/gl_API.xml @@ -5468,21 +5468,21 @@ <param name="shader" type="GLuint"/> <return type="GLboolean"/> <glx ignore="true"/> </function> <function name="LinkProgram" es2="2.0"> <param name="program" type="GLuint"/> <glx ignore="true"/> </function> - <function name="ShaderSource" es2="2.0"> + <function name="ShaderSource" es2="2.0" marshal="custom"> <param name="shader" type="GLuint"/> <param name="count" type="GLsizei"/> <param name="string" type="const GLchar * const *"/> <param name="length" type="const GLint *"/> <glx ignore="true"/> </function> <function name="UseProgram" es2="2.0"> <param name="program" type="GLuint"/> <glx ignore="true"/> diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources index 65e1fc9..bae89b9 100644 --- a/src/mesa/Makefile.sources +++ b/src/mesa/Makefile.sources @@ -125,20 +125,21 @@ MAIN_FILES = \ main/histogram.h \ main/image.c \ main/image.h \ main/imports.c \ main/imports.h \ main/light.c \ main/light.h \ main/lines.c \ main/lines.h \ main/macros.h \ + main/marshal.c \ main/marshal_generated.c \ main/marshal_generated.h \ main/matrix.c \ main/matrix.h \ main/mipmap.c \ main/mipmap.h \ main/mm.c \ main/mm.h \ main/mtypes.h \ main/multisample.c \ diff --git a/src/mesa/main/marshal.c b/src/mesa/main/marshal.c new file mode 100644 index 0000000..c0760d2 --- /dev/null +++ b/src/mesa/main/marshal.c @@ -0,0 +1,123 @@ +/* + * Copyright © 2012 Intel Corporation + * + * Permission is hereby granted, free of charge, to any person obtaining a + * copy of this software and associated documentation files (the "Software"), + * to deal in the Software without restriction, including without limitation + * the rights to use, copy, modify, merge, publish, distribute, sublicense, + * and/or sell copies of the Software, and to permit persons to whom the + * Software is furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice (including the next + * paragraph) shall be included in all copies or substantial portions of the + * Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS + * IN THE SOFTWARE. + */ + +/** \file marshal.c + * + * Custom functions for marshalling GL calls from the main thread to a worker + * thread when automatic code generation isn't appropriate. + */ + +#include "marshal.h" +#include "dispatch.h" +#include "marshal_generated.h" + +struct marshal_cmd_ShaderSource +{ + struct marshal_cmd_base cmd_base; + GLuint shader; + GLsizei count; + /* Followed by GLint length[count], then the contents of all strings, + * concatenated. + */ +}; + + +void +_mesa_unmarshal_ShaderSource(struct gl_context *ctx, + const struct marshal_cmd_ShaderSource *cmd) +{ + const GLint *cmd_length = (const GLint *) (cmd + 1); + const GLchar *cmd_strings = (const GLchar *) (cmd_length + cmd->count); + /* TODO: how to deal with malloc failure? */ + const GLchar * *string = malloc(cmd->count * sizeof(const GLchar *)); + int i; + + for (i = 0; i < cmd->count; ++i) { + string[i] = cmd_strings; + cmd_strings += cmd_length[i]; + } + CALL_ShaderSource(ctx->CurrentServerDispatch, + (cmd->shader, cmd->count, string, cmd_length)); + free(string); +} + + +static size_t +measure_ShaderSource_strings(GLsizei count, const GLchar * const *string, + const GLint *length_in, GLint *length_out) +{ + int i; + size_t total_string_length = 0; + + for (i = 0; i < count; ++i) { + if (length_in == NULL || length_in[i] < 0) { + if (string[i]) + length_out[i] = strlen(string[i]); + } else { + length_out[i] = length_in[i]; + } + total_string_length += length_out[i]; + } + return total_string_length; +} + + +void GLAPIENTRY +_mesa_marshal_ShaderSource(GLuint shader, GLsizei count, + const GLchar * const *string, const GLint *length) +{ + /* TODO: how to report an error if count < 0? */ + + GET_CURRENT_CONTEXT(ctx); + /* TODO: how to deal with malloc failure? */ + const size_t fixed_cmd_size = sizeof(struct marshal_cmd_ShaderSource); + STATIC_ASSERT(sizeof(struct marshal_cmd_ShaderSource) % sizeof(GLint) == 0); + size_t length_size = count * sizeof(GLint); + GLint *length_tmp = malloc(length_size); + size_t total_string_length = + measure_ShaderSource_strings(count, string, length, length_tmp); + size_t total_cmd_size = fixed_cmd_size + length_size + total_string_length; + + if (total_cmd_size <= MARSHAL_MAX_CMD_SIZE) { + struct marshal_cmd_ShaderSource *cmd = + _mesa_glthread_allocate_command(ctx, DISPATCH_CMD_ShaderSource, + total_cmd_size); + GLint *cmd_length = (GLint *) (cmd + 1); + GLchar *cmd_strings = (GLchar *) (cmd_length + count); + int i; + + cmd->shader = shader; + cmd->count = count; + memcpy(cmd_length, length_tmp, length_size); + for (i = 0; i < count; ++i) { + memcpy(cmd_strings, string[i], cmd_length[i]); + cmd_strings += cmd_length[i]; + } + _mesa_post_marshal_hook(ctx); + } else { + _mesa_glthread_finish(ctx); + CALL_ShaderSource(ctx->CurrentServerDispatch, + (shader, count, string, length_tmp)); + } + free(length_tmp); +} diff --git a/src/mesa/main/marshal.h b/src/mesa/main/marshal.h index ad32b6b..0b69d66 100644 --- a/src/mesa/main/marshal.h +++ b/src/mesa/main/marshal.h @@ -102,11 +102,21 @@ static inline void _mesa_post_marshal_hook(struct gl_context *ctx) { /* This can be enabled for debugging whether a failure is a synchronization * problem between the main thread and the worker thread, or a failure in * how we actually marshal. */ if (false) _mesa_glthread_finish(ctx); } +struct marshal_cmd_ShaderSource; + +void GLAPIENTRY +_mesa_marshal_ShaderSource(GLuint shader, GLsizei count, + const GLchar * const *string, const GLint *length); + +void +_mesa_unmarshal_ShaderSource(struct gl_context *ctx, + const struct marshal_cmd_ShaderSource *cmd); + #endif /* MARSHAL_H */ -- 2.7.4 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev