February 2024 - Wine-commits

Alexandre Julliard : server: Don't update the entry point in the image information for ARM64EC modules.
by Alexandre Julliard 08 Feb '24

08 Feb '24

Module: wine Branch: master Commit: e05c6c821104d45344fc532a15f96360a24a0174 URL: https://gitlab.winehq.org/wine/wine/-/commit/e05c6c821104d45344fc532a15f963… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 8 13:10:44 2024 +0100 server: Don't update the entry point in the image information for ARM64EC modules. --- dlls/ntdll/tests/wow64.c | 39 ++++++++++++++++++++++++++++++++++----- dlls/ntdll/unix/server.c | 5 ++--- include/wine/server_protocol.h | 5 ++--- server/debugger.c | 2 +- server/mapping.c | 8 ++++++-- server/process.c | 13 +------------ server/protocol.def | 1 - server/request.h | 5 ++--- server/trace.c | 4 +--- 9 files changed, 49 insertions(+), 33 deletions(-)

1 0

Alexandre Julliard : ntdll: Redirect the module entry point for ARM64EC modules.
by Alexandre Julliard 08 Feb '24

08 Feb '24

Module: wine Branch: master Commit: 8c703f3e2f8162364564a0d47307887d9de5f2a4 URL: https://gitlab.winehq.org/wine/wine/-/commit/8c703f3e2f8162364564a0d4730788… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 8 14:00:20 2024 +0100 ntdll: Redirect the module entry point for ARM64EC modules. --- dlls/ntdll/tests/wow64.c | 1 - dlls/ntdll/unix/loader.c | 19 ++++++++----------- dlls/ntdll/unix/unix_private.h | 1 + dlls/ntdll/unix/virtual.c | 11 ++++++----- 4 files changed, 15 insertions(+), 17 deletions(-) diff --git a/dlls/ntdll/tests/wow64.c b/dlls/ntdll/tests/wow64.c index 941b127a2f6..a1beb9a4997 100644 --- a/dlls/ntdll/tests/wow64.c +++ b/dlls/ntdll/tests/wow64.c @@ -275,7 +275,6 @@ static void test_process_machine( HANDLE process, HANDLE thread, win32_entry is the redirected x86-64 -> ARM64EC one */ ok( image.TransferAddress != entry_point, "wrong entry %p\n", image.TransferAddress ); ok( image.TransferAddress != win32_entry, "wrong entry %p\n", image.TransferAddress ); - todo_wine ok( win32_entry != entry_point, "wrong win32 entry %p\n", win32_entry ); } } diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 05284115b67..88b24420ec1 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -1585,24 +1585,23 @@ static void load_ntdll_wow64_functions( HMODULE module ) /*********************************************************************** - * redirect_arm64ec_ptr + * redirect_arm64ec_rva * - * Redirect a function pointer through the arm64ec redirection table. + * Redirect an address through the arm64ec redirection table. */ -static void *redirect_arm64ec_ptr( void *module, void *ptr, - const IMAGE_ARM64EC_REDIRECTION_ENTRY *map, ULONG map_count ) +ULONG_PTR redirect_arm64ec_rva( void *base, ULONG_PTR rva, const IMAGE_ARM64EC_METADATA *metadata ) { - int min = 0, max = map_count - 1; - ULONG_PTR rva = (char *)ptr - (char *)module; + const IMAGE_ARM64EC_REDIRECTION_ENTRY *map = get_rva( base, metadata->RedirectionMetadata ); + int min = 0, max = metadata->RedirectionMetadataCount - 1; while (min <= max) { int pos = (min + max) / 2; - if (map[pos].Source == rva) return get_rva( module, map[pos].Destination ); + if (map[pos].Source == rva) return map[pos].Destination; if (map[pos].Source < rva) min = pos + 1; else max = pos - 1; } - return ptr; + return rva; } @@ -1615,13 +1614,11 @@ static void redirect_ntdll_functions( HMODULE module ) { const IMAGE_LOAD_CONFIG_DIRECTORY *loadcfg; const IMAGE_ARM64EC_METADATA *metadata; - const IMAGE_ARM64EC_REDIRECTION_ENTRY *map; if (!(loadcfg = get_module_data_dir( module, IMAGE_DIRECTORY_ENTRY_LOAD_CONFIG, NULL ))) return; if (!(metadata = (void *)loadcfg->CHPEMetadataPointer)) return; - if (!(map = get_rva( module, metadata->RedirectionMetadata ))) return; #define REDIRECT(name) \ - p##name = redirect_arm64ec_ptr( module, p##name, map, metadata->RedirectionMetadataCount ) + p##name = get_rva( module, redirect_arm64ec_rva( module, (char *)p##name - (char *)module, metadata )) REDIRECT( DbgUiRemoteBreakin ); REDIRECT( KiRaiseUserExceptionDispatcher ); REDIRECT( KiUserExceptionDispatcher ); diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index af746b220e8..cbf72651a9a 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -191,6 +191,7 @@ extern BOOL is_builtin_path( const UNICODE_STRING *path, WORD *machine ); extern NTSTATUS load_main_exe( const WCHAR *name, const char *unix_name, const WCHAR *curdir, USHORT load_machine, WCHAR **image, void **module ); extern NTSTATUS load_start_exe( WCHAR **image, void **module ); +extern ULONG_PTR redirect_arm64ec_rva( void *module, ULONG_PTR rva, const IMAGE_ARM64EC_METADATA *metadata ); extern void start_server( BOOL debug ); extern unsigned int server_call_unlocked( void *req_ptr ); diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index cfe19eae89d..02c0bebe879 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -2488,7 +2488,7 @@ static void alloc_arm64ec_map(void) * update_arm64ec_ranges */ static void update_arm64ec_ranges( struct file_view *view, IMAGE_NT_HEADERS *nt, - const IMAGE_DATA_DIRECTORY *dir ) + const IMAGE_DATA_DIRECTORY *dir, UINT *entry_point ) { const IMAGE_ARM64EC_METADATA *metadata; const IMAGE_CHPE_RANGE_ENTRY *map; @@ -2501,6 +2501,7 @@ static void update_arm64ec_ranges( struct file_view *view, IMAGE_NT_HEADERS *nt, if (!arm64ec_view) alloc_arm64ec_map(); commit_arm64ec_map( view ); metadata = (void *)(base + (cfg->CHPEMetadataPointer - nt->OptionalHeader.ImageBase)); + *entry_point = redirect_arm64ec_rva( base, nt->OptionalHeader.AddressOfEntryPoint, metadata ); if (!metadata->CodeMap) return; map = (void *)(base + metadata->CodeMap); @@ -2865,11 +2866,11 @@ static NTSTATUS map_image_into_view( struct file_view *view, const WCHAR *filena (!machine && main_image_info.Machine == IMAGE_FILE_MACHINE_AMD64))) { update_arm64x_mapping( view, nt, dir, sections ); - /* reload changed data from NT header */ - image_info->machine = nt->FileHeader.Machine; - image_info->entry_point = nt->OptionalHeader.AddressOfEntryPoint; + /* reload changed machine from NT header */ + image_info->machine = nt->FileHeader.Machine; } - if (image_info->machine == IMAGE_FILE_MACHINE_AMD64) update_arm64ec_ranges( view, nt, dir ); + if (image_info->machine == IMAGE_FILE_MACHINE_AMD64) + update_arm64ec_ranges( view, nt, dir, &image_info->entry_point ); } #endif if (machine && machine != nt->FileHeader.Machine) return STATUS_NOT_SUPPORTED;

