[Bug 39442] New: Using Spice required to avoid failure in mmdevapi:capture

https://bugs.winehq.org/show_bug.cgi?id=39442

Bug ID: 39442
           Summary: Using Spice required to avoid failure in
                    mmdevapi:capture
           Product: Wine-Testbot
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

The Windows 7+ VMs all use an ich6 virtual soundcard (they don't support any of
ac97, es1370 or sb16 ones).

But they have failures in mmdevapi:capture such as:

capture.c:586: Returned latency: 5.8050 ms
capture.c:178: Test failed: Position 375 expected 0
capture.c:186: Wait'ed position 375 pad 0 flags 1, amount of frames locked: 448
capture.c:228: Test failed: Position 1719 expected 823
[...]

It turns out that the way the VM is configured for remote access impacts the
result of the tests:
 * If configured for VNC then the tests fail as described, even if no client is
connected.
 * If configured for Spice access then the tests succeed, even if no client is
connected. And if a client is connected the sound is carried over with pretty
good reliability.

So the way one accesses the VM changes the behavior in the guest. That seems
wrong and I reported it to QEmu:
https://bugs.launchpad.net/qemu/+bug/1499908

I have updated the w7pro64 configuration on 2015/09/25 and it now only has one
intermittent Master volume failure left in mmdevapi:capture. The change does
not seem to have negatively impacted the other tests.

So it seems the change can safely be propagated to the other Windows 7+ VMs.
There may be one issue though, which is that combining QXL with Spice causes
the Windows 10 VM to freeze very quickly. Hopefully that's specific to Windows
10.

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

https://bugs.winehq.org/show_bug.cgi?id=39442

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://bugs.launchpad.net/
                   |                            |qemu/+bug/1499908

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

https://bugs.winehq.org/show_bug.cgi?id=39442

--- Comment #1 from Hans Leidekker  ---
In reply to François Gouget from comment #0)
> The Windows 7+ VMs all use an ich6 virtual soundcard (they don't support any
> of ac97, es1370 or sb16 ones).

Why do you think that Windows 7 doesn't support ac97? I managed to switch
Windows 7 to ac97 and it makes the test failures go away.

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

https://bugs.winehq.org/show_bug.cgi?id=39442

--- Comment #2 from Hans Leidekker  ---
(In reply to Hans Leidekker from comment #1)
> In reply to François Gouget from comment #0)
> > The Windows 7+ VMs all use an ich6 virtual soundcard (they don't support any
> > of ac97, es1370 or sb16 ones).
>
> Why do you think that Windows 7 doesn't support ac97? I managed to switch
> Windows 7 to ac97 and it makes the test failures go away.

Looks like it's supported on 32-bit Windows but not on 64-bit.

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

https://bugs.winehq.org/show_bug.cgi?id=39442

--- Comment #3 from François Gouget  ---
Created attachment 56963
  --> https://bugs.winehq.org/attachment.cgi?id=56963
Source code to reproduce the issue

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=39442

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #4 from François Gouget  ---
All VMs use Spice now, including the Windows 10 ones, the only exception being
wtbwvista (probably because it has not been updated in a long time).

I don't see the failures mentioned in the original report anymore. There is
still a couple failure on Windows 7 but it happens on real hardware too (see
the Windows 8 tests on cw-rx460). So it's not related to VNC vs. Spice.

capture.c:395: Test marked todo: only PulseAudio gets here
capture.c:414: Test failed: Valid IAudioCaptureClient_GetBuffer returns
08890001

Windows 10 >= 1709 also has some failures (one of which happens on real
hardware) but again that does not looks related to the original errors.

So I think this is fixed.

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=39442

--- Comment #5 from François Gouget  ---
Closing.

[Bug 39442] Using Spice required to avoid failure in mmdevapi:capture

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=39442

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #6 from François Gouget  ---
Closing (really this time).