16 Jul
2023
16 Jul
'23
10:42 p.m.
On Sun Jul 16 10:32:02 2023 +0000, Vladislav Timonin wrote:
Stared at this for a bit yesterday, narrowed it down to `ASM_GetSimpleAudioVolume`, calling `get_audio_session_wrapper` through `AUDDRV_GetAudioSessionWrapper` "fixes" the tests. For some reason, the sessions list changes address halfway through the test, and it ends up creating a new session instead.
00d0:fixme:mmdevapi:get_audio_session ({faaab7e1-480b-4230-b5e6-8e09a3cc3adb}, 000000000038F110, 0, 000000000021FAC8) sessions 00000002EB239100 00d0:fixme:mmdevapi:get_audio_session ({faaab7e1-480b-4230-b5e6-8e09a3cc3adb}, 000000000038DD70, 0, 000000000021FAC8) sessions 00000002EB239100 00d0:fixme:mmdevapi:get_audio_session ({faaab7e1-480b-4230-b5e6-8e09a3cc3adb}, 000000000038DD70, 2, 0000000000383B70) sessions 00000002EB239100 00d0:fixme:mmdevapi:get_audio_session ({faaab7e1-480b-4230-b5e6-8e09a3cc3adb}, 000000000038F110, 2, 0000000000383C40) sessions 00000002EB239100 00cc:render: 21 tests executed (0 marked as todo, 0 as flaky, 0 failures), 0 skipped.00d0:fixme:mmdevapi:get_audio_session ({ccd800bb-46ca-418f-8020-0988753a38c4}, 000000000038F110, 0, 000000000021FAC8) sessions 000000020F413260 00d0:fixme:mmdevapi:get_audio_session ({ccd800bb-46ca-418f-8020-0988753a38c4}, 000000000038DD70, 0, 000000000021FAC8) sessions 000000020F413260 00d0:fixme:mmdevapi:get_audio_session ({ccd800bb-46ca-418f-8020-0988753a38c4}, 000000000038DD70, 2, 0000000000383B70) sessions 00000002EB239100 00d0:fixme:mmdevapi:get_audio_session ({ccd800bb-46ca-418f-8020-0988753a38c4}, 000000000038F110, 2, 0000000000383C40) sessions 00000002EB239100 00cc:render: 21 tests executed (0 marked as todo, 0 as flaky, 1 failure), 0 skipped.
Thanks for investigating!
I actually added the trace message to reveal the weird behavior, by comparing the output produced by the good and the broken code.
However, the result was (and still is) confusing as I would expect other session related tests to fail as well...