[PATCH 0/3] MR406: win32u: Introduce new add_modes device manager callback.
From: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> --- dlls/winex11.drv/desktop.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/winex11.drv/desktop.c b/dlls/winex11.drv/desktop.c index 162b524d504..3e589f9d940 100644 --- a/dlls/winex11.drv/desktop.c +++ b/dlls/winex11.drv/desktop.c @@ -335,6 +335,16 @@ void X11DRV_init_desktop( Window win, unsigned int width, unsigned int height ) max_width = primary_rect.right; max_height = primary_rect.bottom; + /* Initialize virtual desktop display settings handler */ + settings_handler.name = "Virtual Desktop"; + settings_handler.priority = 1000; + settings_handler.get_id = X11DRV_desktop_get_id; + settings_handler.get_modes = X11DRV_desktop_get_modes; + settings_handler.free_modes = X11DRV_desktop_free_modes; + settings_handler.get_current_mode = X11DRV_desktop_get_current_mode; + settings_handler.set_current_mode = X11DRV_desktop_set_current_mode; + X11DRV_Settings_SetHandler( &settings_handler ); + /* Initialize virtual desktop mode display device handler */ desktop_handler.name = "Virtual Desktop"; desktop_handler.get_gpus = X11DRV_desktop_get_gpus; @@ -346,16 +356,6 @@ void X11DRV_init_desktop( Window win, unsigned int width, unsigned int height ) desktop_handler.register_event_handlers = NULL; TRACE("Display device functions are now handled by: Virtual Desktop\n"); X11DRV_DisplayDevices_Init( TRUE ); - - /* Initialize virtual desktop display settings handler */ - settings_handler.name = "Virtual Desktop"; - settings_handler.priority = 1000; - settings_handler.get_id = X11DRV_desktop_get_id; - settings_handler.get_modes = X11DRV_desktop_get_modes; - settings_handler.free_modes = X11DRV_desktop_free_modes; - settings_handler.get_current_mode = X11DRV_desktop_get_current_mode; - settings_handler.set_current_mode = X11DRV_desktop_set_current_mode; - X11DRV_Settings_SetHandler( &settings_handler ); } -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/406
From: Rémi Bernon <rbernon(a)codeweavers.com> And use it to enumerate display modes in winex11 and wineandroid. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> --- dlls/win32u/sysparams.c | 106 +++++++++++++++++++++++++++++------- dlls/wineandroid.drv/init.c | 6 ++ dlls/winex11.drv/display.c | 16 ++++++ include/wine/gdi_driver.h | 1 + 4 files changed, 109 insertions(+), 20 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 9580452b50f..0b7e7aefd61 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -178,6 +178,8 @@ static const WCHAR x_panningW[] = {'X','P','a','n','n','i','n','g',0}; static const WCHAR y_panningW[] = {'Y','P','a','n','n','i','n','g',0}; static const WCHAR orientationW[] = {'O','r','i','e','n','t','a','t','i','o','n',0}; static const WCHAR fixed_outputW[] = {'F','i','x','e','d','O','u','t','p','u','t',0}; +static const WCHAR driver_extraW[] = {'D','r','i','v','e','r','E','x','t','r','a',0}; +static const WCHAR mode_countW[] = {'M','o','d','e','C','o','u','n','t',0}; static const char guid_devclass_displayA[] = "{4D36E968-E325-11CE-BFC1-08002BE10318}"; static const WCHAR guid_devclass_displayW[] = @@ -420,19 +422,20 @@ static void release_display_device_init_mutex( HANDLE mutex ) NtClose( mutex ); } -static BOOL write_adapter_mode( HKEY adapter_key, const DEVMODEW *mode ) +static BOOL write_adapter_mode( HKEY adapter_key, DWORD index, const DEVMODEW *mode ) { - static const WCHAR default_settingsW[] = {'D','e','f','a','u','l','t','S','e','t','t','i','n','g','s','.',0}; WCHAR bufferW[MAX_PATH]; + char buffer[MAX_PATH]; + HKEY hkey; + BOOL ret; -#define set_mode_field( name, field, flag ) \ - do \ - { \ - lstrcpyW( bufferW, default_settingsW ); \ - lstrcatW( bufferW, (name) ); \ - if (!set_reg_value( adapter_key, bufferW, REG_DWORD, &mode->field, sizeof(mode->field) )) \ - return FALSE; \ - } while (0) + sprintf( buffer, "Modes\\%08X", index ); + if (!(hkey = reg_create_key( adapter_key, bufferW, asciiz_to_unicode( bufferW, buffer ) - sizeof(WCHAR), + REG_OPTION_VOLATILE, NULL ))) + return FALSE; + +#define set_mode_field( name, field, flag ) \ + if (!(ret = set_reg_value( hkey, (name), REG_DWORD, &mode->field, sizeof(mode->field) ))) goto done; set_mode_field( bits_per_pelW, dmBitsPerPel, DM_BITSPERPEL ); set_mode_field( x_resolutionW, dmPelsWidth, DM_PELSWIDTH ); @@ -443,26 +446,34 @@ static BOOL write_adapter_mode( HKEY adapter_key, const DEVMODEW *mode ) set_mode_field( fixed_outputW, dmDisplayFixedOutput, DM_DISPLAYFIXEDOUTPUT ); set_mode_field( x_panningW, dmPosition.x, DM_POSITION ); set_mode_field( y_panningW, dmPosition.y, DM_POSITION ); + ret = set_reg_value( hkey, driver_extraW, REG_BINARY, mode + 1, mode->dmDriverExtra ); #undef set_mode_field - return TRUE; +done: + NtClose( hkey ); + return ret; } -static BOOL read_adapter_mode( HKEY adapter_key, DEVMODEW *mode ) +static BOOL read_adapter_mode( HKEY adapter_key, DWORD index, DEVMODEW *mode ) { - static const WCHAR default_settingsW[] = {'D','e','f','a','u','l','t','S','e','t','t','i','n','g','s','.',0}; char value_buf[offsetof(KEY_VALUE_PARTIAL_INFORMATION, Data[sizeof(DWORD)])]; KEY_VALUE_PARTIAL_INFORMATION *value = (void *)value_buf; WCHAR bufferW[MAX_PATH]; + char buffer[MAX_PATH]; + HKEY hkey; + BOOL ret; + + sprintf( buffer, "Modes\\%08X", index ); + if (!(hkey = reg_open_key( adapter_key, bufferW, asciiz_to_unicode( bufferW, buffer ) - sizeof(WCHAR) ))) + return FALSE; #define query_mode_field( name, field, flag ) \ do \ { \ - lstrcpyW( bufferW, default_settingsW ); \ - lstrcatW( bufferW, (name) ); \ - if (!query_reg_value( adapter_key, bufferW, value, sizeof(value_buf) ) || \ - value->Type != REG_DWORD) return FALSE; \ + ret = query_reg_value( hkey, (name), value, sizeof(value_buf) ) && \ + value->Type == REG_DWORD; \ + if (!ret) goto done; \ mode->field = *(const DWORD *)value->Data; \ mode->dmFields |= (flag); \ } while (0) @@ -479,6 +490,13 @@ static BOOL read_adapter_mode( HKEY adapter_key, DEVMODEW *mode ) #undef query_mode_field + ret = query_reg_value( hkey, driver_extraW, value, sizeof(value_buf) ) && + value->Type == REG_BINARY; + if (ret && value->DataLength <= mode->dmDriverExtra) + memcpy( mode + 1, value->Data, mode->dmDriverExtra ); + +done: + NtClose( hkey ); return TRUE; } @@ -494,7 +512,7 @@ static BOOL read_registry_settings( const WCHAR *adapter_path, DEVMODEW *mode ) else if (!(hkey = reg_open_key( config_key, adapter_path, lstrlenW( adapter_path ) * sizeof(WCHAR) ))) ret = FALSE; else { - ret = read_adapter_mode( hkey, mode ); + ret = read_adapter_mode( hkey, ENUM_REGISTRY_SETTINGS, mode ); NtClose( hkey ); } @@ -514,7 +532,7 @@ static BOOL write_registry_settings( const WCHAR *adapter_path, const DEVMODEW * if (!(hkey = reg_open_key( config_key, adapter_path, lstrlenW( adapter_path ) * sizeof(WCHAR) ))) ret = FALSE; else { - ret = write_adapter_mode( hkey, mode ); + ret = write_adapter_mode( hkey, ENUM_REGISTRY_SETTINGS, mode ); NtClose( hkey ); } @@ -909,6 +927,8 @@ struct device_manager_ctx unsigned int video_count; unsigned int monitor_count; unsigned int output_count; + unsigned int mode_count; + unsigned int driver_extra; HANDLE mutex; WCHAR gpuid[128]; WCHAR gpu_guid[64]; @@ -998,6 +1018,8 @@ static void add_gpu( const struct gdi_gpu *gpu, void *param ) gpu_index = ctx->gpu_count++; ctx->adapter_count = 0; ctx->monitor_count = 0; + ctx->mode_count = 0; + ctx->driver_extra = 0; if (!enum_key && !(enum_key = reg_create_key( NULL, enum_keyW, sizeof(enum_keyW), 0, NULL ))) return; @@ -1127,6 +1149,8 @@ static void add_adapter( const struct gdi_adapter *adapter, void *param ) adapter_index = ctx->adapter_count++; video_index = ctx->video_count++; ctx->monitor_count = 0; + ctx->mode_count = 0; + ctx->driver_extra = 0; len = asciiz_to_unicode( bufferW, "\\Registry\\Machine\\System\\CurrentControlSet\\" "Control\\Video\\" ) / sizeof(WCHAR) - 1; @@ -1282,11 +1306,36 @@ static void add_monitor( const struct gdi_monitor *monitor, void *param ) if (hkey) NtClose( hkey ); } +static void add_mode( const DEVMODEW *mode, void *param ) +{ + struct device_manager_ctx *ctx = param; + + if (!ctx->adapter_count) + { + static const struct gdi_adapter default_adapter = + { + .state_flags = DISPLAY_DEVICE_ATTACHED_TO_DESKTOP | DISPLAY_DEVICE_PRIMARY_DEVICE | DISPLAY_DEVICE_VGA_COMPATIBLE, + }; + TRACE( "adding default fake adapter\n" ); + add_adapter( &default_adapter, ctx ); + } + + if (write_adapter_mode( ctx->adapter_key, ctx->mode_count, mode )) + { + ctx->driver_extra = max( ctx->driver_extra, mode->dmDriverExtra ); + set_reg_value( ctx->adapter_key, driver_extraW, REG_DWORD, &ctx->driver_extra, sizeof(ctx->driver_extra) ); + + ctx->mode_count++; + set_reg_value( ctx->adapter_key, mode_countW, REG_DWORD, &ctx->mode_count, sizeof(ctx->mode_count) ); + } +} + static const struct gdi_device_manager device_manager = { add_gpu, add_adapter, add_monitor, + add_mode, }; static void release_display_manager_ctx( struct device_manager_ctx *ctx ) @@ -1431,22 +1480,39 @@ static BOOL update_display_cache(void) if (!user_driver->pUpdateDisplayDevices( &device_manager, TRUE, &ctx )) { + static const DEVMODEW modes[] = + { + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 32, .dmPelsWidth = 640, .dmPelsHeight = 480, .dmDisplayFrequency = 60, }, + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 32, .dmPelsWidth = 800, .dmPelsHeight = 600, .dmDisplayFrequency = 60, }, + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 32, .dmPelsWidth = 1024, .dmPelsHeight = 768, .dmDisplayFrequency = 60, }, + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 16, .dmPelsWidth = 640, .dmPelsHeight = 480, .dmDisplayFrequency = 60, }, + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 16, .dmPelsWidth = 800, .dmPelsHeight = 600, .dmDisplayFrequency = 60, }, + { .dmFields = DM_DISPLAYORIENTATION|DM_BITSPERPEL|DM_PELSWIDTH|DM_PELSHEIGHT|DM_DISPLAYFLAGS|DM_DISPLAYFREQUENCY|DM_POSITION, + .dmBitsPerPel = 16, .dmPelsWidth = 1024, .dmPelsHeight = 768, .dmDisplayFrequency = 60, }, + }; static const struct gdi_gpu gpu; static const struct gdi_adapter adapter = { .state_flags = DISPLAY_DEVICE_ATTACHED_TO_DESKTOP | DISPLAY_DEVICE_PRIMARY_DEVICE | DISPLAY_DEVICE_VGA_COMPATIBLE, }; - DEVMODEW mode = {.dmPelsWidth = 1024, .dmPelsHeight = 768}; + DEVMODEW mode = modes[2]; struct gdi_monitor monitor = { .state_flags = DISPLAY_DEVICE_ACTIVE | DISPLAY_DEVICE_ATTACHED, .rc_monitor = {.right = mode.dmPelsWidth, .bottom = mode.dmPelsHeight}, .rc_work = {.right = mode.dmPelsWidth, .bottom = mode.dmPelsHeight}, }; + UINT i; add_gpu( &gpu, &ctx ); add_adapter( &adapter, &ctx ); add_monitor( &monitor, &ctx ); + for (i = 0; i < ARRAY_SIZE(modes); ++i) add_mode( modes + i, &ctx ); } release_display_manager_ctx( &ctx ); diff --git a/dlls/wineandroid.drv/init.c b/dlls/wineandroid.drv/init.c index a82932bf65f..e23f2b62f7e 100644 --- a/dlls/wineandroid.drv/init.c +++ b/dlls/wineandroid.drv/init.c @@ -288,9 +288,15 @@ BOOL ANDROID_UpdateDisplayDevices( const struct gdi_device_manager *device_manag .rc_work = monitor_rc_work, .state_flags = DISPLAY_DEVICE_ACTIVE | DISPLAY_DEVICE_ATTACHED, }; + const DEVMODEW mode = + { + .dmFields = DM_PELSWIDTH | DM_PELSHEIGHT | DM_BITSPERPEL | DM_DISPLAYFLAGS | DM_DISPLAYFREQUENCY, + .dmBitsPerPel = screen_bpp, .dmPelsWidth = screen_width, .dmPelsHeight = screen_height, .dmDisplayFrequency = 60, + }; device_manager->add_gpu( &gpu, param ); device_manager->add_adapter( &adapter, param ); device_manager->add_monitor( &gdi_monitor, param ); + device_manager->add_mode( &mode, param ); force_display_devices_refresh = FALSE; } diff --git a/dlls/winex11.drv/display.c b/dlls/winex11.drv/display.c index cbf1e7f70cd..174411c7915 100644 --- a/dlls/winex11.drv/display.c +++ b/dlls/winex11.drv/display.c @@ -1026,6 +1026,8 @@ BOOL X11DRV_UpdateDisplayDevices( const struct gdi_device_manager *device_manage struct gdi_gpu *gpus; INT gpu_count, adapter_count, monitor_count; INT gpu, adapter, monitor; + DEVMODEW *modes, *mode; + DWORD mode_count; if (!force && !force_display_devices_refresh) return TRUE; force_display_devices_refresh = FALSE; @@ -1060,6 +1062,20 @@ BOOL X11DRV_UpdateDisplayDevices( const struct gdi_device_manager *device_manage } handler->free_monitors(monitors, monitor_count); + + if (!settings_handler.get_modes( adapters[adapter].id, EDS_ROTATEDMODE, &modes, &mode_count )) + continue; + + qsort( modes, mode_count, sizeof(*modes) + modes[0].dmDriverExtra, mode_compare ); + + for (mode = modes; mode_count; mode_count--) + { + TRACE( "mode: %p\n", mode ); + device_manager->add_mode( mode, param ); + mode = (DEVMODEW *)((char *)mode + sizeof(*modes) + modes[0].dmDriverExtra); + } + + settings_handler.free_modes( modes ); } handler->free_adapters(adapters); diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h index 77ccc431e51..9acfaf1f65f 100644 --- a/include/wine/gdi_driver.h +++ b/include/wine/gdi_driver.h @@ -266,6 +266,7 @@ struct gdi_device_manager void (*add_gpu)( const struct gdi_gpu *gpu, void *param ); void (*add_adapter)( const struct gdi_adapter *adapter, void *param ); void (*add_monitor)( const struct gdi_monitor *monitor, void *param ); + void (*add_mode)( const DEVMODEW *mode, void *param ); }; struct tagUPDATELAYEREDWINDOWINFO; -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/406
From: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> --- dlls/win32u/sysparams.c | 29 +++++++++++++++++++++++++++-- 1 file changed, 27 insertions(+), 2 deletions(-) diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 0b7e7aefd61..820de2743cf 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -203,6 +203,8 @@ static const WCHAR guid_devinterface_monitorW[] = {'{','E','6','F','0','7','B','5','F','-','E','E','9','7','-','4','A','9','0','-', 'B','0','7','6','-','3','3','F','5','7','B','F','4','E','A','A','7','}',0}; +#define NEXT_DEVMODEW(mode) ((DEVMODEW *)((char *)((mode) + 1) + (mode)->dmDriverExtra)) + /* Cached display device information */ struct display_device { @@ -220,6 +222,8 @@ struct adapter struct display_device dev; unsigned int id; const WCHAR *config_key; + unsigned int mode_count; + DEVMODEW *modes; }; struct monitor @@ -545,8 +549,9 @@ static BOOL read_display_adapter_settings( unsigned int index, struct adapter *i char buffer[4096]; KEY_VALUE_PARTIAL_INFORMATION *value = (void *)buffer; WCHAR *value_str = (WCHAR *)value->Data; + DWORD i, driver_extra = 0, size; + DEVMODEW *mode; HKEY hkey; - DWORD size; if (!enum_key && !(enum_key = reg_open_key( NULL, enum_keyW, sizeof(enum_keyW) ))) return FALSE; @@ -585,10 +590,28 @@ static BOOL read_display_adapter_settings( unsigned int index, struct adapter *i /* Interface name */ info->dev.interface_name[0] = 0; + /* ModeCount / DriverExtra */ + if (query_reg_value( hkey, mode_countW, value, sizeof(buffer) ) && value->Type == REG_DWORD) + info->mode_count = *(const DWORD *)value->Data; + if (query_reg_value( hkey, driver_extraW, value, sizeof(buffer) ) && value->Type == REG_DWORD) + driver_extra = *(const DWORD *)value->Data; + + /* Modes */ + if ((info->modes = calloc( info->mode_count, sizeof(DEVMODEW) + driver_extra ))) + { + for (i = 0, mode = info->modes; i < info->mode_count; i++) + { + mode->dmDriverExtra = driver_extra; + if (!read_adapter_mode( hkey, i, mode )) break; + mode = NEXT_DEVMODEW(mode); + } + info->mode_count = i; + } + /* DeviceID */ size = query_reg_value( hkey, gpu_idW, value, sizeof(buffer) ); NtClose( hkey ); - if (!size || value->Type != REG_SZ) return FALSE; + if (!size || value->Type != REG_SZ || !info->mode_count || !info->modes) return FALSE; if (!(hkey = reg_open_key( enum_key, value_str, value->DataLength - sizeof(WCHAR) ))) return FALSE; @@ -1375,6 +1398,7 @@ static void clear_display_devices(void) { adapter = LIST_ENTRY( list_head( &adapters ), struct adapter, entry ); list_remove( &adapter->entry ); + free( adapter->modes ); free( adapter ); } } @@ -1413,6 +1437,7 @@ static BOOL update_display_cache_from_registry(void) if (!read_display_adapter_settings( adapter_id, adapter )) { + free( adapter->modes ); free( adapter ); break; } -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/406
The third commit is breaking things on macOS. ``` $ ./wine programs/wordpad/wordpad.exe 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config ... ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/406#note_3603
On Fri Jul 8 09:31:20 2022 +0000, Huw Davies wrote:
The third commit is breaking things on macOS. ``` $ ./wine programs/wordpad/wordpad.exe 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config 007c:err:system:update_display_cache failed to read display config ... ``` Thanks, I guess I should try figuring how it lists modes exactly and use the callback directly there too...
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/406#note_3605
participants (2)
-
Huw Davies (＠huw) -
Rémi Bernon