1 0

Alexandre Julliard : server: Add hybrid flag in image mapping information.
by Alexandre Julliard 08 Feb '24

08 Feb '24

Module: wine Branch: master Commit: de988eeea0801289adb9dbd2420a66d142382ab5 URL: https://gitlab.winehq.org/wine/wine/-/commit/de988eeea0801289adb9dbd2420a66… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 8 15:39:46 2024 +0100 server: Add hybrid flag in image mapping information. --- include/wine/server_protocol.h | 5 ++-- server/mapping.c | 68 ++++++++++++++++++++++++++++++++++-------- server/protocol.def | 3 +- 3 files changed, 60 insertions(+), 16 deletions(-) diff --git a/include/wine/server_protocol.h b/include/wine/server_protocol.h index 0286ee7056e..e039cf59e72 100644 --- a/include/wine/server_protocol.h +++ b/include/wine/server_protocol.h @@ -844,7 +844,8 @@ typedef struct unsigned char contains_code : 1; unsigned char wine_builtin : 1; unsigned char wine_fakedll : 1; - unsigned char padding : 5; + unsigned char is_hybrid : 1; + unsigned char padding : 4; unsigned char image_flags; unsigned int loader_flags; unsigned int header_size; @@ -6506,7 +6507,7 @@ union generic_reply /* ### protocol_version begin ### */ -#define SERVER_PROTOCOL_VERSION 787 +#define SERVER_PROTOCOL_VERSION 788 /* ### protocol_version end ### */ diff --git a/server/mapping.c b/server/mapping.c index 13604455729..6b0de1b8b94 100644 --- a/server/mapping.c +++ b/server/mapping.c @@ -654,11 +654,10 @@ static int build_shared_mapping( struct mapping *mapping, int fd, return 0; } -/* load the CLR header from its section */ -static int load_clr_header( IMAGE_COR20_HEADER *hdr, size_t va, size_t size, int unix_fd, - IMAGE_SECTION_HEADER *sec, unsigned int nb_sec ) +/* load a data directory header from its section */ +static int load_data_dir( void *dir, size_t dir_size, size_t va, size_t size, int unix_fd, + IMAGE_SECTION_HEADER *sec, unsigned int nb_sec ) { - ssize_t ret; size_t map_size, file_size; off_t file_start; unsigned int i; @@ -672,18 +671,41 @@ static int load_clr_header( IMAGE_COR20_HEADER *hdr, size_t va, size_t size, int get_section_sizes( &sec[i], &map_size, &file_start, &file_size ); if (size >= map_size) continue; if (va - sec[i].VirtualAddress >= map_size - size) continue; - file_size = min( file_size, map_size ); - size = min( size, sizeof(*hdr) ); - ret = pread( unix_fd, hdr, min( size, file_size ), file_start + va - sec[i].VirtualAddress ); - if (ret <= 0) break; - if (ret < sizeof(*hdr)) memset( (char *)hdr + ret, 0, sizeof(*hdr) - ret ); - return (hdr->MajorRuntimeVersion > COR_VERSION_MAJOR_V2 || - (hdr->MajorRuntimeVersion == COR_VERSION_MAJOR_V2 && - hdr->MinorRuntimeVersion >= COR_VERSION_MINOR)); + if (size > dir_size) size = dir_size; + if (size > file_size) size = file_size; + return pread( unix_fd, dir, size, file_start + va - sec[i].VirtualAddress ); } return 0; } +/* load the CLR header from its section */ +static int load_clr_header( IMAGE_COR20_HEADER *hdr, size_t va, size_t size, int unix_fd, + IMAGE_SECTION_HEADER *sec, unsigned int nb_sec ) +{ + int ret = load_data_dir( hdr, sizeof(*hdr), va, size, unix_fd, sec, nb_sec ); + + if (ret <= 0) return 0; + if (ret < sizeof(*hdr)) memset( (char *)hdr + ret, 0, sizeof(*hdr) - ret ); + return (hdr->MajorRuntimeVersion > COR_VERSION_MAJOR_V2 || + (hdr->MajorRuntimeVersion == COR_VERSION_MAJOR_V2 && + hdr->MinorRuntimeVersion >= COR_VERSION_MINOR)); +} + +/* load the LOAD_CONFIG header from its section */ +static int load_cfg_header( IMAGE_LOAD_CONFIG_DIRECTORY64 *cfg, size_t va, size_t size, + int unix_fd, IMAGE_SECTION_HEADER *sec, unsigned int nb_sec ) +{ + unsigned int cfg_size; + int ret = load_data_dir( cfg, sizeof(*cfg), va, size, unix_fd, sec, nb_sec ); + + if (ret <= 0) return 0; + cfg_size = ret; + if (cfg_size < offsetof( IMAGE_LOAD_CONFIG_DIRECTORY64, Size ) + sizeof(cfg_size)) return 0; + if (cfg_size > cfg->Size) cfg_size = cfg->Size; + if (cfg_size < sizeof(*cfg)) memset( (char *)cfg + cfg_size, 0, sizeof(*cfg) - cfg_size ); + return 1; +} + /* retrieve the mapping parameters for an executable (PE) image */ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_size, int unix_fd ) { @@ -707,9 +729,14 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s IMAGE_OPTIONAL_HEADER64 hdr64; } opt; } nt; + union + { + IMAGE_LOAD_CONFIG_DIRECTORY32 cfg32; + IMAGE_LOAD_CONFIG_DIRECTORY64 cfg64; + } cfg; off_t pos; int size, has_relocs; - size_t mz_size, clr_va = 0, clr_size = 0; + size_t mz_size, clr_va = 0, clr_size = 0, cfg_va, cfg_size; unsigned int i; /* load the headers */ @@ -753,6 +780,9 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s clr_va = nt.opt.hdr32.DataDirectory[IMAGE_DIRECTORY_ENTRY_COM_DESCRIPTOR].VirtualAddress; clr_size = nt.opt.hdr32.DataDirectory[IMAGE_DIRECTORY_ENTRY_COM_DESCRIPTOR].Size; } + cfg_va = nt.opt.hdr32.DataDirectory[IMAGE_DIRECTORY_ENTRY_LOAD_CONFIG].VirtualAddress; + cfg_size = nt.opt.hdr32.DataDirectory[IMAGE_DIRECTORY_ENTRY_LOAD_CONFIG].Size; + mapping->image.base = nt.opt.hdr32.ImageBase; mapping->image.entry_point = nt.opt.hdr32.AddressOfEntryPoint; mapping->image.map_size = ROUND_SIZE( nt.opt.hdr32.SizeOfImage ); @@ -801,6 +831,9 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s clr_va = nt.opt.hdr64.DataDirectory[IMAGE_DIRECTORY_ENTRY_COM_DESCRIPTOR].VirtualAddress; clr_size = nt.opt.hdr64.DataDirectory[IMAGE_DIRECTORY_ENTRY_COM_DESCRIPTOR].Size; } + cfg_va = nt.opt.hdr64.DataDirectory[IMAGE_DIRECTORY_ENTRY_LOAD_CONFIG].VirtualAddress; + cfg_size = nt.opt.hdr64.DataDirectory[IMAGE_DIRECTORY_ENTRY_LOAD_CONFIG].Size; + mapping->image.base = nt.opt.hdr64.ImageBase; mapping->image.entry_point = nt.opt.hdr64.AddressOfEntryPoint; mapping->image.map_size = ROUND_SIZE( nt.opt.hdr64.SizeOfImage ); @@ -834,6 +867,7 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s return STATUS_INVALID_IMAGE_FORMAT; } + mapping->image.is_hybrid = 0; mapping->image.padding = 0; mapping->image.map_addr = get_fd_map_address( mapping->fd ); mapping->image.image_charact = nt.FileHeader.Characteristics; @@ -875,6 +909,14 @@ static unsigned int get_image_params( struct mapping *mapping, file_pos_t file_s } } + if (load_cfg_header( &cfg.cfg64, cfg_va, cfg_size, unix_fd, sec, nt.FileHeader.NumberOfSections )) + { + if (nt.opt.hdr32.Magic == IMAGE_NT_OPTIONAL_HDR32_MAGIC) + mapping->image.is_hybrid = !!cfg.cfg32.CHPEMetadataPointer; + else + mapping->image.is_hybrid = !!cfg.cfg64.CHPEMetadataPointer; + } + if (!build_shared_mapping( mapping, unix_fd, sec, nt.FileHeader.NumberOfSections )) return STATUS_INVALID_FILE_FOR_SECTION; diff --git a/server/protocol.def b/server/protocol.def index ee4648350db..3c452e08e2a 100644 --- a/server/protocol.def +++ b/server/protocol.def @@ -860,7 +860,8 @@ typedef struct unsigned char contains_code : 1; unsigned char wine_builtin : 1; unsigned char wine_fakedll : 1; - unsigned char padding : 5; + unsigned char is_hybrid : 1; + unsigned char padding : 4; unsigned char image_flags; unsigned int loader_flags; unsigned int header_size;

