wine-commits August 2021

wine-commits@winehq.org

1 participants
1051 discussions

Alexandre Julliard : ntdll: Add a Wine-specific NtQueryVirtualMemory() query to retrieve the Unix call table.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: d91d05b80a4f7ccd7447c4514e5f8298bfcb177b URL: https://source.winehq.org/git/wine.git/?a=commit;h=d91d05b80a4f7ccd7447c451… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Aug 10 17:56:14 2021 +0200 ntdll: Add a Wine-specific NtQueryVirtualMemory() query to retrieve the Unix call table. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/virtual.c | 41 ++++++++++++++++++++++++++++++++++++++++- include/winternl.h | 4 +++- 2 files changed, 43 insertions(+), 2 deletions(-) diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index ee1f331d778..3532f5c8746 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -657,6 +657,29 @@ void *get_builtin_so_handle( void *module ) } +/*********************************************************************** + * get_builtin_unix_funcs + */ +NTSTATUS get_builtin_unix_funcs( void *module, BOOL wow, void **funcs ) +{ + const char *ptr_name = wow ? "__wine_unix_call_wow64_funcs" : "__wine_unix_call_funcs"; + sigset_t sigset; + NTSTATUS status = STATUS_DLL_NOT_FOUND; + struct builtin_module *builtin; + + server_enter_uninterrupted_section( &virtual_mutex, &sigset ); + LIST_FOR_EACH_ENTRY( builtin, &builtin_modules, struct builtin_module, entry ) + { + if (builtin->module != module) continue; + *funcs = dlsym( builtin->unix_handle, ptr_name ); + status = STATUS_SUCCESS; + break; + } + server_leave_uninterrupted_section( &virtual_mutex, &sigset ); + return status; +} + + /*********************************************************************** * get_builtin_unix_info */ @@ -4289,6 +4312,8 @@ NTSTATUS WINAPI NtQueryVirtualMemory( HANDLE process, LPCVOID addr, MEMORY_INFORMATION_CLASS info_class, PVOID buffer, SIZE_T len, SIZE_T *res_len ) { + NTSTATUS status; + TRACE("(%p, %p, info_class=%d, %p, %ld, %p)\n", process, addr, info_class, buffer, len, res_len); @@ -4311,13 +4336,27 @@ NTSTATUS WINAPI NtQueryVirtualMemory( HANDLE process, LPCVOID addr, if (handle) { - NTSTATUS status = get_builtin_init_funcs( handle, buffer, len, res_len ); + status = get_builtin_init_funcs( handle, buffer, len, res_len ); release_builtin_module( module ); return status; } } return STATUS_INVALID_HANDLE; + case MemoryWineUnixFuncs: + case MemoryWineUnixWow64Funcs: + if (len != sizeof(UINT64)) return STATUS_INFO_LENGTH_MISMATCH; + if (process == GetCurrentProcess()) + { + void *module = (void *)addr; + void *funcs = NULL; + + status = get_builtin_unix_funcs( module, info_class == MemoryWineUnixWow64Funcs, &funcs ); + if (!status) *(UINT64 *)buffer = (UINT_PTR)funcs; + return status; + } + return STATUS_INVALID_HANDLE; + default: FIXME("(%p,%p,info_class=%d,%p,%ld,%p) Unknown information class\n", process, addr, info_class, buffer, len, res_len); diff --git a/include/winternl.h b/include/winternl.h index 075b1863f72..41bc56d186d 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -1919,7 +1919,9 @@ typedef enum _MEMORY_INFORMATION_CLASS { MemoryBasicInformationCapped, MemoryPhysicalContiguityInformation, #ifdef __WINESRC__ - MemoryWineImageInitFuncs = 1000 + MemoryWineImageInitFuncs = 1000, + MemoryWineUnixFuncs, + MemoryWineUnixWow64Funcs, #endif } MEMORY_INFORMATION_CLASS;

1 0

