Mantis Bugtracker

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000330 [Yate - Yet Another Telephony Engine] module minor have not tried 2013-01-10 20:49 2013-02-01 18:57
Reporter iev View Status public  
Assigned To
Priority normal Resolution open  
Status acknowledged   Product Version 4.x
Summary 0000330: Yate Client on Windows - stuttered sound when using secondary buffer
Description Module dsoundchan
Voice stutters when primary=no in dsoundchan.conf
Lots of debug messages like
<dsound:NOTE> Slip detected, changing write offs from 466 to 1106, p=390 w=1026
<dsound:NOTE> Slip detected, changing write offs from 786 to 1266, p=612 w=1186
<dsound:NOTE> Slip detected, changing write offs from 626 to 20, p=580 w=1220
<dsound:NOTE> Slip detected, changing write offs from 340 to 1020, p=302 w=940
when debug logging to file enabled.

I suppose size of secondary buffer is set too small -
<dsound:INFO> DirectSound buffer size 1280
<dsound:INFO> DirectSound buffer size 32768
when using primary buffer.
Sorry, but I could not check it yet because could not build the module.

Additional Information When using default settings of Yate client other sound-producing programs fail when call is accepted in Yate client.
As the cause of it is use of primary buffer in directsound by Yate client, I decided to try using secondary buffer.
Tags No tags attached.
Attached Files ? file icon dsoundchan.patch [^] (2,544 bytes) 2013-01-30 20:06

- Relationships

-  Notes
iev (reporter)
2013-01-17 20:24
edited on: 2013-01-17 20:36

At last I've managed to rebuild from source and can positively confirm that
changing line 236 of modules/client/dsoundchan.cpp from
bdesc.dwBufferBytes = 4*m_chunk;
bdesc.dwBufferBytes = 32768; //4*m_chunk;
fixes stuttering sound on secondary dsound buffer.

iev (reporter)
2013-01-30 20:13

Proposed patch:
1. Add config variable "bufsize" for setting secondary buffer size, 32768 by default
2. Use secondary buffer by default to enhance compatibility with oher programs.

Tested on Windows XP SP3.

- Issue History
Date Modified Username Field Change
2013-01-10 20:49 iev New Issue
2013-01-17 20:24 iev Note Added: 0000506
2013-01-17 20:36 iev Note Edited: 0000506
2013-01-30 20:06 iev File Added: dsoundchan.patch
2013-01-30 20:13 iev Note Added: 0000507
2013-01-30 20:14 iev Issue Monitored: iev
2013-02-01 18:57 paulc Status new => acknowledged

Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker