November 2022 - Wine-commits

Francisco Casas : vkd3d-shader/hlsl: Properly free new store node memory if init_deref() fails.
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: 6f6ba8aa56a7d71861e522a9aa63cc47dc52d45d URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/6f6ba8aa56a7d71861e522a9aa63c… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Wed Sep 14 16:57:13 2022 -0300 vkd3d-shader/hlsl: Properly free new store node memory if init_deref() fails. --- libs/vkd3d-shader/hlsl.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libs/vkd3d-shader/hlsl.c b/libs/vkd3d-shader/hlsl.c index eddbf2c8..8aa289ac 100644 --- a/libs/vkd3d-shader/hlsl.c +++ b/libs/vkd3d-shader/hlsl.c @@ -885,7 +885,10 @@ struct hlsl_ir_store *hlsl_new_store_index(struct hlsl_ctx *ctx, const struct hl init_node(&store->node, HLSL_IR_STORE, NULL, loc); if (!init_deref(ctx, &store->lhs, lhs->var, lhs->path_len + !!idx)) + { + vkd3d_free(store); return NULL; + } for (i = 0; i < lhs->path_len; ++i) hlsl_src_from_node(&store->lhs.path[i], lhs->path[i].node); if (idx)

1 0

Francisco Casas : vkd3d-shader/hlsl: Don't allocate object types as constant registers.
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: 3153ce3145d9f36b0dab8928cd0e85415bf00dea URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/3153ce3145d9f36b0dab8928cd0e8… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Wed Sep 7 16:51:56 2022 -0400 vkd3d-shader/hlsl: Don't allocate object types as constant registers. --- libs/vkd3d-shader/hlsl_codegen.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 27af49cf..18c26771 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -2103,6 +2103,9 @@ static void allocate_const_registers(struct hlsl_ctx *ctx, struct hlsl_ir_functi { if (var->is_uniform && var->last_read) { + if (var->data_type->reg_size == 0) + continue; + if (var->data_type->reg_size > 4) var->reg = allocate_range(ctx, &liveness, 1, UINT_MAX, var->data_type->reg_size); else

1 0

Francisco Casas : vkd3d-shader/hlsl: Validate that extern structs don't contain objects SM < 5.
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: 2fa913ccaa96c2676d7b65dfea92c38abd56dda5 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/2fa913ccaa96c2676d7b65dfea92c… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Tue Nov 1 19:13:39 2022 -0300 vkd3d-shader/hlsl: Validate that extern structs don't contain objects SM < 5. It is worth noting that these checks should also be included for declarations inside cbuffers, once they are implemented. --- libs/vkd3d-shader/hlsl.y | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index f77b0b61..0e97d4d1 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -1887,6 +1887,26 @@ static void initialize_var_components(struct hlsl_ctx *ctx, struct list *instrs, } } +static bool type_has_object_components(struct hlsl_type *type, bool must_be_in_struct) +{ + if (type->type == HLSL_CLASS_OBJECT) + return !must_be_in_struct; + if (type->type == HLSL_CLASS_ARRAY) + return type_has_object_components(type->e.array.type, must_be_in_struct); + + if (type->type == HLSL_CLASS_STRUCT) + { + unsigned int i; + + for (i = 0; i < type->e.record.field_count; ++i) + { + if (type_has_object_components(type->e.record.fields[i].type, false)) + return true; + } + } + return false; +} + static struct list *declare_vars(struct hlsl_ctx *ctx, struct hlsl_type *basic_type, DWORD modifiers, struct list *var_list) { @@ -2022,6 +2042,13 @@ static struct list *declare_vars(struct hlsl_ctx *ctx, struct hlsl_type *basic_t if (!(modifiers & HLSL_STORAGE_STATIC)) var->modifiers |= HLSL_STORAGE_UNIFORM; + if (ctx->profile->major_version < 5 && (var->modifiers & HLSL_STORAGE_UNIFORM) && + type_has_object_components(var->data_type, true)) + { + hlsl_error(ctx, &var->loc, VKD3D_SHADER_ERROR_HLSL_INVALID_TYPE, + "Target profile doesn't support objects as struct members in uniform variables.\n"); + } + if ((func = hlsl_get_func_decl(ctx, var->name))) { hlsl_error(ctx, &var->loc, VKD3D_SHADER_ERROR_HLSL_REDEFINED,

1 0

Francisco Casas : vkd3d-shader/hlsl: Validate that statics don't contain both resources and numerics.
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: 6873b7130417b32474705339e6c94c6364d88115 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/6873b7130417b32474705339e6c94… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Tue Nov 1 19:35:12 2022 -0300 vkd3d-shader/hlsl: Validate that statics don't contain both resources and numerics. --- libs/vkd3d-shader/hlsl.y | 27 +++++++++++++++++++++++++++ tests/hlsl-invalid.shader_test | 14 ++++++++++++++ 2 files changed, 41 insertions(+) diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index 0e97d4d1..6461ade5 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -1907,6 +1907,26 @@ static bool type_has_object_components(struct hlsl_type *type, bool must_be_in_s return false; } +static bool type_has_numeric_components(struct hlsl_type *type) +{ + if (type->type <= HLSL_CLASS_LAST_NUMERIC) + return true; + if (type->type == HLSL_CLASS_ARRAY) + return type_has_numeric_components(type->e.array.type); + + if (type->type == HLSL_CLASS_STRUCT) + { + unsigned int i; + + for (i = 0; i < type->e.record.field_count; ++i) + { + if (type_has_numeric_components(type->e.record.fields[i].type)) + return true; + } + } + return false; +} + static struct list *declare_vars(struct hlsl_ctx *ctx, struct hlsl_type *basic_type, DWORD modifiers, struct list *var_list) { @@ -2076,6 +2096,13 @@ static struct list *declare_vars(struct hlsl_ctx *ctx, struct hlsl_type *basic_t "Semantics are not allowed on local variables."); } + if ((var->modifiers & HLSL_STORAGE_STATIC) && type_has_numeric_components(var->data_type) + && type_has_object_components(var->data_type, false)) + { + hlsl_error(ctx, &var->loc, VKD3D_SHADER_ERROR_HLSL_INVALID_TYPE, + "Static variables cannot have both numeric and resource components."); + } + if ((type->modifiers & HLSL_MODIFIER_CONST) && !v->initializer.args_count && !(modifiers & (HLSL_STORAGE_STATIC | HLSL_STORAGE_UNIFORM))) { diff --git a/tests/hlsl-invalid.shader_test b/tests/hlsl-invalid.shader_test index a7364b1e..1b39d7f0 100644 --- a/tests/hlsl-invalid.shader_test +++ b/tests/hlsl-invalid.shader_test @@ -250,3 +250,17 @@ float4 main() : sv_target a.a = 1; return a.a; } + +[pixel shader fail] +struct apple +{ + sampler sam; + float4 aa; +}; + +static struct apple a; + +float4 main() : sv_target +{ + return 1.0; +}

1 0

Francisco Casas : vkd3d-shader/hlsl: Use the base type of the array elements in write_sm1_type().
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: 90e6e418a38ac03ad8ca2045070d761d7ad9c84f URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/90e6e418a38ac03ad8ca2045070d7… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Mon Sep 5 18:06:44 2022 -0300 vkd3d-shader/hlsl: Use the base type of the array elements in write_sm1_type(). --- libs/vkd3d-shader/hlsl_sm1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/vkd3d-shader/hlsl_sm1.c b/libs/vkd3d-shader/hlsl_sm1.c index 2219ef56..ba22925e 100644 --- a/libs/vkd3d-shader/hlsl_sm1.c +++ b/libs/vkd3d-shader/hlsl_sm1.c @@ -272,7 +272,7 @@ static void write_sm1_type(struct vkd3d_bytecode_buffer *buffer, struct hlsl_typ } } - type->bytecode_offset = put_u32(buffer, vkd3d_make_u32(sm1_class(type), sm1_base_type(type))); + type->bytecode_offset = put_u32(buffer, vkd3d_make_u32(sm1_class(type), sm1_base_type(array_type))); put_u32(buffer, vkd3d_make_u32(type->dimy, type->dimx)); put_u32(buffer, vkd3d_make_u32(array_size, field_count)); put_u32(buffer, fields_offset);

