Thank you for all these reviews. I have submitted v4 to rebase on master and fix whitespace issues. I'll address all your issues in v5.

On Thu, Apr 11, 2019 at 3:35 PM Jacek Caban <jacek@codeweavers.com> wrote:

On 4/11/19 8:26 PM, Derek Lesho wrote:
> @@ -3431,7 +3433,8 @@ void WINAPI ExReleaseResourceForThreadLite( PERESOURCE resource, ERESOURCE_THREA
> */
> void WINAPI KeEnterCriticalRegion(void)
> {
> - FIXME(": stub\n");
> + TRACE(": semi-stub\n");
> + KeGetCurrentThread()->critical_region = TRUE;
> }

According to MSDN, those may be called recursively, so you'd need a
counter here instead.

Thanks,

Jacek