[PATCH resend 1/5] ntoskrnl.exe/tests: Add tests with and without report IDs.

Zebediah Figura (she/her) zfigura at codeweavers.com
Wed Jun 16 10:59:01 CDT 2021


On 6/16/21 10:20 AM, Rémi Bernon wrote:
> On 6/16/21 4:42 PM, Zebediah Figura (she/her) wrote:
>> On 6/16/21 4:37 AM, Rémi Bernon wrote:
>>> Well, not so less frequent I guess, but this is still caused by some
>>> failure to add the devices and it happens already now.
>>>
>>> Next patch succeeds there for instance (except for the bus arrival
>>> messages like I said):
>>>
>>> https://testbot.winehq.org/JobDetails.pl?Key=92615
>>>
>>
>> I think the problem may just be that Windows sometimes takes a *long*
>> time to install PnP drivers. Not sure what it's spending all that time
>> doing, but we may have to increase some timeouts...
>>
> 
> I've tried increasing the timeouts to things like 1s, and it didn't
> help. Maybe it's still too low though.
> 

1s is probably not even close to enough. I'd try 30s for a start. Though 
measuring the actual time is probably a good idea too.

(Yes, it's a long time, and that sucks, but I think the benefits far 
outweigh the costs. Plus most of the test VMs don't seem to take nearly 
that long.)



More information about the wine-devel mailing list