<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 3, 2014 at 11:24 AM, George Joseph <span dir="ltr"><<a href="mailto:george.joseph@fairview5.com" target="_blank">george.joseph@fairview5.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div>As I was playing around with TOS/COS in pjsip last week I noticed some inconsistencies that I'd like to correct...</div>
<div><br></div><div>There's a 'tos' parameter on the transport object but not only does it actually set DSCP instead of TOS, it sets it in pjproject only and therefore for signalling only.</div>
<div><br></div><div>There are tos_audio and tos_video parameters on endpoint which do set tos for the rtp engine but they don't accept hex or symbolic values as chan_sip does.</div><div><br></div><div>So, given that DSCP has been around for a while and that's what the symbolic values represent anyway, Id like to do the following...</div>

<div><br></div><div>In transport, rename tos and cos to dscp_sip and cos_sip respectively and for dscp_sip,  create a new function ast_str2dscp() as a companion to the existing ast_str2tos that resolves the symbolics.  The new function is needed because while the existing ast_str2tos function takes in a string representation of DSCP, it left-shifts it 2 bits to go into the rtp_engine's tos field.  pjproject only accepts dscp.</div>

<div><br></div><div>In endpoint, rename tos_audio and tos_video to dscp_audio and dscp_video respectively, then use ast_str2tos to resolve the symbolics.</div><div><br></div><div>I could also add the new parameters and leave the old ones there for backwards compatibility for a while if that makes sense.</div>

<div><br></div><div>What do you think?</div><div><a href="https://issues.asterisk.org/jira/browse/ASTERISK-23235" target="_blank">https://issues.asterisk.org/jira/browse/ASTERISK-23235</a><br></div><div><br></div><div><br>
</div></div></blockquote><div><br></div><div>I agree with this.</div><div><br></div><div>*waits for spears and arrows*</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div></div><div><br>
</div></div>
<br>--<br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">
<span style="color:rgb(0,0,0);font-family:monospace;font-size:medium">--------------------------------------------------</span><br style="color:rgb(0,0,0);font-family:monospace;font-size:medium"><span style="color:rgb(0,0,0);font-family:monospace;font-size:medium">Malcolm Davenport</span><br style="color:rgb(0,0,0);font-family:monospace;font-size:medium">
<span style="color:rgb(0,0,0);font-family:monospace;font-size:medium">Digium, Inc. | Senior Product Manager</span><br style="color:rgb(0,0,0);font-family:monospace;font-size:medium"><span style="color:rgb(0,0,0);font-family:monospace;font-size:medium">445 Jan Davis Drive NW - Huntsville, AL 35806 - US</span><br style="color:rgb(0,0,0);font-family:monospace;font-size:medium">
<span style="color:rgb(0,0,0);font-family:monospace;font-size:medium">Tel/Fax: +</span><span style="color:rgb(0,0,0);font-family:monospace;font-size:medium"><a style="color:rgb(0,0,139);text-decoration:none">1 256 428 6252</a></span><br style="color:rgb(0,0,0);font-family:monospace;font-size:medium">
<span style="color:rgb(0,0,139);font-family:monospace;font-size:medium"><a href="mailto:malcolmd@digium.com" target="_blank">malcolmd@digium.com</a></span><br></div>
</div></div>