<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[Freeswitch-trunk][17151] </title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<div id="header">FreeSWITCH Subversion</div>
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://fisheye.freeswitch.org/changelog/FreeSWITCH?cs=17151">17151</a></dd>
<dt>Author</dt> <dd>mikej</dd>
<dt>Date</dt> <dd>2010-03-30 18:20:26 -0500 (Tue, 30 Mar 2010)</dd>
</dl>

<h3>Log Message</h3>
<pre>added load/noload options for the supported codecs</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#freeswitchtrunkconfautoload_configssangoma_codecconfxml">freeswitch/trunk/conf/autoload_configs/sangoma_codec.conf.xml</a></li>
<li><a href="#freeswitchtrunksrcmodcodecsmod_sangoma_codecMakefile">freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/Makefile</a></li>
<li><a href="#freeswitchtrunksrcmodcodecsmod_sangoma_codecmod_sangoma_codecc">freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/mod_sangoma_codec.c</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="freeswitchtrunkconfautoload_configssangoma_codecconfxml"></a>
<div class="modfile"><h4>Modified: freeswitch/trunk/conf/autoload_configs/sangoma_codec.conf.xml (17150 => 17151)</h4>
<pre class="diff"><span>
<span class="info">--- freeswitch/trunk/conf/autoload_configs/sangoma_codec.conf.xml        2010-03-30 23:20:18 UTC (rev 17150)
+++ freeswitch/trunk/conf/autoload_configs/sangoma_codec.conf.xml        2010-03-30 23:20:26 UTC (rev 17151)
</span><span class="lines">@@ -1,11 +1,23 @@
</span><span class="cx"> &lt;configuration name=&quot;sangoma_codec.conf&quot; description=&quot;Sangoma Codec Configuration&quot;&gt;
</span><span class="cx"> 
</span><span class="cx">         &lt;settings&gt;
</span><del>-                &lt;!-- nothing to put here yet --&gt;
</del><ins>+                &lt;!--
+                Comma separated list of codecs to register with FreeSWITCH, 
+                by default (if this parameter is not set) all available codecs are registered.
+                Valid codec values are: PCMU,PCMA,L16,G729,G726-32,GSM
+                &lt;param name=&quot;load&quot; value=&quot;all&quot;/&gt;
+                --&gt;
+
+                &lt;!-- List of codecs to not register with FreeSWITCH, by default this is empty,
+                but you may want to not load PCMU and PCMA or may be others to not use your 
+                vocallo resources in codecs that are done well and fast in software.        
+                &lt;param name=&quot;noload&quot; value=&quot;PCMU,PCMA&quot;/&gt;
+                --&gt;
</ins><span class="cx">         &lt;/settings&gt;
</span><span class="cx"> 
</span><span class="cx">         &lt;vocallos&gt; 
</span><span class="cx"> 
</span><ins>+                &lt;!-- The name of the vocallo is the ethernet device name as displayed by ifconfig --&gt;
</ins><span class="cx">                 &lt;vocallo name=&quot;eth5&quot;&gt;
</span><span class="cx">                         &lt;!-- Starting UDP port for the vocallo --&gt;
</span><span class="cx">                         &lt;param name=&quot;baseudp&quot; value=&quot;5000&quot;/&gt;
</span></span></pre></div>
<a id="freeswitchtrunksrcmodcodecsmod_sangoma_codecMakefile"></a>
<div class="modfile"><h4>Modified: freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/Makefile (17150 => 17151)</h4>
<pre class="diff"><span>
<span class="info">--- freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/Makefile        2010-03-30 23:20:18 UTC (rev 17150)
+++ freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/Makefile        2010-03-30 23:20:26 UTC (rev 17151)
</span><span class="lines">@@ -1,3 +1,3 @@
</span><span class="cx"> BASE=../../../..
</span><del>-LOCAL_LDFLAGS += -lsangoma_transcode
</del><ins>+LOCAL_LDFLAGS += -lsng-tc
</ins><span class="cx"> include $(BASE)/build/modmake.rules
</span></span></pre></div>
<a id="freeswitchtrunksrcmodcodecsmod_sangoma_codecmod_sangoma_codecc"></a>
<div class="modfile"><h4>Modified: freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/mod_sangoma_codec.c (17150 => 17151)</h4>
<pre class="diff"><span>
<span class="info">--- freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/mod_sangoma_codec.c        2010-03-30 23:20:18 UTC (rev 17150)
+++ freeswitch/trunk/src/mod/codecs/mod_sangoma_codec/mod_sangoma_codec.c        2010-03-30 23:20:26 UTC (rev 17151)
</span><span class="lines">@@ -35,7 +35,7 @@
</span><span class="cx"> #include &quot;switch.h&quot;
</span><span class="cx"> #include &quot;g711.h&quot;
</span><span class="cx"> 
</span><del>-#include &lt;sangoma_transcode.h&gt;
</del><ins>+#include &lt;sng_tc/sng_tc.h&gt;
</ins><span class="cx"> 
</span><span class="cx"> #ifdef __linux__
</span><span class="cx"> /* for ethernet device query */
</span><span class="lines">@@ -101,13 +101,21 @@
</span><span class="cx">         { SNGTC_CODEC_G729AB,  18,  &quot;G729&quot;,    &quot;Sangoma G729&quot;,      40, 8000,   10000, 80,  160, 10,  1 },
</span><span class="cx">         { SNGTC_CODEC_G726_32, 122, &quot;G726-32&quot;, &quot;Sangoma G.726 32k&quot;, 40, 32000,  10000, 80,  160, 40,  1 },
</span><span class="cx">         { SNGTC_CODEC_GSM_FR,  3,   &quot;GSM&quot;,     &quot;Sangoma GSM&quot;,       20, 13200,  20000, 160, 320, 33,  0 },
</span><ins>+#if 0
</ins><span class="cx">         /* FIXME: grandstream crashes with iLBC implementation */
</span><span class="cx">         { SNGTC_CODEC_ILBC,    97,  &quot;iLBC&quot;,    &quot;Sangoma ILBC&quot;,      -1, -1,     -1,    -1,  -1,  -1,  0 },
</span><span class="cx">         /* FIXME: sampling rate seems wrong with this, audioooo soooundssssss sloooooow ... */
</span><span class="cx">         { SNGTC_CODEC_G722,    9,   &quot;G722&quot;,    &quot;Sangoma G722&quot;,      20, 64000,  20000, 160, 640, 160, 0 },
</span><ins>+#endif
</ins><span class="cx">         { -1,                  -1,  NULL,      NULL,                -1, -1,     -1,    -1,  -1,      -1 },
</span><span class="cx"> };
</span><span class="cx"> 
</span><ins>+/* default codec list to load, users may override, special codec 'all' loads everything available unless listed in noload */
+static char g_codec_load_list[1024] = &quot;all&quot;;
+
+/* default codec list to NOT load, users may override */
+static char g_codec_noload_list[1024] = &quot;&quot;;
+
</ins><span class="cx"> struct codec_data {
</span><span class="cx">         /* sngtc request and reply */
</span><span class="cx">         sngtc_codec_request_t request;
</span><span class="lines">@@ -206,8 +214,8 @@
</span><span class="cx">         rtp_session = switch_rtp_new(local_ip, rtp_port, 
</span><span class="cx">                         codec_ip, codec_reply_leg-&gt;codec_udp_port, 
</span><span class="cx">                         iana,
</span><del>-                        codec_reg_leg-&gt;ms*8, /* samples per interval, FIXME: do based on sampling rate */
-                        codec_reg_leg-&gt;ms*1000, /* ms per packet */
</del><ins>+                        sess-&gt;impl-&gt;samples_per_packet,
+                        codec_reg_leg-&gt;ms*1000, /* microseconds per packet */
</ins><span class="cx">                         flags, NULL, &amp;err, g_pool);
</span><span class="cx"> 
</span><span class="cx">         if (!rtp_session) {
</span><span class="lines">@@ -845,6 +853,21 @@
</span><span class="cx"> 
</span><span class="cx">         if ((settings = switch_xml_child(cfg, &quot;settings&quot;))) {
</span><span class="cx">                 /* nothing here yet */
</span><ins>+                for (param = switch_xml_child(settings, &quot;param&quot;); param; param = param-&gt;next) {
+                                char *var = (char *)switch_xml_attr_soft(param, &quot;name&quot;);
+                                char *val = (char *)switch_xml_attr_soft(param, &quot;value&quot;);
+
+                                /* this parameter overrides the default list of codecs to load */
+                                if (!strcasecmp(var, &quot;load&quot;)) {
+                                        strncpy(g_codec_load_list, val, sizeof(g_codec_load_list)-1);
+                                        g_codec_load_list[sizeof(g_codec_load_list)-1] = 0;
+                                } else if (!strcasecmp(var, &quot;noload&quot;)) {
+                                        strncpy(g_codec_noload_list, val, sizeof(g_codec_noload_list)-1);
+                                        g_codec_noload_list[sizeof(g_codec_noload_list)-1] = 0;
+                                } else {
+                                        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, &quot;Ignored unknown Sangoma codec setting %s\n&quot;, var);
+                                }
+                }
</ins><span class="cx">         }
</span><span class="cx"> 
</span><span class="cx">         if ((vocallos = switch_xml_child(cfg, &quot;vocallos&quot;))) {
</span><span class="lines">@@ -888,7 +911,7 @@
</span><span class="cx">                                         }
</span><span class="cx">                                         g_init_cfg.host_nic_vocallo_cfg[vidx].vocallo_ip = ntohl(vocallo_base_ip.s_addr);
</span><span class="cx">                                 } else {
</span><del>-                                        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, &quot;Ignored unknown Sangoma codec setting %s\n&quot;, val);
</del><ins>+                                        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_WARNING, &quot;Ignored unknown Sangoma vocallo setting %s\n&quot;, var);
</ins><span class="cx">                                 }
</span><span class="cx">                         }
</span><span class="cx"> 
</span><span class="lines">@@ -966,11 +989,28 @@
</span><span class="cx">         *module_interface = switch_loadable_module_create_module_interface(pool, modname);
</span><span class="cx"> 
</span><span class="cx"> 
</span><ins>+        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, &quot;Loading codecs, load='%s', noload='%s'\n&quot;, g_codec_load_list, g_codec_noload_list);
</ins><span class="cx">         for (c = 0; g_codec_map[c].codec_id != -1; c++) {
</span><span class="cx"> 
</span><ins>+                /* check if the codec is in the load list, otherwise skip it */
+                if (strcasecmp(g_codec_load_list, &quot;all&quot;) &amp;&amp; !strcasestr(g_codec_load_list, g_codec_map[c].iana_name)) {
+                        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, &quot;Not loading codec %s because was not found in the load list\n&quot;, 
+                                        g_codec_map[c].iana_name);
+                        continue;
+                }
+
+                /* load it unless is named in the noload list */
+                if (strcasestr(g_codec_noload_list, g_codec_map[c].iana_name)) {
+                        switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, &quot;Not loading codec %s because was not found in the noload list\n&quot;, 
+                                        g_codec_map[c].iana_name);
+                        continue;
+                }
+
+                switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, &quot;Registering implementations for codec %s\n&quot;, g_codec_map[c].iana_name);
+
</ins><span class="cx">                 /* let know the library which iana to use */
</span><span class="cx">                 sngtc_set_iana_code_based_on_codec_id(g_codec_map[c].codec_id, g_codec_map[c].iana);
</span><del>-                switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_NOTICE, &quot;Mapped codec %d to IANA %d\n&quot;, g_codec_map[c].codec_id, g_codec_map[c].iana);
</del><ins>+                switch_log_printf(SWITCH_CHANNEL_LOG, SWITCH_LOG_DEBUG, &quot;Mapped codec %d to IANA %d\n&quot;, g_codec_map[c].codec_id, g_codec_map[c].iana);
</ins><span class="cx"> 
</span><span class="cx">                 /* SWITCH_ADD_CODEC allocates a codec interface structure from the pool the core gave us and adds it to the internal interface 
</span><span class="cx">                  * list the core keeps, gets a codec id and set the given codec name to it.
</span></span></pre>
</div>
</div>
<div id="footer">See you at ClueCon</div>

</body>
</html>