November 2021 - Wine-Devel - list.winehq.org

Re: [PATCH vkd3d] vkd3d-shader/hlsl: Unify texture and sampler allocation functions.

by Matteo Bruni

On Tue, Nov 9, 2021 at 1:54 PM Matteo Bruni <mbruni(a)codeweavers.com> wrote: > Based on top of 219079. Eh, not quite, there is also 219117 in play here. I mostly wanted to show the idea, but let me resend something easier to apply...

Nov. 9, 2021

[PATCH 5/5] win32u: Move NtUserCountClipboardFormats implementation from user32.

by Jacek Caban

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/clipboard.c | 23 +-------------- dlls/user32/user32.spec | 2 +- dlls/win32u/Makefile.in | 1 + dlls/win32u/clipboard.c | 55 ++++++++++++++++++++++++++++++++++++ dlls/win32u/driver.c | 34 +++++++++++++++++++--- dlls/win32u/gdiobj.c | 1 + dlls/win32u/win32u.spec | 2 +- dlls/win32u/win32u_private.h | 3 ++ dlls/win32u/wrappers.c | 5 ++++ include/ntuser.h | 1 + 10 files changed, 99 insertions(+), 28 deletions(-) create mode 100644 dlls/win32u/clipboard.c

Nov. 9, 2021

[PATCH 4/5] win32u: Move null user driver implementation from user32.

by Jacek Caban

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/driver.c | 209 +---------------------------- dlls/win32u/driver.c | 310 ++++++++++++++++++++++++++++++++++++++++++- 2 files changed, 310 insertions(+), 209 deletions(-)

Nov. 9, 2021

[PATCH 3/5] win32u: Use user_driver_funcs to pass driver to __wine_set_display_driver.

by Jacek Caban

Nov. 9, 2021

[PATCH 2/5] user32: Use user_driver_funcs to expose user driver function from drivers.

by Jacek Caban

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- The ultimate plan is for driver to register themselves by a direct Unix call to win32u. dlls/user32/driver.c | 154 +++++++++-------- dlls/user32/user32.spec | 1 + dlls/wineandroid.drv/android.h | 27 +++ dlls/wineandroid.drv/init.c | 132 ++++----------- dlls/wineandroid.drv/wineandroid.drv.spec | 24 --- dlls/winemac.drv/gdi.c | 148 ++++++----------- dlls/winemac.drv/macdrv.h | 52 ++++++ dlls/winemac.drv/mouse.c | 2 +- dlls/winemac.drv/winemac.drv.spec | 39 ----- dlls/winex11.drv/display.c | 1 - dlls/winex11.drv/event.c | 2 - dlls/winex11.drv/init.c | 194 +++++++++++----------- dlls/winex11.drv/winex11.drv.spec | 41 ----- dlls/winex11.drv/x11drv.h | 50 ++++++ dlls/winex11.drv/x11drv_main.c | 1 + dlls/winex11.drv/xim.c | 2 - include/wine/gdi_driver.h | 2 + 17 files changed, 393 insertions(+), 479 deletions(-)

Nov. 9, 2021

