January 2022 - Wine-commits

Henri Verbeet : wined3d: Discard texture resources before unloading them in wined3d_texture_destroy_object().
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 8b4ae5951defedc8b9b74a212c5e8401062a5ba0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8b4ae5951defedc8b9b74a21… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jan 20 17:30:53 2022 +0100 wined3d: Discard texture resources before unloading them in wined3d_texture_destroy_object(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/texture.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index a638ec4eeae..34d76249084 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -1507,6 +1507,16 @@ static void wined3d_texture_destroy_object(void *object) heap_free(texture->dirty_regions); } + /* Discard the contents of resources with CPU access, to avoid downloading + * them to SYSMEM on unload. */ + if (resource->access & WINED3D_RESOURCE_ACCESS_CPU) + { + for (i = 0; i < sub_count; ++i) + { + wined3d_texture_validate_location(texture, i, WINED3D_LOCATION_DISCARDED); + wined3d_texture_invalidate_location(texture, i, ~WINED3D_LOCATION_DISCARDED); + } + } resource->resource_ops->resource_unload(resource); }

1 0

Henri Verbeet : wined3d: Use wined3d_mask_from_size() in wined3d_ffp_get_vs_settings().
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 074dd545363bc36f35201357968cb7f1adc8a523 URL: https://source.winehq.org/git/wine.git/?a=commit;h=074dd545363bc36f35201357… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jan 20 17:30:54 2022 +0100 wined3d: Use wined3d_mask_from_size() in wined3d_ffp_get_vs_settings(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/utils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 5e375f43a48..4724393905b 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -6734,7 +6734,7 @@ void wined3d_ffp_get_vs_settings(const struct wined3d_context *context, settings->texgen[i] = state->texture_states[i][WINED3D_TSS_TEXCOORD_INDEX]; } if (d3d_info->full_ffp_varyings) - settings->texcoords = (1u << WINED3D_MAX_TEXTURES) - 1; + settings->texcoords = wined3d_mask_from_size(WINED3D_MAX_TEXTURES); if (d3d_info->emulated_flatshading) settings->flatshading = state->render_states[WINED3D_RS_SHADEMODE] == WINED3D_SHADE_FLAT; @@ -6783,7 +6783,7 @@ void wined3d_ffp_get_vs_settings(const struct wined3d_context *context, settings->texgen[i] = state->texture_states[i][WINED3D_TSS_TEXCOORD_INDEX]; } if (d3d_info->full_ffp_varyings) - settings->texcoords = (1u << WINED3D_MAX_TEXTURES) - 1; + settings->texcoords = wined3d_mask_from_size(WINED3D_MAX_TEXTURES); for (i = 0; i < WINED3D_MAX_ACTIVE_LIGHTS; ++i) {

1 0

Henri Verbeet : wined3d: Use wined3d_mask_from_size() in wined3d_ffp_get_fs_settings().
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 0a1b3fc136831cf7710bc9755f56da04fb93c93b URL: https://source.winehq.org/git/wine.git/?a=commit;h=0a1b3fc136831cf7710bc975… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jan 20 17:30:55 2022 +0100 wined3d: Use wined3d_mask_from_size() in wined3d_ffp_get_fs_settings(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index 4724393905b..69c73487240 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -6549,7 +6549,7 @@ void wined3d_ffp_get_fs_settings(const struct wined3d_context *context, const st } else { - settings->texcoords_initialized = (1u << WINED3D_MAX_TEXTURES) - 1; + settings->texcoords_initialized = wined3d_mask_from_size(WINED3D_MAX_TEXTURES); } settings->pointsprite = state->render_states[WINED3D_RS_POINTSPRITEENABLE]

1 0

