[PATCH v2 09/10] winmm: Reimplement joystick APIs on top of dinput.

[Rémi Bernon]

On 12/9/21 17:11, Alexandre Julliard wrote:
> Rémi Bernon <rbernon at codeweavers.com> writes:
> 
>> Based on a patch from Zebediah Figura <zfigura at codeweavers.com>.
>>
>> Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
>> ---
>>   dlls/dinput8/tests/hid.c |  51 ------
>>   dlls/winmm/Makefile.in   |   2 +-
>>   dlls/winmm/joystick.c    | 355 +++++++++++++++++++++++++++++++++------
>>   dlls/winmm/winemm.h      |   3 +
>>   dlls/winmm/winmm.c       |   2 +
>>   5 files changed, 308 insertions(+), 105 deletions(-)
> 
> This is causing a deadlock in the tests:
> 
> tools/runtest -q -P wine -T . -M dplayx.dll -p dlls/dplayx/tests/dplayx_test.exe dplayx && touch dlls/dplayx/tests/dplayx.ok
> dplayx.c:6945: Tests skipped: Run in interactive mode to run all dplayx tests.
> 00fc:err:sync:RtlpWaitForCriticalSection section 7BC64440 "dlls/ntdll/loader.c: loader_section" wait timed out in thread 00fc, blocked by 0024, retrying (60 sec)
> 00fc:err:sync:RtlpWaitForCriticalSection section 7BC64440 "dlls/ntdll/loader.c: loader_section" wait timed out in thread 00fc, blocked by 0024, retrying (60 sec)
> ...
> 
> 

Crap, sorry, looks like this will require some changes in dinput 
internal thread shutdown. Maybe the first patches are fine, and 
hopefully I'll have something for tomorrow?
-- 
Rémi Bernon <rbernon at codeweavers.com>