On Thu Mar 21 15:24:43 2024 +0000, Jinoh Kang wrote:
Right, well as things have changed a lot since then as we now have
loops anyway everywhere, what about merging the seq lock within it, like this: https://gitlab.winehq.org/rbernon/wine/-/commit/6897f75798835c3a0bf2e2c7f428... Looks fine, although I'd use `STATUS_PENDING` instead of `STATUS_RETRY /* return STATUS_RETRY to enter reader loops */`. Also perhaps we want to pass-through NTSTATUS from `get_thread_session_object_info` instead of discarding it, so that we can return things like `0xc0010000 | ERROR_INVALID_WINDOW_HANDLE`.
You mean STATUS_PENDING all the time, or both STATUS_PENDING / STATUS_RETRY depending on the case? I'd prefer not to have two different statuses, so that caller loops can be simpler.