[PATCH v6 0/1] MR4698: winewayland.drv: Add unaccelerated pointer support

[Grigory Vasilyev (＠h0tc0d3)]

Adds the registry key
`HKEY_CURRENT_USER\Software\Wine\Wayland Driver\unaccelerated_pointer`
witch allows raw input. This makes it easier
to calculate the same sensitivity in different games,
use sensitivity calculators, and easily change values
when changing DPI and do not depend on the compositor or OS.

For example, you want to set the sensitivity to half as much,
and sensitivity curves in libinput are more difficult
to calculate than in the games themselves.

--
  v6: winewayland.drv: Add unaccelerated pointer support

https://gitlab.winehq.org/wine/wine/-/merge_requests/4698

[PATCH v6 1/1] winewayland.drv: Add unaccelerated pointer support

[Grigory Vasilyev]

From: Grigory Vasilyev

Adds the registry key
HKEY_CURRENT_USER\Software\Wine\Wayland Driver\unaccelerated_pointer
witch allows raw input. This makes it easier
to calculate the same sensitivity in different games,
use sensitivity calculators, and easily change values
when changing DPI and do not depend on the compositor or OS.

For example, you want to set the sensitivity to half as much,
and sensitivity curves in libinput are more difficult
to calculate than in the games themselves.
---
 dlls/winewayland.drv/Makefile.in       |  1 +
 dlls/winewayland.drv/dllmain.c         | 41 ++++++++++++++++++++++++++
 dlls/winewayland.drv/unixlib.h         |  1 +
 dlls/winewayland.drv/wayland_pointer.c | 23 ++++++++++++---
 dlls/winewayland.drv/waylanddrv_main.c | 10 +++++++
 5 files changed, 72 insertions(+), 4 deletions(-)

diff --git a/dlls/winewayland.drv/Makefile.in b/dlls/winewayland.drv/Makefile.in
index 9ad1ad6889d..bee478257ae 100644
--- a/dlls/winewayland.drv/Makefile.in
+++ b/dlls/winewayland.drv/Makefile.in
@@ -2,6 +2,7 @@ MODULE = winewayland.drv
 UNIXLIB = winewayland.so
 UNIX_CFLAGS = $(EGL_CFLAGS) $(WAYLAND_CLIENT_CFLAGS) $(WAYLAND_EGL_CFLAGS) $(XKBCOMMON_CFLAGS) $(XKBREGISTRY_CFLAGS)
 UNIX_LIBS = -lwin32u $(WAYLAND_CLIENT_LIBS) $(WAYLAND_EGL_LIBS) $(XKBCOMMON_LIBS) $(XKBREGISTRY_LIBS) $(PTHREAD_LIBS) -lm
+IMPORTS = advapi32

SOURCES = \
    display.c \
diff --git a/dlls/winewayland.drv/dllmain.c b/dlls/winewayland.drv/dllmain.c
index d040620957b..ba0fef07b74 100644
--- a/dlls/winewayland.drv/dllmain.c
+++ b/dlls/winewayland.drv/dllmain.c
@@ -20,6 +20,7 @@

#include "waylanddrv_dll.h"

+#include "winreg.h"
 #include "wine/debug.h"

