Re: [PATCH 0/5] MR3103: server: Create and use a desktop shared mapping for GetCursorPos.

[Jinoh Kang (＠iamahuman)]

On Wed Jun 28 11:53:28 2023 +0000, Rémi Bernon wrote:
> I personally like that it looks like server requests, it makes
> wineserver communication stand out compared to other function calls.

Yes, not to mention that it introduces an unnecessary read fence (which is a bit costly on ARM platforms) at the start of the loop. The read fence cannot be reordered after the `__SHARED_READ_SEQ( x ) != __seq` comparison, so it's difficult to make it conditional without introducing even more complexity or overhead.