Re: [PATCH 1/1] win32u: Fix data race in NtUserGetProcessDpiAwarenessContext.

[Zebediah Figura]

On 10/3/22 07:45, Jinoh Kang wrote:
> @@ -5326,14 +5326,17 @@ BOOL WINAPI NtUserSetProcessDpiAwarenessContext( ULONG awareness, ULONG unknown
>    */
>   ULONG WINAPI NtUserGetProcessDpiAwarenessContext( HANDLE process )
>   {
> - DPI_AWARENESS val;
> -
> if (process && process != GetCurrentProcess())
> {
>     WARN( "not supported on other process %p\n", process );
>     return NTUSER_DPI_UNAWARE;
> }
>
> - if (!dpi_awareness) return NTUSER_DPI_UNAWARE;
> - return dpi_awareness;
> - val = ReadAcquire( &dpi_awareness );
> - if (!val) return NTUSER_DPI_UNAWARE;
> - return val;

Why does this need acquire semantics?