WINE_DEFAULT_DEBUG_CHANNEL(waylanddrv);
@@ -38,12 +39,52 @@ static DWORD WINAPI wayland_read_events_thread(void *arg)
 BOOL WINAPI DllMain(HINSTANCE instance, DWORD reason, void *reserved)
 {
     DWORD tid;
+    DWORD regRes;
+    DWORD regValueSize;
+    LPWSTR regValue = NULL;
+    HKEY hSubKey = NULL;
+    BOOL unaccelerated_pointer = FALSE;

if (reason != DLL_PROCESS_ATTACH) return TRUE;

DisableThreadLibraryCalls(instance);
     if (__wine_init_unix_call()) return FALSE;

+    regRes = RegOpenKeyExW(HKEY_CURRENT_USER, L"Software\Wine\Wayland Driver", 0, KEY_READ, &hSubKey);
+    if (regRes != ERROR_SUCCESS)
+    {
+        WINE_TRACE("Registry key HKCU\Software\Wine\Wayland Driver not exist.\n");
+        goto close_registry;
+    }
+
+    regRes = RegQueryValueExW(hSubKey, L"unaccelerated_pointer", NULL, NULL, NULL, &regValueSize);
+    if (regRes != ERROR_SUCCESS)
+    {
+        WINE_ERR("Can't get value size for HKCU\Software\Wine\Wayland Driver\unaccelerated_pointer. Error: %ld\n", regRes);
+        goto close_registry;
+    }
+
+    regValue = calloc(regValueSize + 1, sizeof(*regValue));
+
+    regRes = RegQueryValueExW(hSubKey, L"unaccelerated_pointer", NULL, NULL, (LPBYTE)regValue, &regValueSize);
+    if (regRes != ERROR_SUCCESS)
+    {
+        WINE_ERR("Can't get value for HKCU\Software\Wine\Wayland Driver\unaccelerated_pointer. Error: %ld\n", regRes);
+        free(regValue);
+        goto close_registry;
+    }
+
+    WINE_TRACE("Registry HKCU\Software\Wine\Wayland Driver\unaccelerated_pointer value=%hs.\n", regValue);
+    if(*regValue)
+        unaccelerated_pointer = TRUE;
+
+    free(regValue);
+
+close_registry:
+    RegCloseKey(hSubKey);
+
+    WAYLANDDRV_UNIX_CALL(set_unaccelerated_pointer, &unaccelerated_pointer);
+
     if (WAYLANDDRV_UNIX_CALL(init, NULL))
         return FALSE;

diff --git a/dlls/winewayland.drv/unixlib.h b/dlls/winewayland.drv/unixlib.h
index dc3bfdf8893..53b8afada1d 100644
--- a/dlls/winewayland.drv/unixlib.h
+++ b/dlls/winewayland.drv/unixlib.h
@@ -27,6 +27,7 @@ enum waylanddrv_unix_func
 {
     waylanddrv_unix_func_init,
     waylanddrv_unix_func_read_events,
+    waylanddrv_unix_func_set_unaccelerated_pointer,
     waylanddrv_unix_func_count,
 };

diff --git a/dlls/winewayland.drv/wayland_pointer.c b/dlls/winewayland.drv/wayland_pointer.c
index df76a4e9954..05aea799e45 100644
--- a/dlls/winewayland.drv/wayland_pointer.c
+++ b/dlls/winewayland.drv/wayland_pointer.c
@@ -34,6 +34,8 @@

WINE_DEFAULT_DEBUG_CHANNEL(waylanddrv);

+extern BOOL waylanddrv_unaccelerated_pointer;
+
 static HWND wayland_pointer_get_focused_hwnd(void)
 {
     struct wayland_pointer *pointer = &process_wayland.pointer;
@@ -259,15 +261,28 @@ static void relative_pointer_v1_relative_motion(void *data,
     POINT screen, origin;
     struct wayland_surface *surface;
     RECT window_rect;
+    double delta_x;
+    double delta_y;

if (!(hwnd = wayland_pointer_get_focused_hwnd())) return;
     if (!(surface = wayland_surface_lock_hwnd(hwnd))) return;

window_rect = surface->window.rect;

+    if(waylanddrv_unaccelerated_pointer)
+    {
+        delta_x = wl_fixed_to_double(dx_unaccel);
+        delta_y = wl_fixed_to_double(dy_unaccel);
+    }
+    else
+    {
+        delta_x = wl_fixed_to_double(dx);
+        delta_y = wl_fixed_to_double(dy);
+    }
+
     wayland_surface_coords_to_window(surface,
-                                     wl_fixed_to_double(dx),
-                                     wl_fixed_to_double(dy),
+                                     delta_x,
+                                     delta_y,
                                      (int *)&screen.x, (int *)&screen.y);

pthread_mutex_unlock(&surface->mutex);
@@ -313,8 +328,8 @@ static void relative_pointer_v1_relative_motion(void *data,
     input.mi.dy = screen.y;
     input.mi.dwFlags = MOUSEEVENTF_MOVE;

-    TRACE("hwnd=%p wayland_dxdy=%.2f,%.2f screen_dxdy=%d,%d\n",
-          hwnd, wl_fixed_to_double(dx), wl_fixed_to_double(dy),
+    TRACE("hwnd=%p unaccelerated_pointer=%d wayland_dxdy=%.2f,%.2f screen_dxdy=%d,%d\n",
+          hwnd, waylanddrv_unaccelerated_pointer, delta_x, delta_y,
           (int)screen.x, (int)screen.y);

NtUserSendHardwareInput(hwnd, 0, &input, 0);
diff --git a/dlls/winewayland.drv/waylanddrv_main.c b/dlls/winewayland.drv/waylanddrv_main.c
index 7d02497295a..21d5c45b49e 100644
--- a/dlls/winewayland.drv/waylanddrv_main.c
+++ b/dlls/winewayland.drv/waylanddrv_main.c
@@ -46,6 +46,14 @@ static const struct user_driver_funcs waylanddrv_funcs =
     .pwine_get_wgl_driver = WAYLAND_wine_get_wgl_driver,
 };

+BOOL waylanddrv_unaccelerated_pointer;
+
+static NTSTATUS waylanddrv_unix_set_unaccelerated_pointer(void *arg)
+{
+    waylanddrv_unaccelerated_pointer = arg ? *((BOOL *)arg) : FALSE;
+    return 0;
+}
+
 static NTSTATUS waylanddrv_unix_init(void *arg)
 {
     /* Set the user driver functions now so that they are available during
@@ -75,6 +83,7 @@ const unixlib_entry_t __wine_unix_call_funcs[] =
 {
     waylanddrv_unix_init,
     waylanddrv_unix_read_events,
+    waylanddrv_unix_set_unaccelerated_pointer,
 };

C_ASSERT(ARRAYSIZE(__wine_unix_call_funcs) == waylanddrv_unix_func_count);
@@ -85,6 +94,7 @@ const unixlib_entry_t __wine_unix_call_wow64_funcs[] =
 {
     waylanddrv_unix_init,
     waylanddrv_unix_read_events,
+    waylanddrv_unix_set_unaccelerated_pointer,
 };

C_ASSERT(ARRAYSIZE(__wine_unix_call_wow64_funcs) == waylanddrv_unix_func_count);