1 0

Alexandre Julliard : ntdll: Use the current machine by default to create an ARM64X process.
by Alexandre Julliard 08 Feb '24

08 Feb '24

Module: wine Branch: master Commit: 1456b31eecfef91fc8ffc8c2de29ab69c5b5463b URL: https://gitlab.winehq.org/wine/wine/-/commit/1456b31eecfef91fc8ffc8c2de29ab… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 8 15:46:17 2024 +0100 ntdll: Use the current machine by default to create an ARM64X process. --- dlls/ntdll/unix/loader.c | 3 ++- dlls/ntdll/unix/process.c | 7 ++++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 88b24420ec1..06959fa95df 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -480,7 +480,8 @@ char *get_alternate_wineloader( WORD machine ) if (machine == current_machine) return NULL; - if (machine == IMAGE_FILE_MACHINE_AMD64) /* try the 64-bit loader */ + /* try the 64-bit loader */ + if (current_machine == IMAGE_FILE_MACHINE_I386 && machine == IMAGE_FILE_MACHINE_AMD64) { size_t len = strlen(wineloader); diff --git a/dlls/ntdll/unix/process.c b/dlls/ntdll/unix/process.c index 71eab55920d..b58d69fb28a 100644 --- a/dlls/ntdll/unix/process.c +++ b/dlls/ntdll/unix/process.c @@ -801,7 +801,12 @@ NTSTATUS WINAPI NtCreateUserProcess( HANDLE *process_handle_ptr, HANDLE *thread_ } goto done; } - if (!machine) machine = pe_info.machine; + if (!machine) + { + machine = pe_info.machine; + if (is_arm64ec() && pe_info.is_hybrid && machine == IMAGE_FILE_MACHINE_ARM64) + machine = main_image_info.Machine; + } if (!(startup_info = create_startup_info( attr.ObjectName, process_flags, params, &pe_info, &startup_info_size ))) goto done; env_size = get_env_size( params, &winedebug );

