- Wine-Commits - list.winehq.org

Francisco Casas : vkd3d-shader/hlsl: Avoid invalid input/output copies for non-numeric types.
by Alexandre Julliard May 1, 2023

May 1, 2023

Module: vkd3d Branch: master Commit: 61c72a4fd1a2aaf7b688b27f81dd09790b8dcc69 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/61c72a4fd1a2aaf7b688b27f81dd0… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Wed Apr 5 12:45:33 2023 -0400 vkd3d-shader/hlsl: Avoid invalid input/output copies for non-numeric types. --- libs/vkd3d-shader/hlsl_codegen.c | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 42f8ab3b..314ef4ff 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -263,6 +263,16 @@ static void prepend_input_copy(struct hlsl_ctx *ctx, struct list *instrs, struct struct hlsl_ir_var *var = lhs->src.var; unsigned int i; + if (type->class > HLSL_CLASS_LAST_NUMERIC) + { + struct vkd3d_string_buffer *string; + if (!(string = hlsl_type_to_string(ctx, type))) + return; + hlsl_fixme(ctx, &var->loc, "Input semantics for type %s.", string->buffer); + hlsl_release_string_buffer(ctx, string); + return; + } + vector_type = hlsl_get_vector_type(ctx, type->base_type, hlsl_type_minor_size(type)); for (i = 0; i < hlsl_type_major_size(type); ++i) @@ -358,6 +368,16 @@ static void append_output_copy(struct hlsl_ctx *ctx, struct list *instrs, struct struct hlsl_ir_var *var = rhs->src.var; unsigned int i; + if (type->class > HLSL_CLASS_LAST_NUMERIC) + { + struct vkd3d_string_buffer *string; + if (!(string = hlsl_type_to_string(ctx, type))) + return; + hlsl_fixme(ctx, &var->loc, "Output semantics for type %s.", string->buffer); + hlsl_release_string_buffer(ctx, string); + return; + } + vector_type = hlsl_get_vector_type(ctx, type->base_type, hlsl_type_minor_size(type)); for (i = 0; i < hlsl_type_major_size(type); ++i)

1 0

Francisco Casas : tests: Allow invalid vertex shader tests.
by Alexandre Julliard May 1, 2023

May 1, 2023

Module: vkd3d Branch: master Commit: 9cc1c7fe9d834ad080a606f954db2cce41464863 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/9cc1c7fe9d834ad080a606f954db2… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Tue Apr 4 19:23:26 2023 -0400 tests: Allow invalid vertex shader tests. --- tests/shader_runner.c | 21 ++++++++++++++++++++- 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/tests/shader_runner.c b/tests/shader_runner.c index 961d2fe0..214b1213 100644 --- a/tests/shader_runner.c +++ b/tests/shader_runner.c @@ -87,6 +87,7 @@ enum parse_state STATE_SHADER_PIXEL, STATE_SHADER_PIXEL_TODO, STATE_SHADER_VERTEX, + STATE_SHADER_VERTEX_TODO, STATE_TEST, }; @@ -824,7 +825,9 @@ void run_shader_tests(struct shader_runner *runner, const struct shader_runner_o break; case STATE_SHADER_VERTEX: - compile_shader(runner, shader_source, shader_source_len, "vs", expect_hr); + case STATE_SHADER_VERTEX_TODO: + todo_if (state == STATE_SHADER_VERTEX_TODO) + compile_shader(runner, shader_source, shader_source_len, "vs", expect_hr); free(runner->vs_source); runner->vs_source = shader_source; shader_source = NULL; @@ -1031,6 +1034,21 @@ void run_shader_tests(struct shader_runner *runner, const struct shader_runner_o state = STATE_SHADER_VERTEX; expect_hr = S_OK; } + else if (!strcmp(line, "[vertex shader todo]\n")) + { + state = STATE_SHADER_VERTEX_TODO; + expect_hr = S_OK; + } + else if (!strcmp(line, "[vertex shader fail]\n")) + { + state = STATE_SHADER_VERTEX; + expect_hr = E_FAIL; + } + else if (!strcmp(line, "[vertex shader fail todo]\n")) + { + state = STATE_SHADER_VERTEX_TODO; + expect_hr = E_FAIL; + } else if (!strcmp(line, "[input layout]\n")) { state = STATE_INPUT_LAYOUT; @@ -1061,6 +1079,7 @@ void run_shader_tests(struct shader_runner *runner, const struct shader_runner_o case STATE_SHADER_PIXEL: case STATE_SHADER_PIXEL_TODO: case STATE_SHADER_VERTEX: + case STATE_SHADER_VERTEX_TODO: { size_t len = strlen(line);

1 0

Francisco Casas : tests: Expect S_OK result on [vertex shader].
by Alexandre Julliard May 1, 2023

May 1, 2023

Module: vkd3d Branch: master Commit: 877fd3f0b449838ac79e39509abc1619f5723a1d URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/877fd3f0b449838ac79e39509abc1… Author: Francisco Casas <fcasas(a)codeweavers.com> Date: Tue Apr 4 18:59:43 2023 -0400 tests: Expect S_OK result on [vertex shader]. --- tests/shader_runner.c | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/shader_runner.c b/tests/shader_runner.c index 8e66948e..961d2fe0 100644 --- a/tests/shader_runner.c +++ b/tests/shader_runner.c @@ -1029,6 +1029,7 @@ void run_shader_tests(struct shader_runner *runner, const struct shader_runner_o else if (!strcmp(line, "[vertex shader]\n")) { state = STATE_SHADER_VERTEX; + expect_hr = S_OK; } else if (!strcmp(line, "[input layout]\n")) {

1 0

Alexandre Julliard : Wine release 8.7
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: website Branch: master Commit: f880748eff7e65ee7a2c05cece463590ee51ea99 URL: https://gitlab.winehq.org/winehq/winehq/-/commit/f880748eff7e65ee7a2c05cece… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 28 22:08:51 2023 +0200 Wine release 8.7 Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- news/en/2023042801.xml | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/news/en/2023042801.xml b/news/en/2023042801.xml new file mode 100644 index 00000000..f56b3074 --- /dev/null +++ b/news/en/2023042801.xml @@ -0,0 +1,15 @@ +<news> +<date>April 28, 2023</date> +<title>Wine 8.7 Released</title> +<body> + The Wine development release 8.7 is now available. + <a href="{$root}/announce/8.7">What's new</a> in this release: +<ul> + <li>DXBC shader parsing delegated to vkd3d.</li> + <li>More spool file improvements in the PostScript driver.</li> + <li>Various bug fixes.</li> +</ul> + +The source is <a href="//dl.winehq.org/wine/source/8.x/wine-8.7.tar.xz">available now</a>. +Binary packages are in the process of being built, and will appear soon at their respective <a href="{$root}/download">download locations</a>. +</body></news>

1 0

Tag wine-8.7 : Alexandre Julliard : Release 8.7
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: wine Branch: refs/tags/wine-8.7 Tag: 1608d8c273f9ed956f4a8aca6ba4fd2d004f3a7d URL: https://gitlab.winehq.org/wine/wine/-/tags/1608d8c273f9ed956f4a8aca6ba4fd2d… Tagger: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 28 22:06:24 2023 +0200 Release 8.7

1 0

Nikolay Sivov : vkd3d-shader/hlsl: Implement D3DCOLORtoUBYTE4() function.
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: vkd3d Branch: master Commit: 7ba373946bed1fa63cade358f776a29773c2ddf0 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/7ba373946bed1fa63cade358f776a… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Apr 21 07:49:29 2023 +0200 vkd3d-shader/hlsl: Implement D3DCOLORtoUBYTE4() function. --- Makefile.am | 1 + libs/vkd3d-shader/hlsl.y | 46 +++++++++++++++++++++++++++++++ tests/hlsl-d3dcolor-to-ubyte4.shader_test | 24 ++++++++++++++++ 3 files changed, 71 insertions(+) diff --git a/Makefile.am b/Makefile.am index df25d1b9..b9fd06ee 100644 --- a/Makefile.am +++ b/Makefile.am @@ -77,6 +77,7 @@ vkd3d_shader_tests = \ tests/hlsl-clamp.shader_test \ tests/hlsl-comma.shader_test \ tests/hlsl-cross.shader_test \ + tests/hlsl-d3dcolor-to-ubyte4.shader_test \ tests/hlsl-discard.shader_test \ tests/hlsl-dot.shader_test \ tests/hlsl-duplicate-modifiers.shader_test \ diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index f96897cb..d516f3b5 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -3395,6 +3395,51 @@ static bool intrinsic_trunc(struct hlsl_ctx *ctx, return !!add_unary_arithmetic_expr(ctx, params->instrs, HLSL_OP1_TRUNC, arg, loc); } +static bool intrinsic_d3dcolor_to_ubyte4(struct hlsl_ctx *ctx, + const struct parse_initializer *params, const struct vkd3d_shader_location *loc) +{ + struct hlsl_ir_node *arg = params->args[0], *ret, *c; + struct hlsl_type *arg_type = arg->data_type; + struct hlsl_ir_swizzle *swizzle; + + if (arg_type->class != HLSL_CLASS_SCALAR && !(arg_type->class == HLSL_CLASS_VECTOR && arg_type->dimx == 4)) + { + struct vkd3d_string_buffer *string; + + if ((string = hlsl_type_to_string(ctx, arg_type))) + { + hlsl_error(ctx, loc, VKD3D_SHADER_ERROR_HLSL_INVALID_TYPE, "Wrong argument type '%s'.", string->buffer); + hlsl_release_string_buffer(ctx, string); + } + + return false; + } + + if (!(arg = intrinsic_float_convert_arg(ctx, params, arg, loc))) + return false; + + if (!(c = hlsl_new_float_constant(ctx, 255.0f + (0.5f / 256.0f), loc))) + return false; + list_add_tail(params->instrs, &c->entry); + + if (arg_type->class == HLSL_CLASS_VECTOR) + { + if (!(swizzle = hlsl_new_swizzle(ctx, HLSL_SWIZZLE(Z, Y, X, W), 4, arg, loc))) + return false; + list_add_tail(params->instrs, &swizzle->node.entry); + + arg = &swizzle->node; + } + + if (!(ret = add_binary_arithmetic_expr(ctx, params->instrs, HLSL_OP2_MUL, arg, c, loc))) + return false; + + if (ctx->profile->major_version >= 4) + return !!add_unary_arithmetic_expr(ctx, params->instrs, HLSL_OP1_TRUNC, ret, loc); + + return true; +} + static const struct intrinsic_function { const char *name; @@ -3406,6 +3451,7 @@ static const struct intrinsic_function intrinsic_functions[] = { /* Note: these entries should be kept in alphabetical order. */ + {"D3DCOLORtoUBYTE4", 1, true, intrinsic_d3dcolor_to_ubyte4}, {"abs", 1, true, intrinsic_abs}, {"all", 1, true, intrinsic_all}, {"any", 1, true, intrinsic_any}, diff --git a/tests/hlsl-d3dcolor-to-ubyte4.shader_test b/tests/hlsl-d3dcolor-to-ubyte4.shader_test new file mode 100644 index 00000000..e31ef61c --- /dev/null +++ b/tests/hlsl-d3dcolor-to-ubyte4.shader_test @@ -0,0 +1,24 @@ +[require] +shader model >= 4.0 + +[pixel shader] +float4 main(uniform float4 u) : sv_target +{ + return D3DCOLORtoUBYTE4(u); +} + +[test] +uniform 0 float4 -0.5 6.5 7.5 3.4 +draw quad +probe all rgba (1912.0, 1657.0, -127.0, 867.0) 1 + +[pixel shader] +float4 main(uniform float4 u) : sv_target +{ + return D3DCOLORtoUBYTE4(u.x); +} + +[test] +uniform 0 float4 -0.5 6.5 7.5 3.4 +draw quad +probe all rgba (-127.0, -127.0, -127.0, -127.0) 1

1 0

Conor McCarthy : vkd3d: Synchronise concurrent descriptor heap binding by multiple command lists.
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: vkd3d Branch: master Commit: 5366ca7001b51cca2d4ee485435b667b62d983ba URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/5366ca7001b51cca2d4ee485435b6… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Thu Apr 27 15:05:58 2023 +1000 vkd3d: Synchronise concurrent descriptor heap binding by multiple command lists. It is possible for multiple command lists to use the same heap, and submit it simultaneously to multiple d3d12 queues. --- libs/vkd3d/command.c | 10 +++++++++- libs/vkd3d/resource.c | 4 +++- libs/vkd3d/vkd3d_private.h | 3 ++- 3 files changed, 14 insertions(+), 3 deletions(-) diff --git a/libs/vkd3d/command.c b/libs/vkd3d/command.c index 094466d2..81ddccb4 100644 --- a/libs/vkd3d/command.c +++ b/libs/vkd3d/command.c @@ -3176,7 +3176,11 @@ static void command_list_flush_vk_heap_updates(struct d3d12_command_list *list) unsigned int i; for (i = 0; i < list->descriptor_heap_count; ++i) - d3d12_desc_flush_vk_heap_updates(list->descriptor_heaps[i], device); + { + vkd3d_mutex_lock(&list->descriptor_heaps[i]->vk_sets_mutex); + d3d12_desc_flush_vk_heap_updates_locked(list->descriptor_heaps[i], device); + vkd3d_mutex_unlock(&list->descriptor_heaps[i]->vk_sets_mutex); + } list->descriptor_heap_count = 0; } @@ -3215,6 +3219,8 @@ static void d3d12_command_list_bind_descriptor_heap(struct d3d12_command_list *l list->descriptor_heaps[list->descriptor_heap_count++] = heap; } + vkd3d_mutex_lock(&heap->vk_sets_mutex); + for (set = 0; set < ARRAY_SIZE(heap->vk_descriptor_sets); ++set) { VkDescriptorSet vk_descriptor_set = heap->vk_descriptor_sets[set].vk_set; @@ -3225,6 +3231,8 @@ static void d3d12_command_list_bind_descriptor_heap(struct d3d12_command_list *l VK_CALL(vkCmdBindDescriptorSets(list->vk_command_buffer, bindings->vk_bind_point, rs->vk_pipeline_layout, rs->vk_set_count + set, 1, &vk_descriptor_set, 0, NULL)); } + + vkd3d_mutex_unlock(&heap->vk_sets_mutex); } static void d3d12_command_list_update_heap_descriptors(struct d3d12_command_list *list, diff --git a/libs/vkd3d/resource.c b/libs/vkd3d/resource.c index d12191a9..213b2dbb 100644 --- a/libs/vkd3d/resource.c +++ b/libs/vkd3d/resource.c @@ -2323,7 +2323,7 @@ static void d3d12_desc_write_vk_heap(struct d3d12_descriptor_heap *descriptor_he writes->count = i; } -void d3d12_desc_flush_vk_heap_updates(struct d3d12_descriptor_heap *descriptor_heap, struct d3d12_device *device) +void d3d12_desc_flush_vk_heap_updates_locked(struct d3d12_descriptor_heap *descriptor_heap, struct d3d12_device *device) { const struct vkd3d_vk_device_procs *vk_procs = &device->vk_procs; struct d3d12_desc *descriptors, *src; @@ -3724,6 +3724,7 @@ static ULONG STDMETHODCALLTYPE d3d12_descriptor_heap_Release(ID3D12DescriptorHea } VK_CALL(vkDestroyDescriptorPool(device->vk_device, heap->vk_descriptor_pool, NULL)); + vkd3d_mutex_destroy(&heap->vk_sets_mutex); vkd3d_free(heap); @@ -3946,6 +3947,7 @@ static HRESULT d3d12_descriptor_heap_init(struct d3d12_descriptor_heap *descript return hr; d3d12_descriptor_heap_vk_descriptor_sets_init(descriptor_heap, device, desc); + vkd3d_mutex_init(&descriptor_heap->vk_sets_mutex); d3d12_device_add_ref(descriptor_heap->device = device); diff --git a/libs/vkd3d/vkd3d_private.h b/libs/vkd3d/vkd3d_private.h index 0af643b0..79474414 100644 --- a/libs/vkd3d/vkd3d_private.h +++ b/libs/vkd3d/vkd3d_private.h @@ -988,13 +988,14 @@ struct d3d12_descriptor_heap VkDescriptorPool vk_descriptor_pool; struct d3d12_descriptor_heap_vk_set vk_descriptor_sets[VKD3D_SET_INDEX_COUNT]; + struct vkd3d_mutex vk_sets_mutex; unsigned int volatile dirty_list_head; uint8_t DECLSPEC_ALIGN(sizeof(void *)) descriptors[]; }; -void d3d12_desc_flush_vk_heap_updates(struct d3d12_descriptor_heap *descriptor_heap, struct d3d12_device *device); +void d3d12_desc_flush_vk_heap_updates_locked(struct d3d12_descriptor_heap *descriptor_heap, struct d3d12_device *device); static inline struct d3d12_descriptor_heap *d3d12_desc_get_descriptor_heap(const struct d3d12_desc *descriptor) {

1 0

Nikolay Sivov : vkd3d-shader/hlsl: Partially implement trunc().
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: vkd3d Branch: master Commit: 4b3707aeb41e6a7ee5fdef7545abe57dd81ac6a6 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/4b3707aeb41e6a7ee5fdef7545abe… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Fri Apr 14 23:00:57 2023 +0200 vkd3d-shader/hlsl: Partially implement trunc(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- Makefile.am | 1 + libs/vkd3d-shader/hlsl.c | 1 + libs/vkd3d-shader/hlsl.h | 1 + libs/vkd3d-shader/hlsl.y | 12 ++++++++++++ libs/vkd3d-shader/tpf.c | 5 +++++ tests/hlsl-trunc.shader_test | 44 ++++++++++++++++++++++++++++++++++++++++++++ 6 files changed, 64 insertions(+) diff --git a/Makefile.am b/Makefile.am index 8e94658d..df25d1b9 100644 --- a/Makefile.am +++ b/Makefile.am @@ -123,6 +123,7 @@ vkd3d_shader_tests = \ tests/hlsl-struct-semantics.shader_test \ tests/hlsl-ternary.shader_test \ tests/hlsl-transpose.shader_test \ + tests/hlsl-trunc.shader_test \ tests/hlsl-type-names.shader_test \ tests/hlsl-vector-indexing.shader_test \ tests/hlsl-vector-indexing-uniform.shader_test \ diff --git a/libs/vkd3d-shader/hlsl.c b/libs/vkd3d-shader/hlsl.c index f0b6a5ef..19656431 100644 --- a/libs/vkd3d-shader/hlsl.c +++ b/libs/vkd3d-shader/hlsl.c @@ -2295,6 +2295,7 @@ const char *debug_hlsl_expr_op(enum hlsl_ir_expr_op op) [HLSL_OP1_SIN] = "sin", [HLSL_OP1_SIN_REDUCED] = "sin_reduced", [HLSL_OP1_SQRT] = "sqrt", + [HLSL_OP1_TRUNC] = "trunc", [HLSL_OP2_ADD] = "+", [HLSL_OP2_BIT_AND] = "&", diff --git a/libs/vkd3d-shader/hlsl.h b/libs/vkd3d-shader/hlsl.h index 2ff18536..2e9d3f5e 100644 --- a/libs/vkd3d-shader/hlsl.h +++ b/libs/vkd3d-shader/hlsl.h @@ -492,6 +492,7 @@ enum hlsl_ir_expr_op HLSL_OP1_SIN, HLSL_OP1_SIN_REDUCED, /* Reduced range [-pi, pi] */ HLSL_OP1_SQRT, + HLSL_OP1_TRUNC, HLSL_OP2_ADD, HLSL_OP2_BIT_AND, diff --git a/libs/vkd3d-shader/hlsl.y b/libs/vkd3d-shader/hlsl.y index 6c3bca34..f96897cb 100644 --- a/libs/vkd3d-shader/hlsl.y +++ b/libs/vkd3d-shader/hlsl.y @@ -3384,6 +3384,17 @@ static bool intrinsic_transpose(struct hlsl_ctx *ctx, return true; } +static bool intrinsic_trunc(struct hlsl_ctx *ctx, + const struct parse_initializer *params, const struct vkd3d_shader_location *loc) +{ + struct hlsl_ir_node *arg; + + if (!(arg = intrinsic_float_convert_arg(ctx, params, params->args[0], loc))) + return false; + + return !!add_unary_arithmetic_expr(ctx, params->instrs, HLSL_OP1_TRUNC, arg, loc); +} + static const struct intrinsic_function { const char *name; @@ -3433,6 +3444,7 @@ intrinsic_functions[] = {"tex2D", -1, false, intrinsic_tex2D}, {"tex3D", -1, false, intrinsic_tex3D}, {"transpose", 1, true, intrinsic_transpose}, + {"trunc", 1, true, intrinsic_trunc}, }; static int intrinsic_function_name_compare(const void *a, const void *b) diff --git a/libs/vkd3d-shader/tpf.c b/libs/vkd3d-shader/tpf.c index 0908fba3..3debe952 100644 --- a/libs/vkd3d-shader/tpf.c +++ b/libs/vkd3d-shader/tpf.c @@ -4031,6 +4031,11 @@ static void write_sm4_expr(struct hlsl_ctx *ctx, write_sm4_unary_op(buffer, VKD3D_SM4_OP_SQRT, &expr->node, arg1, 0); break; + case HLSL_OP1_TRUNC: + assert(type_is_float(dst_type)); + write_sm4_unary_op(buffer, VKD3D_SM4_OP_ROUND_Z, &expr->node, arg1, 0); + break; + case HLSL_OP2_ADD: switch (dst_type->base_type) { diff --git a/tests/hlsl-trunc.shader_test b/tests/hlsl-trunc.shader_test new file mode 100644 index 00000000..419608fb --- /dev/null +++ b/tests/hlsl-trunc.shader_test @@ -0,0 +1,44 @@ +[pixel shader] +float4 main(uniform float4 u) : sv_target +{ + return trunc(u); +} + +[test] +uniform 0 float4 -0.5 6.5 7.5 3.4 +draw quad +probe all rgba (0.0, 6.0, 7.0, 3.0) +uniform 0 float4 -1.5 6.5 7.5 3.4 +draw quad +probe all rgba (-1.0, 6.0, 7.0, 3.0) + +[pixel shader] +float4 main(uniform float4 u) : sv_target +{ + float a = trunc(u.r); + int2 b = trunc(u.gb); + float4 res = float4(b, a, u.a); + return trunc(res); +} + +[test] +uniform 0 float4 -0.5 6.5 7.5 3.4 +draw quad +probe all rgba (6.0, 7.0, 0.0, 3.0) + +[require] +shader model >= 4.0 + +[pixel shader] +float4 main(uniform int4 u) : sv_target +{ + float a = trunc(u.r); + int2 b = trunc(u.gb); + float4 res = float4(b, a, u.a); + return trunc(res); +} + +[test] +uniform 0 int4 -1 6 7 3 +draw quad +probe all rgba (6.0, 7.0, -1.0, 3.0)

1 0

Conor McCarthy : vkd3d: Track all descriptor heaps bound during command list recording and flush their writes.
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: vkd3d Branch: master Commit: fa63da6030cce4d7c1e1a5b255d495264a8f8695 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/fa63da6030cce4d7c1e1a5b255d49… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Thu Apr 27 14:59:15 2023 +1000 vkd3d: Track all descriptor heaps bound during command list recording and flush their writes. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54895 --- libs/vkd3d/command.c | 75 +++++++++++++++++++++------------------------- libs/vkd3d/vkd3d_private.h | 2 ++ 2 files changed, 36 insertions(+), 41 deletions(-) diff --git a/libs/vkd3d/command.c b/libs/vkd3d/command.c index fb6b40b2..094466d2 100644 --- a/libs/vkd3d/command.c +++ b/libs/vkd3d/command.c @@ -3159,6 +3159,27 @@ static void d3d12_command_list_update_descriptor_tables(struct d3d12_command_lis } } +static bool contains_heap(struct d3d12_descriptor_heap **heap_array, unsigned int count, + const struct d3d12_descriptor_heap *query) +{ + unsigned int i; + + for (i = 0; i < count; ++i) + if (heap_array[i] == query) + return true; + return false; +} + +static void command_list_flush_vk_heap_updates(struct d3d12_command_list *list) +{ + struct d3d12_device *device = list->device; + unsigned int i; + + for (i = 0; i < list->descriptor_heap_count; ++i) + d3d12_desc_flush_vk_heap_updates(list->descriptor_heaps[i], device); + list->descriptor_heap_count = 0; +} + static void d3d12_command_list_bind_descriptor_heap(struct d3d12_command_list *list, enum vkd3d_pipeline_bind_point bind_point, struct d3d12_descriptor_heap *heap) { @@ -3183,6 +3204,17 @@ static void d3d12_command_list_bind_descriptor_heap(struct d3d12_command_list *l bindings->sampler_heap_id = heap->serial_id; } + if (!contains_heap(list->descriptor_heaps, list->descriptor_heap_count, heap)) + { + if (list->descriptor_heap_count == ARRAY_SIZE(list->descriptor_heaps)) + { + /* Descriptors can be written after binding. */ + FIXME("Flushing descriptor updates while list %p is not closed.\n", list); + command_list_flush_vk_heap_updates(list); + } + list->descriptor_heaps[list->descriptor_heap_count++] = heap; + } + for (set = 0; set < ARRAY_SIZE(heap->vk_descriptor_sets); ++set) { VkDescriptorSet vk_descriptor_set = heap->vk_descriptor_sets[set].vk_set; @@ -5927,6 +5959,7 @@ static HRESULT d3d12_command_list_init(struct d3d12_command_list *list, struct d list->update_descriptors = device->use_vk_heaps ? d3d12_command_list_update_heap_descriptors : d3d12_command_list_update_descriptors; + list->descriptor_heap_count = 0; if (SUCCEEDED(hr = d3d12_command_allocator_allocate_command_buffer(allocator, list))) { @@ -6187,44 +6220,6 @@ static void d3d12_command_queue_submit_locked(struct d3d12_command_queue *queue) } } -static bool contains_heap(const struct d3d12_descriptor_heap **heap_array, unsigned int count, - const struct d3d12_descriptor_heap *query) -{ - unsigned int i; - - for (i = 0; i < count; ++i) - if (heap_array[i] == query) - return true; - return false; -} - -static void pipeline_bindings_flush_vk_heap_updates(struct vkd3d_pipeline_bindings *bindings, - struct d3d12_device *device) -{ - /* Only two heaps are strictly allowed, but more could be supported with a hack. */ - const struct d3d12_descriptor_heap *heap_array[3]; - struct d3d12_descriptor_heap *descriptor_heap; - unsigned int i, count; - uint64_t mask; - - mask = bindings->descriptor_table_active_mask & bindings->root_signature->descriptor_table_mask; - - for (i = 0, count = 0; i < ARRAY_SIZE(bindings->descriptor_tables); ++i) - { - if (!(mask & (1ull << i)) || !bindings->descriptor_tables[i]) - continue; - - descriptor_heap = d3d12_desc_get_descriptor_heap(bindings->descriptor_tables[i]); - /* Another thread could be writing unused descriptors, so try to check each heap only once. Flushing - * any updates added after the first flush will only delay execution of the command list. */ - if (contains_heap(heap_array, count, descriptor_heap)) - continue; - if (count < ARRAY_SIZE(heap_array)) - heap_array[count++] = descriptor_heap; - d3d12_desc_flush_vk_heap_updates(descriptor_heap, device); - } -} - static void STDMETHODCALLTYPE d3d12_command_queue_ExecuteCommandLists(ID3D12CommandQueue *iface, UINT command_list_count, ID3D12CommandList * const *command_lists) { @@ -6258,9 +6253,7 @@ static void STDMETHODCALLTYPE d3d12_command_queue_ExecuteCommandLists(ID3D12Comm return; } - if (cmd_list->state) - pipeline_bindings_flush_vk_heap_updates(&cmd_list->pipeline_bindings[cmd_list->state->vk_bind_point], - cmd_list->device); + command_list_flush_vk_heap_updates(cmd_list); buffers[i] = cmd_list->vk_command_buffer; } diff --git a/libs/vkd3d/vkd3d_private.h b/libs/vkd3d/vkd3d_private.h index 88dc5772..0af643b0 100644 --- a/libs/vkd3d/vkd3d_private.h +++ b/libs/vkd3d/vkd3d_private.h @@ -1407,6 +1407,8 @@ struct d3d12_command_list VkDeviceSize so_counter_buffer_offsets[D3D12_SO_BUFFER_SLOT_COUNT]; void (*update_descriptors)(struct d3d12_command_list *list, enum vkd3d_pipeline_bind_point bind_point); + struct d3d12_descriptor_heap *descriptor_heaps[64]; + unsigned int descriptor_heap_count; struct vkd3d_private_store private_store; };

1 0

Connor McAdams : uiautomationcore: Implement IUIAutomation::GetFocusedElement{BuildCacheRequest}.
by Alexandre Julliard April 28, 2023

April 28, 2023

Module: wine Branch: master Commit: 5bd409070556980edb2683c6a1d99beee7cb7cec URL: https://gitlab.winehq.org/wine/wine/-/commit/5bd409070556980edb2683c6a1d99b… Author: Connor McAdams <cmcadams(a)codeweavers.com> Date: Fri Mar 24 13:14:05 2023 -0400 uiautomationcore: Implement IUIAutomation::GetFocusedElement{BuildCacheRequest}. Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/uiautomationcore/tests/uiautomation.c | 208 +++++++++++++++++++++++++++++ dlls/uiautomationcore/uia_com_client.c | 60 ++++++++- 2 files changed, 264 insertions(+), 4 deletions(-)

1 0