[PATCH 0/1] MR9840: winepulse: A potential solution to audio crackling

[Dzmitry Keremsha (＠Vyrolian)]

I have noticed that winepulse leaves too little headroom when draining the buffer, which very often leads to crackling that can't be fully resolved by manipulating latency.

Let's check some math:

Let's assume that the buffer (tlength) is at 33ms, and minreq is at 13.3ms. If 13.2 ms were drained, the refill would be denied. If the next timer trigger takes 14ms due to jitter, that leaves us with a 6.1ms margin. As such, if we get any jitter longer than that during refill, we get an underflow. I think this is roughly what leads to crackles.

My solution increases timer polling to 5x the frequency, so if 13.2ms were drained, the next timer trigger would happen at around 16ms. That would leave us with a 17ms headroom.

The math might not be 100% representation of what happens, but I think it's quite close to reality. On practice, this does indeed help to fully get rid of crackling for an indefinite amount of time, when before they were unevitable.

I understand that this might be not the most elegant approach, the 5x increase is arbitrary. I also had to remove the adv_bytes logic and use to_write to move pointers within pulse_write instead.

I think this problem is quite urgent, because while this does not happen on every system, it's frequent enough that we see "Audio: Crackling" reports on most games in protondb and there are many forum threads about audio crackling. From my experience, the only solution was switching to winealsa, which is very consistent and does not crackle at all. My experience has been very consistent with forum threads when it came to the nature of crackling. I first noticed it in Balatro, which is a very light game and should not be crackling at all on any setup. Some games have very frequent crackling every 20-30 seconds, or a buzzsaw crackling every 10-15 minutes.

I hope my idea brings us closer to the ultimate fix, I really look forward to your feedback!

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9840

[PATCH 1/1] winepulse patch

[Dzmitry Keremsha]

From: Dzmitry Keremsha <[e-mail]>

---
 dlls/winepulse.drv/pulse.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/dlls/winepulse.drv/pulse.c b/dlls/winepulse.drv/pulse.c
index 7c8d872c604..93f10499c01 100644
--- a/dlls/winepulse.drv/pulse.c
+++ b/dlls/winepulse.drv/pulse.c
@@ -1180,7 +1180,7 @@ static NTSTATUS pulse_create_stream(void *args)
 
     stream->bufsize_frames = ceil((params->duration / 10000000.) * params->fmt->nSamplesPerSec);
     bufsize_bytes = stream->bufsize_frames * pa_frame_size(&stream->ss);
-    stream->mmdev_period_usec = params->period / 10;
+    stream->mmdev_period_usec = params->period / 50;
 
     stream->share = params->share;
     stream->flags = params->flags;
@@ -1451,6 +1451,9 @@ static void pulse_write(struct pulse_stream *stream)
         to_write = stream->real_bufsize_bytes - stream->pa_offs_bytes;
         TRACE("writing small chunk of %u bytes\n", to_write);
         write_buffer(stream, buf, to_write);
+        stream->held_bytes -= to_write;
+        stream->lcl_offs_bytes += to_write;
+        stream->lcl_offs_bytes %= stream->real_bufsize_bytes;
         stream->pa_held_bytes -= to_write;
         to_write = bytes - to_write;
         stream->pa_offs_bytes = 0;
@@ -1461,6 +1464,9 @@ static void pulse_write(struct pulse_stream *stream)
 
     TRACE("writing main chunk of %u bytes\n", to_write);
     write_buffer(stream, buf, to_write);
+    stream->held_bytes -= to_write;
+    stream->lcl_offs_bytes += to_write;
+    stream->lcl_offs_bytes %= stream->real_bufsize_bytes;
     stream->pa_offs_bytes += to_write;
     stream->pa_offs_bytes %= stream->real_bufsize_bytes;
     stream->pa_held_bytes -= to_write;
@@ -1572,7 +1578,6 @@ static NTSTATUS pulse_timer_loop(void *args)
     struct pulse_stream *stream = handle_get_stream(params->stream);
     LARGE_INTEGER delay;
     pa_usec_t last_time;
-    UINT32 adv_bytes;
     int success;
 
     pulse_lock();
@@ -1634,12 +1639,6 @@ static NTSTATUS pulse_timer_loop(void *args)
                 if (stream->dataflow == eRender)
                 {
                     pulse_write(stream);
-
-                    /* regardless of what PA does, advance one period */
-                    adv_bytes = min(stream->period_bytes, stream->held_bytes);
-                    stream->lcl_offs_bytes += adv_bytes;
-                    stream->lcl_offs_bytes %= stream->real_bufsize_bytes;
-                    stream->held_bytes -= adv_bytes;
                 }
                 else if(stream->dataflow == eCapture)
                 {
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/9840