1 0

Alexandre Julliard : ntdll: Fix RtlWow64GetCurrentMachine() result on ARM64EC.
by Alexandre Julliard 08 Feb '24

08 Feb '24

Module: wine Branch: master Commit: cf7c48e0366ec27dd946f2fcec12cb66f2e0cb23 URL: https://gitlab.winehq.org/wine/wine/-/commit/cf7c48e0366ec27dd946f2fcec12cb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 8 16:30:24 2024 +0100 ntdll: Fix RtlWow64GetCurrentMachine() result on ARM64EC. --- dlls/ntdll/loader.c | 5 ----- dlls/ntdll/ntdll_misc.h | 12 ++++++++++++ dlls/ntdll/process.c | 9 +++++---- dlls/ntdll/tests/wow64.c | 38 ++++++++++++++++---------------------- 4 files changed, 33 insertions(+), 31 deletions(-) diff --git a/dlls/ntdll/loader.c b/dlls/ntdll/loader.c index 4fc8af2662e..55065117a01 100644 --- a/dlls/ntdll/loader.c +++ b/dlls/ntdll/loader.c @@ -51,19 +51,14 @@ WINE_DECLARE_DEBUG_CHANNEL(imports); #ifdef __i386__ static const WCHAR pe_dir[] = L"\\i386-windows"; -static const USHORT current_machine = IMAGE_FILE_MACHINE_I386; #elif defined __x86_64__ static const WCHAR pe_dir[] = L"\\x86_64-windows"; -static const USHORT current_machine = IMAGE_FILE_MACHINE_AMD64; #elif defined __arm__ static const WCHAR pe_dir[] = L"\\arm-windows"; -static const USHORT current_machine = IMAGE_FILE_MACHINE_ARMNT; #elif defined __aarch64__ static const WCHAR pe_dir[] = L"\\aarch64-windows"; -static const USHORT current_machine = IMAGE_FILE_MACHINE_ARM64; #else static const WCHAR pe_dir[] = L""; -static const USHORT current_machine = IMAGE_FILE_MACHINE_UNKNOWN; #endif /* we don't want to include winuser.h */ diff --git a/dlls/ntdll/ntdll_misc.h b/dlls/ntdll/ntdll_misc.h index e4094d341d8..58e29837cd0 100644 --- a/dlls/ntdll/ntdll_misc.h +++ b/dlls/ntdll/ntdll_misc.h @@ -41,6 +41,18 @@ #define NTDLL_TLS_ERRNO 16 /* TLS slot for _errno() */ +#ifdef __i386__ +static const USHORT current_machine = IMAGE_FILE_MACHINE_I386; +#elif defined(__x86_64__) +static const USHORT current_machine = IMAGE_FILE_MACHINE_AMD64; +#elif defined(__arm__) +static const USHORT current_machine = IMAGE_FILE_MACHINE_ARMNT; +#elif defined(__aarch64__) +static const USHORT current_machine = IMAGE_FILE_MACHINE_ARM64; +#else +static const USHORT current_machine = IMAGE_FILE_MACHINE_UNKNOWN; +#endif + #if defined(__i386__) || defined(__x86_64__) || defined(__arm__) || defined(__aarch64__) static const UINT_PTR page_size = 0x1000; #else diff --git a/dlls/ntdll/process.c b/dlls/ntdll/process.c index dcdb26f81d1..102e0c930a9 100644 --- a/dlls/ntdll/process.c +++ b/dlls/ntdll/process.c @@ -94,10 +94,11 @@ NTSTATUS WINAPI RtlWow64EnableFsRedirectionEx( ULONG disable, ULONG *old_value ) */ USHORT WINAPI RtlWow64GetCurrentMachine(void) { - USHORT current, native; - - RtlWow64GetProcessMachines( GetCurrentProcess(), &current, &native ); - return current ? current : native; + USHORT machine = current_machine; +#ifdef _WIN64 + if (NtCurrentTeb()->WowTebOffset) RtlWow64GetCurrentCpuArea( &machine, NULL, NULL ); +#endif + return machine; } diff --git a/dlls/ntdll/tests/wow64.c b/dlls/ntdll/tests/wow64.c index a1beb9a4997..aebff700940 100644 --- a/dlls/ntdll/tests/wow64.c +++ b/dlls/ntdll/tests/wow64.c @@ -289,7 +289,7 @@ static void test_query_architectures(void) STARTUPINFOA si = { sizeof(si) }; NTSTATUS status; HANDLE process; - ULONG len; + ULONG i, len; #ifdef __arm64ec__ BOOL is_arm64ec = TRUE; #else @@ -383,27 +383,21 @@ static void test_query_architectures(void) } if (pRtlWow64IsWowGuestMachineSupported) { - BOOLEAN ret = 0xcc; - status = pRtlWow64IsWowGuestMachineSupported( IMAGE_FILE_MACHINE_I386, &ret ); - ok( !status, "failed %lx\n", status ); - ok( ret == (native_machine == IMAGE_FILE_MACHINE_AMD64 || - native_machine == IMAGE_FILE_MACHINE_ARM64), "wrong result %u\n", ret ); - ret = 0xcc; - status = pRtlWow64IsWowGuestMachineSupported( IMAGE_FILE_MACHINE_ARMNT, &ret ); - ok( !status, "failed %lx\n", status ); - ok( !ret || native_machine == IMAGE_FILE_MACHINE_ARM64, "wrong result %u\n", ret ); - ret = 0xcc; - status = pRtlWow64IsWowGuestMachineSupported( IMAGE_FILE_MACHINE_AMD64, &ret ); - ok( !status, "failed %lx\n", status ); - ok( !ret || native_machine == IMAGE_FILE_MACHINE_ARM64, "wrong result %u\n", ret ); - ret = 0xcc; - status = pRtlWow64IsWowGuestMachineSupported( IMAGE_FILE_MACHINE_ARM64, &ret ); - ok( !status, "failed %lx\n", status ); - ok( !ret, "wrong result %u\n", ret ); - ret = 0xcc; - status = pRtlWow64IsWowGuestMachineSupported( 0xdead, &ret ); - ok( !status, "failed %lx\n", status ); - ok( !ret, "wrong result %u\n", ret ); + static const WORD machines[] = { IMAGE_FILE_MACHINE_I386, IMAGE_FILE_MACHINE_ARMNT, + IMAGE_FILE_MACHINE_AMD64, IMAGE_FILE_MACHINE_ARM64, 0xdead }; + + for (i = 0; i < ARRAY_SIZE(machines); i++) + { + BOOLEAN ret = 0xcc; + status = pRtlWow64IsWowGuestMachineSupported( machines[i], &ret ); + ok( !status, "failed %lx\n", status ); + if (is_machine_32bit( machines[i] ) && !is_machine_32bit( native_machine )) + ok( ret || machines[i] == IMAGE_FILE_MACHINE_ARMNT || + broken(current_machine == IMAGE_FILE_MACHINE_I386), /* win10-1607 wow64 */ + "%04x: got %u\n", machines[i], ret ); + else + ok( !ret, "%04x: got %u\n", machines[i], ret ); + } } }

