Repository navigation
Safe driver cleanup and concurrent DPC/ISR diagnosis for alpha.8 - #50
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Branch cleanup: git cherry verified the original research commit is preserved in this PR (- 44bd300). The redundant research/dpc-isr-capture-technique branch has now been deleted locally and remotely; the research document remains in this pushed PR branch. |
Protect all bound driver packages and revalidate cleanup in the elevated worker without /uninstall or /force. Require repeated overlapping ICMP/ETW evidence for affinity or ancillary-device disable. VM testing reproduced a WPF dispatcher deadlock at worker startup; run worker entry points off the blocked dispatcher and cover the real process handshake with a regression test. Extract TraceEvent native content in single-file builds. Update the accepted #32 scope and alpha.8 release notes.
|
Final local checks pass: dotnet format --verify-no-changes; 2 Python archive tests and regeneration check; 956 Release tests passed, 14 opt-in live tests skipped. Updated packaged VM smoke additionally requires resolved module names: vmbus.sys, ntoskrnl.exe, tcpip.sys, storvsp.sys, fastfat.sys, storport.sys, classpnp.sys, symcryptk.dll, ntfs.sys observed. No eligible removal/disable was requested; driver store unchanged and VM reset. |
…CI formatting TraceEvent 3.1.21 does not populate the code-address index used by its ISR/DPC extension helpers. Resolve routine addresses directly against kernel module ranges at the event timestamp; VM smoke now resolves vmbus.sys, tcpip.sys, storport.sys and other modules instead of only (unresolved). Require at least one resolved module in the packaged smoke test.
Summary
Implements the scope explicitly approved by the owner in this session; supersedes the original recommend-only scope in #32.
/uninstallor/force; explicit user confirmation and no automatic undo claim. Optional official RAPR link..sysmodule mapping.Verification
No real driver removal/device-disable or affinity benefit has been validated on physical hardware. Repeated co-occurrence is not proof of causation. The release remains an alpha preview.
Closes #32.