- wine-gitlab - winehq.org

[PATCH 0/1] MR1837: wined3d: Recognize AMD as a vendor string
by Ryan Hendrickson (＠rhendric) 19 Dec '22

19 Dec '22

I have an AMD Ryzen 7 5700U, which when my system was running Mesa 22.0.4 identified itself in `glxinfo` as: ``` Vendor: AMD (0x1002) Device: AMD RENOIR (LLVM 13.0.1, DRM 3.47, 5.19.17) (0x164c) ``` After a recent system upgrade that took Mesa to 22.2.3, `glxinfo` now gives: ``` Vendor: AMD (0x1002) Device: RENOIR (renoir, LLVM 14.0.6, DRM 3.48, 6.0.12) (0x164c) ``` Wine programs also started reliably crashing on me, with the following message that did not appear before: ``` 0130:fixme:d3d:wined3d_guess_card_vendor Received unrecognized GL_VENDOR "AMD". Returning HW_VENDOR_NVIDIA. ``` I don't really know what I'm doing but that message suggested this patch to me, which does resolve the crashes and doesn't seem obviously wrong. Maybe there's some other table to which the new identifying information of this APU needs to be added, but keeping this check for future-proofing might still be a good idea in that case? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1837

4 5

[PATCH 0/2] MR1824: Remove broken from some color tests that can also fail on Linux.
by Alexandre Julliard (＠julliard) 19 Dec '22

19 Dec '22

They fail on the Gitlab CI. An alternative would be to mark them todo_wine but it's not clear that there's anything we want to do here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1824

4 6

[PATCH v3 0/8] MR1842: dinput/tests: Wait for each device interface to start / stop and misc test cleanup.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53525 -- v3: dinput/tests: Remove unnecessary goto and label. dinput/tests: Wait for each device interface to start / stop. dinput/tests: Add a device count parameter to hid_device_(start|stop). dinput/tests: Use semaphores to count and notify rawinput messages. dinput/tests: Avoid using INFINITE timeouts. dinput/tests: Remove old physical joystick tests. dinput/tests: Unregister rawinput devices after tests. hidclass.sys: Make sure a desktop is created for the driver process. https://gitlab.winehq.org/wine/wine/-/merge_requests/1842

3 10

[PATCH 0/3] MR1844: win32u/tests: Add some pointer related syscall tests.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Pretty much the same as https://gitlab.winehq.org/wine/wine/-/merge_requests/1843, but on win32u side, to figure and validate the syscall entry signatures. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1844

2 3

[PATCH v2 0/5] MR1843: user32: Add more EnableMouseInPointer / GetPointerType / GetPointerInfo tests.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Checking the expected basic behavior, for https://bugs.winehq.org/show_bug.cgi?id=51537 and https://bugs.winehq.org/show_bug.cgi?id=53847. -- v2: user32/tests: Test GetPointer(Frame)Info(History) behavior. user32/tests: Move old GetPointerType tests into test_GetPointerInfo. user32/tests: Test GetPointerInfo behavior with EnableMouseInPointer. user32/tests: Test IsMouseInPointerEnabled behavior. user32/tests: Test EnableMouseInPointer behavior. https://gitlab.winehq.org/wine/wine/-/merge_requests/1843

2 6

[PATCH v2 0/7] MR1842: dinput/tests: Wait for each device interface to start / stop and misc test cleanup.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53525 -- v2: dinput/tests: Wait for each device interface to start / stop. dinput/tests: Add a device count parameter to hid_device_(start|stop). dinput/tests: Use semaphores to count and notify rawinput messages. dinput/tests: Avoid using INFINITE timeouts. dinput/tests: Remove old physical joystick tests. dinput/tests: Unregister rawinput devices after tests. hidclass.sys: Make sure a desktop is created for the driver process. https://gitlab.winehq.org/wine/wine/-/merge_requests/1842

3 10

[PATCH 0/7] MR1843: user32: Add more EnableMouseInPointer / GetPointerType / GetPointerInfo tests.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Checking the expected basic behavior, for https://bugs.winehq.org/show_bug.cgi?id=51537 and https://bugs.winehq.org/show_bug.cgi?id=53847. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1843

