This fixes a regression introduced by ebbcc10b.
Prior to that commit, buffers which were not CPU-accessible would not have the
WINED3D_BUFFER_USE_BO flag set, and accordingly
buffer_resource_sub_resource_map() would simply return the SYSMEM location,
ignoring the DISCARD and NOOVERWRITE flags.

However, the "accelerated" path in wined3d_cs_map_upload_bo() only checks for
the DISCARD flag, assuming that it is only set for dynamic GPU-accessible
buffers, and would subsequently try to allocate non-mappable memory and then map
it.

This commit avoids the accelerated path for such buffers, once again matching
our old behaviour. According to [1], this also matches Windows: DISCARD on
SYSTEMMEM buffers is ignored on both AMD and NVidia, and NOOVERWRITE is ignored
on NVidia (but not AMD).

[1] https://www.winehq.org/mailman3/hyperkitty/list/wine-devel@winehq.org/message/XACSAVAGYTJUZDFLX4JWO665VTB6M7LX/

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

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53743Signed-off-by: Zebediah Figura <zfigura@codeweavers.com>

Signed-off-by: Zebediah Figura <zfigura@codeweavers.com>

Signed-off-by: Zebediah Figura <zfigura@codeweavers.com>

DISCARD maps on deferred contexts do not seem to offer the same synchronization
guarantees as do DISCARD maps on the immediate context, at least not with
respect to subsequent NOOVERWRITE maps on the immediate context. This is not
particularly surprising, seeing as it is probably far more usual to only map a
resource on one kind of context or the other.

The test currently fails on Windows 11 on NVidia drivers, but it also fails on
Mesa, for what I think are the same reasons.

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

By backporting the corresponding window management tweaks from the
d3d8:device test.

Split the check_rt_color() function to allow conditional todos.

It fails on the Gitlab CI.

Although the test has never been observed to fail on Windows, I think the
failure is genuinely a race in the test.

File handles (like events) are signaled in order to mark that an I/O operation
has completed. In this case the I/O operation includes manipulating data on both
ends of the pipe, and as part of that may signal the other end. Internally,
however, the file handle must logically happen *after* all of this processing
has taken place, not least because (given the Windows I/O architecture) it is
the job of the I/O manager, not the IRP handler.

Since the purpose of the test is probably just to check that the file handle
will be signaled after a synchronous I/O operation has completed, we may as well
check it from the client thread, after we know for sure that it has.

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

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

The check was dropped in e392e0ac,
leaving the dwFlags variable unused.

This reverts commit 4608e1e1.

The ASF Reader filter suffers from defects which are difficult to solve,
and will require a better implementation of wmvcore in order to work
correctly. In the meantime, revert the class registration so that the
File Source filter is preferred instead.

This will break games that tried to use the filter directly and which
may have started to work correctly, but they are less frequent than the
ones simply playing wmv videos with quartz.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53640
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53748
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54191

This reapplies the change from 67d0038e
that was reverted in 06b19322, causing
a regression.

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

We get conhost.exe:tty failures on testbot:
- not for version prior to w1064v1909 (these tests require
  pseudo-consoles, which are not implemented in these Windows
  version).
- for w1064v1909
- not for W10 versions after w1064v1909.

So, we assume that pseudo-console support is immature in this
Windows version: skip all the remaining test at the first
discrepancy point.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53465Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Instead of uploading all currently bound textures. Besides being redundant, this
would generate mipmaps for an uninitialized texture, and subsequently trying to
render from nonzero mipmap levels would yield uninitialized data.

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

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

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

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

Signed-off-by: Gabriel Ivăncescu <gabrielopcode@gmail.com>

Fixes a regression introduced by
aa8f97e8.

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

udev_add_device declares a variable (product) even when it is not used
on all platforms. Use the same condition for the declaration that already
guards the code.

init_argv0_dir has conditional code for different operating systems. In
case of FreeBSD a variable remains uninitialized in the error case, yet
is then used. Fix that by handling the error case.

It can be activated without moving.