Henri Verbeet : wined3d: Properly compare integers in wined3d_pipeline_layout_vk_compare().
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 9b98d86f292891e55ab33e830470f281ce9a10ae URL: https://source.winehq.org/git/wine.git/?a=commit;h=9b98d86f292891e55ab33e83… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jan 20 17:30:56 2022 +0100 wined3d: Properly compare integers in wined3d_pipeline_layout_vk_compare(). As pointed out by Giovanni, modular subtraction doesn't produce a total order; in particular, it's not transitive. For the values likely to be encountered by wined3d_pipeline_layout_vk_compare(), this is perhaps unlikely to be an issue in practice. However, that's not necessarily the case for e.g. handles or masks. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context_vk.c | 5 +++-- dlls/wined3d/wined3d_private.h | 5 +++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c index fd6d810f0e2..cf6247387ca 100644 --- a/dlls/wined3d/context_vk.c +++ b/dlls/wined3d/context_vk.c @@ -1797,9 +1797,10 @@ static int wined3d_pipeline_layout_vk_compare(const void *key, const struct wine const struct wined3d_pipeline_layout_key_vk *a = key; const struct wined3d_pipeline_layout_key_vk *b = &WINE_RB_ENTRY_VALUE(entry, const struct wined3d_pipeline_layout_vk, entry)->key; + int ret; - if (a->binding_count != b->binding_count) - return a->binding_count - b->binding_count; + if ((ret = wined3d_uint32_compare(a->binding_count, b->binding_count))) + return ret; return memcmp(a->bindings, b->bindings, a->binding_count * sizeof(*a->bindings)); } diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 5fa51a844ac..c9d83b3fbc7 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -447,6 +447,11 @@ static inline unsigned int wined3d_popcount(unsigned int x) #endif } +static inline int wined3d_uint32_compare(uint32_t x, uint32_t y) +{ + return (x > y) - (x < y); +} + #define ORM_BACKBUFFER 0 #define ORM_FBO 1

1 0