1 0

Conor McCarthy : vkd3d-shader/dxil: Handle hyperbolic trigonometric functions in sm6_parser_emit_dx_unary().
by Alexandre Julliard 07 Feb '24

07 Feb '24

Module: vkd3d Branch: master Commit: 625e28957479971fc41fef16a65b3736e4492b07 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/625e28957479971fc41fef16a65b3… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Mon Jan 22 15:28:44 2024 +1000 vkd3d-shader/dxil: Handle hyperbolic trigonometric functions in sm6_parser_emit_dx_unary(). --- libs/vkd3d-shader/d3d_asm.c | 3 +++ libs/vkd3d-shader/dxil.c | 12 ++++++++++++ libs/vkd3d-shader/spirv.c | 6 ++++++ libs/vkd3d-shader/vkd3d_shader_private.h | 3 +++ tests/hlsl/trigonometry.shader_test | 12 ++++++------ 5 files changed, 30 insertions(+), 6 deletions(-) diff --git a/libs/vkd3d-shader/d3d_asm.c b/libs/vkd3d-shader/d3d_asm.c index 3ae736a8..3f86bd45 100644 --- a/libs/vkd3d-shader/d3d_asm.c +++ b/libs/vkd3d-shader/d3d_asm.c @@ -171,10 +171,13 @@ static const char * const shader_opcode_names[] = [VKD3DSIH_GATHER4_S ] = "gather4_s", [VKD3DSIH_GEO ] = "ge", [VKD3DSIH_GEU ] = "ge_unord", + [VKD3DSIH_HCOS ] = "hcos", [VKD3DSIH_HS_CONTROL_POINT_PHASE ] = "hs_control_point_phase", [VKD3DSIH_HS_DECLS ] = "hs_decls", [VKD3DSIH_HS_FORK_PHASE ] = "hs_fork_phase", [VKD3DSIH_HS_JOIN_PHASE ] = "hs_join_phase", + [VKD3DSIH_HSIN ] = "hsin", + [VKD3DSIH_HTAN ] = "htan", [VKD3DSIH_IADD ] = "iadd", [VKD3DSIH_IBFE ] = "ibfe", [VKD3DSIH_IDIV ] = "idiv", diff --git a/libs/vkd3d-shader/dxil.c b/libs/vkd3d-shader/dxil.c index 89d972ca..a001f6f0 100644 --- a/libs/vkd3d-shader/dxil.c +++ b/libs/vkd3d-shader/dxil.c @@ -342,6 +342,9 @@ enum dx_intrinsic_opcode DX_ACOS = 15, DX_ASIN = 16, DX_ATAN = 17, + DX_HCOS = 18, + DX_HSIN = 19, + DX_HTAN = 20, DX_EXP = 21, DX_FRC = 22, DX_LOG = 23, @@ -3690,6 +3693,12 @@ static enum vkd3d_shader_opcode map_dx_unary_op(enum dx_intrinsic_opcode op) return VKD3DSIH_ASIN; case DX_ATAN: return VKD3DSIH_ATAN; + case DX_HCOS: + return VKD3DSIH_HCOS; + case DX_HSIN: + return VKD3DSIH_HSIN; + case DX_HTAN: + return VKD3DSIH_HTAN; case DX_EXP: return VKD3DSIH_EXP; case DX_FRC: @@ -4263,6 +4272,9 @@ static const struct sm6_dx_opcode_info sm6_dx_op_table[] = [DX_FMIN ] = {"g", "RR", sm6_parser_emit_dx_binary}, [DX_FRC ] = {"g", "R", sm6_parser_emit_dx_unary}, [DX_IBFE ] = {"m", "iiR", sm6_parser_emit_dx_tertiary}, + [DX_HCOS ] = {"g", "R", sm6_parser_emit_dx_unary}, + [DX_HSIN ] = {"g", "R", sm6_parser_emit_dx_unary}, + [DX_HTAN ] = {"g", "R", sm6_parser_emit_dx_unary}, [DX_IMAX ] = {"m", "RR", sm6_parser_emit_dx_binary}, [DX_IMIN ] = {"m", "RR", sm6_parser_emit_dx_binary}, [DX_ISFINITE ] = {"1", "g", sm6_parser_emit_dx_unary}, diff --git a/libs/vkd3d-shader/spirv.c b/libs/vkd3d-shader/spirv.c index 5f7a33da..a3baeea7 100644 --- a/libs/vkd3d-shader/spirv.c +++ b/libs/vkd3d-shader/spirv.c @@ -7008,6 +7008,9 @@ static enum GLSLstd450 spirv_compiler_map_ext_glsl_instruction( {VKD3DSIH_FIRSTBIT_LO, GLSLstd450FindILsb}, {VKD3DSIH_FIRSTBIT_SHI, GLSLstd450FindSMsb}, {VKD3DSIH_FRC, GLSLstd450Fract}, + {VKD3DSIH_HCOS, GLSLstd450Cosh}, + {VKD3DSIH_HSIN, GLSLstd450Sinh}, + {VKD3DSIH_HTAN, GLSLstd450Tanh}, {VKD3DSIH_IMAX, GLSLstd450SMax}, {VKD3DSIH_IMIN, GLSLstd450SMin}, {VKD3DSIH_LOG, GLSLstd450Log2}, @@ -9587,6 +9590,9 @@ static int spirv_compiler_handle_instruction(struct spirv_compiler *compiler, case VKD3DSIH_ACOS: case VKD3DSIH_ASIN: case VKD3DSIH_ATAN: + case VKD3DSIH_HCOS: + case VKD3DSIH_HSIN: + case VKD3DSIH_HTAN: case VKD3DSIH_DFMA: case VKD3DSIH_DMAX: case VKD3DSIH_DMIN: diff --git a/libs/vkd3d-shader/vkd3d_shader_private.h b/libs/vkd3d-shader/vkd3d_shader_private.h index 3d043c24..1030adf9 100644 --- a/libs/vkd3d-shader/vkd3d_shader_private.h +++ b/libs/vkd3d-shader/vkd3d_shader_private.h @@ -366,10 +366,13 @@ enum vkd3d_shader_opcode VKD3DSIH_GATHER4_S, VKD3DSIH_GEO, VKD3DSIH_GEU, + VKD3DSIH_HCOS, VKD3DSIH_HS_CONTROL_POINT_PHASE, VKD3DSIH_HS_DECLS, VKD3DSIH_HS_FORK_PHASE, VKD3DSIH_HS_JOIN_PHASE, + VKD3DSIH_HSIN, + VKD3DSIH_HTAN, VKD3DSIH_IADD, VKD3DSIH_IBFE, VKD3DSIH_IDIV, diff --git a/tests/hlsl/trigonometry.shader_test b/tests/hlsl/trigonometry.shader_test index c9102e0a..034f861b 100644 --- a/tests/hlsl/trigonometry.shader_test +++ b/tests/hlsl/trigonometry.shader_test @@ -86,10 +86,10 @@ float4 main() : sv_target [test] uniform 0 float4 -6.28318531 -0.88137359 0.88137359 6.28318531 -todo draw quad +todo(sm<6) draw quad probe all rgba (-267.744894, -1.0, 1.0, 267.744894) 2 uniform 0 float4 -0.0 0.0 -90.0 90.0 -todo draw quad +todo(sm<6) draw quad % mingw does not support "inf" for scanf(), but numbers beyond FLOAT_MAX consistently result in inf. probe all rgba (0.0, 0.0, -1.0e39, 1.0e39) 1 @@ -104,10 +104,10 @@ float4 main() : sv_target [test] uniform 0 float4 -1.76274717 -1.3169579 1.3169579 1.76274717 -todo draw quad +todo(sm<6) draw quad probe all rgba (3.0, 2.0, 2.0, 3.0) 2 uniform 0 float4 -0.0 0.0 -90.0 90.0 -todo draw quad +todo(sm<6) draw quad probe all rgba (1.0, 1.0, 1.0e39, 1.0e39) 1 @@ -121,8 +121,8 @@ float4 main() : sv_target [test] uniform 0 float4 -1.57079633 -0.54930614 0.54930614 1.57079633 -todo draw quad +todo(sm<6) draw quad probe all rgba (-0.91715234, -0.5, 0.5, 0.91715234) 2 uniform 0 float4 -10.0 -0.0 0.0 10.0 -todo draw quad +todo(sm<6) draw quad probe all rgba (-1.0, 0.0, 0.0, 1.0) 1

1 0

Giovanni Mascellani : tests: Work around a Metal bug.
by Alexandre Julliard 07 Feb '24

07 Feb '24

Module: vkd3d Branch: master Commit: 4ba324d37c8fa7ca47525f2525ded391a0b851cc URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/4ba324d37c8fa7ca47525f2525ded… Author: Giovanni Mascellani <gmascellani(a)codeweavers.com> Date: Fri Feb 2 11:17:36 2024 +0100 tests: Work around a Metal bug. This test currently hit a Metal bug when run on Apple Silicon with MoltenVK and fails. We don't have an easy way to mark shader runner tests as buggy and we're not interested in tracking that bug anyway, so I'm just working around it. --- tests/hlsl/switch.shader_test | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/hlsl/switch.shader_test b/tests/hlsl/switch.shader_test index 543d44e2..63c9c09d 100644 --- a/tests/hlsl/switch.shader_test +++ b/tests/hlsl/switch.shader_test @@ -527,7 +527,7 @@ float4 main() : sv_target { switch (v.x) { - case 0: + case 1: for (j = 0; j < v.z; j++) { c += 1.0f; @@ -535,7 +535,7 @@ float4 main() : sv_target continue; } break; - case 1: + case 0: c += 2.0f; break; } @@ -545,10 +545,10 @@ float4 main() : sv_target } [test] -uniform 0 uint4 0 0 3 1 +uniform 0 uint4 1 0 3 1 draw quad probe all rgba (10.0, 11.0, 12.0, 13.0) -uniform 0 uint4 1 0 3 1 +uniform 0 uint4 0 0 3 1 draw quad probe all rgba (7.0, 8.0, 9.0, 10.0)

1 0

Conor McCarthy : vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS6 in CheckFeatureSupport().
by Alexandre Julliard 07 Feb '24

07 Feb '24

Module: vkd3d Branch: master Commit: db4a98c0d27eee24857d2f1a3b7fd45e77a8a0bc URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/db4a98c0d27eee24857d2f1a3b7fd… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Tue Jan 30 15:44:05 2024 +1000 vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS6 in CheckFeatureSupport(). --- libs/vkd3d/device.c | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c index cf8b506f..2a7024ec 100644 --- a/libs/vkd3d/device.c +++ b/libs/vkd3d/device.c @@ -3348,6 +3348,31 @@ static HRESULT STDMETHODCALLTYPE d3d12_device_CheckFeatureSupport(ID3D12Device7 return S_OK; } + case D3D12_FEATURE_D3D12_OPTIONS6: + { + D3D12_FEATURE_DATA_D3D12_OPTIONS6 *data = feature_data; + + if (feature_data_size != sizeof(*data)) + { + WARN("Invalid size %u.\n", feature_data_size); + return E_INVALIDARG; + } + + data->AdditionalShadingRatesSupported = FALSE; + data->PerPrimitiveShadingRateSupportedWithViewportIndexing = FALSE; + data->VariableShadingRateTier = D3D12_VARIABLE_SHADING_RATE_TIER_NOT_SUPPORTED; + data->ShadingRateImageTileSize = 0; + data->BackgroundProcessingSupported = FALSE; + + TRACE("Additional shading rates support %#x.\n", data->AdditionalShadingRatesSupported); + TRACE("Per-primitive shading rates with viewport indexing %#x.\n", + data->PerPrimitiveShadingRateSupportedWithViewportIndexing); + TRACE("Variable shading rate tier %#x.\n", data->VariableShadingRateTier); + TRACE("Shading rate image tile size %#x.\n", data->ShadingRateImageTileSize); + TRACE("Background processing support %#x.\n", data->BackgroundProcessingSupported); + return S_OK; + } + default: FIXME("Unhandled feature %#x.\n", feature); return E_NOTIMPL;

1 0

Conor McCarthy : vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS7 in CheckFeatureSupport().
by Alexandre Julliard 07 Feb '24

07 Feb '24

Module: vkd3d Branch: master Commit: 5fd3076c67813be7be2d2e1805174f359e286cd9 URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/5fd3076c67813be7be2d2e1805174… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Wed Jan 31 16:53:56 2024 +1000 vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS7 in CheckFeatureSupport(). --- libs/vkd3d/device.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c index 2a7024ec..299ecd6d 100644 --- a/libs/vkd3d/device.c +++ b/libs/vkd3d/device.c @@ -3373,6 +3373,24 @@ static HRESULT STDMETHODCALLTYPE d3d12_device_CheckFeatureSupport(ID3D12Device7 return S_OK; } + case D3D12_FEATURE_D3D12_OPTIONS7: + { + D3D12_FEATURE_DATA_D3D12_OPTIONS7 *data = feature_data; + + if (feature_data_size != sizeof(*data)) + { + WARN("Invalid size %u.\n", feature_data_size); + return E_INVALIDARG; + } + + data->MeshShaderTier = D3D12_MESH_SHADER_TIER_NOT_SUPPORTED; + data->SamplerFeedbackTier = D3D12_SAMPLER_FEEDBACK_TIER_NOT_SUPPORTED; + + TRACE("Mesh shading tier %#x.\n", data->MeshShaderTier); + TRACE("Sampler feedback tier %#x.\n", data->SamplerFeedbackTier); + return S_OK; + } + default: FIXME("Unhandled feature %#x.\n", feature); return E_NOTIMPL;

1 0

Conor McCarthy : vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS8 in CheckFeatureSupport().
by Alexandre Julliard 07 Feb '24

07 Feb '24

Module: vkd3d Branch: master Commit: 8ea67b0ac61b0122826ff9f61b34fc20b3ead7ff URL: https://gitlab.winehq.org/wine/vkd3d/-/commit/8ea67b0ac61b0122826ff9f61b34f… Author: Conor McCarthy <cmccarthy(a)codeweavers.com> Date: Wed Jan 31 17:09:59 2024 +1000 vkd3d: Handle D3D12_FEATURE_D3D12_OPTIONS8 in CheckFeatureSupport(). --- libs/vkd3d/device.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/libs/vkd3d/device.c b/libs/vkd3d/device.c index 299ecd6d..b1c07eb2 100644 --- a/libs/vkd3d/device.c +++ b/libs/vkd3d/device.c @@ -3391,6 +3391,22 @@ static HRESULT STDMETHODCALLTYPE d3d12_device_CheckFeatureSupport(ID3D12Device7 return S_OK; } + case D3D12_FEATURE_D3D12_OPTIONS8: + { + D3D12_FEATURE_DATA_D3D12_OPTIONS8 *data = feature_data; + + if (feature_data_size != sizeof(*data)) + { + WARN("Invalid size %u.\n", feature_data_size); + return E_INVALIDARG; + } + + data->UnalignedBlockTexturesSupported = FALSE; + + TRACE("Unaligned block texture support %#x.\n", data->UnalignedBlockTexturesSupported); + return S_OK; + } + default: FIXME("Unhandled feature %#x.\n", feature); return E_NOTIMPL;

1 0