[PATCH v2 1/3] dinput/tests: Enforce HID report descriptor buffer size.

Rémi Bernon rbernon at codeweavers.com
Wed Apr 27 06:37:53 CDT 2022


On 4/27/22 13:03, Marvin wrote:
> Hi,
> 
> While running your changed tests, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> 
> Full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=113536
> 
> Your paranoid android.
> 
> 
> === w7u_adm (32 bit report) ===
> 
> dinput:
> device8: Timeout
> 
> === w1064_tsign (64 bit report) ===
> 
> dinput:
> hotplug.c:927: Test failed: 1: controller removed handler not invoked
> hotplug.c:927: Test failed: 1: controller removed handler not invoked
> 

I can repro this locally too sometimes, seems to be pre-existing 
somehow, I'll investigate.

> === w1064_tsign (64 bit report) ===
> 
> dinput:
> joystick8: Timeout
> 
> === debian11 (32 bit Arabic:Morocco report) ===
> 
> dinput:
> force_feedback.c:5474: Test failed: WaitForSingleObject returned 0x102
> force_feedback.c:5474: Test failed: GetOverlappedResult returned 996
> force_feedback.c:5474: Test failed: IOCTL_WINETEST_HID_WAIT_EXPECT failed, last error 996
> driver_bus.c:139: Test failed: hid.c:5474 expect[0]: missing (code 0xb000f id 6 len 2)

That's also a pre-existing race, or the timeout isn't big enough, I'll 
try to fix it in a later patch.

-- 
Rémi Bernon <rbernon at codeweavers.com>



More information about the wine-devel mailing list