Signed-off-by: Dmitry Timoshkov <dmitry@baikal.ru>

Fixes: 010008f6
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56615

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=44863

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=44863

This improves performance for the game "Grounded", on a AMD Radeon RX 6700 XT,
with radv from Mesa 22.3.6. Testing was done with the "cb_access_map_w" option
enabled, which also improves performance with the game by itself.

Grounded generally makes about 4000 draw calls per frame, which seems not
atypical. This change makes it submit at most an extra 8 times per frame, but in
practice due to WINED3D_PERIODIC_SUBMIT_MAX_BUFFERS it submits less (usually
only 2-3).

The most demanding game I've seen made about 20,000 draw calls per frame, at
which point the overhead of adapter_vk_draw_primitive() itself becomes a serious
bottleneck. For such a game we would submit 40 more times per frame with these
settings, although WINED3D_PERIODIC_SUBMIT_MAX_BUFFERS means we would likely
submit less than that. In any case if submission itself becomes a bottleneck, we
should offload it to a separate thread.

Credit goes to Philip Rebohle and his work on DXVK for helping me to notice that
periodic submission might make a difference.

Part of beginning a render pass involves executing an RTV barrier, which itself
needs to call wined3d_context_vk_get_command_buffer(). However, that function
may decide to submit the command buffer, in order to prevent resource buildup,
or [in the future] because it has been some length of time since the last
submission.

Therefore we cannot retrieve and store a VkCommandBuffer pointer before
executing an RTV barrier and then use it later.

This fixes the following C++ compiler error

mfmediaengine.h:1216:18: error: expected ',' or '...' before 'protected'
 1216 |            BOOL *protected) = 0;
      |                  ^~~~~~~~~

After commit 898ab8da wine would no longer
build on musl.

Issue is that apparently TCSETS2 isn't defined when including sys/ioctl.h.

A little digging shows that glibc goes ahead and includes asm/ioctls.h in
sys/ioctl.h, providing said macro.  Musl on the other hand doesn't and relies
on bits/ioctl.h, which lacks said macro.
Signed-off-by: Fotios Valasiadis <fvalasiad@gmail.com>

NtUserCreateWindowEx will call apply_window_pos and drivers
WindowPosChanging callback, which already creates window data.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56429
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56607

Fixes a regression from bb2e02ab.

This appear to be introduced with commit 12f73ed9.

When This->stride is negative (bottom up image) it converts the multiplication to an UINT.
Thus causing the pointer to be incorrect when y > 0.

And only refresh the list if the device handle isn't known.