Fix/test assert fail

[planetchili]

Fixing assertion fires cause by recent changes (during tests only). Tune timings on one flakey test. Fixing tests hanging in ETLTests group.

[Copilot]

Pull request overview

This PR addresses test-only assertion failures and hangs by refactoring cross-module logging linkage to avoid heap/ABI issues, improving playback-session teardown behavior under backpressure, and tuning/adjusting tests to reduce flakiness and prevent teardown deadlocks.

Changes:

• Refactor logging bridge APIs to use raw pointers/function pointers across module boundaries and add explicit “sever bridge” behavior during teardown.
• Split playback trace-session stopping into a request/finalize model and add backpressure release to prevent output-thread hangs.
• Update ETL/backpressure-related tests to avoid indefinite waits and reduce flakiness.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
IntelPresentMon/SampleClient/LogSetup.cpp	Switches to pointer-based logging link helper.
IntelPresentMon/PresentMonService/MockPresentMonSession.h	Splits stop into request/finalize and adds join-pending flag.
IntelPresentMon/PresentMonService/MockPresentMonSession.cpp	Implements request/finalize stop flow and triggers backpressure release on stop.
IntelPresentMon/PresentMonService/FrameBroadcaster.h	Adds timeout-aware push warning and a helper to release backpressure across PIDs.
IntelPresentMon/PresentMonMiddleware/Middleware.h	Makes middleware destructor noexcept.
IntelPresentMon/PresentMonMiddleware/Middleware.cpp	Ensures StopTracking attempts during teardown with exception containment/logging.
IntelPresentMon/PresentMonMiddleware/LogSetup.h	Changes copy-channel setup API to raw pointer and adds sever-bridge function.
IntelPresentMon/PresentMonMiddleware/LogSetup.cpp	Implements sever-bridge behavior and tags copy driver for targeted removal.
IntelPresentMon/PresentMonAPI2Tests/TestProcess.h	Adds bounded wait for service exit and terminates on timeout.
IntelPresentMon/PresentMonAPI2Tests/Logging.cpp	Moves tests to use pointer-based logging link function.
IntelPresentMon/PresentMonAPI2Tests/InterimBroadcasterTests.cpp	Tunes query/window sleeps for flakiness/backpressure timing.
IntelPresentMon/PresentMonAPI2Tests/EtlTests.cpp	Replaces first-frame helper with bounded polling and drains backpressure ring during teardown.
IntelPresentMon/PresentMonAPI2Loader/Implementation.cpp	Adds optional resolution + export wrapper for pmLinkLoggingPtrs_.
IntelPresentMon/PresentMonAPI2/PresentMonAPI.cpp	Introduces pmLinkLoggingPtrs_, refactors linking internals, and severs copy bridge on unlink.
IntelPresentMon/PresentMonAPI2/Internal.h	Changes LoggingSingletons to function pointers and documents preferred ABI-safe API.
IntelPresentMon/Core/source/infra/LogSetup.cpp	Switches to pointer-based logging link helper.
IntelPresentMon/CommonUtilities/log/CopyDriver.h	Changes CopyDriver to hold a raw channel pointer.
IntelPresentMon/CommonUtilities/log/CopyDriver.cpp	Guards Submit/Flush against null channel pointer.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.