Alexandre Julliard : ntdll: Load the Unix dll at the same time as the PE one, but don't map it yet.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: c29f6504747c3a80135085a7ae9ac470d2950c09 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c29f6504747c3a80135085a7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Aug 10 16:55:21 2021 +0200 ntdll: Load the Unix dll at the same time as the PE one, but don't map it yet. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/loader.c | 17 +++++++++-------- dlls/ntdll/unix/unix_private.h | 3 ++- dlls/ntdll/unix/virtual.c | 33 ++++++++++++++++++++++++++++----- 3 files changed, 39 insertions(+), 14 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 5f1cbda51a9..f5135a19f40 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -1121,21 +1121,17 @@ static NTSTATUS CDECL init_unix_lib( void *module, DWORD reason, const void *ptr if (!entry) { - if (!name) return STATUS_DLL_NOT_FOUND; - if (!(handle = dlopen( name, RTLD_NOW ))) return STATUS_DLL_NOT_FOUND; + if (!name || !handle) return STATUS_DLL_NOT_FOUND; if (!(nt = dlsym( handle, "__wine_spec_nt_header" )) || !(entry = dlsym( handle, "__wine_init_unix_lib" ))) - { - dlclose( handle ); - set_builtin_unix_info( module, NULL, NULL, NULL ); return STATUS_INVALID_IMAGE_FORMAT; - } + TRACE( "loaded %s for %p\n", debugstr_a(name), module ); unix_module = (void *)((nt->OptionalHeader.ImageBase + 0xffff) & ~0xffff); map_so_dll( nt, unix_module ); fixup_ntdll_imports( name, unix_module ); - set_builtin_unix_info( module, NULL, handle, entry ); + set_builtin_unix_entry( module, entry ); } init_func = entry; return init_func( module, reason, ptr_in, ptr_out ); @@ -1417,8 +1413,13 @@ static NTSTATUS find_builtin_dll( UNICODE_STRING *nt_name, void **module, SIZE_T done: if (status >= 0 && ext) { + void *handle; + strcpy( ext, ".so" ); - set_builtin_unix_info( *module, ptr, NULL, NULL ); + if ((handle = dlopen( ptr, RTLD_NOW ))) + { + if (set_builtin_unix_handle( *module, ptr, handle )) dlclose( handle ); + } } free( file ); return status; diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 7d89d99d120..0fce580b120 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -231,7 +231,8 @@ extern void virtual_fill_image_information( const pe_image_info_t *pe_info, SECTION_IMAGE_INFORMATION *info ) DECLSPEC_HIDDEN; extern void *get_builtin_so_handle( void *module ) DECLSPEC_HIDDEN; extern NTSTATUS get_builtin_unix_info( void *module, const char **name, void **handle, void **entry ) DECLSPEC_HIDDEN; -extern NTSTATUS set_builtin_unix_info( void *module, const char *name, void *handle, void *entry ) DECLSPEC_HIDDEN; +extern NTSTATUS set_builtin_unix_handle( void *module, const char *name, void *handle ) DECLSPEC_HIDDEN; +extern NTSTATUS set_builtin_unix_entry( void *module, void *entry ) DECLSPEC_HIDDEN; extern NTSTATUS get_thread_ldt_entry( HANDLE handle, void *data, ULONG len, ULONG *ret_len ) DECLSPEC_HIDDEN; extern void *get_native_context( CONTEXT *context ) DECLSPEC_HIDDEN; diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index 63b322a0c68..ee1f331d778 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -682,9 +682,9 @@ NTSTATUS get_builtin_unix_info( void *module, const char **name, void **handle, /*********************************************************************** - * set_builtin_unix_info + * set_builtin_unix_handle */ -NTSTATUS set_builtin_unix_info( void *module, const char *name, void *handle, void *entry ) +NTSTATUS set_builtin_unix_handle( void *module, const char *name, void *handle ) { sigset_t sigset; NTSTATUS status = STATUS_DLL_NOT_FOUND; @@ -696,10 +696,8 @@ NTSTATUS set_builtin_unix_info( void *module, const char *name, void *handle, vo if (builtin->module != module) continue; if (!builtin->unix_handle) { - free( builtin->unix_name ); - builtin->unix_name = name ? strdup( name ) : NULL; + builtin->unix_name = strdup( name ); builtin->unix_handle = handle; - builtin->unix_entry = entry; status = STATUS_SUCCESS; } else status = STATUS_IMAGE_ALREADY_LOADED; @@ -710,6 +708,31 @@ NTSTATUS set_builtin_unix_info( void *module, const char *name, void *handle, vo } +/*********************************************************************** + * set_builtin_unix_entry + */ +NTSTATUS set_builtin_unix_entry( void *module, void *entry ) +{ + sigset_t sigset; + NTSTATUS status = STATUS_DLL_NOT_FOUND; + struct builtin_module *builtin; + + server_enter_uninterrupted_section( &virtual_mutex, &sigset ); + LIST_FOR_EACH_ENTRY( builtin, &builtin_modules, struct builtin_module, entry ) + { + if (builtin->module != module) continue; + if (builtin->unix_handle) + { + builtin->unix_entry = entry; + status = STATUS_SUCCESS; + } + break; + } + server_leave_uninterrupted_section( &virtual_mutex, &sigset ); + return status; +} + + /*********************************************************************** * free_ranges_lower_bound *

1 0

Alexandre Julliard : ntdll: Don't try to load .so dlls for a different machine.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: ccd9008d8d0a33769d8b328d5bc6775f940ef0bd URL: https://source.winehq.org/git/wine.git/?a=commit;h=ccd9008d8d0a33769d8b328d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Aug 10 16:42:09 2021 +0200 ntdll: Don't try to load .so dlls for a different machine. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/loader.c | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/dlls/ntdll/unix/loader.c b/dlls/ntdll/unix/loader.c index 55fc73d2d06..5f1cbda51a9 100644 --- a/dlls/ntdll/unix/loader.c +++ b/dlls/ntdll/unix/loader.c @@ -1286,12 +1286,14 @@ static NTSTATUS open_builtin_pe_file( const char *name, OBJECT_ATTRIBUTES *attr, * open_builtin_so_file */ static NTSTATUS open_builtin_so_file( const char *name, OBJECT_ATTRIBUTES *attr, void **module, - SECTION_IMAGE_INFORMATION *image_info, BOOL prefer_native ) + SECTION_IMAGE_INFORMATION *image_info, + WORD machine, BOOL prefer_native ) { NTSTATUS status; int fd; *module = NULL; + if (machine != current_machine) return STATUS_DLL_NOT_FOUND; if ((fd = open( name, O_RDONLY )) == -1) return STATUS_DLL_NOT_FOUND; if (check_library_arch( fd )) @@ -1362,7 +1364,7 @@ static NTSTATUS find_builtin_dll( UNICODE_STRING *nt_name, void **module, SIZE_T status = open_builtin_pe_file( ptr, &attr, module, size_ptr, image_info, machine, prefer_native ); if (status != STATUS_DLL_NOT_FOUND) goto done; strcpy( file + pos + len + 1, ".so" ); - status = open_builtin_so_file( ptr, &attr, module, image_info, prefer_native ); + status = open_builtin_so_file( ptr, &attr, module, image_info, machine, prefer_native ); if (status != STATUS_DLL_NOT_FOUND) goto done; /* now as a program */ @@ -1376,7 +1378,7 @@ static NTSTATUS find_builtin_dll( UNICODE_STRING *nt_name, void **module, SIZE_T status = open_builtin_pe_file( ptr, &attr, module, size_ptr, image_info, machine, prefer_native ); if (status != STATUS_DLL_NOT_FOUND) goto done; strcpy( file + pos + len + 1, ".so" ); - status = open_builtin_so_file( ptr, &attr, module, image_info, prefer_native ); + status = open_builtin_so_file( ptr, &attr, module, image_info, machine, prefer_native ); if (status != STATUS_DLL_NOT_FOUND) goto done; } @@ -1393,11 +1395,10 @@ static NTSTATUS find_builtin_dll( UNICODE_STRING *nt_name, void **module, SIZE_T ptr = prepend( ptr, dll_paths[i], strlen(dll_paths[i]) ); if (status != STATUS_DLL_NOT_FOUND) goto done; strcpy( file + pos + len + 1, ".so" ); - status = open_builtin_so_file( ptr, &attr, module, image_info, prefer_native ); + status = open_builtin_so_file( ptr, &attr, module, image_info, machine, prefer_native ); if (status != STATUS_DLL_NOT_FOUND) goto done; file[pos + len + 1] = 0; ptr = prepend( file + pos, dll_paths[i], strlen(dll_paths[i]) ); - if (status != STATUS_DLL_NOT_FOUND) goto done; status = open_builtin_pe_file( ptr, &attr, module, size_ptr, image_info, machine, prefer_native ); if (status == STATUS_IMAGE_MACHINE_TYPE_MISMATCH) { @@ -1406,7 +1407,7 @@ static NTSTATUS find_builtin_dll( UNICODE_STRING *nt_name, void **module, SIZE_T } if (status != STATUS_DLL_NOT_FOUND) goto done; strcpy( file + pos + len + 1, ".so" ); - status = open_builtin_so_file( ptr, &attr, module, image_info, prefer_native ); + status = open_builtin_so_file( ptr, &attr, module, image_info, machine, prefer_native ); if (status == STATUS_IMAGE_MACHINE_TYPE_MISMATCH) found_image = TRUE; else if (status != STATUS_DLL_NOT_FOUND) goto done; } @@ -1777,7 +1778,7 @@ static void load_ntdll(void) if (status == STATUS_DLL_NOT_FOUND) { sprintf( name, "%s/ntdll.dll.so", ntdll_dir ); - status = open_builtin_so_file( name, &attr, &module, &info, FALSE ); + status = open_builtin_so_file( name, &attr, &module, &info, current_machine, FALSE ); } if (status == STATUS_IMAGE_NOT_AT_BASE) relocate_ntdll( module ); else if (status) fatal_error( "failed to load %s error %x\n", name, status );

1 0

Alexandre Julliard : ntdll: Make release_builtin_module() static.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 18408b18f3ebd9630a02f0bd16864f5e5118b71a URL: https://source.winehq.org/git/wine.git/?a=commit;h=18408b18f3ebd9630a02f0bd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Aug 10 16:26:31 2021 +0200 ntdll: Make release_builtin_module() static. Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/unix/unix_private.h | 1 - dlls/ntdll/unix/virtual.c | 5 ++--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/ntdll/unix/unix_private.h b/dlls/ntdll/unix/unix_private.h index 61c290c156d..7d89d99d120 100644 --- a/dlls/ntdll/unix/unix_private.h +++ b/dlls/ntdll/unix/unix_private.h @@ -229,7 +229,6 @@ extern void virtual_set_force_exec( BOOL enable ) DECLSPEC_HIDDEN; extern void virtual_set_large_address_space(void) DECLSPEC_HIDDEN; extern void virtual_fill_image_information( const pe_image_info_t *pe_info, SECTION_IMAGE_INFORMATION *info ) DECLSPEC_HIDDEN; -extern NTSTATUS release_builtin_module( void *module ) DECLSPEC_HIDDEN; extern void *get_builtin_so_handle( void *module ) DECLSPEC_HIDDEN; extern NTSTATUS get_builtin_unix_info( void *module, const char **name, void **handle, void **entry ) DECLSPEC_HIDDEN; extern NTSTATUS set_builtin_unix_info( void *module, const char *name, void *handle, void *entry ) DECLSPEC_HIDDEN; diff --git a/dlls/ntdll/unix/virtual.c b/dlls/ntdll/unix/virtual.c index f055be94238..63b322a0c68 100644 --- a/dlls/ntdll/unix/virtual.c +++ b/dlls/ntdll/unix/virtual.c @@ -615,7 +615,7 @@ static void add_builtin_module( void *module, void *handle ) /*********************************************************************** * release_builtin_module */ -NTSTATUS release_builtin_module( void *module ) +static void release_builtin_module( void *module ) { struct builtin_module *builtin; @@ -630,9 +630,8 @@ NTSTATUS release_builtin_module( void *module ) free( builtin->unix_name ); free( builtin ); } - return STATUS_SUCCESS; + break; } - return STATUS_INVALID_PARAMETER; }

1 0

Andrew Boyarshin : include: Fix IRequestDictionary IID.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 32ec7d7b94d1c2a476553c1e50f59ac6ecb5c8f1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=32ec7d7b94d1c2a476553c1e… Author: Andrew Boyarshin <andrew.boyarshin(a)gmail.com> Date: Fri Jul 23 14:49:36 2021 +0700 include: Fix IRequestDictionary IID. IRequestDictionary had the wrong UUID specified (IResponse one). Found by using MIDL instead of WIDL to compile Wine headers: duplicated IID caused compilation error in MIDL. Signed-off-by: Andrew Boyarshin <andrew.boyarshin(a)gmail.com> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msxml3/tests/domdoc.c | 2 +- include/asptlb.idl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index df30104a156..a7890d56f71 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -13170,7 +13170,7 @@ static HRESULT WINAPI transformdest_QueryInterface(IUnknown *iface, REFIID riid, IsEqualIID(riid, &IID_IServiceProvider) || IsEqualIID(riid, &IID_IStream) || IsEqualIID(riid, &IID_ISequentialStream) || - IsEqualIID(riid, &IID_IRequestDictionary); + IsEqualIID(riid, &IID_IResponse); todo_wine_if(IsEqualIID(riid, &IID_IXMLDOMDocument)) ok(known_iid, "Unexpected riid %s\n", wine_dbgstr_guid(riid)); diff --git a/include/asptlb.idl b/include/asptlb.idl index b38bc5d1fe9..07f250702dd 100644 --- a/include/asptlb.idl +++ b/include/asptlb.idl @@ -23,7 +23,7 @@ import "oaidl.idl"; [ object, dual, - uuid(D97A6DA0-A864-11cf-83BE-00A0C90C2BD8) + uuid(D97A6DA0-A85F-11DF-83AE-00A0C90C2BD8) ] interface IRequestDictionary : IDispatch {

1 0

Luke Deller : kernelbase: Translate FILE_FLAG_SEQUENTIAL_SCAN.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 8294b2c8adc127794b6f16580059721abc4d4589 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8294b2c8adc127794b6f1658… Author: Luke Deller <luke(a)deller.id.au> Date: Wed Aug 4 21:29:35 2021 +1000 kernelbase: Translate FILE_FLAG_SEQUENTIAL_SCAN. Translate FILE_FLAG_SEQUENTIAL_SCAN into the corresponding flag for NtCreateFile options Signed-off-by: Luke Deller <luke(a)deller.id.au> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/kernelbase/file.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/dlls/kernelbase/file.c b/dlls/kernelbase/file.c index e0a75c2ad08..d21ad299d1f 100644 --- a/dlls/kernelbase/file.c +++ b/dlls/kernelbase/file.c @@ -723,6 +723,8 @@ static UINT get_nt_file_options( DWORD attributes ) options |= FILE_SYNCHRONOUS_IO_NONALERT; if (attributes & FILE_FLAG_RANDOM_ACCESS) options |= FILE_RANDOM_ACCESS; + if (attributes & FILE_FLAG_SEQUENTIAL_SCAN) + options |= FILE_SEQUENTIAL_ONLY; if (attributes & FILE_FLAG_WRITE_THROUGH) options |= FILE_WRITE_THROUGH; return options; @@ -3227,6 +3229,7 @@ HANDLE WINAPI DECLSPEC_HOTPATCH OpenFileById( HANDLE handle, LPFILE_ID_DESCRIPTO if (flags & FILE_FLAG_NO_BUFFERING) options |= FILE_NO_INTERMEDIATE_BUFFERING; if (!(flags & FILE_FLAG_OVERLAPPED)) options |= FILE_SYNCHRONOUS_IO_NONALERT; if (flags & FILE_FLAG_RANDOM_ACCESS) options |= FILE_RANDOM_ACCESS; + if (flags & FILE_FLAG_SEQUENTIAL_SCAN) options |= FILE_SEQUENTIAL_ONLY; flags &= FILE_ATTRIBUTE_VALID_FLAGS; objectName.Length = sizeof(ULONGLONG);

1 0

Luke Deller : msvcrt: Translate file open access pattern hints.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 1ca67c8ba20886e1a71bb4a2aa94638beaba4534 URL: https://source.winehq.org/git/wine.git/?a=commit;h=1ca67c8ba20886e1a71bb4a2… Author: Luke Deller <luke(a)deller.id.au> Date: Wed Aug 4 21:29:34 2021 +1000 msvcrt: Translate file open access pattern hints. Translate access pattern hints supplied to fopen / _open into the corresponding attributes for the lower level CreateFileW call. Signed-off-by: Luke Deller <luke(a)deller.id.au> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msvcrt/file.c | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index 6e955099090..b0eeaf2a351 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -22,8 +22,6 @@ * License along with this library; if not, write to the Free Software * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA * - * TODO - * Use the file flag hints O_SEQUENTIAL, O_RANDOM, O_SHORT_LIVED */ #include <direct.h> @@ -1609,8 +1607,12 @@ static int msvcrt_get_flags(const wchar_t* mode, int *open_flags, int* stream_fl case 'w': break; case 'S': + if (!(*open_flags & _O_RANDOM)) + *open_flags |= _O_SEQUENTIAL; + break; case 'R': - FIXME("ignoring cache optimization flag: %c\n", mode[-1]); + if (!(*open_flags & _O_SEQUENTIAL)) + *open_flags |= _O_RANDOM; break; default: ERR("incorrect mode flag: %c\n", mode[-1]); @@ -2269,6 +2271,13 @@ int CDECL _wsopen_dispatch( const wchar_t* path, int oflags, int shflags, int pm sharing |= FILE_SHARE_DELETE; } + if (oflags & _O_RANDOM) + attrib |= FILE_FLAG_RANDOM_ACCESS; + if (oflags & _O_SEQUENTIAL) + attrib |= FILE_FLAG_SEQUENTIAL_SCAN; + if (oflags & _O_SHORT_LIVED) + attrib |= FILE_ATTRIBUTE_TEMPORARY; + sa.nLength = sizeof( SECURITY_ATTRIBUTES ); sa.lpSecurityDescriptor = NULL; sa.bInheritHandle = !(oflags & _O_NOINHERIT);

1 0

Rémi Bernon : ntoskrnl.exe/tests: Restore default mute threshold.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: ae01a6f6adcbe59913c6338b7ec2bc8422c49d91 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ae01a6f6adcbe59913c6338b… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Aug 10 11:31:18 2021 +0200 ntoskrnl.exe/tests: Restore default mute threshold. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntoskrnl.exe/tests/utils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntoskrnl.exe/tests/utils.h b/dlls/ntoskrnl.exe/tests/utils.h index aa84890f6af..f73f6f938ca 100644 --- a/dlls/ntoskrnl.exe/tests/utils.h +++ b/dlls/ntoskrnl.exe/tests/utils.h @@ -51,7 +51,7 @@ static int winetest_debug; static int winetest_report_success; /* silence todos and skips above this threshold */ -static int winetest_mute_threshold = 1; +static int winetest_mute_threshold = 42; /* counts how many times a given line printed a message */ static LONG line_counters[16384];

1 0

Rémi Bernon : hidclass.sys: Use a simpler ring buffer with ref-counted reports.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 325984ded50b354686e5a454aa5aca3aafa81432 URL: https://source.winehq.org/git/wine.git/?a=commit;h=325984ded50b354686e5a454… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Aug 10 11:31:17 2021 +0200 hidclass.sys: Use a simpler ring buffer with ref-counted reports. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/hidclass.sys/Makefile.in | 1 - dlls/hidclass.sys/buffer.c | 204 ------------------------------------------ dlls/hidclass.sys/device.c | 162 +++++++++++++++++++++++++-------- dlls/hidclass.sys/hid.h | 25 +++--- 4 files changed, 136 insertions(+), 256 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=325984ded50b354686e5…

1 0

Rémi Bernon : hidclass.sys: Process pending IRP queue using last read packet.
by Alexandre Julliard 10 Aug '21

10 Aug '21

Module: wine Branch: master Commit: 0a104844eb4ab402be7c419f05b430f107fc0763 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0a104844eb4ab402be7c419f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Aug 10 11:31:16 2021 +0200 hidclass.sys: Process pending IRP queue using last read packet. Instead of requiring the ring buffer to keep previously read packets. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/hidclass.sys/buffer.c | 34 ---------------------------------- dlls/hidclass.sys/device.c | 45 ++++++++++++++++++--------------------------- dlls/hidclass.sys/hid.h | 1 - 3 files changed, 18 insertions(+), 62 deletions(-) diff --git a/dlls/hidclass.sys/buffer.c b/dlls/hidclass.sys/buffer.c index 720ca4eeb20..59c0edf29f9 100644 --- a/dlls/hidclass.sys/buffer.c +++ b/dlls/hidclass.sys/buffer.c @@ -154,40 +154,6 @@ void RingBuffer_ReadNew(struct ReportRingBuffer *ring, UINT index, void *output, } } -void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) -{ - int pointer; - void *ret = NULL; - - EnterCriticalSection(&ring->lock); - if (index >= ring->pointer_alloc || ring->pointers[index] == POINTER_UNUSED - || ring->end == ring->start) - { - LeaveCriticalSection(&ring->lock); - *size = 0; - return; - } - - pointer = ring->pointers[index]; - - if (pointer == ring->end) - pointer--; - - if (pointer < 0) - pointer = ring->size - 1; - - ret = &ring->buffer[pointer * ring->buffer_size]; - memcpy(output, ret, ring->buffer_size); - if (pointer == ring->pointers[index]) - { - ring->pointers[index]++; - if (ring->pointers[index] == ring->size) - ring->pointers[index] = 0; - } - LeaveCriticalSection(&ring->lock); - *size = ring->buffer_size; -} - UINT RingBuffer_AddPointer(struct ReportRingBuffer *ring) { UINT idx; diff --git a/dlls/hidclass.sys/device.c b/dlls/hidclass.sys/device.c index a3c82d03269..8510f25ae41 100644 --- a/dlls/hidclass.sys/device.c +++ b/dlls/hidclass.sys/device.c @@ -82,10 +82,13 @@ static void WINAPI read_cancel_routine(DEVICE_OBJECT *device, IRP *irp) static void hid_device_queue_input( DEVICE_OBJECT *device, HID_XFER_PACKET *packet ) { BASE_DEVICE_EXTENSION *ext = device->DeviceExtension; + struct hid_preparsed_data *preparsed = ext->u.pdo.preparsed_data; + HID_XFER_PACKET *read_packet, *last_packet = packet; struct hid_report_queue *queue; RAWINPUT *rawinput; ULONG size; KIRQL irql; + IRP *irp; size = offsetof( RAWINPUT, data.hid.bRawData[packet->reportBufferLen] ); if (!(rawinput = malloc( size ))) ERR( "Failed to allocate rawinput data!\n" ); @@ -114,37 +117,27 @@ static void hid_device_queue_input( DEVICE_OBJECT *device, HID_XFER_PACKET *pack LIST_FOR_EACH_ENTRY( queue, &ext->u.pdo.report_queues, struct hid_report_queue, entry ) RingBuffer_Write( queue->buffer, packet ); KeReleaseSpinLock( &ext->u.pdo.report_queues_lock, irql ); -} - -static void HID_Device_processQueue(DEVICE_OBJECT *device) -{ - BASE_DEVICE_EXTENSION *ext = device->DeviceExtension; - struct hid_preparsed_data *preparsed = ext->u.pdo.preparsed_data; - struct hid_report_queue *queue; - HID_XFER_PACKET *packet; - UINT buffer_size; - IRP *irp; - packet = malloc( sizeof(*packet) + preparsed->caps.InputReportByteLength ); + if (!(read_packet = malloc( sizeof(*packet) + preparsed->caps.InputReportByteLength ))) + { + ERR( "Failed to allocate read_packet!\n" ); + return; + } - while((irp = pop_irp_from_queue(ext))) + while ((irp = pop_irp_from_queue( ext ))) { queue = irp->Tail.Overlay.OriginalFileObject->FsContext; - RingBuffer_Read( queue->buffer, 0, packet, &buffer_size ); - if (buffer_size) - { - memcpy( irp->AssociatedIrp.SystemBuffer, packet + 1, preparsed->caps.InputReportByteLength ); - irp->IoStatus.Information = packet->reportBufferLen; - irp->IoStatus.Status = STATUS_SUCCESS; - } - else - { - irp->IoStatus.Information = 0; - irp->IoStatus.Status = STATUS_UNSUCCESSFUL; - } + RingBuffer_ReadNew( queue->buffer, 0, read_packet, &size ); + if (!size) packet = last_packet; + else packet = read_packet; + + memcpy( irp->AssociatedIrp.SystemBuffer, packet + 1, preparsed->caps.InputReportByteLength ); + irp->IoStatus.Information = packet->reportBufferLen; + irp->IoStatus.Status = STATUS_SUCCESS; IoCompleteRequest( irp, IO_NO_INCREMENT ); } - free(packet); + + free( read_packet ); } static DWORD CALLBACK hid_device_thread(void *args) @@ -187,7 +180,6 @@ static DWORD CALLBACK hid_device_thread(void *args) packet->reportBufferLen = io.Information; hid_device_queue_input( device, packet ); - HID_Device_processQueue(device); } rc = WaitForSingleObject(ext->u.pdo.halt_event, @@ -229,7 +221,6 @@ static DWORD CALLBACK hid_device_thread(void *args) packet->reportBufferLen = io.Information; hid_device_queue_input( device, packet ); - HID_Device_processQueue(device); } if (exit_now) diff --git a/dlls/hidclass.sys/hid.h b/dlls/hidclass.sys/hid.h index d886672ec10..6bfba100007 100644 --- a/dlls/hidclass.sys/hid.h +++ b/dlls/hidclass.sys/hid.h @@ -96,7 +96,6 @@ struct hid_report_queue void RingBuffer_Write(struct ReportRingBuffer *buffer, void *data) DECLSPEC_HIDDEN; UINT RingBuffer_AddPointer(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN; void RingBuffer_RemovePointer(struct ReportRingBuffer *ring, UINT index) DECLSPEC_HIDDEN; -void RingBuffer_Read(struct ReportRingBuffer *ring, UINT index, void *output, UINT *size) DECLSPEC_HIDDEN; void RingBuffer_ReadNew(struct ReportRingBuffer *buffer, UINT index, void *output, UINT *size) DECLSPEC_HIDDEN; UINT RingBuffer_GetBufferSize(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN; UINT RingBuffer_GetSize(struct ReportRingBuffer *buffer) DECLSPEC_HIDDEN;

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits August 2021