[PATCH vkd3d] vkd3d-shader/hlsl: Unify texture and sampler allocation functions.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> --- Hacking a bit on the compiler helped. Register reservation / allocation is actually in a much better shape than I thought, functionality-wise. I still dislike the way object allocation is done but there is no ground or reason for blocking patches on that account. What do you think about this patch instead? Ugly? Based on top of 219079. libs/vkd3d-shader/hlsl_codegen.c | 102 +++++++++++-------------------- 1 file changed, 36 insertions(+), 66 deletions(-) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index b802f3a2..7c0b0f95 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -1202,118 +1202,88 @@ static void allocate_buffers(struct hlsl_ctx *ctx) } } -static const struct hlsl_ir_var *get_reserved_sampler(struct hlsl_ctx *ctx, uint32_t index) +static const struct hlsl_ir_var *get_reserved_object(struct hlsl_ctx *ctx, char type, uint32_t index) { const struct hlsl_ir_var *var; LIST_FOR_EACH_ENTRY(var, &ctx->extern_vars, const struct hlsl_ir_var, extern_entry) { - if (var->last_read && var->reg_reservation.type == 's' && var->reg_reservation.index == index) + if (var->last_read && var->reg_reservation.type == type && var->reg_reservation.index == index) return var; } return NULL; } -static void allocate_samplers(struct hlsl_ctx *ctx) +static const struct object_type_info { - struct hlsl_ir_var *var; - uint32_t index = 0; - - LIST_FOR_EACH_ENTRY(var, &ctx->extern_vars, struct hlsl_ir_var, extern_entry) - { - if (!var->last_read || var->data_type->type != HLSL_CLASS_OBJECT - || var->data_type->base_type != HLSL_TYPE_SAMPLER) - continue; - - if (var->reg_reservation.type == 's') - { - const struct hlsl_ir_var *reserved_sampler = get_reserved_sampler(ctx, var->reg_reservation.index); - - if (reserved_sampler && reserved_sampler != var) - { - hlsl_error(ctx, var->loc, VKD3D_SHADER_ERROR_HLSL_OVERLAPPING_RESERVATIONS, - "Multiple samplers bound to s%u.", var->reg_reservation.index); - hlsl_note(ctx, reserved_sampler->loc, VKD3D_SHADER_LOG_ERROR, - "Sampler '%s' is already bound to s%u.", reserved_sampler->name, - var->reg_reservation.index); - } - - var->reg.id = var->reg_reservation.index; - var->reg.allocated = true; - TRACE("Allocated reserved %s to s%u.\n", var->name, index); - } - else if (!var->reg_reservation.type) - { - while (get_reserved_sampler(ctx, index)) - ++index; - - var->reg.id = index; - var->reg.allocated = true; - TRACE("Allocated %s to s%u.\n", var->name, index); - ++index; - } - else - { - hlsl_error(ctx, var->loc, VKD3D_SHADER_ERROR_HLSL_INVALID_RESERVATION, - "Samplers must be bound to register type 's'."); - } - } + enum hlsl_base_type type; + char *name; + char reg_name; } +object_types[] = +{ + { HLSL_TYPE_SAMPLER, "sampler", 's' }, + { HLSL_TYPE_TEXTURE, "texture", 't' }, +}; -static const struct hlsl_ir_var *get_reserved_texture(struct hlsl_ctx *ctx, uint32_t index) +static const struct object_type_info *get_object_type_info(enum hlsl_base_type type) { - const struct hlsl_ir_var *var; + unsigned int i; - LIST_FOR_EACH_ENTRY(var, &ctx->extern_vars, const struct hlsl_ir_var, extern_entry) - { - if (var->last_read && var->reg_reservation.type == 't' && var->reg_reservation.index == index) - return var; - } + for (i = 0; i < ARRAY_SIZE(object_types); ++i) + if (type == object_types[i].type) + return &object_types[i]; + + WARN("No type info for object type %u.\n", type); return NULL; } -static void allocate_textures(struct hlsl_ctx *ctx) +static void allocate_objects(struct hlsl_ctx *ctx, enum hlsl_base_type type) { + const struct object_type_info *type_info = get_object_type_info(type); struct hlsl_ir_var *var; uint32_t index = 0; LIST_FOR_EACH_ENTRY(var, &ctx->extern_vars, struct hlsl_ir_var, extern_entry) { if (!var->last_read || var->data_type->type != HLSL_CLASS_OBJECT - || var->data_type->base_type != HLSL_TYPE_TEXTURE) + || var->data_type->base_type != type) continue; - if (var->reg_reservation.type == 't') + if (var->reg_reservation.type == type_info->reg_name) { - const struct hlsl_ir_var *reserved_texture = get_reserved_texture(ctx, var->reg_reservation.index); + const struct hlsl_ir_var *reserved_object = get_reserved_object(ctx, type_info->reg_name, + var->reg_reservation.index); - if (reserved_texture && reserved_texture != var) + if (reserved_object && reserved_object != var) { hlsl_error(ctx, var->loc, VKD3D_SHADER_ERROR_HLSL_OVERLAPPING_RESERVATIONS, - "Multiple textures bound to t%u.", var->reg_reservation.index); - hlsl_note(ctx, reserved_texture->loc, VKD3D_SHADER_LOG_ERROR, - "Texture '%s' is already bound to t%u.", reserved_texture->name, + "Multiple %ss bound to %c%u.", type_info->name, type_info->reg_name, var->reg_reservation.index); + hlsl_note(ctx, reserved_object->loc, VKD3D_SHADER_LOG_ERROR, + "Object '%s' is already bound to %c%u.", reserved_object->name, + type_info->reg_name, var->reg_reservation.index); } var->reg.id = var->reg_reservation.index; var->reg.allocated = true; - TRACE("Allocated reserved %s to t%u.\n", var->name, var->reg_reservation.index); + TRACE("Allocated reserved %s to %c%u.\n", var->name, type_info->reg_name, var->reg_reservation.index); } else if (!var->reg_reservation.type) { - while (get_reserved_texture(ctx, index)) + while (get_reserved_object(ctx, type_info->reg_name, index)) ++index; var->reg.id = index; var->reg.allocated = true; - TRACE("Allocated %s to t%u.\n", var->name, index); + TRACE("Allocated %s to %c%u.\n", var->name, type_info->reg_name, index); ++index; } else { hlsl_error(ctx, var->loc, VKD3D_SHADER_ERROR_HLSL_INVALID_RESERVATION, - "Textures must be bound to register type 't'."); + "Object of type '%s' must be bound to register type '%c'.", + type_info->name, type_info->reg_name); } } } @@ -1434,10 +1404,10 @@ int hlsl_emit_dxbc(struct hlsl_ctx *ctx, struct hlsl_ir_function_decl *entry_fun else { allocate_buffers(ctx); - allocate_textures(ctx); + allocate_objects(ctx, HLSL_TYPE_TEXTURE); } allocate_semantic_registers(ctx); - allocate_samplers(ctx); + allocate_objects(ctx, HLSL_TYPE_SAMPLER); if (ctx->result) return ctx->result; -- 2.26.3

Nov. 9, 2021

[PATCH vkd3d] vkd3d-shader/hlsl: Fix texture reservation trace.

by Matteo Bruni

Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> --- libs/vkd3d-shader/hlsl_codegen.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libs/vkd3d-shader/hlsl_codegen.c b/libs/vkd3d-shader/hlsl_codegen.c index 24b8205c..0bf18c2b 100644 --- a/libs/vkd3d-shader/hlsl_codegen.c +++ b/libs/vkd3d-shader/hlsl_codegen.c @@ -1240,7 +1240,7 @@ static void allocate_textures(struct hlsl_ctx *ctx) var->reg.id = var->reg_reservation.index; var->reg.allocated = true; - TRACE("Allocated reserved %s to t%u.\n", var->name, index); + TRACE("Allocated reserved %s to t%u.\n", var->name, var->reg_reservation.index); } else if (!var->reg_reservation.type) { -- 2.26.3

Nov. 9, 2021

[PATCH 1/5] user32: Introduce user_driver_funcs struct.

by Jacek Caban

And use it instead of USER_DRIVER. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> --- dlls/user32/driver.c | 24 +++++++-------- dlls/user32/painting.c | 9 ++---- dlls/user32/sysparams.c | 1 - dlls/user32/user_private.h | 56 ++-------------------------------- dlls/user32/win.c | 6 +--- dlls/user32/winpos.c | 9 ++---- include/wine/gdi_driver.h | 61 ++++++++++++++++++++++++++++++++++++++ 7 files changed, 78 insertions(+), 88 deletions(-)

Nov. 9, 2021

Re: [PATCH vkd3d v3 06/14] tests: Return a failure code if unable to parse a shader_test file.

by Henri Verbeet

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>

Nov. 9, 2021

Re: [PATCH vkd3d v3 05/14] tests: Add a shader test for texture sampling.

by Henri Verbeet

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>

Nov. 9, 2021