Re: [PATCH v2 0/6] MR788: winevulkan: Move dispatchable handles allocation to PE code.

[Jacek Caban (＠jacek)]

On Tue Sep  6 19:22:58 2022 +0000, Alexandre Julliard wrote:
> This is breaking tests here:
> tools/runtest -q -P wine -T . -M vulkan-1.dll -p
> dlls/vulkan-1/tests/vulkan-1_test.exe vulkan && touch dlls/vulkan-1/tests/vulkan.ok
> wine: Unhandled page fault on read access to 00000008 at address
> 682C43C8 (thread 016c), starting debugger...
> Unhandled exception: page fault on read access to 0x00000008 in 32-bit
> code (0x682c43c8).
> Register dump:
>  CS:0023 SS:002b DS:002b ES:002b FS:0063 GS:006b
>  EIP:682c43c8 ESP:0064fc10 EBP:0064fc78 EFLAGS:00010202(  R- --  I   - -
> - )
>  EAX:00000000 EBX:00000000 ECX:0064fc90 EDX:00000000
>  ESI:00000000 EDI:00137d60
> Stack dump:
> 0x0064fc10:  00d705a0 00000000 000000fd 0064fc4c
> 0x0064fc20:  00d704e8 00137d60 0064fc68 00000000
> 0x0064fc30:  00d705a8 0064fc4c 00000000 00d70570
> 0x0064fc40:  00130000 00000000 00d70598 00d70570
> 0x0064fc50:  00d705a0 00000000 00000000 0064fc48
> 0x0064fc60:  00d704e8 00137d60 0064fc90 00000000
> Backtrace:
> =>0 0x682c43c8 vkDestroyCommandPool+0x28(device=<internal error>,
> handle=<internal error>, allocator=<internal error>)
> [Z:\home\julliard\wine\wine\dlls\winevulkan\loader.c:527] in winevulkan (0x0064fc78)
>   1 0x004020f2
> test_destroy_command_pool+0x162(vk_physical_device=00D704E8)
> [Z:\home\julliard\wine\wine\dlls\vulkan-1\tests\vulkan.c:346] in
> vulkan-1_test (0x0064fcf8)
>   2 0x00403694 for_each_device_instance+0x164(extension_count=<internal
> error>, enabled_extensions=<internal error>,
> test_func_instance=00000000, test_func=00401F90)
> [Z:\home\julliard\wine\wine\dlls\vulkan-1\tests\vulkan.c:905] in
> vulkan-1_test (0x0064fd48)
>   3 0x00403bd3 create_instance(test_func=<internal error>)
> [Z:\home\julliard\wine\wine\dlls\vulkan-1\tests\vulkan.c:920] in
> vulkan-1_test (0x0064fdb8)
>   4 0x00403bd3 test_unsupported_instance_extensions(test_func=<internal
> error>) [Z:\home\julliard\wine\wine\dlls\vulkan-1\tests\vulkan.c:920] in
> vulkan-1_test (0x0064fdb8)
>   5 0x00403bd3 func_vulkan+0x4c3()
> [Z:\home\julliard\wine\wine\dlls\vulkan-1\tests\vulkan.c:920] in
> vulkan-1_test (0x0064fdb8)
>   6 0x00403f53 run_test+0xe3(name="vulkan")
> [Z:\home\julliard\wine\wine\include\wine\test.h:674] in vulkan-1_test (0x0064fe18)
>   7 0x00405aa9 main+0x1f9(argc=<internal error>, argv=<internal error>)
> [Z:\home\julliard\wine\wine\include\wine\test.h:780] in vulkan-1_test (0x0064fee8)
>   8 0x0040585f mainCRTStartup+0x7f()
> [Z:\home\julliard\wine\wine\dlls\msvcrt\crt_main.c:60] in vulkan-1_test (0x0064ff30)
>   9 0x7b62a2f0 in kernel32 (+0x2a2f0) (0x0064ff48)
>   10 0x7bc5c7d7 in ntdll (+0x5c7d7) (0x0064ff5c)
>   11 0x7bc5ceb0 RtlCreateUserThread(entry=004057E0, arg=7FFD1000)
> [Z:\home\julliard\wine\wine\dlls\ntdll\thread.c:261] in ntdll (0x0064ffec)
> 0x682c43c8 vkDestroyCommandPool+0x28
> [Z:\home\julliard\wine\wine\dlls\winevulkan\loader.c:527] in winevulkan:
> movl	0x8(%esi),%eax
> 527	    LIST_FOR_EACH_ENTRY_SAFE(buffer, cursor,
> &cmd_pool->command_buffers, struct VkCommandBuffer_T, pool_link)

Sorry about that, the new version should fix it.