Henri Verbeet : wined3d: Properly compare integers in wined3d_so_desc_compare().
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: d8781586753eee1bb795b1d5ece8002c2c586129 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d8781586753eee1bb795b1d5… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Jan 20 17:30:57 2022 +0100 wined3d: Properly compare integers in wined3d_so_desc_compare(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/device.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 93c738ad955..71b5aaa5742 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -5733,11 +5733,11 @@ static int wined3d_so_desc_compare(const void *key, const struct wine_rb_entry * unsigned int i; int ret; - if ((ret = (k->element_count - desc->element_count))) + if ((ret = wined3d_uint32_compare(k->element_count, desc->element_count))) return ret; - if ((ret = (k->buffer_stride_count - desc->buffer_stride_count))) + if ((ret = wined3d_uint32_compare(k->buffer_stride_count, desc->buffer_stride_count))) return ret; - if ((ret = (k->rasterizer_stream_idx - desc->rasterizer_stream_idx))) + if ((ret = wined3d_uint32_compare(k->rasterizer_stream_idx, desc->rasterizer_stream_idx))) return ret; for (i = 0; i < k->element_count; ++i) @@ -5745,25 +5745,25 @@ static int wined3d_so_desc_compare(const void *key, const struct wine_rb_entry * const struct wined3d_stream_output_element *b = &desc->elements[i]; const struct wined3d_stream_output_element *a = &k->elements[i]; - if ((ret = (a->stream_idx - b->stream_idx))) + if ((ret = wined3d_uint32_compare(a->stream_idx, b->stream_idx))) return ret; if ((ret = (!a->semantic_name - !b->semantic_name))) return ret; if (a->semantic_name && (ret = strcmp(a->semantic_name, b->semantic_name))) return ret; - if ((ret = (a->semantic_idx - b->semantic_idx))) + if ((ret = wined3d_uint32_compare(a->semantic_idx, b->semantic_idx))) return ret; - if ((ret = (a->component_idx - b->component_idx))) + if ((ret = wined3d_uint32_compare(a->component_idx, b->component_idx))) return ret; - if ((ret = (a->component_count - b->component_count))) + if ((ret = wined3d_uint32_compare(a->component_count, b->component_count))) return ret; - if ((ret = (a->output_slot - b->output_slot))) + if ((ret = wined3d_uint32_compare(a->output_slot, b->output_slot))) return ret; } for (i = 0; i < k->buffer_stride_count; ++i) { - if ((ret = (k->buffer_strides[i] - desc->buffer_strides[i]))) + if ((ret = wined3d_uint32_compare(k->buffer_strides[i], desc->buffer_strides[i]))) return ret; }

1 0

Michael Stefaniuc : dmusic: Call SetSynkSink() first to initialize the Synth.
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 7217973cb90730b9eeb859dc057acb4e46ecb21b URL: https://source.winehq.org/git/wine.git/?a=commit;h=7217973cb90730b9eeb859dc… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Thu Jan 20 21:20:19 2022 +0100 dmusic: Call SetSynkSink() first to initialize the Synth. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmusic/port.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/dmusic/port.c b/dlls/dmusic/port.c index 7bd2afdb9a3..165bd520191 100644 --- a/dlls/dmusic/port.c +++ b/dlls/dmusic/port.c @@ -851,13 +851,13 @@ HRESULT synth_port_create(IDirectMusic8Impl *parent, DMUS_PORTPARAMS *port_param hr = CoCreateInstance(&CLSID_DirectMusicSynthSink, NULL, CLSCTX_INPROC_SERVER, &IID_IDirectMusicSynthSink, (void**)&obj->synth_sink); if (SUCCEEDED(hr)) - hr = IDirectMusicSynth_SetMasterClock(obj->synth, obj->pLatencyClock); + hr = IDirectMusicSynth_SetSynthSink(obj->synth, obj->synth_sink); if (SUCCEEDED(hr)) - hr = IDirectMusicSynthSink_SetMasterClock(obj->synth_sink, obj->pLatencyClock); + hr = IDirectMusicSynth_SetMasterClock(obj->synth, obj->pLatencyClock); if (SUCCEEDED(hr)) - hr = IDirectMusicSynth_SetSynthSink(obj->synth, obj->synth_sink); + hr = IDirectMusicSynthSink_SetMasterClock(obj->synth_sink, obj->pLatencyClock); if (SUCCEEDED(hr)) hr = IDirectMusicSynth_Open(obj->synth, port_params);

1 0

Michael Stefaniuc : dmsynth: Pass the master clock on to the sink.
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 98e6e2429f43e9f9ca233c25e7e3b37fd99b3e97 URL: https://source.winehq.org/git/wine.git/?a=commit;h=98e6e2429f43e9f9ca233c25… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Thu Jan 20 21:20:20 2022 +0100 dmsynth: Pass the master clock on to the sink. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmsynth/synth.c | 7 +++++-- dlls/dmusic/port.c | 3 --- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/dmsynth/synth.c b/dlls/dmsynth/synth.c index 3d3af80d005..51b33e8df8d 100644 --- a/dlls/dmsynth/synth.c +++ b/dlls/dmsynth/synth.c @@ -305,9 +305,12 @@ static HRESULT WINAPI IDirectMusicSynth8Impl_SetMasterClock(IDirectMusicSynth8 * { IDirectMusicSynth8Impl *This = impl_from_IDirectMusicSynth8(iface); - FIXME("(%p)->(%p): stub\n", This, clock); + TRACE("(%p)->(%p)\n", This, clock); - return S_OK; + if (!This->sink) + return DMUS_E_NOSYNTHSINK; + + return IDirectMusicSynthSink_SetMasterClock(This->sink, clock); } static HRESULT WINAPI IDirectMusicSynth8Impl_GetLatencyClock(IDirectMusicSynth8 *iface, diff --git a/dlls/dmusic/port.c b/dlls/dmusic/port.c index 165bd520191..88c7c60a50d 100644 --- a/dlls/dmusic/port.c +++ b/dlls/dmusic/port.c @@ -856,9 +856,6 @@ HRESULT synth_port_create(IDirectMusic8Impl *parent, DMUS_PORTPARAMS *port_param if (SUCCEEDED(hr)) hr = IDirectMusicSynth_SetMasterClock(obj->synth, obj->pLatencyClock); - if (SUCCEEDED(hr)) - hr = IDirectMusicSynthSink_SetMasterClock(obj->synth_sink, obj->pLatencyClock); - if (SUCCEEDED(hr)) hr = IDirectMusicSynth_Open(obj->synth, port_params);

1 0

Michael Stefaniuc : dmusic: Get rid of the SynthPortImpl typedef.
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: a708c25b84055d3416eb5e142624d783c15b5f87 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a708c25b84055d3416eb5e14… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Thu Jan 20 21:20:21 2022 +0100 dmusic: Get rid of the SynthPortImpl typedef. and rename the iface to obj helpers. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmusic/port.c | 96 +++++++++++++++++++++++++++--------------------------- 1 file changed, 48 insertions(+), 48 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=a708c25b84055d3416eb…

1 0

Michael Stefaniuc : dmusic: Remove SynthPortImpl from the method and vtbl names.
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 6fe37b465649bfff2085ea871ffad0fff7ee7d61 URL: https://source.winehq.org/git/wine.git/?a=commit;h=6fe37b465649bfff2085ea87… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Thu Jan 20 21:20:22 2022 +0100 dmusic: Remove SynthPortImpl from the method and vtbl names. Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmusic/port.c | 163 +++++++++++++++++++++++++++-------------------------- 1 file changed, 83 insertions(+), 80 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=6fe37b465649bfff2085…

1 0

Nikolay Sivov : mf/samplegrabber: Cancel pending sample timer.
by Alexandre Julliard 20 Jan '22

20 Jan '22

Module: wine Branch: master Commit: 8679255eccd57e2a1a785a7efac04e0c84cb9055 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8679255eccd57e2a1a785a7e… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jan 20 13:46:55 2022 +0300 mf/samplegrabber: Cancel pending sample timer. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Giovanni Mascellani <gmascellani(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mf/samplegrabber.c | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/dlls/mf/samplegrabber.c b/dlls/mf/samplegrabber.c index e0287e1e1db..e7d3be68df3 100644 --- a/dlls/mf/samplegrabber.c +++ b/dlls/mf/samplegrabber.c @@ -849,11 +849,7 @@ static ULONG WINAPI sample_grabber_sink_Release(IMFMediaSink *iface) if (grabber->clock) IMFPresentationClock_Release(grabber->clock); if (grabber->timer) - { - if (grabber->cancel_key) - IMFTimer_CancelTimer(grabber->timer, grabber->cancel_key); IMFTimer_Release(grabber->timer); - } if (grabber->cancel_key) IUnknown_Release(grabber->cancel_key); if (grabber->stream_event_queue) @@ -973,10 +969,21 @@ static HRESULT WINAPI sample_grabber_sink_GetStreamSinkById(IMFMediaSink *iface, return hr; } +static void sample_grabber_cancel_timer(struct sample_grabber *grabber) +{ + if (grabber->timer && grabber->cancel_key) + { + IMFTimer_CancelTimer(grabber->timer, grabber->cancel_key); + IUnknown_Release(grabber->cancel_key); + grabber->cancel_key = NULL; + } +} + static void sample_grabber_set_presentation_clock(struct sample_grabber *grabber, IMFPresentationClock *clock) { if (grabber->clock) { + sample_grabber_cancel_timer(grabber); IMFPresentationClock_RemoveClockStateSink(grabber->clock, &grabber->IMFClockStateSink_iface); IMFPresentationClock_Release(grabber->clock); if (grabber->timer) @@ -1122,6 +1129,9 @@ static HRESULT sample_grabber_set_state(struct sample_grabber *grabber, enum sin hr = MF_E_INVALID_STATE_TRANSITION; else { + if (state == SINK_STATE_STOPPED) + sample_grabber_cancel_timer(grabber); + if (state == SINK_STATE_RUNNING && grabber->state == SINK_STATE_STOPPED) { /* Every transition to running state sends a bunch requests to build up initial queue. */

1 0