History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: XCL-130
Type: New Feature New Feature
Status: Open Open
Priority: Minor Minor
Assignee: Alexander Chemeris
Reporter: Alexander Chemeris
Votes: 0
Watchers: 1
Operations

If you were logged in you would be able to see more operations.
sipXtapi: sipXcallLib

Support for G.729 and G.723 codecs with Intel IPP

Created: 2007-03-10 06:41   Updated: 2007-05-01 17:46
Component/s: sipXtapi
Affects Version/s: None
Fix Version/s: None

Original Estimate: Unknown Remaining Estimate: Unknown Time Spent: Unknown
File Attachments: 1. Zip Archive IPP codecs.zip (15 kb)
2. File IPPcodecs.diff (56 kb)
3. Zip Archive newpatch.zip (28 kb)

Environment: sipXtapi branch
Issue Links:
Composition
 
This issue has part:
XCL-96 Support of G.729A codec Major Closed


 Description  « Hide
This patch was originally created by Dennis Surovtsev. I did not check is it working or not and how it conform to coding style guideline. Feel free to use it as inspiration or clean up it and resubmit. I would happily check in G.729 and G.723 support if somebody provide good patch.

 All   Comments   Work Log   Change History      Sort Order:
Jaroslav Libak - 2007-03-13 16:41
Ok here is what I've got. Patches + readme files are in the attachment.
These are modified patches of Dennis Surovtsev. They contain some bugfixes, code style changes.

G729A works for me using Intel IPP 5.1. I have tested it by dialing another "phone" - ReceiveCall_dbgNoVideo.exe
Then also by dialing a test number of a voip operator, and also by calling another number of the same operator and having a conversation with myself on another phone.

If there is silence, then rtp frames of size 0 or 2 appear, these are rejected, I dont know what to do with these right now.
Also if there is silence, then decoder sometimes decodes 240 bytes instead of 320 bytes (160 samples), but decoder pretends 160 samples were decoded (but there are no audio glitches mysteriously) as the USC decoder doesnt return the number of decoded bytes in a variable. It returns instead number of input bytes processed.

Alexander needs to have a look at this code and maybe fix some bugs or do style changes. It should also be tested by more people than 1.

G723.1 - Dennis wrote the patches as if this decoder decoded both 723.1 and 723 but I have read 723 is completely different so I dont know how that would work. So the code was modified to work with 723.1 only.
There are some changes in the code, but it is untested and probably wont work (729A didnt work in the beginning either). Someone needs to test this. I dont need this codec right now.

The readme.txt contains everything you should need to get it to compile.

Jaroslav Libak - 2007-05-01 17:46
Newer version of the patch, just 2 diff files, for the latest revision.