Summary: | ASTERISK-06194: PATCH: Set IP TOS separately for SIP packets, RTP audio packets, and RTP video packets | ||
Reporter: | Jeffrey C. Ollie (jcollie) | Labels: | |
Date Opened: | 2006-01-25 22:41:27.000-0600 | Date Closed: | 2008-01-15 16:29:48.000-0600 |
Priority: | Major | Regression? | No |
Status: | Closed/Complete | Components: | Channels/chan_sip/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) qos-1.2-patch.txt ( 1) qos-trunk-patch.txt ( 2) qos-trunk-patch-2.txt ( 3) qos-trunk-patch-8938.txt | |
Description: | Currently, Asterisk lets you set the same IP TOS for all packets that a SIP channel could generate (SIP call signalling packets, RTP audio packets, and RTP video packets). However, different types of packets have different quality of service requirements. For more information, see the "Enterprise QoS Solution Reference Network Design Guide" from Cisco: http://www.cisco.com/application/pdf/en/us/guest/netsol/ns432/c649/ccmigration_09186a008049b062.pdf ****** ADDITIONAL INFORMATION ****** Attached are patches for branch/1.2 and trunk that allow chan_sip to set different IP TOS values for the different kids of packets. | ||
Comments: | By: Olle Johansson (oej) 2006-01-26 02:04:43.000-0600 I do like this patch! THank you. Some comments: 1) The error messages needs to be corrected, they refer to the old values 2) We should create a README.tos and move your explanation from sip.conf there and refer to it. The sip.conf.sample is growing into an unmanagable file as is. 3) We cannot change the default behaviour of today really. So let's enable the new recommended values in sip.conf.sample and keep it at zero in the source. (It's a policy of the dev team to be very careful in changing default behaviour). We won't add new features to 1.2, so keep updating the trunk version. By: Jeffrey C. Ollie (jcollie) 2006-01-26 14:26:12.000-0600 Here's a new patch against rev 8749 of trunk that addresses your comments. By: Olle Johansson (oej) 2006-01-26 14:41:45.000-0600 That's fast. THank you. I will look at it over the weekend. By: Olle Johansson (oej) 2006-01-28 08:38:31.000-0600 Do we have to deprecate the old tos setting values or can we match them to the new values? Removing them harms backwards compatibility. If we need to, we have to, but if it is possible to have a translation path that would be good. We will have to implement the same setting values in chan_iax2.c as well. By: Olle Johansson (oej) 2006-01-28 08:40:39.000-0600 Sorry, did not see the changes in acl.c. The old settings is already there. But this means that other configs that use tos settings will get the output as well. We need to update the iax.conf sample config I guess, maybe others. By: Jeffrey C. Ollie (jcollie) 2006-01-30 15:03:31.000-0600 Updated patch... README.tos now addresses sip.conf, iax.conf, and iaxprov.conf, the only places that I could find TOS settings in the config files. Also updated chan_iax2.c and iax2-provision.c to give proper errors. Also updated iax2-provision.c to use ast_str2tos rather than it's own (outdated) copy of the TOS parsing code. By: Olle Johansson (oej) 2006-01-31 07:24:37.000-0600 Opening a new branch for this issue to keep the patch up to date while waiting for review http://svn.digium.com/svn/asterisk/team/oej/iptos By: Olle Johansson (oej) 2006-01-31 07:25:08.000-0600 Btw, the new settings also comply with SIP forum's PBX interoperability recommendations. By: Jeffrey C. Ollie (jcollie) 2006-01-31 07:44:16.000-0600 I found a draft of the "IP PBX / Service Provider Interoperability" document that OEJ mentions at: http://data.memberclicks.com/bbattach/123566/sf-draft-twg-IP_PBX_SP_Interop-sibley-v4.pdf The QoS information is in Section 13 on page 16. The main page for the SIP Forum's IP PBX / SP Interop TG can be found at: http://www.sipforum.org/content/view/179/213/ The archives of their mailing list can be found here: http://sipforum.org/pipermail/techwg/index.html By: Olle Johansson (oej) 2006-02-09 08:21:58.000-0600 I am amazed by the number of test reports we get. Updated to current trunk. By: Jeffrey C. Ollie (jcollie) 2006-02-09 08:43:57.000-0600 Hmm... yes, would be nice to get test reports from someone else. I've been running a patched 1.2 in production (unfortunately on a relatively lightly loaded system) with no ill effects for a while now. By: Olle Johansson (oej) 2006-03-04 17:03:42.000-0600 Any idea what we can use for UDPTL for T.38 - the same as Audio or something else? By: Jeffrey C. Ollie (jcollie) 2006-03-06 21:38:58.000-0600 I can't find anything that specifically states what DSCP packets containing T.38 data should be marked with. However, it appears that Cisco devices would use the same DSCP as they would use for audio data. So I would think that EF would be the best choice. By: Olle Johansson (oej) 2006-03-07 01:08:28.000-0600 Thanks for the feedback. By: Olle Johansson (oej) 2006-03-07 15:21:29.000-0600 This is ready for svn trunk. By: Olle Johansson (oej) 2006-03-07 15:22:08.000-0600 The code is in the /team/oej/iptos branch. By: Jared Smith (jsmith) 2006-03-13 13:08:19.000-0600 I've tested this as part of Olle's test-this-branch branch, and it looks great! I only have 2 minor concerns: 1) The debug messages point to the wrong README file (README.tos instead of ip-tos.txt) 2) If I issue an "IAX2 reload" from the Asterisk CLI, I get the following debug message, even though tos isn't set to lowdelay in iax.conf: Mar 13 12:03:38 WARNING[32280]: acl.c:262 ast_str2tos: tos value lowdelay is deprecated. See doc/README.tos for more information. Other than those two little things, I think this is probably ready for trunk (in my humble opinion). By: Olle Johansson (oej) 2006-03-27 21:29:59.000-0600 Committed to svn trunk revision 15435 - Changed the settings to IANA registered types - Made a conversion table to/from string to value Thanks for your work! By: Olle Johansson (oej) 2006-03-27 21:30:58.000-0600 - removing branch "team/oej/iptos" By: Digium Subversion (svnbot) 2008-01-15 16:29:48.000-0600 Repository: asterisk Revision: 8973 A team/oej/iptos/ ------------------------------------------------------------------------ r8973 | oej | 2008-01-15 16:29:47 -0600 (Tue, 15 Jan 2008) | 2 lines Issue ASTERISK-6194 - update of IP TOS settings for IAX and SIP ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=8973 |