- Wine-Devel - list.winehq.org

[PATCH 2/4] shell32: the menu wIDs in hmenu were rebased by idCmdFirst
by Damjan Jovanovic April 30, 2021

April 30, 2021

Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com> --- dlls/shell32/shlview_cmenu.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)

1 0

[PATCH 1/4] shell32/tests: add more tests for Shell_MergeMenus()
by Damjan Jovanovic April 30, 2021

April 30, 2021

Show Shell_MergeMenus() really adds the offset instead of changing IDs to start with it, and that it really honours the maximum allowed value. Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com> --- dlls/shell32/tests/shlfileop.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+)

1 0

[PATCH] winegstreamer: Delay duration query until caps have been acquired.
by Zebediah Figura April 29, 2021

April 29, 2021

From: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> --- dlls/quartz/tests/waveparser.c | 2 +- dlls/winegstreamer/wg_parser.c | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/quartz/tests/waveparser.c b/dlls/quartz/tests/waveparser.c index f3ea8f2b482..f78e15e5c19 100644 --- a/dlls/quartz/tests/waveparser.c +++ b/dlls/quartz/tests/waveparser.c @@ -929,7 +929,7 @@ static void test_seeking(void) duration = 0xdeadbeef; hr = IMediaSeeking_GetDuration(seeking, &duration); ok(hr == S_OK, "Got hr %#x.\n", hr); - todo_wine ok(duration > 0, "Got duration %s.\n", wine_dbgstr_longlong(duration)); + ok(duration > 0, "Got duration %s.\n", wine_dbgstr_longlong(duration)); stop = current = 0xdeadbeef; hr = IMediaSeeking_GetStopPosition(seeking, &stop); diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index d6aceff8cfe..c06e5af8160 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -1553,7 +1553,6 @@ static HRESULT CDECL wg_parser_connect(struct wg_parser *parser, uint64_t file_s { struct wg_parser_stream *stream = parser->streams[i]; - stream->duration = query_duration(stream->their_src); while (!stream->has_caps && !parser->error) pthread_cond_wait(&parser->init_cond, &parser->mutex); if (parser->error) @@ -1561,6 +1560,12 @@ static HRESULT CDECL wg_parser_connect(struct wg_parser *parser, uint64_t file_s pthread_mutex_unlock(&parser->mutex); return E_FAIL; } + /* GStreamer doesn't actually provide any guarantees about when duration + * is available, even for seekable streams. However, many elements (e.g. + * avidemux, wavparse, qtdemux) in practice record duration before + * fixing caps, so as a heuristic, wait until we get caps before trying + * to query for duration. */ + stream->duration = query_duration(stream->their_src); } pthread_mutex_unlock(&parser->mutex); -- 2.30.2

1 0

[PATCH 3/4] d3d11/tests: Initialise "constant" in test_device_context_state() (Valgrind).
by Henri Verbeet April 29, 2021

April 29, 2021

We're mostly just using this to make sure the value we read back is that same as the one we set, so in that sense the actual value is inconsequential. Nevertheless, randomness is generally undesirable in the tests. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> --- dlls/d3d11/tests/d3d11.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/d3d11/tests/d3d11.c b/dlls/d3d11/tests/d3d11.c index ca60170d080..191df67a41d 100644 --- a/dlls/d3d11/tests/d3d11.c +++ b/dlls/d3d11/tests/d3d11.c @@ -6775,6 +6775,7 @@ static void test_device_context_state(void) 0x00000008, 0x00000000, 0x00000008, 0x58454853, 0x00000020, 0x00050050, 0x00000008, 0x0100086a, 0x0400009b, 0x00000001, 0x00000001, 0x00000001, 0x0100003e, }; + static const struct vec4 constant = {1.257f, 1.885f, 2.513f, 3.770f}; ID3DDeviceContextState *context_state, *previous_context_state, *tmp_context_state, *context_state2; UINT ib_offset, vb_offset, vb_stride, so_offset, offset, stride, sample_mask, stencil_ref, count; @@ -6814,7 +6815,6 @@ static void test_device_context_state(void) ID3D11Texture2D *texture; enum DXGI_FORMAT format; float blend_factor[4]; - struct vec4 constant; DWORD data_size; BOOL pred_value; ULONG refcount; -- 2.20.1

2 1

[PATCH 4/4] wined3d: Only read "*rect_count" when "rects" is non-NULL in wined3d_device_context_get_scissor_rects() (Valgrind).
by Henri Verbeet April 29, 2021

April 29, 2021

When "rects" is NULL, "*rect_count" is potentially uninitialised. That's fairly benign because the resulting "count" value is inconsequential in that case, but it's also unnecessary, and easy to avoid. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> --- dlls/wined3d/device.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index f3d7cff74bf..813ee773478 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1722,8 +1722,7 @@ void CDECL wined3d_device_context_get_scissor_rects(const struct wined3d_device_ TRACE("context %p, rect_count %p, rects %p.\n", context, rect_count, rects); - count = rect_count ? min(*rect_count, state->scissor_rect_count) : 1; - if (count && rects) + if (rects && (count = rect_count ? min(*rect_count, state->scissor_rect_count) : 1)) memcpy(rects, state->scissor_rects, count * sizeof(*rects)); if (rect_count) *rect_count = state->scissor_rect_count; -- 2.20.1

1 0

[PATCH 2/4] wined3d: Free the program variants array in shader_spirv_destroy() (Valgrind).
by Henri Verbeet April 29, 2021

April 29, 2021

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> --- dlls/wined3d/shader_spirv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/wined3d/shader_spirv.c b/dlls/wined3d/shader_spirv.c index eb2647db12d..7617ee9d6ef 100644 --- a/dlls/wined3d/shader_spirv.c +++ b/dlls/wined3d/shader_spirv.c @@ -1019,6 +1019,7 @@ static void shader_spirv_destroy(struct wined3d_shader *shader) shader_spirv_invalidate_contexts_graphics_program_variant(&device_vk->d, variant_vk); VK_CALL(vkDestroyShaderModule(device_vk->vk_device, variant_vk->vk_module, NULL)); } + heap_free(program_vk->variants); vkd3d_shader_free_scan_descriptor_info(&program_vk->descriptor_info); shader->backend_data = NULL; -- 2.20.1

1 0

[PATCH 1/4] wined3d: Avoid accessing the bo_user list from outside the command stream thread (Valgrind).
by Henri Verbeet April 29, 2021

April 29, 2021

Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> --- dlls/wined3d/adapter_gl.c | 69 ++++++++++++++++------------------- dlls/wined3d/adapter_vk.c | 77 ++++++++++++++++++--------------------- 2 files changed, 68 insertions(+), 78 deletions(-) diff --git a/dlls/wined3d/adapter_gl.c b/dlls/wined3d/adapter_gl.c index 990991e2e3d..8787324571a 100644 --- a/dlls/wined3d/adapter_gl.c +++ b/dlls/wined3d/adapter_gl.c @@ -4755,6 +4755,7 @@ struct wined3d_view_gl_destroy_ctx { struct wined3d_device *device; const struct wined3d_gl_view *gl_view; + struct wined3d_bo_user *bo_user; struct wined3d_bo_gl *counter_bo; void *object; struct wined3d_view_gl_destroy_ctx *free; @@ -4787,13 +4788,15 @@ static void wined3d_view_gl_destroy_object(void *object) checkGLcall("delete resources"); context_release(context); } + if (ctx->bo_user) + list_remove(&ctx->bo_user->entry); heap_free(ctx->object); heap_free(ctx->free); } -static void wined3d_view_gl_destroy(struct wined3d_device *device, - const struct wined3d_gl_view *gl_view, struct wined3d_bo_gl *counter_bo, void *object) +static void wined3d_view_gl_destroy(struct wined3d_device *device, const struct wined3d_gl_view *gl_view, + struct wined3d_bo_user *bo_user, struct wined3d_bo_gl *counter_bo, void *object) { struct wined3d_view_gl_destroy_ctx *ctx, c; @@ -4801,6 +4804,7 @@ static void wined3d_view_gl_destroy(struct wined3d_device *device, ctx = &c; ctx->device = device; ctx->gl_view = gl_view; + ctx->bo_user = bo_user; ctx->counter_bo = counter_bo; ctx->object = object; ctx->free = ctx != &c ? ctx : NULL; @@ -4813,21 +4817,16 @@ static void wined3d_view_gl_destroy(struct wined3d_device *device, static void adapter_gl_destroy_rendertarget_view(struct wined3d_rendertarget_view *view) { struct wined3d_rendertarget_view_gl *view_gl = wined3d_rendertarget_view_gl(view); - struct wined3d_device *device = view_gl->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = view_gl->v.resource; TRACE("view_gl %p.\n", view_gl); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); + /* Take a reference to the resource, in case releasing the resource + * would cause the device to be destroyed. */ + wined3d_resource_incref(resource); wined3d_rendertarget_view_cleanup(&view_gl->v); - wined3d_view_gl_destroy(device, &view_gl->gl_view, NULL, view_gl); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_gl_destroy(resource->device, &view_gl->gl_view, NULL, NULL, view_gl); + wined3d_resource_decref(resource); } static HRESULT adapter_gl_create_shader_resource_view(const struct wined3d_view_desc *desc, @@ -4859,22 +4858,20 @@ static HRESULT adapter_gl_create_shader_resource_view(const struct wined3d_view_ static void adapter_gl_destroy_shader_resource_view(struct wined3d_shader_resource_view *view) { struct wined3d_shader_resource_view_gl *view_gl = wined3d_shader_resource_view_gl(view); - struct wined3d_device *device = view_gl->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = view_gl->v.resource; TRACE("view_gl %p.\n", view_gl); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); - list_remove(&view_gl->bo_user.entry); + /* Take a reference to the resource. There are two reasons for this: + * - Releasing the resource could in turn cause the device to be + * destroyed, but we still need the device for + * wined3d_view_vk_destroy(). + * - We shouldn't free buffer resources until after we've removed the + * view from its bo_user list. */ + wined3d_resource_incref(resource); wined3d_shader_resource_view_cleanup(&view_gl->v); - wined3d_view_gl_destroy(device, &view_gl->gl_view, NULL, view_gl); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_gl_destroy(resource->device, &view_gl->gl_view, &view_gl->bo_user, NULL, view_gl); + wined3d_resource_decref(resource); } static HRESULT adapter_gl_create_unordered_access_view(const struct wined3d_view_desc *desc, @@ -4906,22 +4903,20 @@ static HRESULT adapter_gl_create_unordered_access_view(const struct wined3d_view static void adapter_gl_destroy_unordered_access_view(struct wined3d_unordered_access_view *view) { struct wined3d_unordered_access_view_gl *view_gl = wined3d_unordered_access_view_gl(view); - struct wined3d_device *device = view_gl->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = view_gl->v.resource; TRACE("view_gl %p.\n", view_gl); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); - list_remove(&view_gl->bo_user.entry); + /* Take a reference to the resource. There are two reasons for this: + * - Releasing the resource could in turn cause the device to be + * destroyed, but we still need the device for + * wined3d_view_vk_destroy(). + * - We shouldn't free buffer resources until after we've removed the + * view from its bo_user list. */ + wined3d_resource_incref(resource); wined3d_unordered_access_view_cleanup(&view_gl->v); - wined3d_view_gl_destroy(device, &view_gl->gl_view, &view_gl->counter_bo, view_gl); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_gl_destroy(resource->device, &view_gl->gl_view, &view_gl->bo_user, &view_gl->counter_bo, view_gl); + wined3d_resource_decref(resource); } static HRESULT adapter_gl_create_sampler(struct wined3d_device *device, const struct wined3d_sampler_desc *desc, diff --git a/dlls/wined3d/adapter_vk.c b/dlls/wined3d/adapter_vk.c index 0abae24fd1f..99d7547b834 100644 --- a/dlls/wined3d/adapter_vk.c +++ b/dlls/wined3d/adapter_vk.c @@ -1304,6 +1304,7 @@ struct wined3d_view_vk_destroy_ctx struct wined3d_device_vk *device_vk; VkBufferView *vk_buffer_view; VkImageView *vk_image_view; + struct wined3d_bo_user *bo_user; struct wined3d_bo_vk *vk_counter_bo; VkBufferView *vk_counter_view; uint64_t *command_buffer_id; @@ -1350,6 +1351,8 @@ static void wined3d_view_vk_destroy_object(void *object) TRACE("Destroyed image view 0x%s.\n", wine_dbgstr_longlong(*ctx->vk_image_view)); } } + if (ctx->bo_user) + list_remove(&ctx->bo_user->entry); if (ctx->vk_counter_bo && ctx->vk_counter_bo->vk_buffer) wined3d_context_vk_destroy_bo(wined3d_context_vk(context), ctx->vk_counter_bo); if (ctx->vk_counter_view) @@ -1374,7 +1377,7 @@ static void wined3d_view_vk_destroy_object(void *object) } static void wined3d_view_vk_destroy(struct wined3d_device *device, VkBufferView *vk_buffer_view, - VkImageView *vk_image_view, struct wined3d_bo_vk *vk_counter_bo, + VkImageView *vk_image_view, struct wined3d_bo_user *bo_user, struct wined3d_bo_vk *vk_counter_bo, VkBufferView *vk_counter_view, uint64_t *command_buffer_id, void *view_vk) { struct wined3d_view_vk_destroy_ctx *ctx, c; @@ -1384,6 +1387,7 @@ static void wined3d_view_vk_destroy(struct wined3d_device *device, VkBufferView ctx->device_vk = wined3d_device_vk(device); ctx->vk_buffer_view = vk_buffer_view; ctx->vk_image_view = vk_image_view; + ctx->bo_user = bo_user; ctx->vk_counter_bo = vk_counter_bo; ctx->vk_counter_view = vk_counter_view; ctx->command_buffer_id = command_buffer_id; @@ -1398,22 +1402,17 @@ static void wined3d_view_vk_destroy(struct wined3d_device *device, VkBufferView static void adapter_vk_destroy_rendertarget_view(struct wined3d_rendertarget_view *view) { struct wined3d_rendertarget_view_vk *view_vk = wined3d_rendertarget_view_vk(view); - struct wined3d_device *device = view_vk->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = view_vk->v.resource; TRACE("view_vk %p.\n", view_vk); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); + /* Take a reference to the resource, in case releasing the resource + * would cause the device to be destroyed. */ + wined3d_resource_incref(resource); wined3d_rendertarget_view_cleanup(&view_vk->v); - wined3d_view_vk_destroy(device, NULL, &view_vk->vk_image_view, - NULL, NULL, &view_vk->command_buffer_id, view_vk); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_vk_destroy(resource->device, NULL, &view_vk->vk_image_view, + NULL, NULL, NULL, &view_vk->command_buffer_id, view_vk); + wined3d_resource_decref(resource); } static HRESULT adapter_vk_create_shader_resource_view(const struct wined3d_view_desc *desc, @@ -1445,30 +1444,28 @@ static HRESULT adapter_vk_create_shader_resource_view(const struct wined3d_view_ static void adapter_vk_destroy_shader_resource_view(struct wined3d_shader_resource_view *view) { struct wined3d_shader_resource_view_vk *srv_vk = wined3d_shader_resource_view_vk(view); - struct wined3d_device *device = srv_vk->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = srv_vk->v.resource; struct wined3d_view_vk *view_vk = &srv_vk->view_vk; VkBufferView *vk_buffer_view = NULL; VkImageView *vk_image_view = NULL; TRACE("srv_vk %p.\n", srv_vk); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); - if (srv_vk->v.resource->type == WINED3D_RTYPE_BUFFER) + /* Take a reference to the resource. There are two reasons for this: + * - Releasing the resource could in turn cause the device to be + * destroyed, but we still need the device for + * wined3d_view_vk_destroy(). + * - We shouldn't free buffer resources until after we've removed the + * view from its bo_user list. */ + wined3d_resource_incref(resource); + if (resource->type == WINED3D_RTYPE_BUFFER) vk_buffer_view = &view_vk->u.vk_buffer_view; else vk_image_view = &view_vk->u.vk_image_info.imageView; - list_remove(&view_vk->bo_user.entry); wined3d_shader_resource_view_cleanup(&srv_vk->v); - wined3d_view_vk_destroy(device, vk_buffer_view, vk_image_view, - NULL, NULL, &view_vk->command_buffer_id, srv_vk); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_vk_destroy(resource->device, vk_buffer_view, vk_image_view, + &view_vk->bo_user, NULL, NULL, &view_vk->command_buffer_id, srv_vk); + wined3d_resource_decref(resource); } static HRESULT adapter_vk_create_unordered_access_view(const struct wined3d_view_desc *desc, @@ -1500,30 +1497,28 @@ static HRESULT adapter_vk_create_unordered_access_view(const struct wined3d_view static void adapter_vk_destroy_unordered_access_view(struct wined3d_unordered_access_view *view) { struct wined3d_unordered_access_view_vk *uav_vk = wined3d_unordered_access_view_vk(view); - struct wined3d_device *device = uav_vk->v.resource->device; - unsigned int swapchain_count = device->swapchain_count; + struct wined3d_resource *resource = uav_vk->v.resource; struct wined3d_view_vk *view_vk = &uav_vk->view_vk; VkBufferView *vk_buffer_view = NULL; VkImageView *vk_image_view = NULL; TRACE("uav_vk %p.\n", uav_vk); - /* Take a reference to the device, in case releasing the view's resource - * would cause the device to be destroyed. However, swapchain resources - * don't take a reference to the device, and we wouldn't want to increment - * the refcount on a device that's in the process of being destroyed. */ - if (swapchain_count) - wined3d_device_incref(device); - if (uav_vk->v.resource->type == WINED3D_RTYPE_BUFFER) + /* Take a reference to the resource. There are two reasons for this: + * - Releasing the resource could in turn cause the device to be + * destroyed, but we still need the device for + * wined3d_view_vk_destroy(). + * - We shouldn't free buffer resources until after we've removed the + * view from its bo_user list. */ + wined3d_resource_incref(resource); + if (resource->type == WINED3D_RTYPE_BUFFER) vk_buffer_view = &view_vk->u.vk_buffer_view; else vk_image_view = &view_vk->u.vk_image_info.imageView; - list_remove(&view_vk->bo_user.entry); wined3d_unordered_access_view_cleanup(&uav_vk->v); - wined3d_view_vk_destroy(device, vk_buffer_view, vk_image_view, &uav_vk->counter_bo, - &uav_vk->vk_counter_view, &view_vk->command_buffer_id, uav_vk); - if (swapchain_count) - wined3d_device_decref(device); + wined3d_view_vk_destroy(resource->device, vk_buffer_view, vk_image_view, &view_vk->bo_user, + &uav_vk->counter_bo, &uav_vk->vk_counter_view, &view_vk->command_buffer_id, uav_vk); + wined3d_resource_decref(resource); } static HRESULT adapter_vk_create_sampler(struct wined3d_device *device, const struct wined3d_sampler_desc *desc, -- 2.20.1

1 0

[PATCH v2 1/2] winegstreamer: Delay duration query until caps have been acquired.
by Connor McAdams April 29, 2021

April 29, 2021

Signed-off-by: Connor McAdams <cmcadams(a)codeweavers.com> --- dlls/winegstreamer/wg_parser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 5529321490e..f90362981df 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -1553,9 +1553,9 @@ static HRESULT CDECL wg_parser_connect(struct wg_parser *parser, uint64_t file_s { struct wg_parser_stream *stream = parser->streams[i]; - stream->duration = query_duration(stream->their_src); while (!stream->has_caps && !parser->error) pthread_cond_wait(&parser->init_cond, &parser->mutex); + stream->duration = query_duration(stream->their_src); if (parser->error) { pthread_mutex_unlock(&parser->mutex); -- 2.25.1

2 3

[PATCH 10/10] msvcrt: Import round implementation from musl.
by Piotr Caban April 29, 2021

April 29, 2021

Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> --- configure.ac | 1 - dlls/msvcrt/math.c | 31 ++++++++++++++++++++++++++++++- dlls/msvcrt/unixlib.c | 13 ------------- dlls/msvcrt/unixlib.h | 1 - 4 files changed, 30 insertions(+), 16 deletions(-)

1 0

[PATCH 09/10] msvcrt: Import lround implementation from musl.
by Piotr Caban April 29, 2021

April 29, 2021

Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> --- configure.ac | 1 - dlls/msvcrt/math.c | 9 ++++++++- dlls/msvcrt/unixlib.c | 13 ------------- dlls/msvcrt/unixlib.h | 1 - 4 files changed, 8 insertions(+), 16 deletions(-)

1 0