[asterisk-bugs] [Asterisk 0011962]: [patch] Simple file recording buffer - great performance gain

[noreply at bugs.digium.com]

The following issue requires your FEEDBACK. 
====================================================================== 
http://bugs.digium.com/view.php?id=11962 
====================================================================== 
Reported By:                garlew
Assigned To:                Corydon76
====================================================================== 
Project:                    Asterisk
Issue ID:                   11962
Category:                   Resources/res_monitor
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             02-09-2008 05:33 CST
Last Modified:              04-02-2008 13:41 CDT
====================================================================== 
Summary:                    [patch] Simple file recording buffer - great
performance gain
Description: 
By default there is no recording buffer in Asterisk. There were attempts to
record  to RAM-disk etc. It turns out creating a simple 32K buffer for each
recorded stream greatly increases number of files Asterisk can record
simultaneously without RAM-disk. I'm not putting any numbers here because
it depends on disk subsystem used.

Also recording to wav format unnecesarily calls update_header for every
frame written - it is sufficient to update header when closing the stream.
Below is strace output for one frame wav write before the patch.
[pid 30797] write(26, "\344\343\1\0", 4) = 4
[pid 30797] write(26, "\300\343\1\0", 4) = 4
[pid 30797] write(26,
"\30\0\10\0x\0\330\377\370\3778\0\270\377\30\0\350\377\230"..., 320) = 320

====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 04-02-08 13:41  
---------------------------------------------------------------------- 
As this is a new feature, the patch needs to be updated to apply against
trunk.  As soon as that is done, this can go in. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-02-08 13:41  Corydon76      Note Added: 0084948                          
04-02-08 13:41  Corydon76      Assigned To               => Corydon76       
04-02-08 13:41  Corydon76      Status                   new => feedback     
======================================================================