3 8

[PATCH v4 0/7] MR1842: dinput/tests: Wait for each device interface to start / stop and misc test cleanup.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53525 -- v4: https://gitlab.winehq.org/wine/wine/-/merge_requests/1842

2 8

[PATCH 0/6] MR1842: dinput/tests: Wait for each device interface to start / stop and misc test cleanup.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=53525 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1842

4 9

[PATCH 0/1] MR1841: hidclass.sys: Make sure a desktop is created for the driver process.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

This was previously automatically done when user32 was loaded, but since the move of __wine_send_input to win32u, we do not need to load it any more. Somehow it is still loaded when prefix is initialized or updated, and the dinput tests work in some cases, but not in some other. This makes sure we have a desktop or WM_INPUT messages won't be sent by wineserver. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1841

2 2

Re: [PATCH v2 0/2] MR1839: mf/tests: Test SetOutputType for WMV decoder.
by Rémi Bernon (＠rbernon) 19 Dec '22

19 Dec '22

On Mon Dec 19 11:04:02 2022 +0000, Ziqing Hui wrote: > Testbot complains about too much printing. Should I remove some tests? > For example, test only one input type for SetOutputType tests. I'd suggest to shorten the test messages, like for instance `SetOutputType returned unexpected hr 0x80004001 for flag 0.` shorten to `0x0: SetOutputType returned 0x80004001.` with wine_push_context for the flags, or as an even more extreme measure, if necessary, `0x0: got hr 0x80004001.`. If that's not enough then yes, you can probably think about removing some tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1839#note_19835

1 0

Re: [PATCH v2 0/1] MR1769: winecrt0: Avoid predefined BOOL in get_relocbase
by Alexandre Julliard (＠julliard) 19 Dec '22

19 Dec '22

This should be superseded by f69c11117ee56cb664bd0ade1392c5553ff2af30. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1769#note_19833

1 0

[PATCH 0/1] MR1840: ntdll: Only compile get_current_teb() on linux.
by Huw Davies (＠huw) 19 Dec '22

19 Dec '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1840

2 1

Re: [PATCH v2 0/2] MR1839: mf/tests: Test SetOutputType for WMV decoder.
by Ziqing Hui (＠zhui) 19 Dec '22

19 Dec '22

Testbot complains about too much printing. Should I remove some tests? For example, test only one input type for SetOutputType tests. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1839#note_19832

1 0

[PATCH 0/2] MR1839: mf/tests: Test SetOutputType for WMV decoder.
by Ziqing Hui (＠zhui) 19 Dec '22

19 Dec '22

-- https://gitlab.winehq.org/wine/wine/-/merge_requests/1839

3 3

[PATCH 0/1] MR1773: localui: Add additional safety check (Coverity)
by Fabian Maurer (＠DarkShadow44) 19 Dec '22

19 Dec '22

Should not change the logic, but makes Coverity happy and seems safer Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1773

3 7

[PATCH v5 0/7] MR51: vkd3d-shader/hlsl: Improve constant handling in copy prop, and extend the use of aoffimmi modifiers.
by Francisco Casas (＠fcasas) 19 Dec '22

19 Dec '22

If a hlsl_ir_load loads a variable whose components are stored from different instructions, copy propagation doesn't replace it. But if all these instructions are constants (which currently is the case for value constructors), the load can be replaced with a constant value, which is what the first patch of this series does. For instance, this shader: ``` sampler s; Texture2D t; float4 main() : sv_target { return t.Gather(s, float2(0.6, 0.6), int2(0, 0)); } ``` results in the following IR before applying the patch: ``` float | 6.00000024e-01 float | 6.00000024e-01 uint | 0 | = (<constructor-2>[@4].x @2) uint | 1 | = (<constructor-2>[@6].x @3) float2 | <constructor-2> int | 0 int | 0 uint | 0 | = (<constructor-5>[@11].x @9) uint | 1 | = (<constructor-5>[@13].x @10) int2 | <constructor-5> float4 | gather_red(resource = t, sampler = s, coords = @8, offset = @15) | return | = (<output-sv_target0> @16) ``` and this IR afterwards: ``` float2 | {6.00000024e-01 6.00000024e-01 } int2 | {0 0 } float4 | gather_red(resource = t, sampler = s, coords = @2, offset = @3) | return | = (<output-sv_target0> @4) ``` This is required to write texel_offsets as aoffimmi modifiers in the sm4 backend, since it expects the texel_offset arguments to be hlsl_ir_constant. This series also: * Allows Gather() methods to use aoffimmi modifiers instead of an additional source register (which is the only way allowed for shader model 4.1), when possible. * Adds support to texel_offsets in the Load() method via aoffimmi modifiers (the only allowed method). -- v5: vkd3d-shader/hlsl: Propagate swizzle chains. vkd3d-shader/hlsl: Replace swizzles with constants in copy prop. tests: Test constant propagation through swizzles. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/51

3 10

[PATCH 0/2] MR58: vkd3d-shader/hlsl: Reinterpret minimum precision types as their regular counterparts.
by Francisco Casas (＠fcasas) 19 Dec '22

19 Dec '22

Reinterpret min16float, min10float, min16int, min12int, and min16uint as their regular counterparts: float, float, int, int, uint, respectively. A proper implementation would require adding minimum precision indicators to all the dxbc-tpf instructions that use these types. Consider the output of fxc 10.1 with the following shader: ```hlsl uniform int i; float4 main() : sv_target { min16float4 a = {0, 1, 2, i}; min16int2 b = {4, i}; min10float3 c = {6.4, 7, i}; min12int d = 9.4; min16uint4x2 e = {14.4, 15, 16, 17, 18, 19, 20, i}; return mul(e, b) + a + c.xyzx + d; } ``` ``` ps_5_0 dcl_globalFlags refactoringAllowed | enableMinimumPrecision dcl_constantbuffer CB0[1], immediateIndexed dcl_output o0.xyzw dcl_temps 3 imad r0.xyz {min16u}, l(15, 17, 19, 0) {def32 as min16u}, cb0[0].xxxx {def32 as min16u}, l(56, 64, 72, 0) {def32 as min16u} utof r0.xyz {min16f}, r0.xyzx {min16u} imad r1.x {min16u}, cb0[0].x {def32 as min16u}, cb0[0].x {def32 as min16u}, l(80) {def32 as min16u} utof r0.w {min16f}, r1.x {min16u} mov r1.xyz {min16f}, l(0,1.000000,2.000000,0) {def32 as min16f} itof r1.w {min16f}, cb0[0].x add r0.xyzw {min16f}, r0.xyzw {min16f}, r1.xyzw {min16f} mov r2.y {min2_8f}, l(7.000000) {def32 as min2_8f} itof r2.z {min2_8f}, cb0[0].x add r0.yz {min16f}, r0.yyzy {min16f}, r2.yyzy {min2_8f as min16f} add r0.xyzw {min16f}, r0.xyzw {min16f}, l(15.400000, 9.000000, 9.000000, 15.400000) {def32 as min16f} mov o0.xyzw, r0.xyzw {min16f as def32} ret ``` However, if the graphics driver doesn't have minimum precision support, it ignores the minimum precision indicators and runs at 32-bit precision, which is equivalent as working with regular types. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/58

3 5

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

I've given a cursory review to the implementation, and supplied some inline feedback. Besides that, I think it would be a good idea to split up the commit more, and try to apply one fixup at a time. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19811

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > if (!url) > return E_FAIL; > > - srcLen = lstrlenW(url) + 1; > + /* > + * First check for known, valid and typo free scheme > + */ > + for (pos=1; pos<ARRAY_SIZE(url_scheme); pos++) > + { > + len = wcslen(url_scheme[pos]); > + if ( (len <= wcslen(url)) && (!_wcsnicmp(url, url_scheme[pos], len)) ) > + { > + /* > + * check if string fits into maxChars > + */ > + if (len+1 >= maxChars) This comment doesn't seem to tell us anything more than the following line does. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19810

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > */ > HRESULT WINAPI UrlFixupW(PCWSTR url, PWSTR translatedUrl, DWORD maxChars) > { > - DWORD srcLen; > + DWORD srcLen, dstLen, len; > + DWORD colon_pos = 0, pos = 0; > + wchar_t *helper_str = NULL; > + PWSTR save_str = translatedUrl; We use WCHAR, not wchar_t. It'd also be nice to use a consistent style for all of variable names in this function (wrt snake case vs camel case). In new code I think we prefer snake case. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19808

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > + * First check for known, valid and typo free scheme > + */ > + for (pos=1; pos<ARRAY_SIZE(url_scheme); pos++) > + { > + len = wcslen(url_scheme[pos]); > + if ( (len <= wcslen(url)) && (!_wcsnicmp(url, url_scheme[pos], len)) ) > + { > + /* > + * check if string fits into maxChars > + */ > + if (len+1 >= maxChars) > + return S_FALSE; > + > + lstrcpynW(save_str, url_scheme[pos], len+1); > + url += len; > + goto scheme_done; Can we use a helper function to fix up the scheme instead of using a goto? This will probably also help make the function easier to follow, by virtue of being less monolithic. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19809

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > - srcLen = lstrlenW(url) + 1; > + /* > + * First check for known, valid and typo free scheme > + */ > + for (pos=1; pos<ARRAY_SIZE(url_scheme); pos++) > + { > + len = wcslen(url_scheme[pos]); > + if ( (len <= wcslen(url)) && (!_wcsnicmp(url, url_scheme[pos], len)) ) > + { > + /* > + * check if string fits into maxChars > + */ > + if (len+1 >= maxChars) > + return S_FALSE; > + > + lstrcpynW(save_str, url_scheme[pos], len+1); We already know it fits, so this can just be memcpy(). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19807

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > + DWORD colon_pos = 0, pos = 0; > + wchar_t *helper_str = NULL; > + PWSTR save_str = translatedUrl; > > - FIXME("%s, %p, %ld stub\n", wine_dbgstr_w(url), translatedUrl, maxChars); > + const WCHAR *url_scheme[] = { L"", L"ftp:", L"https:", L"gopher:", L"mailto:", L"news:", > + L"nntp:", L"telnet:", L"wais:", L"file:", L"mk:", > + L"http:", L"shell:", L"snews:", L"local:", > + L"javascript:", L"vbscript:", L"about:", L"res:", > + L"ms-shell-rooted:", L"ms-shell-idlist:", L"hcp:"}; > + > + const WCHAR *url_scheme_short[] = { L"", L"ftp", L"https", L"gopher", L"mailto", L"snews", > + L"nntp", L"telnet", L"wais", L"file", L"mk", > + L"http", L"ms-shell-rooted", L"news", L"local", > + L"javascript", L"vbscript", L"about", L"res", > + L"ms-shell-idlist", L"shell", L"hcp"}; These are constant arrays, so should be "static const WCHAR *const". Also, it seems that one array is the same as the other without the final colon; can we avoid defining two arrays? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19806

1 0

Re: [PATCH v3 0/3] MR1825: kernelbase/path.c: UrlFixupW()
by Zebediah Figura (＠zfigura) 18 Dec '22

18 Dec '22

Zebediah Figura (@zfigura) commented about dlls/kernelbase/path.c: > > -HRESULT WINAPI UrlFixupW(const WCHAR *url, WCHAR *translatedUrl, DWORD maxChars) > +/* > + * from Documentation: > + * https://docs.microsoft.com/en-us/windows/desktop/api/shlwapi/nf-shlwapi-url… > + * > + * UrlFixupW attempts to correct a URL whose protocol identifier is incorrect. > + * For example, htttp will be changed to http. > + * > + * LWSTDAPI UrlFixupW( > + * [in] PCWSTR pcszUrl, > + * [out] PWSTR pszTranslatedUrl, > + * DWORD cchMax > + * ); > +*/ > +HRESULT WINAPI UrlFixupW(PCWSTR url, PWSTR translatedUrl, DWORD maxChars) Without trying to get bogged down in style, this change is somewhat counterproductive; we tend to avoid P* typedefs in new code, and we also have been abandoning the use of "documentation" style headers, which rarely convey any useful information. (This also doesn't really belong in a commit which should only affect tests.) -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1825#note_19805

1 0

2025

2024

2023

2022

wine-gitlab