wine/vkd3d
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/vkd3d.git synced 2024-11-21 16:46:41 -08:00
Code Issues Packages Projects Releases Wiki Activity

vkd3d-shader: Report error messages from vkd3d_shader_serialize_root_signature().

Browse Source 
Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Henri Verbeet 2020-07-30 14:59:58 +04:30 committed by Alexandre Julliard
parent 33eb944bf4
commit 1279d5df9e
5 changed files with 192 additions and 84 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
include/vkd3d_shader.h
View File

@ -756,9 +756,8 @@ int vkd3d_shader_parse_root_signature(const struct vkd3d_shader_code *dxbc,
struct vkd3d_shader_versioned_root_signature_desc *root_signature, char **messages); struct vkd3d_shader_versioned_root_signature_desc *root_signature, char **messages);
void vkd3d_shader_free_root_signature(struct vkd3d_shader_versioned_root_signature_desc *root_signature); void vkd3d_shader_free_root_signature(struct vkd3d_shader_versioned_root_signature_desc *root_signature);
/* FIXME: Add support for returning error messages (ID3DBlob). */
int vkd3d_shader_serialize_root_signature(const struct vkd3d_shader_versioned_root_signature_desc *root_signature, int vkd3d_shader_serialize_root_signature(const struct vkd3d_shader_versioned_root_signature_desc *root_signature,
struct vkd3d_shader_code *dxbc); struct vkd3d_shader_code *dxbc, char **messages);
int vkd3d_shader_convert_root_signature(struct vkd3d_shader_versioned_root_signature_desc *dst, int vkd3d_shader_convert_root_signature(struct vkd3d_shader_versioned_root_signature_desc *dst,
enum vkd3d_shader_root_signature_version version, const struct vkd3d_shader_versioned_root_signature_desc *src); enum vkd3d_shader_root_signature_version version, const struct vkd3d_shader_versioned_root_signature_desc *src);
@ -788,7 +787,8 @@ typedef int (*PFN_vkd3d_shader_parse_root_signature)(const struct vkd3d_shader_c
typedef void (*PFN_vkd3d_shader_free_root_signature)(struct vkd3d_shader_versioned_root_signature_desc *root_signature); typedef void (*PFN_vkd3d_shader_free_root_signature)(struct vkd3d_shader_versioned_root_signature_desc *root_signature);
typedef int (*PFN_vkd3d_shader_serialize_root_signature)( typedef int (*PFN_vkd3d_shader_serialize_root_signature)(
const struct vkd3d_shader_versioned_root_signature_desc *root_signature, struct vkd3d_shader_code *dxbc); const struct vkd3d_shader_versioned_root_signature_desc *root_signature,
struct vkd3d_shader_code *dxbc, char **messages);
typedef int (*PFN_vkd3d_shader_convert_root_signature)(struct vkd3d_shader_versioned_root_signature_desc *dst, typedef int (*PFN_vkd3d_shader_convert_root_signature)(struct vkd3d_shader_versioned_root_signature_desc *dst,
enum vkd3d_shader_root_signature_version version, const struct vkd3d_shader_versioned_root_signature_desc *src); enum vkd3d_shader_root_signature_version version, const struct vkd3d_shader_versioned_root_signature_desc *src);

221
libs/vkd3d-shader/dxbc.c
View File

@ -2768,6 +2768,8 @@ static unsigned int versioned_root_signature_get_flags(const struct vkd3d_shader
struct root_signature_writer_context struct root_signature_writer_context
{ {
struct vkd3d_shader_message_context message_context;
DWORD *data; DWORD *data;
size_t position; size_t position;
size_t capacity; size_t capacity;
@ -2813,33 +2815,38 @@ static size_t get_chunk_offset(struct root_signature_writer_context *context)
static int shader_write_root_signature_header(struct root_signature_writer_context *context) static int shader_write_root_signature_header(struct root_signature_writer_context *context)
{ {
if (!write_dword(context, TAG_DXBC)) if (!write_dword(context, TAG_DXBC))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
/* The checksum is computed when all data is generated. */ /* The checksum is computed when all data is generated. */
if (!write_dwords(context, 4, 0x00000000)) if (!write_dwords(context, 4, 0x00000000))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, 0x00000001)) if (!write_dword(context, 0x00000001))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
context->total_size_position = context->position; context->total_size_position = context->position;
if (!write_dword(context, 0xffffffff)) /* total size */ if (!write_dword(context, 0xffffffff)) /* total size */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, 1)) /* chunk count */ if (!write_dword(context, 1)) /* chunk count */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
/* chunk offset */ /* chunk offset */
if (!write_dword(context, (context->position + 1) * sizeof(DWORD))) if (!write_dword(context, (context->position + 1) * sizeof(DWORD)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, TAG_RTS0)) if (!write_dword(context, TAG_RTS0))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, 0xffffffff)) /* chunk size */ if (!write_dword(context, 0xffffffff)) /* chunk size */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
context->chunk_position = context->position; context->chunk_position = context->position;
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature header.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_descriptor_ranges(struct root_signature_writer_context *context, static int shader_write_descriptor_ranges(struct root_signature_writer_context *context,
@ -2851,18 +2858,23 @@ static int shader_write_descriptor_ranges(struct root_signature_writer_context *
for (i = 0; i < table->descriptor_range_count; ++i) for (i = 0; i < table->descriptor_range_count; ++i)
{ {
if (!write_dword(context, ranges[i].range_type)) if (!write_dword(context, ranges[i].range_type))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].descriptor_count)) if (!write_dword(context, ranges[i].descriptor_count))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].base_shader_register)) if (!write_dword(context, ranges[i].base_shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].register_space)) if (!write_dword(context, ranges[i].register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].descriptor_table_offset)) if (!write_dword(context, ranges[i].descriptor_table_offset))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
} }
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature descriptor ranges.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_descriptor_ranges1(struct root_signature_writer_context *context, static int shader_write_descriptor_ranges1(struct root_signature_writer_context *context,
@ -2874,79 +2886,109 @@ static int shader_write_descriptor_ranges1(struct root_signature_writer_context
for (i = 0; i < table->descriptor_range_count; ++i) for (i = 0; i < table->descriptor_range_count; ++i)
{ {
if (!write_dword(context, ranges[i].range_type)) if (!write_dword(context, ranges[i].range_type))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].descriptor_count)) if (!write_dword(context, ranges[i].descriptor_count))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].base_shader_register)) if (!write_dword(context, ranges[i].base_shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].register_space)) if (!write_dword(context, ranges[i].register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].flags)) if (!write_dword(context, ranges[i].flags))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, ranges[i].descriptor_table_offset)) if (!write_dword(context, ranges[i].descriptor_table_offset))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
} }
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature descriptor ranges.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_descriptor_table(struct root_signature_writer_context *context, static int shader_write_descriptor_table(struct root_signature_writer_context *context,
const struct vkd3d_shader_root_descriptor_table *table) const struct vkd3d_shader_root_descriptor_table *table)
{ {
if (!write_dword(context, table->descriptor_range_count)) if (!write_dword(context, table->descriptor_range_count))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, get_chunk_offset(context) + sizeof(DWORD))) /* offset */ if (!write_dword(context, get_chunk_offset(context) + sizeof(DWORD))) /* offset */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
return shader_write_descriptor_ranges(context, table); return shader_write_descriptor_ranges(context, table);
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root descriptor table.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_descriptor_table1(struct root_signature_writer_context *context, static int shader_write_descriptor_table1(struct root_signature_writer_context *context,
const struct vkd3d_shader_root_descriptor_table1 *table) const struct vkd3d_shader_root_descriptor_table1 *table)
{ {
if (!write_dword(context, table->descriptor_range_count)) if (!write_dword(context, table->descriptor_range_count))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, get_chunk_offset(context) + sizeof(DWORD))) /* offset */ if (!write_dword(context, get_chunk_offset(context) + sizeof(DWORD))) /* offset */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
return shader_write_descriptor_ranges1(context, table); return shader_write_descriptor_ranges1(context, table);
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root descriptor table.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_root_constants(struct root_signature_writer_context *context, static int shader_write_root_constants(struct root_signature_writer_context *context,
const struct vkd3d_shader_root_constants *constants) const struct vkd3d_shader_root_constants *constants)
{ {
if (!write_dword(context, constants->shader_register)) if (!write_dword(context, constants->shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, constants->register_space)) if (!write_dword(context, constants->register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, constants->value_count)) if (!write_dword(context, constants->value_count))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root constants.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_root_descriptor(struct root_signature_writer_context *context, static int shader_write_root_descriptor(struct root_signature_writer_context *context,
const struct vkd3d_shader_root_descriptor *descriptor) const struct vkd3d_shader_root_descriptor *descriptor)
{ {
if (!write_dword(context, descriptor->shader_register)) if (!write_dword(context, descriptor->shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, descriptor->register_space)) if (!write_dword(context, descriptor->register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root descriptor.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_root_descriptor1(struct root_signature_writer_context *context, static int shader_write_root_descriptor1(struct root_signature_writer_context *context,
const struct vkd3d_shader_root_descriptor1 *descriptor) const struct vkd3d_shader_root_descriptor1 *descriptor)
{ {
if (!write_dword(context, descriptor->shader_register)) if (!write_dword(context, descriptor->shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, descriptor->register_space)) if (!write_dword(context, descriptor->register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, descriptor->flags)) if (!write_dword(context, descriptor->flags))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root descriptor.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_root_parameters(struct root_signature_writer_context *context, static int shader_write_root_parameters(struct root_signature_writer_context *context,
@ -2961,11 +3003,11 @@ static int shader_write_root_parameters(struct root_signature_writer_context *co
for (i = 0; i < parameter_count; ++i) for (i = 0; i < parameter_count; ++i)
{ {
if (!write_dword(context, versioned_root_signature_get_parameter_type(desc, i))) if (!write_dword(context, versioned_root_signature_get_parameter_type(desc, i)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, versioned_root_signature_get_parameter_shader_visibility(desc, i))) if (!write_dword(context, versioned_root_signature_get_parameter_shader_visibility(desc, i)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, 0xffffffff)) /* offset */ if (!write_dword(context, 0xffffffff)) /* offset */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
} }
for (i = 0; i < parameter_count; ++i) for (i = 0; i < parameter_count; ++i)
@ -2993,6 +3035,9 @@ static int shader_write_root_parameters(struct root_signature_writer_context *co
break; break;
default: default:
FIXME("Unrecognized type %#x.\n", versioned_root_signature_get_parameter_type(desc, i)); FIXME("Unrecognized type %#x.\n", versioned_root_signature_get_parameter_type(desc, i));
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_INVALID_ROOT_PARAMETER_TYPE,
"Invalid/unrecognised root signature root parameter type %#x.",
versioned_root_signature_get_parameter_type(desc, i));
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
@ -3001,6 +3046,11 @@ static int shader_write_root_parameters(struct root_signature_writer_context *co
} }
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature root parameters.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_static_samplers(struct root_signature_writer_context *context, static int shader_write_static_samplers(struct root_signature_writer_context *context,
@ -3012,34 +3062,39 @@ static int shader_write_static_samplers(struct root_signature_writer_context *co
for (i = 0; i < versioned_root_signature_get_static_sampler_count(desc); ++i) for (i = 0; i < versioned_root_signature_get_static_sampler_count(desc); ++i)
{ {
if (!write_dword(context, samplers[i].filter)) if (!write_dword(context, samplers[i].filter))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].address_u)) if (!write_dword(context, samplers[i].address_u))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].address_v)) if (!write_dword(context, samplers[i].address_v))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].address_w)) if (!write_dword(context, samplers[i].address_w))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_float(context, samplers[i].mip_lod_bias)) if (!write_float(context, samplers[i].mip_lod_bias))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].max_anisotropy)) if (!write_dword(context, samplers[i].max_anisotropy))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].comparison_func)) if (!write_dword(context, samplers[i].comparison_func))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].border_colour)) if (!write_dword(context, samplers[i].border_colour))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_float(context, samplers[i].min_lod)) if (!write_float(context, samplers[i].min_lod))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_float(context, samplers[i].max_lod)) if (!write_float(context, samplers[i].max_lod))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].shader_register)) if (!write_dword(context, samplers[i].shader_register))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].register_space)) if (!write_dword(context, samplers[i].register_space))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, samplers[i].shader_visibility)) if (!write_dword(context, samplers[i].shader_visibility))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
} }
return VKD3D_OK; return VKD3D_OK;
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature static samplers.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int shader_write_root_signature(struct root_signature_writer_context *context, static int shader_write_root_signature(struct root_signature_writer_context *context,
@ -3049,30 +3104,36 @@ static int shader_write_root_signature(struct root_signature_writer_context *con
int ret; int ret;
if (!write_dword(context, desc->version)) if (!write_dword(context, desc->version))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, versioned_root_signature_get_parameter_count(desc))) if (!write_dword(context, versioned_root_signature_get_parameter_count(desc)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, get_chunk_offset(context) + 4 * sizeof(DWORD))) /* offset */ if (!write_dword(context, get_chunk_offset(context) + 4 * sizeof(DWORD))) /* offset */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, versioned_root_signature_get_static_sampler_count(desc))) if (!write_dword(context, versioned_root_signature_get_static_sampler_count(desc)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
samplers_offset_position = context->position; samplers_offset_position = context->position;
if (!write_dword(context, 0xffffffff)) /* offset */ if (!write_dword(context, 0xffffffff)) /* offset */
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if (!write_dword(context, versioned_root_signature_get_flags(desc))) if (!write_dword(context, versioned_root_signature_get_flags(desc)))
return VKD3D_ERROR_OUT_OF_MEMORY; goto fail;
if ((ret = shader_write_root_parameters(context, desc)) < 0) if ((ret = shader_write_root_parameters(context, desc)) < 0)
return ret; return ret;
context->data[samplers_offset_position] = get_chunk_offset(context); context->data[samplers_offset_position] = get_chunk_offset(context);
return shader_write_static_samplers(context, desc); return shader_write_static_samplers(context, desc);
fail:
vkd3d_shader_error(&context->message_context, VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY,
"Out of memory while writing root signature.");
return VKD3D_ERROR_OUT_OF_MEMORY;
} }
static int validate_descriptor_table_v_1_0(const struct vkd3d_shader_root_descriptor_table *descriptor_table) static int validate_descriptor_table_v_1_0(const struct vkd3d_shader_root_descriptor_table *descriptor_table,
struct vkd3d_shader_message_context *message_context)
{ {
bool have_srv_uav_cbv = false; bool have_srv_uav_cbv = false;
bool have_sampler = false; bool have_sampler = false;
@ -3095,6 +3156,8 @@ static int validate_descriptor_table_v_1_0(const struct vkd3d_shader_root_descri
else else
{ {
WARN("Invalid descriptor range type %#x.\n", r->range_type); WARN("Invalid descriptor range type %#x.\n", r->range_type);
vkd3d_shader_error(message_context, VKD3D_SHADER_ERROR_RS_INVALID_DESCRIPTOR_RANGE_TYPE,
"Invalid root signature descriptor range type %#x.", r->range_type);
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
} }
@ -3102,13 +3165,16 @@ static int validate_descriptor_table_v_1_0(const struct vkd3d_shader_root_descri
if (have_srv_uav_cbv && have_sampler) if (have_srv_uav_cbv && have_sampler)
{ {
WARN("Samplers cannot be mixed with CBVs/SRVs/UAVs in descriptor tables.\n"); WARN("Samplers cannot be mixed with CBVs/SRVs/UAVs in descriptor tables.\n");
vkd3d_shader_error(message_context, VKD3D_SHADER_ERROR_RS_MIXED_DESCRIPTOR_RANGE_TYPES,
"Encountered both CBV/SRV/UAV and sampler descriptor ranges in the same root descriptor table.");
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
return VKD3D_OK; return VKD3D_OK;
} }
static int validate_descriptor_table_v_1_1(const struct vkd3d_shader_root_descriptor_table1 *descriptor_table) static int validate_descriptor_table_v_1_1(const struct vkd3d_shader_root_descriptor_table1 *descriptor_table,
struct vkd3d_shader_message_context *message_context)
{ {
bool have_srv_uav_cbv = false; bool have_srv_uav_cbv = false;
bool have_sampler = false; bool have_sampler = false;
@ -3131,6 +3197,8 @@ static int validate_descriptor_table_v_1_1(const struct vkd3d_shader_root_descri
else else
{ {
WARN("Invalid descriptor range type %#x.\n", r->range_type); WARN("Invalid descriptor range type %#x.\n", r->range_type);
vkd3d_shader_error(message_context, VKD3D_SHADER_ERROR_RS_INVALID_DESCRIPTOR_RANGE_TYPE,
"Invalid root signature descriptor range type %#x.", r->range_type);
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
} }
@ -3138,13 +3206,16 @@ static int validate_descriptor_table_v_1_1(const struct vkd3d_shader_root_descri
if (have_srv_uav_cbv && have_sampler) if (have_srv_uav_cbv && have_sampler)
{ {
WARN("Samplers cannot be mixed with CBVs/SRVs/UAVs in descriptor tables.\n"); WARN("Samplers cannot be mixed with CBVs/SRVs/UAVs in descriptor tables.\n");
vkd3d_shader_error(message_context, VKD3D_SHADER_ERROR_RS_MIXED_DESCRIPTOR_RANGE_TYPES,
"Encountered both CBV/SRV/UAV and sampler descriptor ranges in the same root descriptor table.");
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
return VKD3D_OK; return VKD3D_OK;
} }
static int validate_root_signature_desc(const struct vkd3d_shader_versioned_root_signature_desc *desc) static int validate_root_signature_desc(const struct vkd3d_shader_versioned_root_signature_desc *desc,
struct vkd3d_shader_message_context *message_context)
{ {
int ret = VKD3D_OK; int ret = VKD3D_OK;
unsigned int i; unsigned int i;
@ -3157,9 +3228,9 @@ static int validate_root_signature_desc(const struct vkd3d_shader_versioned_root
if (type == VKD3D_SHADER_ROOT_PARAMETER_TYPE_DESCRIPTOR_TABLE) if (type == VKD3D_SHADER_ROOT_PARAMETER_TYPE_DESCRIPTOR_TABLE)
{ {
if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0)
ret = validate_descriptor_table_v_1_0(&desc->u.v_1_0.parameters[i].u.descriptor_table); ret = validate_descriptor_table_v_1_0(&desc->u.v_1_0.parameters[i].u.descriptor_table, message_context);
else else
ret = validate_descriptor_table_v_1_1(&desc->u.v_1_1.parameters[i].u.descriptor_table); ret = validate_descriptor_table_v_1_1(&desc->u.v_1_1.parameters[i].u.descriptor_table, message_context);
} }
if (ret < 0) if (ret < 0)
@ -3170,37 +3241,46 @@ static int validate_root_signature_desc(const struct vkd3d_shader_versioned_root
} }
int vkd3d_shader_serialize_root_signature(const struct vkd3d_shader_versioned_root_signature_desc *root_signature, int vkd3d_shader_serialize_root_signature(const struct vkd3d_shader_versioned_root_signature_desc *root_signature,
struct vkd3d_shader_code *dxbc) struct vkd3d_shader_code *dxbc, char **messages)
{ {
struct root_signature_writer_context context; struct root_signature_writer_context context;
size_t total_size, chunk_size; size_t total_size, chunk_size;
uint32_t checksum[4]; uint32_t checksum[4];
int ret; int ret;
TRACE("root_signature %p, dxbc %p.\n", root_signature, dxbc); TRACE("root_signature %p, dxbc %p, messages %p.\n", root_signature, dxbc, messages);
if (messages)
*messages = NULL;
memset(&context, 0, sizeof(context));
if (!vkd3d_shader_message_context_init(&context.message_context, VKD3D_SHADER_LOG_INFO, NULL))
return VKD3D_ERROR;
if (root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 if (root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0
&& root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) && root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1)
{ {
ret = VKD3D_ERROR_INVALID_ARGUMENT;
WARN("Root signature version %#x not supported.\n", root_signature->version); WARN("Root signature version %#x not supported.\n", root_signature->version);
return VKD3D_ERROR_INVALID_ARGUMENT; vkd3d_shader_error(&context.message_context, VKD3D_SHADER_ERROR_RS_INVALID_VERSION,
"Root signature version %#x is not supported.", root_signature->version);
goto done;
} }
if ((ret = validate_root_signature_desc(root_signature)) < 0) if ((ret = validate_root_signature_desc(root_signature, &context.message_context)) < 0)
return ret; goto done;
memset(dxbc, 0, sizeof(*dxbc)); memset(dxbc, 0, sizeof(*dxbc));
memset(&context, 0, sizeof(context));
if ((ret = shader_write_root_signature_header(&context)) < 0) if ((ret = shader_write_root_signature_header(&context)) < 0)
{ {
vkd3d_free(context.data); vkd3d_free(context.data);
return ret; goto done;
} }
if ((ret = shader_write_root_signature(&context, root_signature)) < 0) if ((ret = shader_write_root_signature(&context, root_signature)) < 0)
{ {
vkd3d_free(context.data); vkd3d_free(context.data);
return ret; goto done;
} }
total_size = context.position * sizeof(DWORD); total_size = context.position * sizeof(DWORD);
@ -3214,7 +3294,14 @@ int vkd3d_shader_serialize_root_signature(const struct vkd3d_shader_versioned_ro
vkd3d_compute_dxbc_checksum(dxbc->code, dxbc->size, checksum); vkd3d_compute_dxbc_checksum(dxbc->code, dxbc->size, checksum);
memcpy((uint32_t *)dxbc->code + 1, checksum, sizeof(checksum)); memcpy((uint32_t *)dxbc->code + 1, checksum, sizeof(checksum));
return VKD3D_OK; ret = VKD3D_OK;
done:
vkd3d_shader_message_context_trace_messages(&context.message_context);
if (messages && !(*messages = vkd3d_shader_message_context_copy_messages(&context.message_context)))
ret = VKD3D_ERROR_OUT_OF_MEMORY;
vkd3d_shader_message_context_cleanup(&context.message_context);
return ret;
} }
static void free_descriptor_ranges(const struct vkd3d_shader_root_parameter *parameters, unsigned int count) static void free_descriptor_ranges(const struct vkd3d_shader_root_parameter *parameters, unsigned int count)

7
libs/vkd3d-shader/vkd3d_shader_private.h
View File

@ -67,7 +67,14 @@ enum vkd3d_shader_error
VKD3D_SHADER_ERROR_DXBC_INVALID_VERSION = 4, VKD3D_SHADER_ERROR_DXBC_INVALID_VERSION = 4,
VKD3D_SHADER_ERROR_DXBC_INVALID_CHUNK_OFFSET = 5, VKD3D_SHADER_ERROR_DXBC_INVALID_CHUNK_OFFSET = 5,
VKD3D_SHADER_ERROR_DXBC_INVALID_CHUNK_SIZE = 6, VKD3D_SHADER_ERROR_DXBC_INVALID_CHUNK_SIZE = 6,
VKD3D_SHADER_ERROR_TPF_MISMATCHED_CF = 1000, VKD3D_SHADER_ERROR_TPF_MISMATCHED_CF = 1000,
VKD3D_SHADER_ERROR_RS_OUT_OF_MEMORY = 3000,
VKD3D_SHADER_ERROR_RS_INVALID_VERSION = 3001,
VKD3D_SHADER_ERROR_RS_INVALID_ROOT_PARAMETER_TYPE = 3002,
VKD3D_SHADER_ERROR_RS_INVALID_DESCRIPTOR_RANGE_TYPE = 3003,
VKD3D_SHADER_ERROR_RS_MIXED_DESCRIPTOR_RANGE_TYPES = 3004,
}; };
enum VKD3D_SHADER_INSTRUCTION_HANDLER enum VKD3D_SHADER_INSTRUCTION_HANDLER

26
libs/vkd3d/vkd3d_main.c
View File

@ -543,6 +543,7 @@ HRESULT vkd3d_serialize_root_signature(const D3D12_ROOT_SIGNATURE_DESC *desc,
struct vkd3d_shader_versioned_root_signature_desc vkd3d_desc; struct vkd3d_shader_versioned_root_signature_desc vkd3d_desc;
struct vkd3d_shader_code dxbc; struct vkd3d_shader_code dxbc;
struct d3d_blob *blob_object; struct d3d_blob *blob_object;
char *messages;
HRESULT hr; HRESULT hr;
int ret; int ret;
@ -565,13 +566,19 @@ HRESULT vkd3d_serialize_root_signature(const D3D12_ROOT_SIGNATURE_DESC *desc,
vkd3d_desc.version = VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0; vkd3d_desc.version = VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0;
vkd3d_desc.u.v_1_0 = *(const struct vkd3d_shader_root_signature_desc *)desc; vkd3d_desc.u.v_1_0 = *(const struct vkd3d_shader_root_signature_desc *)desc;
if ((ret = vkd3d_shader_serialize_root_signature(&vkd3d_desc, &dxbc)) < 0) if ((ret = vkd3d_shader_serialize_root_signature(&vkd3d_desc, &dxbc, &messages)) < 0)
{ {
WARN("Failed to serialize root signature, vkd3d result %d.\n", ret); WARN("Failed to serialize root signature, vkd3d result %d.\n", ret);
if (error_blob) if (error_blob && messages)
FIXME("Ignoring error blob %p.\n", error_blob); {
if (FAILED(hr = d3d_blob_create(messages, strlen(messages), &blob_object)))
ERR("Failed to create error blob, hr %#x.\n", hr);
else
*error_blob = &blob_object->ID3DBlob_iface;
}
return hresult_from_vkd3d_result(ret); return hresult_from_vkd3d_result(ret);
} }
vkd3d_shader_free_messages(messages);
if (FAILED(hr = d3d_blob_create((void *)dxbc.code, dxbc.size, &blob_object))) if (FAILED(hr = d3d_blob_create((void *)dxbc.code, dxbc.size, &blob_object)))
{ {
@ -591,6 +598,7 @@ HRESULT vkd3d_serialize_versioned_root_signature(const D3D12_VERSIONED_ROOT_SIGN
const struct vkd3d_shader_versioned_root_signature_desc *vkd3d_desc; const struct vkd3d_shader_versioned_root_signature_desc *vkd3d_desc;
struct vkd3d_shader_code dxbc; struct vkd3d_shader_code dxbc;
struct d3d_blob *blob_object; struct d3d_blob *blob_object;
char *messages;
HRESULT hr; HRESULT hr;
int ret; int ret;
@ -606,13 +614,19 @@ HRESULT vkd3d_serialize_versioned_root_signature(const D3D12_VERSIONED_ROOT_SIGN
*error_blob = NULL; *error_blob = NULL;
vkd3d_desc = (const struct vkd3d_shader_versioned_root_signature_desc *)desc; vkd3d_desc = (const struct vkd3d_shader_versioned_root_signature_desc *)desc;
if ((ret = vkd3d_shader_serialize_root_signature(vkd3d_desc, &dxbc)) < 0) if ((ret = vkd3d_shader_serialize_root_signature(vkd3d_desc, &dxbc, &messages)) < 0)
{ {
WARN("Failed to serialize root signature, vkd3d result %d.\n", ret); WARN("Failed to serialize root signature, vkd3d result %d.\n", ret);
if (error_blob) if (error_blob && messages)
FIXME("Ignoring error blob %p.\n", error_blob); {
if (FAILED(hr = d3d_blob_create(messages, strlen(messages), &blob_object)))
ERR("Failed to create error blob, hr %#x.\n", hr);
else
*error_blob = &blob_object->ID3DBlob_iface;
}
return hresult_from_vkd3d_result(ret); return hresult_from_vkd3d_result(ret);
} }
vkd3d_shader_free_messages(messages);
if (FAILED(hr = d3d_blob_create((void *)dxbc.code, dxbc.size, &blob_object))) if (FAILED(hr = d3d_blob_create((void *)dxbc.code, dxbc.size, &blob_object)))
{ {

2
tests/vkd3d_shader_api.c
View File

@ -127,7 +127,7 @@ static void test_vkd3d_shader_pfns(void)
pfn_vkd3d_shader_compile = vkd3d_shader_compile; pfn_vkd3d_shader_compile = vkd3d_shader_compile;
pfn_vkd3d_shader_scan = vkd3d_shader_scan; pfn_vkd3d_shader_scan = vkd3d_shader_scan;
rc = pfn_vkd3d_shader_serialize_root_signature(&empty_rs_desc, &dxbc); rc = pfn_vkd3d_shader_serialize_root_signature(&empty_rs_desc, &dxbc, NULL);
ok(rc == VKD3D_OK, "Got unexpected error code %d.\n", rc); ok(rc == VKD3D_OK, "Got unexpected error code %d.\n", rc);
rc = pfn_vkd3d_shader_parse_root_signature(&dxbc, &root_signature_desc, NULL); rc = pfn_vkd3d_shader_parse_root_signature(&dxbc, &root_signature_desc, NULL);
ok(rc == VKD3D_OK, "Got unexpected error code %d.\n", rc); ok(rc == VKD3D_OK, "Got unexpected error code %d.\n", rc);