1 0

Francisco Casas : vkd3d-shader/hlsl: Use reg_size as component count when allocating a single register.
by Alexandre Julliard 10 Nov '22

10 Nov '22

Module: vkd3d Branch: master Commit: f21693b28458f6198d93226fd808eda3cae04c48 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/f21693b28458f6198d93226fd808e… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Mon Oct 31 12:35:16 2022 -0300 vkd3d-shader/hlsl: Use reg_size as component count when allocating a single register. Otherwise, for instance, the added test results in: debug_hlsl_writemask: Assertion `!(writemask & ~VKD3DSP_WRITEMASK_ALL)' failed. Which happens in allocate_variable_temp_register() when the variable's type reg_size is <= 4 but its component count is larger, which may happen if it contains objects. --- libs/vkd3d-shader/hlsl_codegen.c | 8 ++++---- tests/object-references.shader_test | 25 +++++++++++++++++++++++++ 2 files changed, 29 insertions(+), 4 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 18c26771..58d44c4d 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -1927,7 +1927,7 @@ static void allocate_variable_temp_register(struct hlsl_ctx *ctx, struct hlsl_ir var->last_read, var->data_type->reg_size); else var->reg = allocate_register(ctx, liveness, var->first_write, - var->last_read, hlsl_type_component_count(var->data_type)); + var->last_read, var->data_type->reg_size); TRACE("Allocated %s to %s (liveness %u-%u).\n", var->name, debug_register('r', var->reg, var->data_type), var->first_write, var->last_read); } @@ -1946,7 +1946,7 @@ static void allocate_temp_registers_recurse(struct hlsl_ctx *ctx, struct hlsl_bl instr->last_read, instr->data_type->reg_size); else instr->reg = allocate_register(ctx, liveness, instr->index, - instr->last_read, instr->data_type->dimx); + instr->last_read, instr->data_type->reg_size); TRACE("Allocated anonymous expression @%u to %s (liveness %u-%u).\n", instr->index, debug_register('r', instr->reg, instr->data_type), instr->index, instr->last_read); } @@ -2009,7 +2009,7 @@ static void allocate_const_registers_recurse(struct hlsl_ctx *ctx, struct hlsl_b if (reg_size > 4) constant->reg = allocate_range(ctx, liveness, 1, UINT_MAX, reg_size); else - constant->reg = allocate_register(ctx, liveness, 1, UINT_MAX, type->dimx); + constant->reg = allocate_register(ctx, liveness, 1, UINT_MAX, reg_size); TRACE("Allocated constant @%u to %s.\n", instr->index, debug_register('c', constant->reg, type)); if (!hlsl_array_reserve(ctx, (void **)&defs->values, &defs->size, @@ -2111,7 +2111,7 @@ static void allocate_const_registers(struct hlsl_ctx *ctx, struct hlsl_ir_functi else { var->reg = allocate_register(ctx, &liveness, 1, UINT_MAX, 4); - var->reg.writemask = (1u << var->data_type->dimx) - 1; + var->reg.writemask = (1u << var->data_type->reg_size) - 1; } TRACE("Allocated %s to %s.\n", var->name, debug_register('c', var->reg, var->data_type)); } diff --git a/tests/object-references.shader_test b/tests/object-references.shader_test index f3fda472..a33bd14a 100644 --- a/tests/object-references.shader_test +++ b/tests/object-references.shader_test @@ -118,3 +118,28 @@ float4 main() : sv_target { return tex[n].Load(0); } + + +[pixel shader todo] +Texture2D tex; +uniform float f; + +struct apple +{ + Texture2D tex1; + Texture2D tex2; + float3 aa; +}; + +float4 main() : sv_target +{ + struct apple a = {tex, tex, 1.0, 2.0, 3.0}; + + a.aa += f; + return a.aa.xyzx; +} + +[test] +uniform 0 float 10.0 +todo draw quad +todo probe (0, 0) rgba (11.0, 12.0, 13.0, 11.0)

1 0

Zebediah Figura : d3d8: Build without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard 09 Nov '22

09 Nov '22

Module: wine Branch: master Commit: 52c35814bece03b8668a699b87d856af19ba94c3 URL: https://gitlab.winehq.org/wine/wine/-/commit/52c35814bece03b8668a699b87d856… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sat Sep 17 23:32:00 2022 -0500 d3d8: Build without -DWINE_NO_LONG_TYPES. --- dlls/d3d8/Makefile.in | 1 - dlls/d3d8/buffer.c | 24 +++--- dlls/d3d8/device.c | 186 +++++++++++++++++++++--------------------- dlls/d3d8/directx.c | 16 ++-- dlls/d3d8/shader.c | 12 +-- dlls/d3d8/surface.c | 10 +-- dlls/d3d8/swapchain.c | 8 +- dlls/d3d8/texture.c | 34 ++++---- dlls/d3d8/vertexdeclaration.c | 28 +++---- dlls/d3d8/volume.c | 5 +- 10 files changed, 159 insertions(+), 165 deletions(-)

1 0

Zebediah Figura : d3d9: Build without -DWINE_NO_LONG_TYPES.
by Alexandre Julliard 09 Nov '22

09 Nov '22

Module: wine Branch: master Commit: 9b3742bebb52252d9a3bfec35d17808dda2b641b URL: https://gitlab.winehq.org/wine/wine/-/commit/9b3742bebb52252d9a3bfec35d1780… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Sun Sep 18 22:57:13 2022 -0500 d3d9: Build without -DWINE_NO_LONG_TYPES. --- dlls/d3d9/Makefile.in | 1 - dlls/d3d9/buffer.c | 24 +++---- dlls/d3d9/d3d9_main.c | 12 ++-- dlls/d3d9/device.c | 142 +++++++++++++++++++++--------------------- dlls/d3d9/directx.c | 24 +++---- dlls/d3d9/query.c | 11 ++-- dlls/d3d9/shader.c | 12 ++-- dlls/d3d9/stateblock.c | 6 +- dlls/d3d9/surface.c | 12 ++-- dlls/d3d9/swapchain.c | 10 +-- dlls/d3d9/texture.c | 36 +++++------ dlls/d3d9/vertexdeclaration.c | 12 ++-- dlls/d3d9/volume.c | 4 +- 13 files changed, 152 insertions(+), 154 deletions(-)

1 0

Torge Matthies : ntdll: Fix incorrectly sized mov in user_mode_callback_return.
by Alexandre Julliard 09 Nov '22

09 Nov '22

Module: wine Branch: master Commit: efc45c5e3b49932989b9d061d0a60f504ec7fa6e URL: https://gitlab.winehq.org/wine/wine/-/commit/efc45c5e3b49932989b9d061d0a60f… Author: Torge Matthies <tmatthies(a)codeweavers.com> Date: Wed Nov 9 08:06:28 2022 +0100 ntdll: Fix incorrectly sized mov in user_mode_callback_return. Signed-off-by: Torge Matthies <tmatthies(a)codeweavers.com> --- dlls/ntdll/unix/signal_x86_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/signal_x86_64.c b/dlls/ntdll/unix/signal_x86_64.c index 58029602130..263bb64c8cc 100644 --- a/dlls/ntdll/unix/signal_x86_64.c +++ b/dlls/ntdll/unix/signal_x86_64.c @@ -1647,7 +1647,7 @@ __ASM_GLOBAL_FUNC( user_mode_callback_return, "movq 0x10(%rbp),%rsi\n\t" /* ret_ptr */ "movq 0x18(%rbp),%rdi\n\t" /* ret_len */ "movq %rcx,(%rsi)\n\t" - "movq %rdx,(%rdi)\n\t" + "movl %edx,(%rdi)\n\t" "movdqa -0xe0(%rbp),%xmm15\n\t" "movdqa -0xd0(%rbp),%xmm14\n\t" "movdqa -0xc0(%rbp),%xmm13\n\t"

1 0

Martin Storsjö : ntdll: Write ret_len with the right size in user_mode_callback_return on arm64.
by Alexandre Julliard 09 Nov '22

09 Nov '22

Module: wine Branch: master Commit: 464d3c86dcb1bd057bde84765e4b2ca1aafaec23 URL: https://gitlab.winehq.org/wine/wine/-/commit/464d3c86dcb1bd057bde84765e4b2c… Author: Martin Storsjö <martin(a)martin.st> Date: Wed Nov 9 19:41:36 2022 +0200 ntdll: Write ret_len with the right size in user_mode_callback_return on arm64. This fixes a regression since 4069a8b384c15a90a7af66636ec5650eeb53b391. Signed-off-by: Martin Storsjö <martin(a)martin.st> --- dlls/ntdll/unix/signal_arm64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/unix/signal_arm64.c b/dlls/ntdll/unix/signal_arm64.c index bc8225d7ec8..7bce856d616 100644 --- a/dlls/ntdll/unix/signal_arm64.c +++ b/dlls/ntdll/unix/signal_arm64.c @@ -940,7 +940,7 @@ __ASM_GLOBAL_FUNC( user_mode_callback_return, "ldp d14, d15, [x29, #0x90]\n\t" "ldp x5, x6, [x29, #0xa0]\n\t" /* ret_ptr, ret_len */ "str x0, [x5]\n\t" /* ret_ptr */ - "str x1, [x6]\n\t" /* ret_len */ + "str w1, [x6]\n\t" /* ret_len */ "mov x0, x2\n\t" /* status */ "mov sp, x29\n\t" "ldp x29, x30, [sp], #0xc0\n\t"

1 0