From c20fd744fef1ffb938f74dbafbe375c863fecc65 Mon Sep 17 00:00:00 2001 From: Jordan Justen Date: Sat, 4 Nov 2017 16:52:14 -0700 Subject: [PATCH] mesa: Add Mesa ARB_get_program_binary helper functions MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit V2 (Timothy Arceri): - add extra code comment - stop passing around void *binary and just pass program_binary_header *hdr instead. - move to src/mesa/main rather than src/util V3 (Timothy Arceri): - Move more code out of the backend and into the common helpers. Signed-off-by: Jordan Justen Reviewed-by: Tapani Pälli --- src/mesa/Makefile.sources | 2 + src/mesa/main/program_binary.c | 291 +++++++++++++++++++++++++++++++++ src/mesa/main/program_binary.h | 56 +++++++ src/mesa/meson.build | 2 + 4 files changed, 351 insertions(+) create mode 100644 src/mesa/main/program_binary.c create mode 100644 src/mesa/main/program_binary.h diff --git a/src/mesa/Makefile.sources b/src/mesa/Makefile.sources index 6da1e3fef9d..d8b1eb1f995 100644 --- a/src/mesa/Makefile.sources +++ b/src/mesa/Makefile.sources @@ -173,6 +173,8 @@ MAIN_FILES = \ main/points.h \ main/polygon.c \ main/polygon.h \ + main/program_binary.c \ + main/program_binary.h \ main/program_resource.c \ main/program_resource.h \ main/querymatrix.c \ diff --git a/src/mesa/main/program_binary.c b/src/mesa/main/program_binary.c new file mode 100644 index 00000000000..2786487362f --- /dev/null +++ b/src/mesa/main/program_binary.c @@ -0,0 +1,291 @@ +/* + * Mesa 3-D graphics library + * + * Copyright (c) 2017 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 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 program_binary.c + * + * Helper functions for serializing a binary program. + */ + + +#include "compiler/blob.h" +#include "compiler/glsl/serialize.h" +#include "main/errors.h" +#include "main/mtypes.h" +#include "util/crc32.h" +#include "program_binary.h" +#include "program/prog_parameter.h" + +/** + * Mesa supports one binary format, but it must differentiate between formats + * produced by different drivers and different Mesa versions. + * + * Mesa uses a uint32_t value to specify an internal format. The only format + * defined has one uint32_t value of 0, followed by 20 bytes specifying a sha1 + * that uniquely identifies the Mesa driver type and version. + */ + +struct program_binary_header { + /* If internal_format is 0, it must be followed by the 20 byte sha1 that + * identifies the Mesa driver and version supported. If we want to support + * something besides a sha1, then a new internal_format value can be added. + */ + uint32_t internal_format; + uint8_t sha1[20]; + /* Fields following sha1 can be changed since the sha1 will guarantee that + * the binary only works with the same Mesa version. + */ + uint32_t size; + uint32_t crc32; +}; + +/** + * Returns the header size needed for a binary + */ +static unsigned +get_program_binary_header_size(void) +{ + return sizeof(struct program_binary_header); +} + +static bool +write_program_binary(const void *payload, unsigned payload_size, + const void *sha1, void *binary, unsigned binary_size, + GLenum *binary_format) +{ + struct program_binary_header *hdr = binary; + + if (binary_size < sizeof(*hdr)) + return false; + + /* binary_size is the size of the buffer provided by the application. + * Make sure our program (payload) will fit in the buffer. + */ + if (payload_size > binary_size - sizeof(*hdr)) + return false; + + hdr->internal_format = 0; + memcpy(hdr->sha1, sha1, sizeof(hdr->sha1)); + memcpy(hdr + 1, payload, payload_size); + hdr->size = payload_size; + + hdr->crc32 = util_hash_crc32(hdr + 1, payload_size); + *binary_format = GL_PROGRAM_BINARY_FORMAT_MESA; + + return true; +} + +static bool +simple_header_checks(const struct program_binary_header *hdr, unsigned length) +{ + if (hdr == NULL || length < sizeof(*hdr)) + return false; + + if (hdr->internal_format != 0) + return false; + + return true; +} + +static bool +check_crc32(const struct program_binary_header *hdr, unsigned length) +{ + uint32_t crc32; + unsigned crc32_len; + + crc32_len = hdr->size; + if (crc32_len > length - sizeof(*hdr)) + return false; + + crc32 = util_hash_crc32(hdr + 1, crc32_len); + if (hdr->crc32 != crc32) + return false; + + return true; +} + +static bool +is_program_binary_valid(GLenum binary_format, const void *sha1, + const struct program_binary_header *hdr, + unsigned length) +{ + if (binary_format != GL_PROGRAM_BINARY_FORMAT_MESA) + return false; + + if (!simple_header_checks(hdr, length)) + return false; + + if (memcmp(hdr->sha1, sha1, sizeof(hdr->sha1)) != 0) + return false; + + if (!check_crc32(hdr, length)) + return false; + + return true; +} + +/** + * Returns the payload within the binary. + * + * If NULL is returned, then the binary not supported. If non-NULL is + * returned, it will be a pointer contained within the specified `binary` + * buffer. + * + * This can be used to access the payload of `binary` during the + * glProgramBinary call. + */ +static const void* +get_program_binary_payload(GLenum binary_format, const void *sha1, + const void *binary, unsigned length) +{ + const struct program_binary_header *hdr = binary; + if (!is_program_binary_valid(binary_format, sha1, hdr, length)) + return NULL; + return (const uint8_t*)binary + sizeof(*hdr); +} + +static void +write_program_payload(struct gl_context *ctx, struct blob *blob, + struct gl_shader_program *sh_prog) +{ + bool serialize[MESA_SHADER_STAGES]; + for (unsigned stage = 0; stage < MESA_SHADER_STAGES; stage++) { + struct gl_linked_shader *shader = sh_prog->_LinkedShaders[stage]; + serialize[stage] = shader && shader->Program->driver_cache_blob == NULL; + if (serialize[stage]) + ctx->Driver.ProgramBinarySerializeDriverBlob(ctx, shader->Program); + } + + serialize_glsl_program(blob, ctx, sh_prog); + + for (unsigned stage = 0; stage < MESA_SHADER_STAGES; stage++) { + if (!serialize[stage]) + continue; + + struct gl_program *prog = sh_prog->_LinkedShaders[stage]->Program; + ralloc_free(prog->driver_cache_blob); + prog->driver_cache_blob = NULL; + prog->driver_cache_blob_size = 0; + } +} + +static bool +read_program_payload(struct gl_context *ctx, struct blob_reader *blob, + GLenum binary_format, struct gl_shader_program *sh_prog) +{ + if (!deserialize_glsl_program(blob, ctx, sh_prog)) + return false; + + unsigned int stage; + for (stage = 0; stage < ARRAY_SIZE(sh_prog->_LinkedShaders); stage++) { + struct gl_linked_shader *shader = sh_prog->_LinkedShaders[stage]; + if (!shader) + continue; + + ctx->Driver.ProgramBinaryDeserializeDriverBlob(ctx, sh_prog, + shader->Program); + } + + return true; +} + +void +_mesa_get_program_binary_length(struct gl_context *ctx, + struct gl_shader_program *sh_prog, + GLint *length) +{ + struct blob blob; + blob_init_fixed(&blob, NULL, SIZE_MAX); + write_program_payload(ctx, &blob, sh_prog); + *length = get_program_binary_header_size() + blob.size; + blob_finish(&blob); +} + +void +_mesa_get_program_binary(struct gl_context *ctx, + struct gl_shader_program *sh_prog, + GLsizei buf_size, GLsizei *length, + GLenum *binary_format, GLvoid *binary) +{ + struct blob blob; + uint8_t driver_sha1[20]; + unsigned header_size = get_program_binary_header_size(); + + ctx->Driver.GetProgramBinaryDriverSHA1(ctx, driver_sha1); + + blob_init(&blob); + + if (buf_size < header_size) + goto fail; + + write_program_payload(ctx, &blob, sh_prog); + if (blob.size + header_size > buf_size || + blob.out_of_memory) + goto fail; + + bool written = write_program_binary(blob.data, blob.size, driver_sha1, + binary, buf_size, binary_format); + if (!written || blob.out_of_memory) + goto fail; + + *length = header_size + blob.size; + + blob_finish(&blob); + return; + +fail: + _mesa_error(ctx, GL_INVALID_OPERATION, + "glGetProgramBinary(buffer too small)"); + *length = 0; + blob_finish(&blob); +} + +void +_mesa_program_binary(struct gl_context *ctx, struct gl_shader_program *sh_prog, + GLenum binary_format, const GLvoid *binary, + GLsizei length) +{ + uint8_t driver_sha1[20]; + unsigned header_size = get_program_binary_header_size(); + + ctx->Driver.GetProgramBinaryDriverSHA1(ctx, driver_sha1); + + const void *payload = get_program_binary_payload(binary_format, driver_sha1, + binary, length); + + if (payload == NULL) { + sh_prog->data->LinkStatus = linking_failure; + return; + } + + struct blob_reader blob; + blob_reader_init(&blob, payload, length - header_size); + + if (!read_program_payload(ctx, &blob, binary_format, sh_prog)) { + sh_prog->data->LinkStatus = linking_failure; + return; + } + + sh_prog->data->LinkStatus = linking_success; +} diff --git a/src/mesa/main/program_binary.h b/src/mesa/main/program_binary.h new file mode 100644 index 00000000000..9e16b9ec54d --- /dev/null +++ b/src/mesa/main/program_binary.h @@ -0,0 +1,56 @@ +/* + * Mesa 3-D graphics library + * + * Copyright (C) 2004-2007 Brian Paul All Rights Reserved. + * Copyright (C) 2010 VMware, Inc. All Rights Reserved. + * + * 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 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. + */ + + +#ifndef PROGRAM_BINARY_H +#define PROGRAM_BINARY_H + +#include + +#ifdef __cplusplus +extern "C" { +#endif + +void +_mesa_get_program_binary_length(struct gl_context *ctx, + struct gl_shader_program *sh_prog, + GLint *length); + +void +_mesa_get_program_binary(struct gl_context *ctx, + struct gl_shader_program *sh_prog, + GLsizei buf_size, GLsizei *length, + GLenum *binary_format, GLvoid *binary); + +void +_mesa_program_binary(struct gl_context *ctx, struct gl_shader_program *sh_prog, + GLenum binary_format, const GLvoid *binary, + GLsizei length); + +#ifdef __cplusplus +} +#endif + +#endif /* PROGRAM_BINARY_H */ diff --git a/src/mesa/meson.build b/src/mesa/meson.build index 05a3a9ac55d..670c3d22d24 100644 --- a/src/mesa/meson.build +++ b/src/mesa/meson.build @@ -215,6 +215,8 @@ files_libmesa_common = files( 'main/points.h', 'main/polygon.c', 'main/polygon.h', + 'main/program_binary.c', + 'main/program_binary.h', 'main/program_resource.c', 'main/program_resource.h', 'main/querymatrix.c', -- 2.30.2