-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Private/bryan/iotargetopen waitloop #51
Private/bryan/iotargetopen waitloop #51
Conversation
Suggest that you slim down the PR to only perform the |
Do you want certmgr.exe, for installing certificates, to be checked in or have instructions for where to get it?
From: Fredrik Orderud ***@***.***>
Sent: Tuesday, February 20, 2024 10:35 AM
To: forderud/IntelliMouseDriver ***@***.***>
Cc: Bryan Allen Young ***@***.***>; Assign ***@***.***>
Subject: Re: [forderud/IntelliMouseDriver] Private/bryan/iotargetopen waitloop (PR #51)
Suggest that you slim down the PR to only perform the IoRegisterPlugPlayNotification stuff, so that we reduce the amount of code complexity required for fixing issue #44 <#44> .
—
Reply to this email directly, view it on GitHub <#51 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AGBDV7262RLGW4KZDCEE7JTYUTUDZAVCNFSM6AAAAABDRPGEROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJUHAZTCNZYGA> .
You are receiving this because you were assigned. <https://github.com/notifications/beacon/AGBDV77JNIRDTGEDKABBHRDYUTUDZA5CNFSM6AAAAABDRPGEROWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTUQRO2I.gif> Message ID: ***@***.***>
|
|
WdfIoTargetOpen Reliability Experiments
A thunk to pageable code is needed from IoRegisterPnPNotification in all options as WdfIoTargetOpen is pageable. The callback for RegisterPnPNotification runs at DPC.
Basic overview of code flow
This is option No. 3 of 3 options.
Tested for several days on one machine. There have been no failures of WdfIoTargetOpen.
This PR fixes issue #44.