Importing release summary for 1.4.12 release.

git-svn-id: https://origsvn.digium.com/svn/libpri/tags/1.4.12@2270 2fbb986a-6c06-0410-b554-c9c1f0a7f128
Importing files for 1.4.12 release.
2011-07-05 21:11:30 +00:00 · 2011-07-05 21:11:28 +00:00 · 2011-07-05 21:10:48 +00:00
20 changed files with 3235 additions and 1666 deletions
--- a/.cleancount
+++ b/.cleancount
@@ -1 +0,0 @@
-0
--- a/.gitignore
+++ b/.gitignore
@@ -1,12 +0,0 @@
-*.o
-*.o.d
-*.lo
-*.so
-*.a
-libpri.so.*
-pridump
-pritest
-rosetest
-testprilib
-version.c
-
--- a/.gitreview
+++ b/.gitreview
@@ -1,4 +0,0 @@
-[gerrit]
-host=gerrit.asterisk.org
-port=29418
-project=libpri.git
--- a/.version
+++ b/.version
@@ -0,0 +1 @@
+1.4.12
--- a/1800
+++ b/1800
--- a/87
+++ b/87
@@ -10,15 +10,15 @@
 # it under the terms of the GNU General Public License as published by
 # the Free Software Foundation; either version 2 of the License, or
 # (at your option) any later version.
-#
+# 
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
-#
+# 
 # You should have received a copy of the GNU General Public License
 # along with this program; if not, write to the Free Software
-# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. 
 #
 #
 # Uncomment if you want libpri not send PROGRESS_INDICATOR w/ALERTING
@@ -27,8 +27,6 @@
 CC=gcc
 GREP=grep
 AWK=awk
-AR=ar
-RANLIB=ranlib

 OSARCH=$(shell uname -s)
 PROC?=$(shell uname -m)
@@ -67,22 +65,43 @@ STATIC_OBJS= \
 	rose_qsig_name.o \
 	version.o
 DYNAMIC_OBJS= \
-	$(STATIC_OBJS)
-CFLAGS ?= -g
-CFLAGS += $(CPPFLAGS)
-CFLAGS += -Wall -Werror -Wstrict-prototypes -Wmissing-prototypes
-CFLAGS += -fPIC $(ALERTING) $(LIBPRI_OPT) $(COVERAGE_CFLAGS)
+	copy_string.lo \
+	pri.lo \
+	q921.lo \
+	prisched.lo \
+	q931.lo \
+	pri_aoc.lo \
+	pri_cc.lo \
+	pri_facility.lo \
+	asn1_primitive.lo \
+	rose.lo \
+	rose_address.lo \
+	rose_etsi_aoc.lo \
+	rose_etsi_cc.lo \
+	rose_etsi_diversion.lo \
+	rose_etsi_ect.lo \
+	rose_etsi_mwi.lo \
+	rose_other.lo \
+	rose_q931.lo \
+	rose_qsig_aoc.lo \
+	rose_qsig_cc.lo \
+	rose_qsig_ct.lo \
+	rose_qsig_diversion.lo \
+	rose_qsig_mwi.lo \
+	rose_qsig_name.lo \
+	version.lo
+CFLAGS=-Wall -Werror -Wstrict-prototypes -Wmissing-prototypes -g -fPIC $(ALERTING) $(LIBPRI_OPT) $(COVERAGE_CFLAGS)
 INSTALL_PREFIX=$(DESTDIR)
 INSTALL_BASE=/usr
 libdir?=$(INSTALL_BASE)/lib
 ifneq ($(findstring Darwin,$(OSARCH)),)
-  SOFLAGS=$(LDFLAGS) -dynamic -bundle -Xlinker -macosx_version_min -Xlinker 10.4 -Xlinker -undefined -Xlinker dynamic_lookup -force_flat_namespace
+  SOFLAGS=-dynamic -bundle -Xlinker -macosx_version_min -Xlinker 10.4 -Xlinker -undefined -Xlinker dynamic_lookup -force_flat_namespace
  ifeq ($(shell /usr/bin/sw_vers -productVersion | cut -c1-4),10.6)
    SOFLAGS+=/usr/lib/bundle1.o
  endif
  LDCONFIG=/usr/bin/true
 else
-  SOFLAGS=$(LDFLAGS) -shared -Wl,-h$(DYNAMIC_LIBRARY) $(COVERAGE_LDFLAGS)
+  SOFLAGS=-shared -Wl,-h$(DYNAMIC_LIBRARY) $(COVERAGE_LDFLAGS)
  LDCONFIG = /sbin/ldconfig
 endif
 ifneq (,$(findstring X$(OSARCH)X, XLinuxX XGNU/kFreeBSDX XGNUX))
@@ -96,13 +115,11 @@ endif
 endif
 ifeq (${OSARCH},SunOS)
 CFLAGS += -DSOLARIS -I../zaptel-solaris
-LDCONFIG =
+LDCONFIG = 
 LDCONFIG_FLAGS = \# # Trick to comment out the period in the command below
 #INSTALL_PREFIX = /opt/asterisk  # Uncomment out to install in standard Solaris location for 3rd party code
 endif

-UTILITIES= pridump pritest rosetest testprilib
-
 export PRIVERSION

 PRIVERSION:=$(shell GREP=$(GREP) AWK=$(AWK) build_tools/make_version .)
@@ -128,7 +145,7 @@ CFLAGS += -m32
 SOFLAGS += -m32
 endif

-all: $(STATIC_LIBRARY) $(DYNAMIC_LIBRARY) $(UTILITIES)
+all: $(STATIC_LIBRARY) $(DYNAMIC_LIBRARY)

 update:
 	@if [ -d .svn ]; then \
@@ -154,37 +171,37 @@ ifneq (${OSARCH},SunOS)
 	install -m 644 libpri.h $(INSTALL_PREFIX)$(INSTALL_BASE)/include
 	install -m 755 $(DYNAMIC_LIBRARY) $(INSTALL_PREFIX)$(libdir)
 	#if [ -x /usr/sbin/sestatus ] && ( /usr/sbin/sestatus | grep "SELinux status:" | grep -q "enabled"); then /sbin/restorecon -v $(INSTALL_PREFIX)$(libdir)/$(DYNAMIC_LIBRARY); fi
-	( cd $(INSTALL_PREFIX)$(libdir) ; ln -sf $(DYNAMIC_LIBRARY) libpri.so)
+	( cd $(INSTALL_PREFIX)$(libdir) ; ln -sf libpri.so.$(SONAME) libpri.so)
 	install -m 644 $(STATIC_LIBRARY) $(INSTALL_PREFIX)$(libdir)
 	if test $$(id -u) = 0; then $(LDCONFIG) $(LDCONFIG_FLAGS) $(INSTALL_PREFIX)$(libdir); fi
 else
 	install -f $(INSTALL_PREFIX)$(INSTALL_BASE)/include -m 644 libpri.h
 	install -f $(INSTALL_PREFIX)$(libdir) -m 755 $(DYNAMIC_LIBRARY)
-	( cd $(INSTALL_PREFIX)$(libdir) ; ln -sf $(DYNAMIC_LIBRARY) libpri.so)
+	( cd $(INSTALL_PREFIX)$(libdir) ; ln -sf libpri.so.$(SONAME) libpri.so)
 	install -f $(INSTALL_PREFIX)$(libdir) -m 644 $(STATIC_LIBRARY)
 endif

 uninstall:
 	@echo "Removing Libpri"
-	rm -f $(INSTALL_PREFIX)$(libdir)/$(STATIC_LIBRARY)
+	rm -f $(INSTALL_PREFIX)$(libdir)/libpri.so.$(SONAME)
 	rm -f $(INSTALL_PREFIX)$(libdir)/libpri.so
-	rm -f $(INSTALL_PREFIX)$(libdir)/$(DYNAMIC_LIBRARY)
+	rm -f $(INSTALL_PREFIX)$(libdir)/libpri.a
 	rm -f $(INSTALL_PREFIX)$(INSTALL_BASE)/include/libpri.h

-pritest: pritest.o $(STATIC_LIBRARY)
-	$(CC) -o $@ $< $(STATIC_LIBRARY) $(CFLAGS)
+pritest: pritest.o
+	$(CC) -o pritest pritest.o -L. -lpri $(CFLAGS)

 testprilib.o: testprilib.c
-	$(CC) $(CFLAGS) -D_REENTRANT -D_GNU_SOURCE $(MAKE_DEPS) -c -o $@ $<
+	$(CC) $(CFLAGS) -D_REENTRANT -D_GNU_SOURCE -o $@ -c $<

-testprilib: testprilib.o $(STATIC_LIBRARY)
-	$(CC) -o $@ $< $(STATIC_LIBRARY) -lpthread $(CFLAGS)
+testprilib: testprilib.o
+	$(CC) -o testprilib testprilib.o -L. -lpri -lpthread $(CFLAGS)

-pridump: pridump.o $(DYNAMIC_LIBRARY)
-	$(CC) -o $@ $< -L. -lpri $(CFLAGS)
+pridump: pridump.o
+	$(CC) -o pridump pridump.o -L. -lpri $(CFLAGS)

-rosetest: rosetest.o $(STATIC_LIBRARY)
-	$(CC) -o $@ $< $(STATIC_LIBRARY) $(CFLAGS)
+rosetest: rosetest.o
+	$(CC) -o rosetest rosetest.o -L. -lpri $(CFLAGS)

 MAKE_DEPS= -MD -MT $@ -MF .$(subst /,_,$@).d -MP

@@ -195,13 +212,13 @@ MAKE_DEPS= -MD -MT $@ -MF .$(subst /,_,$@).d -MP
 	$(CC) $(CFLAGS) $(MAKE_DEPS) -c -o $@ $<

 $(STATIC_LIBRARY): $(STATIC_OBJS)
-	$(AR) rcs $(STATIC_LIBRARY) $(STATIC_OBJS)
-	$(RANLIB) $(STATIC_LIBRARY)
+	ar rcs $(STATIC_LIBRARY) $(STATIC_OBJS)
+	ranlib $(STATIC_LIBRARY)

 $(DYNAMIC_LIBRARY): $(DYNAMIC_OBJS)
 	$(CC) $(SOFLAGS) -o $@ $(DYNAMIC_OBJS)
 	$(LDCONFIG) $(LDCONFIG_FLAGS) .
-	ln -sf $(DYNAMIC_LIBRARY) libpri.so
+	ln -sf libpri.so.$(SONAME) libpri.so

 version.c: FORCE
 	@build_tools/make_version_c > $@.tmp
@@ -209,9 +226,9 @@ version.c: FORCE
 	@rm -f $@.tmp

 clean:
-	rm -f *.o *.so *.lo
-	rm -f $(STATIC_LIBRARY) $(DYNAMIC_LIBRARY)
-	rm -f $(UTILITIES)
+	rm -f *.o *.so *.lo *.so.$(SONAME)
+	rm -f testprilib $(STATIC_LIBRARY) $(DYNAMIC_LIBRARY)
+	rm -f pritest pridump
 	rm -f .*.d

 .PHONY:
--- a/libpri-1.4.12-summary.html
+++ b/libpri-1.4.12-summary.html
@@ -0,0 +1,325 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml">
+<head><meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /><title>Release Summary - libpri-1.4.12</title></head>
+<body>
+<h1 align="center"><a name="top">Release Summary</a></h1>
+<h3 align="center">libpri-1.4.12</h3>
+<h3 align="center">Date: 2011-07-05</h3>
+<h3 align="center">&lt;asteriskteam@digium.com&gt;</h3>
+<hr/>
+<h2 align="center">Table of Contents</h2>
+<ol>
+   <li><a href="#summary">Summary</a></li>
+   <li><a href="#contributors">Contributors</a></li>
+   <li><a href="#issues">Closed Issues</a></li>
+   <li><a href="#commits">Other Changes</a></li>
+   <li><a href="#diffstat">Diffstat</a></li>
+</ol>
+<hr/>
+<a name="summary"><h2 align="center">Summary</h2></a>
+<center><a href="#top">[Back to Top]</a></center><br/><p>This release includes only bug fixes.  The changes included were made only to address problems that have been identified in this release series.  Users should be able to safely upgrade to this version if this release series is already in use.  Users considering upgrading from a previous release series are strongly encouraged to review the UPGRADE.txt document as well as the CHANGES document for information about upgrading to this release series.</p>
+<p>The data in this summary reflects changes that have been made since the previous release, libpri-1.4.11.</p>
+<hr/>
+<a name="contributors"><h2 align="center">Contributors</h2></a>
+<center><a href="#top">[Back to Top]</a></center><br/><p>This table lists the people who have submitted code, those that have tested patches, as well as those that reported issues on the issue tracker that were resolved in this release.  For coders, the number is how many of their patches (of any size) were committed into this release.  For testers, the number is the number of times their name was listed as assisting with testing a patch.  Finally, for reporters, the number is the number of issues that they reported that were closed by commits that went into this release.</p>
+<table width="100%" border="0">
+<tr>
+<td width="33%"><h3>Coders</h3></td>
+<td width="33%"><h3>Testers</h3></td>
+<td width="33%"><h3>Reporters</h3></td>
+</tr>
+<tr valign="top">
+<td>
+77 rmudgett<br/>
+1 alecdavis<br/>
+1 dvossel<br/>
+1 horape<br/>
+1 jcollie<br/>
+1 mattf<br/>
+1 nic<br/>
+1 paravoid<br/>
+1 russell<br/>
+1 tilghman<br/>
+</td>
+<td>
+10 rmudgett<br/>
+1 cmorford<br/>
+1 ibercom<br/>
+1 jcovert<br/>
+1 jsmith<br/>
+1 lelio<br/>
+1 paravoid<br/>
+1 patrol-cz<br/>
+1 shawkris<br/>
+1 wimpy<br/>
+</td>
+<td>
+2 wuwu<br/>
+1 alecdavis<br/>
+1 bklang<br/>
+1 cmorford<br/>
+1 gelo<br/>
+1 horape<br/>
+1 ibercom<br/>
+1 jcollie<br/>
+1 jcovert<br/>
+1 jims8650<br/>
+1 jmls<br/>
+1 jsmith<br/>
+1 kenner<br/>
+1 kobaz<br/>
+1 lelio<br/>
+1 nic_bellamy<br/>
+1 paravoid<br/>
+1 patrol-cz<br/>
+1 schmoozecom<br/>
+1 shawkris<br/>
+1 tomaso<br/>
+1 wimpy<br/>
+</td>
+</tr>
+</table>
+<hr/>
+<a name="issues"><h2 align="center">Closed Issues</h2></a>
+<center><a href="#top">[Back to Top]</a></center><br/><p>This is a list of all issues from the issue tracker that were closed by changes that went into this release.</p>
+<h3>Category: Channels/chan_dahdi</h3><br/>
+<a href="https://issues.asterisk.org/jira/browse/ASTERISK-16038">ASTERISK-16038</a>: getting warning message every 4 seconds<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2113">2113</a><br/>
+Reporter: jmls<br/>
+Coders: rmudgett<br/>
+<br/>
+<h3>Category: Channels/chan_dahdi/NewFeature</h3><br/>
+<a href="https://issues.asterisk.org/jira/browse/ASTERISK-13410">ASTERISK-13410</a>: [patch] CCBS/CCNR support for QSIG (libpri & chan_dahdi)<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1714">1714</a><br/>
+Reporter: tomaso<br/>
+Testers: rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<h3>Category: General</h3><br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-102">PRI-102</a>: [patch] [regression] Calling Number assignment logic change in libpri 1.4.11.<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1823">1823</a><br/>
+Reporter: ibercom<br/>
+Testers: ibercom<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-103">PRI-103</a>: segfault in pri_schedule_del - ctrl invalid value<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2015">2015</a><br/>
+Reporter: wuwu<br/>
+Testers: rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-105">PRI-105</a>: [patch] ISDN BRI does not recover from line faults<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1918">1918</a><br/>
+Reporter: jcovert<br/>
+Testers: jcovert, rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-107">PRI-107</a>: [patch] Calling name not successfully processed on inbound QSIG PRI calls from Mitel PBX<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1828">1828</a><br/>
+Reporter: jims8650<br/>
+Testers: rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-110">PRI-110</a>: [patch] BRI NT ptmp: Active channels not cleard when i/f goes down<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1982">1982</a><br/>
+Reporter: wimpy<br/>
+Testers: rmudgett, wimpy<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-112">PRI-112</a>: [patch] Libpri crashes when receiving Message Type (100)<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2021">2021</a><br/>
+Reporter: gelo<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-113">PRI-113</a>: [patch] Asterisk is core dumping with LibPRI<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2015">2015</a><br/>
+Reporter: schmoozecom<br/>
+Testers: rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-114">PRI-114</a>: [patch] add extra IE  Q931_IE_TIME_DATE to CONNECT message when in network mode<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2187">2187</a><br/>
+Reporter: wuwu<br/>
+Testers: rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-116">PRI-116</a>: [patch] B410P gets incoming call packets on ISDN but DAHDI doesn't generate the call in Asterisk<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2088">2088</a><br/>
+Reporter: lelio<br/>
+Testers: lelio<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-117">PRI-117</a>: SABME flood on backup D-channel in NFAS configuration<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2111">2111</a><br/>
+Reporter: bklang<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-120">PRI-120</a>: [patch] Swap of master/slave in pri_enslave incorrect<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2192">2192</a><br/>
+Reporter: jcollie<br/>
+Coders: jcollie<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-121">PRI-121</a>: [patch] Caller ID Name always blank<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2206">2206</a><br/>
+Reporter: cmorford<br/>
+Testers: cmorford<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-62">PRI-62</a>: [patch] Inband disconnect setting does nothing<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1794">1794</a><br/>
+Reporter: paravoid<br/>
+Testers: paravoid<br/>
+Coders: paravoid<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-81">PRI-81</a>: [patch] q921_rnr_rx Q921_TIMER_RECOVERY SDL issue<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1845">1845</a><br/>
+Reporter: alecdavis<br/>
+Coders: alecdavis<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-94">PRI-94</a>: [patch] pritest hadn't been ported to DAHDI<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1833">1833</a><br/>
+Reporter: horape<br/>
+Coders: horape<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-95">PRI-95</a>: [patch] LibPRI problem with restart of PBX processor (Testing SVN 1688)<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1962">1962</a><br/>
+Reporter: shawkris<br/>
+Testers: shawkris, rmudgett<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-98">PRI-98</a>: [patch] Regression in libpri 1.4.11 - handling ROSE<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1785">1785</a><br/>
+Reporter: patrol-cz<br/>
+Testers: rmudgett, patrol-cz<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-99">PRI-99</a>: [patch] CallerID name not transmitted on versions of LibPRI after 1.4.10.2<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1810">1810</a><br/>
+Reporter: jsmith<br/>
+Testers: rmudgett, jsmith<br/>
+Coders: rmudgett<br/>
+<br/>
+<h3>Category: NewFeature</h3><br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-122">PRI-122</a>: Option needed for IE_TIME_DATE to be optional on connect<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2266">2266</a><br/>
+Reporter: kenner<br/>
+Coders: rmudgett<br/>
+<br/>
+<a href="https://issues.asterisk.org/jira/browse/PRI-54">PRI-54</a>: [patch] Add pri_new_bri_cb() API - Create BRI D-channel with user defined I/O callbacks and data<br/>
+Revision: <a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1836">1836</a><br/>
+Reporter: nic_bellamy<br/>
+Coders: nic<br/>
+<br/>
+<hr/>
+<a name="commits"><h2 align="center">Commits Not Associated with an Issue</h2></a>
+<center><a href="#top">[Back to Top]</a></center><br/><p>This is a list of all changes that went into this release that did not directly close an issue from the issue tracker.  The commits may have been marked as being related to an issue.  If that is the case, the issue numbers are listed here, as well.</p>
+<table width="100%" border="1">
+<tr><td><b>Revision</b></td><td><b>Author</b></td><td><b>Summary</b></td><td><b>Issues Referenced</b></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1723">1723</a></td><td>rmudgett</td><td>Added ETSI Explicit Call Transfer (ECT) support.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1737">1737</a></td><td>rmudgett</td><td>ETSI Advice Of Charge (AOC) event reporting.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1746">1746</a></td><td>rmudgett</td><td>ETSI Call Waiting support.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1757">1757</a></td><td>rmudgett</td><td>ETSI Malicious Call ID support.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1767">1767</a></td><td>rmudgett</td><td>ETSI Message Waiting Indication (MWI) support.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1776">1776</a></td><td>dvossel</td><td>support for sending ETSI advice of charge</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1829">1829</a></td><td>rmudgett</td><td>Added a libpri version output line as a sanity check for rosetest.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1830">1830</a></td><td>rmudgett</td><td>Only need to output one version of the version message.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1842">1842</a></td><td>rmudgett</td><td>Renamed local struct pri *pri variables to struct pri *ctrl in q921.c.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1848">1848</a></td><td>rmudgett</td><td>Improved Q.921/Q.931 debug output.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1852">1852</a></td><td>rmudgett</td><td>Learn (AGAIN!) why state 7 and state 8 transitions were suppressed.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1853">1853</a></td><td>rmudgett</td><td>No audio on inbound J1 calls.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-35">PRI-35</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1878">1878</a></td><td>mattf</td><td>Add silly fake MDL error Z for kicking L2 to reactivate if a DISC or DM is received in proper states on PTP links.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1883">1883</a></td><td>rmudgett</td><td>Don't silently fail to post our fake MDL-ERROR(Z).</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1884">1884</a></td><td>rmudgett</td><td>Added gripe check to rosetest for invalid operation and error codes.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1894">1894</a></td><td>rmudgett</td><td>Make pri_dump_event() only print the event name.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1904">1904</a></td><td>rmudgett</td><td>Add support to receive ECMA-164 2nd edition OID name ROSE messages.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-37">PRI-37</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1912">1912</a></td><td>rmudgett</td><td>Convert most references of Q931_MAX_TEI to use ARRAY_LEN().</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1923">1923</a></td><td>rmudgett</td><td>Q.921 improvements from comparing Q.921 SDL diagrams with implementation.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1928">1928</a></td><td>rmudgett</td><td>Q.921/Q.931 message debug output improvements.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1929">1929</a></td><td>rmudgett</td><td>Reduced fake MDL-ERROR (Z) message severity to be visible only when debugging enabled.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1955">1955</a></td><td>rmudgett</td><td>Don't crash in __pri_new_tei() if a GR303 subchannel creation fails.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1958">1958</a></td><td>rmudgett</td><td>Balance curly braces in post_handle_q931_message().</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1961">1961</a></td><td>rmudgett</td><td>Prevent a CONNECT message from sending a CONNECT ACKNOWLEDGE in the wrong state.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-95">PRI-95</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1965">1965</a></td><td>rmudgett</td><td>Added more parameter checks to pri_set_timer() and pri_get_timer().</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1976">1976</a></td><td>rmudgett</td><td>Fix spelling error in PTMP agent FSM files.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=1991">1991</a></td><td>rmudgett</td><td>PRI links do not retain active calls if the link comes back before T309 expires.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2001">2001</a></td><td>russell</td><td>Makefile tweaks to allow building for code coverage analysis.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2009">2009</a></td><td>tilghman</td><td>Minor changes to make libpri build on Mac OS X</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2027">2027</a></td><td>rmudgett</td><td>Create two versions of call ptr verify.  One gripes and one does not.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2042">2042</a></td><td>rmudgett</td><td>Dump Q.931 message using the TEI value the message came in with.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2063">2063</a></td><td>rmudgett</td><td>Logically separate Q.921 TEI link processing from D channel control.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2070">2070</a></td><td>rmudgett</td><td>Restructure the Q.931 call record to layer 2 link association.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2077">2077</a></td><td>rmudgett</td><td>Extract the layer 2 link structure out of struct pri.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2078">2078</a></td><td>rmudgett</td><td>Partial support for dynamic interfaces with NFAS.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2079">2079</a></td><td>rmudgett</td><td>Fixes CC agents not automatically clearing if T309 clears the original call.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-46">PRI-46</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2098">2098</a></td><td>rmudgett</td><td>Mainly put space after switch and while keywords.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2101">2101</a></td><td>rmudgett</td><td>Remove all TEIs when NT PTMP starts.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-51">PRI-51</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2102">2102</a></td><td>rmudgett</td><td>Remove unneeded struct q921_link.mdl_error_state member.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2103">2103</a></td><td>rmudgett</td><td>Convert TEI identity defines to enum and create 2str() function.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2104">2104</a></td><td>rmudgett</td><td>Q.921 TEI assignment procedure corrections.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2105">2105</a></td><td>rmudgett</td><td>Added TEI identity check feature to reclaim dead TEIs.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-51">PRI-51</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2109">2109</a></td><td>rmudgett</td><td>Minor MDL handler changes.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2166">2166</a></td><td>rmudgett</td><td>Fix bizarre logic to work as originally intended in q931_notify().</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2167">2167</a></td><td>rmudgett</td><td>Return error if q931_notify() cannot send NOTIFY.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2169">2169</a></td><td>rmudgett</td><td>Fix regression when reorganized for struct pri and struct q921_link.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2170">2170</a></td><td>rmudgett</td><td>Better HOLD/RETRIEVE collision handling.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2172">2172</a></td><td>rmudgett</td><td>Add call transfer exchange of subaddresses support and fix PTMP call transfer signaling.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-47">PRI-47</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2174">2174</a></td><td>rmudgett</td><td>Fix Q.931 retried SETUP not sending facility ies.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2175">2175</a></td><td>rmudgett</td><td>Invalid PTMP redirecting signaling as TE towards NT.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-53">PRI-53</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2183">2183</a></td><td>rmudgett</td><td>Enable LIBPRI_COUNTERS code by default.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-61">PRI-61</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2190">2190</a></td><td>rmudgett</td><td>Add display ie text handling options.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2200">2200</a></td><td>rmudgett</td><td>Fix I-frame retransmission quirks.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-60">PRI-60</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2202">2202</a></td><td>rmudgett</td><td>Crash if NFAS swaps D channels on a call with an active timer.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-58">PRI-58</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2204">2204</a></td><td>rmudgett</td><td>* Added switchtype to ROSE invoke operation not handled message.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2207">2207</a></td><td>rmudgett</td><td>B channel lost by incoming call in BRI NT PTMP mode.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2227">2227</a></td><td>rmudgett</td><td>Add determined remote node type to pri_dump_info_str().</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2232">2232</a></td><td>rmudgett</td><td>Improve the usefulness of pri_dump_info_str() output.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2233">2233</a></td><td>rmudgett</td><td>Fix valgrind reported invalid read/write for display text feature.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2236">2236</a></td><td>rmudgett</td><td>Miscellaneous cleanup before T312 branch merge.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2238">2238</a></td><td>rmudgett</td><td>Implement the mandatory T312 timer for NT PTMP broadcast SETUP calls.</td>
+<td><a href="https://issues.asterisk.org/jira/browse/PRI-32">PRI-32</a></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2258">2258</a></td><td>rmudgett</td><td>CallRerouting response not sent if peer hangs up first.</td>
+<td></td></tr><tr><td><a href="http://svn.digium.com/view/libpri/branches/1.4?view=revision&revision=2262">2262</a></td><td>rmudgett</td><td>Problems with ISDN MWI to phones.</td>
+<td></td></tr></table>
+<hr/>
+<a name="diffstat"><h2 align="center">Diffstat Results</h2></a>
+<center><a href="#top">[Back to Top]</a></center><br/><p>This is a summary of the changes to the source code that went into this release that was generated using the diffstat utility.</p>
+<pre>
+Makefile                          |   39
+doc/cc_ptmp_agent.fsm             |  376 +
+doc/cc_ptmp_agent_flattened.fsm   |  494 ++
+doc/cc_ptmp_monitor.fsm           |  181
+doc/cc_ptmp_monitor_flattened.fsm |  225 +
+doc/cc_ptp_agent.fsm              |  152
+doc/cc_ptp_agent_flattened.fsm    |  200
+doc/cc_ptp_monitor.fsm            |  168
+doc/cc_ptp_monitor_flattened.fsm  |  217 +
+doc/cc_qsig_agent.fsm             |  140
+doc/cc_qsig_agent_flattened.fsm   |  187
+doc/cc_qsig_monitor.fsm           |  249 +
+doc/cc_qsig_monitor_flattened.fsm |  327 +
+libpri.h                          |  900 ++++
+pri.c                             | 1151 ++++-
+pri_aoc.c                         | 1764 ++++++++
+pri_cc.c                          | 7884 ++++++++++++++++++++++++++++++++++++++
+pri_facility.c                    | 1823 +++++++-
+pri_facility.h                    |   87
+pri_internal.h                    |  593 ++
+pri_q921.h                        |  115
+pri_q931.h                        |   27
+prisched.c                        |  228 -
+pritest.c                         |   83
+q921.c                            | 2942 +++++++++-----
+q931.c                            | 3388 ++++++++++++----
+rose.c                            |  536 ++
+rose.h                            |  856 ++++
+rose_etsi_aoc.c                   |    9
+rose_etsi_cc.c                    | 1853 ++++++++
+rose_etsi_mwi.c                   |  686 +++
+rose_internal.h                   |  205
+rose_qsig_cc.c                    |  984 ++++
+rose_qsig_mwi.c                   |    9
+rose_qsig_name.c                  |    1
+rosetest.c                        |  919 ++++
+36 files changed, 27493 insertions(+), 2505 deletions(-)
+</pre><br/>
+<hr/>
+</body>
+</html>
--- a/libpri-1.4.12-summary.txt
+++ b/libpri-1.4.12-summary.txt
@@ -0,0 +1,483 @@
+                                Release Summary
+
+                                 libpri-1.4.12
+
+                                Date: 2011-07-05
+
+                           <asteriskteam@digium.com>
+
+     ----------------------------------------------------------------------
+
+                               Table of Contents
+
+    1. Summary
+    2. Contributors
+    3. Closed Issues
+    4. Other Changes
+    5. Diffstat
+
+     ----------------------------------------------------------------------
+
+                                    Summary
+
+                                 [Back to Top]
+
+   This release includes only bug fixes. The changes included were made only
+   to address problems that have been identified in this release series.
+   Users should be able to safely upgrade to this version if this release
+   series is already in use. Users considering upgrading from a previous
+   release series are strongly encouraged to review the UPGRADE.txt document
+   as well as the CHANGES document for information about upgrading to this
+   release series.
+
+   The data in this summary reflects changes that have been made since the
+   previous release, libpri-1.4.11.
+
+     ----------------------------------------------------------------------
+
+                                  Contributors
+
+                                 [Back to Top]
+
+   This table lists the people who have submitted code, those that have
+   tested patches, as well as those that reported issues on the issue tracker
+   that were resolved in this release. For coders, the number is how many of
+   their patches (of any size) were committed into this release. For testers,
+   the number is the number of times their name was listed as assisting with
+   testing a patch. Finally, for reporters, the number is the number of
+   issues that they reported that were closed by commits that went into this
+   release.
+
+     Coders                   Testers                  Reporters              
+   77 rmudgett              10 rmudgett              2 wuwu                   
+   1 alecdavis              1 cmorford               1 alecdavis              
+   1 dvossel                1 ibercom                1 bklang                 
+   1 horape                 1 jcovert                1 cmorford               
+   1 jcollie                1 jsmith                 1 gelo                   
+   1 mattf                  1 lelio                  1 horape                 
+   1 nic                    1 paravoid               1 ibercom                
+   1 paravoid               1 patrol-cz              1 jcollie                
+   1 russell                1 shawkris               1 jcovert                
+   1 tilghman               1 wimpy                  1 jims8650               
+                                                     1 jmls                   
+                                                     1 jsmith                 
+                                                     1 kenner                 
+                                                     1 kobaz                  
+                                                     1 lelio                  
+                                                     1 nic_bellamy            
+                                                     1 paravoid               
+                                                     1 patrol-cz              
+                                                     1 schmoozecom            
+                                                     1 shawkris               
+                                                     1 tomaso                 
+                                                     1 wimpy                  
+
+     ----------------------------------------------------------------------
+
+                                 Closed Issues
+
+                                 [Back to Top]
+
+   This is a list of all issues from the issue tracker that were closed by
+   changes that went into this release.
+
+  Category: Channels/chan_dahdi
+
+   ASTERISK-16038: getting warning message every 4 seconds
+   Revision: 2113
+   Reporter: jmls
+   Coders: rmudgett
+
+  Category: Channels/chan_dahdi/NewFeature
+
+   ASTERISK-13410: [patch] CCBS/CCNR support for QSIG (libpri & chan_dahdi)
+   Revision: 1714
+   Reporter: tomaso
+   Testers: rmudgett
+   Coders: rmudgett
+
+  Category: General
+
+   PRI-102: [patch] [regression] Calling Number assignment logic change in
+   libpri 1.4.11.
+   Revision: 1823
+   Reporter: ibercom
+   Testers: ibercom
+   Coders: rmudgett
+
+   PRI-103: segfault in pri_schedule_del - ctrl invalid value
+   Revision: 2015
+   Reporter: wuwu
+   Testers: rmudgett
+   Coders: rmudgett
+
+   PRI-105: [patch] ISDN BRI does not recover from line faults
+   Revision: 1918
+   Reporter: jcovert
+   Testers: jcovert, rmudgett
+   Coders: rmudgett
+
+   PRI-107: [patch] Calling name not successfully processed on inbound QSIG
+   PRI calls from Mitel PBX
+   Revision: 1828
+   Reporter: jims8650
+   Testers: rmudgett
+   Coders: rmudgett
+
+   PRI-110: [patch] BRI NT ptmp: Active channels not cleard when i/f goes
+   down
+   Revision: 1982
+   Reporter: wimpy
+   Testers: rmudgett, wimpy
+   Coders: rmudgett
+
+   PRI-112: [patch] Libpri crashes when receiving Message Type (100)
+   Revision: 2021
+   Reporter: gelo
+   Coders: rmudgett
+
+   PRI-113: [patch] Asterisk is core dumping with LibPRI
+   Revision: 2015
+   Reporter: schmoozecom
+   Testers: rmudgett
+   Coders: rmudgett
+
+   PRI-114: [patch] add extra IE Q931_IE_TIME_DATE to CONNECT message when in
+   network mode
+   Revision: 2187
+   Reporter: wuwu
+   Testers: rmudgett
+   Coders: rmudgett
+
+   PRI-116: [patch] B410P gets incoming call packets on ISDN but DAHDI
+   doesn't generate the call in Asterisk
+   Revision: 2088
+   Reporter: lelio
+   Testers: lelio
+   Coders: rmudgett
+
+   PRI-117: SABME flood on backup D-channel in NFAS configuration
+   Revision: 2111
+   Reporter: bklang
+   Coders: rmudgett
+
+   PRI-120: [patch] Swap of master/slave in pri_enslave incorrect
+   Revision: 2192
+   Reporter: jcollie
+   Coders: jcollie
+
+   PRI-121: [patch] Caller ID Name always blank
+   Revision: 2206
+   Reporter: cmorford
+   Testers: cmorford
+   Coders: rmudgett
+
+   PRI-62: [patch] Inband disconnect setting does nothing
+   Revision: 1794
+   Reporter: paravoid
+   Testers: paravoid
+   Coders: paravoid
+
+   PRI-81: [patch] q921_rnr_rx Q921_TIMER_RECOVERY SDL issue
+   Revision: 1845
+   Reporter: alecdavis
+   Coders: alecdavis
+
+   PRI-94: [patch] pritest hadn't been ported to DAHDI
+   Revision: 1833
+   Reporter: horape
+   Coders: horape
+
+   PRI-95: [patch] LibPRI problem with restart of PBX processor (Testing SVN
+   1688)
+   Revision: 1962
+   Reporter: shawkris
+   Testers: shawkris, rmudgett
+   Coders: rmudgett
+
+   PRI-98: [patch] Regression in libpri 1.4.11 - handling ROSE
+   Revision: 1785
+   Reporter: patrol-cz
+   Testers: rmudgett, patrol-cz
+   Coders: rmudgett
+
+   PRI-99: [patch] CallerID name not transmitted on versions of LibPRI after
+   1.4.10.2
+   Revision: 1810
+   Reporter: jsmith
+   Testers: rmudgett, jsmith
+   Coders: rmudgett
+
+  Category: NewFeature
+
+   PRI-122: Option needed for IE_TIME_DATE to be optional on connect
+   Revision: 2266
+   Reporter: kenner
+   Coders: rmudgett
+
+   PRI-54: [patch] Add pri_new_bri_cb() API - Create BRI D-channel with user
+   defined I/O callbacks and data
+   Revision: 1836
+   Reporter: nic_bellamy
+   Coders: nic
+
+     ----------------------------------------------------------------------
+
+                      Commits Not Associated with an Issue
+
+                                 [Back to Top]
+
+   This is a list of all changes that went into this release that did not
+   directly close an issue from the issue tracker. The commits may have been
+   marked as being related to an issue. If that is the case, the issue
+   numbers are listed here, as well.
+
+   +------------------------------------------------------------------------+
+   | Revision | Author   | Summary                             | Issues     |
+   |          |          |                                     | Referenced |
+   |----------+----------+-------------------------------------+------------|
+   | 1723     | rmudgett | Added ETSI Explicit Call Transfer   |            |
+   |          |          | (ECT) support.                      |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1737     | rmudgett | ETSI Advice Of Charge (AOC) event   |            |
+   |          |          | reporting.                          |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1746     | rmudgett | ETSI Call Waiting support.          |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1757     | rmudgett | ETSI Malicious Call ID support.     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1767     | rmudgett | ETSI Message Waiting Indication     |            |
+   |          |          | (MWI) support.                      |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1776     | dvossel  | support for sending ETSI advice of  |            |
+   |          |          | charge                              |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1829     | rmudgett | Added a libpri version output line  |            |
+   |          |          | as a sanity check for rosetest.     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1830     | rmudgett | Only need to output one version of  |            |
+   |          |          | the version message.                |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Renamed local struct pri *pri       |            |
+   | 1842     | rmudgett | variables to struct pri *ctrl in    |            |
+   |          |          | q921.c.                             |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1848     | rmudgett | Improved Q.921/Q.931 debug output.  |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Learn (AGAIN!) why state 7 and      |            |
+   | 1852     | rmudgett | state 8 transitions were            |            |
+   |          |          | suppressed.                         |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1853     | rmudgett | No audio on inbound J1 calls.       | PRI-35     |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Add silly fake MDL error Z for      |            |
+   | 1878     | mattf    | kicking L2 to reactivate if a DISC  |            |
+   |          |          | or DM is received in proper states  |            |
+   |          |          | on PTP links.                       |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1883     | rmudgett | Don't silently fail to post our     |            |
+   |          |          | fake MDL-ERROR(Z).                  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1884     | rmudgett | Added gripe check to rosetest for   |            |
+   |          |          | invalid operation and error codes.  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1894     | rmudgett | Make pri_dump_event() only print    |            |
+   |          |          | the event name.                     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1904     | rmudgett | Add support to receive ECMA-164 2nd | PRI-37     |
+   |          |          | edition OID name ROSE messages.     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1912     | rmudgett | Convert most references of          |            |
+   |          |          | Q931_MAX_TEI to use ARRAY_LEN().    |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Q.921 improvements from comparing   |            |
+   | 1923     | rmudgett | Q.921 SDL diagrams with             |            |
+   |          |          | implementation.                     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1928     | rmudgett | Q.921/Q.931 message debug output    |            |
+   |          |          | improvements.                       |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Reduced fake MDL-ERROR (Z) message  |            |
+   | 1929     | rmudgett | severity to be visible only when    |            |
+   |          |          | debugging enabled.                  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1955     | rmudgett | Don't crash in __pri_new_tei() if a |            |
+   |          |          | GR303 subchannel creation fails.    |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1958     | rmudgett | Balance curly braces in             |            |
+   |          |          | post_handle_q931_message().         |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Prevent a CONNECT message from      |            |
+   | 1961     | rmudgett | sending a CONNECT ACKNOWLEDGE in    | PRI-95     |
+   |          |          | the wrong state.                    |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Added more parameter checks to      |            |
+   | 1965     | rmudgett | pri_set_timer() and                 |            |
+   |          |          | pri_get_timer().                    |            |
+   |----------+----------+-------------------------------------+------------|
+   | 1976     | rmudgett | Fix spelling error in PTMP agent    |            |
+   |          |          | FSM files.                          |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | PRI links do not retain active      |            |
+   | 1991     | rmudgett | calls if the link comes back before |            |
+   |          |          | T309 expires.                       |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2001     | russell  | Makefile tweaks to allow building   |            |
+   |          |          | for code coverage analysis.         |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2009     | tilghman | Minor changes to make libpri build  |            |
+   |          |          | on Mac OS X                         |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Create two versions of call ptr     |            |
+   | 2027     | rmudgett | verify. One gripes and one does     |            |
+   |          |          | not.                                |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2042     | rmudgett | Dump Q.931 message using the TEI    |            |
+   |          |          | value the message came in with.     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2063     | rmudgett | Logically separate Q.921 TEI link   |            |
+   |          |          | processing from D channel control.  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2070     | rmudgett | Restructure the Q.931 call record   |            |
+   |          |          | to layer 2 link association.        |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2077     | rmudgett | Extract the layer 2 link structure  |            |
+   |          |          | out of struct pri.                  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2078     | rmudgett | Partial support for dynamic         |            |
+   |          |          | interfaces with NFAS.               |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Fixes CC agents not automatically   |            |
+   | 2079     | rmudgett | clearing if T309 clears the         | PRI-46     |
+   |          |          | original call.                      |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2098     | rmudgett | Mainly put space after switch and   |            |
+   |          |          | while keywords.                     |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2101     | rmudgett | Remove all TEIs when NT PTMP        | PRI-51     |
+   |          |          | starts.                             |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2102     | rmudgett | Remove unneeded struct              |            |
+   |          |          | q921_link.mdl_error_state member.   |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2103     | rmudgett | Convert TEI identity defines to     |            |
+   |          |          | enum and create 2str() function.    |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2104     | rmudgett | Q.921 TEI assignment procedure      |            |
+   |          |          | corrections.                        |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2105     | rmudgett | Added TEI identity check feature to | PRI-51     |
+   |          |          | reclaim dead TEIs.                  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2109     | rmudgett | Minor MDL handler changes.          |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Fix bizarre logic to work as        |            |
+   | 2166     | rmudgett | originally intended in              |            |
+   |          |          | q931_notify().                      |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2167     | rmudgett | Return error if q931_notify()       |            |
+   |          |          | cannot send NOTIFY.                 |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2169     | rmudgett | Fix regression when reorganized for |            |
+   |          |          | struct pri and struct q921_link.    |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2170     | rmudgett | Better HOLD/RETRIEVE collision      |            |
+   |          |          | handling.                           |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Add call transfer exchange of       |            |
+   | 2172     | rmudgett | subaddresses support and fix PTMP   | PRI-47     |
+   |          |          | call transfer signaling.            |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2174     | rmudgett | Fix Q.931 retried SETUP not sending |            |
+   |          |          | facility ies.                       |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2175     | rmudgett | Invalid PTMP redirecting signaling  | PRI-53     |
+   |          |          | as TE towards NT.                   |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2183     | rmudgett | Enable LIBPRI_COUNTERS code by      | PRI-61     |
+   |          |          | default.                            |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2190     | rmudgett | Add display ie text handling        |            |
+   |          |          | options.                            |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2200     | rmudgett | Fix I-frame retransmission quirks.  | PRI-60     |
+   |----------+----------+-------------------------------------+------------|
+   | 2202     | rmudgett | Crash if NFAS swaps D channels on a | PRI-58     |
+   |          |          | call with an active timer.          |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2204     | rmudgett | * Added switchtype to ROSE invoke   |            |
+   |          |          | operation not handled message.      |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2207     | rmudgett | B channel lost by incoming call in  |            |
+   |          |          | BRI NT PTMP mode.                   |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2227     | rmudgett | Add determined remote node type to  |            |
+   |          |          | pri_dump_info_str().                |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2232     | rmudgett | Improve the usefulness of           |            |
+   |          |          | pri_dump_info_str() output.         |            |
+   |----------+----------+-------------------------------------+------------|
+   |          |          | Fix valgrind reported invalid       |            |
+   | 2233     | rmudgett | read/write for display text         |            |
+   |          |          | feature.                            |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2236     | rmudgett | Miscellaneous cleanup before T312   |            |
+   |          |          | branch merge.                       |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2238     | rmudgett | Implement the mandatory T312 timer  | PRI-32     |
+   |          |          | for NT PTMP broadcast SETUP calls.  |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2258     | rmudgett | CallRerouting response not sent if  |            |
+   |          |          | peer hangs up first.                |            |
+   |----------+----------+-------------------------------------+------------|
+   | 2262     | rmudgett | Problems with ISDN MWI to phones.   |            |
+   +------------------------------------------------------------------------+
+
+     ----------------------------------------------------------------------
+
+                                Diffstat Results
+
+                                 [Back to Top]
+
+   This is a summary of the changes to the source code that went into this
+   release that was generated using the diffstat utility.
+
+ Makefile                          |   39
+ doc/cc_ptmp_agent.fsm             |  376 +
+ doc/cc_ptmp_agent_flattened.fsm   |  494 ++
+ doc/cc_ptmp_monitor.fsm           |  181
+ doc/cc_ptmp_monitor_flattened.fsm |  225 +
+ doc/cc_ptp_agent.fsm              |  152
+ doc/cc_ptp_agent_flattened.fsm    |  200
+ doc/cc_ptp_monitor.fsm            |  168
+ doc/cc_ptp_monitor_flattened.fsm  |  217 +
+ doc/cc_qsig_agent.fsm             |  140
+ doc/cc_qsig_agent_flattened.fsm   |  187
+ doc/cc_qsig_monitor.fsm           |  249 +
+ doc/cc_qsig_monitor_flattened.fsm |  327 +
+ libpri.h                          |  900 ++++
+ pri.c                             | 1151 ++++-
+ pri_aoc.c                         | 1764 ++++++++
+ pri_cc.c                          | 7884 ++++++++++++++++++++++++++++++++++++++
+ pri_facility.c                    | 1823 +++++++-
+ pri_facility.h                    |   87
+ pri_internal.h                    |  593 ++
+ pri_q921.h                        |  115
+ pri_q931.h                        |   27
+ prisched.c                        |  228 -
+ pritest.c                         |   83
+ q921.c                            | 2942 +++++++++-----
+ q931.c                            | 3388 ++++++++++++----
+ rose.c                            |  536 ++
+ rose.h                            |  856 ++++
+ rose_etsi_aoc.c                   |    9
+ rose_etsi_cc.c                    | 1853 ++++++++
+ rose_etsi_mwi.c                   |  686 +++
+ rose_internal.h                   |  205
+ rose_qsig_cc.c                    |  984 ++++
+ rose_qsig_mwi.c                   |    9
+ rose_qsig_name.c                  |    1
+ rosetest.c                        |  919 ++++
+ 36 files changed, 27493 insertions(+), 2505 deletions(-)
+
+     ----------------------------------------------------------------------
--- a/libpri.h
+++ b/libpri.h
@@ -1213,7 +1213,6 @@ typedef struct pri_event_setup_ack {
 	int channel;
 	q931_call *call;
 	struct pri_subcommands *subcmds;
-	int progressmask;
 } pri_event_setup_ack;

 typedef struct pri_event_notify {
@@ -1409,17 +1408,8 @@ const char *pri_facility_error2str(int facility_error_code);
 */
 const char *pri_facility_reject2str(int facility_reject_code);

-/*!
- * \brief Send the ALERTING message.
- *
- * \param pri D channel controller.
- * \param call Q.931 call leg.
- * \param channel Encoded channel id to use.  If zero do not change channel id.
- * \param info Nonzero to include a progress ie indicating inband audio available (ie ringback).
- *
- * \retval 0 on success.
- * \retval -1 on error.
- */
+/* Acknowledge a call and place it on the given channel.  Set info to non-zero if there
+   is in-band data available on the channel */
 int pri_acknowledge(struct pri *pri, q931_call *call, int channel, int info);

 /* Send a digit in overlap mode */
@@ -1429,44 +1419,12 @@ int pri_information(struct pri *pri, q931_call *call, char digit);
 /* Send a keypad facility string of digits */
 int pri_keypad_facility(struct pri *pri, q931_call *call, const char *digits);

-/*!
- * \brief Send the SETUP_ACKNOWLEDGE message.
- *
- * \param pri D channel controller.
- * \param call Q.931 call leg.
- * \param channel Encoded channel id to use.  If zero do not change channel id.
- * \param nonisdn Nonzero to include a progress ie indicating non-end-to-end-ISDN.
- *
- * \retval 0 on success.
- * \retval -1 on error.
- */
+/* Answer the incomplete(call without called number) call on the given channel.
+   Set non-isdn to non-zero if you are not connecting to ISDN equipment */
 int pri_need_more_info(struct pri *pri, q931_call *call, int channel, int nonisdn);

-/*!
- * \brief Send the SETUP_ACKNOWLEDGE message.
- *
- * \param ctrl D channel controller.
- * \param call Q.931 call leg.
- * \param channel Encoded channel id to use.  If zero do not change channel id.
- * \param nonisdn Nonzero to include a progress ie indicating non-end-to-end-ISDN.
- * \param inband Nonzero to include a progress ie indicating inband audio available (ie dialtone).
- *
- * \retval 0 on success.
- * \retval -1 on error.
- */
-int pri_setup_ack(struct pri *ctrl, q931_call *call, int channel, int nonisdn, int inband);
-
-/*!
- * \brief Send the CONNECT message.
- *
- * \param pri D channel controller.
- * \param call Q.931 call leg.
- * \param channel Encoded channel id to use.  If zero do not change channel id.
- * \param nonisdn Nonzero to include a progress ie indicating non-end-to-end-ISDN.
- *
- * \retval 0 on success.
- * \retval -1 on error.
- */
+/* Answer(CONNECT) the call on the given channel.
+   Set non-isdn to non-zero if you are not connecting to ISDN equipment */
 int pri_answer(struct pri *pri, q931_call *call, int channel, int nonisdn);

 /*!
@@ -1734,17 +1692,7 @@ int pri_progress(struct pri *pri, q931_call *c, int channel, int info);
 int pri_progress_with_cause(struct pri *pri, q931_call *c, int channel, int info, int cause);

 #define PRI_PROCEEDING_FULL
-/*!
- * \brief Send the PROCEEDING message.
- *
- * \param pri D channel controller.
- * \param c Q.931 call leg.
- * \param channel Encoded channel id to use.  If zero do not change channel id.
- * \param info Nonzero to include a progress ie indicating inband audio available.
- *
- * \retval 0 on success.
- * \retval -1 on error.
- */
+/* Send call proceeding */
 int pri_proceeding(struct pri *pri, q931_call *c, int channel, int info);

 /* Enable inband progress when a DISCONNECT is received */
@@ -1944,31 +1892,6 @@ int pri_transfer_rsp(struct pri *ctrl, q931_call *call, int invoke_id, int is_su
 */
 void pri_aoc_events_enable(struct pri *ctrl, int enable);

-enum pri_layer2_persistence {
-	PRI_L2_PERSISTENCE_DEFAULT,
-	/*! Immediately bring layer 2 back up if the peer brings layer 2 down. */
-	PRI_L2_PERSISTENCE_KEEP_UP,
-	/*! Leave layer 2 down if the peer brings layer 2 down. */
-	PRI_L2_PERSISTENCE_LEAVE_DOWN,
-#if 0	/* Possible future option.  Would need to define how long to idle before dropping. */
-	/*! Drop layer 2 on D channel idle. */
-	PRI_L2_PERSISTENCE_IDLE_DROP,
-#endif
-};
-
-/*!
- * \brief Set the layer2 persistence option.
- *
- * \param ctrl D channel controller.
- * \param option Layer 2 persistence to apply.
- *
- * \note
- * Not all values are supported by all modes.
- *
- * \return Nothing
- */
-void pri_persistent_layer2_option(struct pri *ctrl, enum pri_layer2_persistence option);
-
 #define PRI_DISPLAY_OPTION_BLOCK		(1 << 0)	/*!< Do not pass display text. */
 #define PRI_DISPLAY_OPTION_NAME_INITIAL	(1 << 1)	/*!< Use display in SETUP/CONNECT for name. */
 #define PRI_DISPLAY_OPTION_NAME_UPDATE	(1 << 2)	/*!< Use display in FACILITY/NOTIFY for COLP name if appropriate. */
@@ -2231,7 +2154,7 @@ enum PRI_TIMERS_AND_COUNTERS {
 	PRI_TIMER_T310,	/*!< Maximum time between receiving a CALL_PROCEEDING and receiving a ALERT/CONNECT/DISCONNECT/PROGRESS */
 	PRI_TIMER_T313,	/*!< Wait for CONNECT acknowledge, CPE side only */
 	PRI_TIMER_T314,
-	PRI_TIMER_T316,	/*!< Time to wait for a RESTART ACK before retransmitting RESTART. (Timer enabled if greater than zero.) */
+	PRI_TIMER_T316,	/*!< Maximum time between transmitting a RESTART and receiving a RESTART ACK */
 	PRI_TIMER_T317,
 	PRI_TIMER_T318,
 	PRI_TIMER_T319,
@@ -2273,7 +2196,6 @@ enum PRI_TIMERS_AND_COUNTERS {
 	PRI_TIMER_QSIG_CC_T4,	/*!< Path reservation supervision timeout. */

 	PRI_TIMER_T312,			/*!< Supervise broadcast SETUP message call reference retention. */
-	PRI_TIMER_N316,			/*!< Number of times to transmit RESTART before giving up if T316 enabled. */

 	/* Must be last in the enum list */
 	PRI_MAX_TIMERS
--- a/pri.c
+++ b/pri.c
@@ -81,7 +81,6 @@ static const struct pri_timer_table pri_timer[] = {
 	{ "T313",           PRI_TIMER_T313,             PRI_ALL_SWITCHES },
 	{ "T314",           PRI_TIMER_T314,             PRI_ALL_SWITCHES },
 	{ "T316",           PRI_TIMER_T316,             PRI_ALL_SWITCHES },
-	{ "N316",           PRI_TIMER_N316,             PRI_ALL_SWITCHES },
 	{ "T317",           PRI_TIMER_T317,             PRI_ALL_SWITCHES },
 	{ "T318",           PRI_TIMER_T318,             PRI_ALL_SWITCHES },
 	{ "T319",           PRI_TIMER_T319,             PRI_ALL_SWITCHES },
@@ -176,7 +175,7 @@ static void pri_default_timers(struct pri *ctrl, int switchtype)

 	ctrl->timers[PRI_TIMER_T200] = 1000;		/* Time between SABME's */
 	ctrl->timers[PRI_TIMER_T201] = ctrl->timers[PRI_TIMER_T200];/* Time between TEI Identity Checks (Default same as T200) */
-	ctrl->timers[PRI_TIMER_T202] = 2 * 1000;	/* Min time between transmission of TEI Identity request messages */
+	ctrl->timers[PRI_TIMER_T202] = 10 * 1000;	/* Min time between transmission of TEI Identity request messages */
 	ctrl->timers[PRI_TIMER_T203] = 10 * 1000;	/* Max time without exchanging packets */

 	ctrl->timers[PRI_TIMER_T303] = 4 * 1000;	/* Length between SETUP retransmissions and timeout */
@@ -185,10 +184,6 @@ static void pri_default_timers(struct pri *ctrl, int switchtype)
 	ctrl->timers[PRI_TIMER_T309] = 6 * 1000;	/* Time to wait before clearing calls in case of D-channel transient event.  Q.931 specifies 6-90 seconds */
 	ctrl->timers[PRI_TIMER_T312] = (4 + 2) * 1000;/* Supervise broadcast SETUP message call reference retention. T303 + 2 seconds */
 	ctrl->timers[PRI_TIMER_T313] = 4 * 1000;	/* Wait for CONNECT acknowledge, CPE side only */
-#if 0	/* Default disable the T316 timer otherwise the user cannot disable it. */
-	ctrl->timers[PRI_TIMER_T316] = 2 * 60 * 1000;	/* RESTART retransmit timer */
-#endif
-	ctrl->timers[PRI_TIMER_N316] = 2;			/* Send RESTART this many times before giving up. */

 	ctrl->timers[PRI_TIMER_TM20] = 2500;		/* Max time awaiting XID response - Q.921 Appendix IV */
 	ctrl->timers[PRI_TIMER_NM20] = 3;			/* Number of XID retransmits - Q.921 Appendix IV */
@@ -295,27 +290,6 @@ static int __pri_write(struct pri *pri, void *buf, int buflen)
 	return res;
 }

-/*!
- * \internal
- * \brief Determine the default layer 2 persistence option.
- *
- * \param ctrl D channel controller.
- *
- * \return Default layer 2 persistence option. (legacy behaviour default)
- */
-static enum pri_layer2_persistence pri_l2_persistence_option_default(struct pri *ctrl)
-{
-	enum pri_layer2_persistence persistence;
-
-	if (PTMP_MODE(ctrl)) {
-		persistence = PRI_L2_PERSISTENCE_LEAVE_DOWN;
-	} else {
-		persistence = PRI_L2_PERSISTENCE_KEEP_UP;
-	}
-
-	return persistence;
-}
-
 /*!
 * \internal
 * \brief Determine the default display text send options.
@@ -584,7 +558,6 @@ static struct pri *pri_ctrl_new(int fd, int node, int switchtype, pri_io_cb rd,
 	ctrl->q931_rxcount = 0;
 	ctrl->q931_txcount = 0;

-	ctrl->l2_persistence = pri_l2_persistence_option_default(ctrl);
 	ctrl->display_flags.send = pri_display_options_send_default(ctrl);
 	ctrl->display_flags.receive = pri_display_options_receive_default(ctrl);
 	switch (switchtype) {
@@ -939,15 +912,7 @@ int pri_need_more_info(struct pri *pri, q931_call *call, int channel, int nonisd
 	if (!pri || !pri_is_call_valid(pri, call)) {
 		return -1;
 	}
-	return q931_setup_ack(pri, call, channel, nonisdn, 0);
-}
-
-int pri_setup_ack(struct pri *ctrl, q931_call *call, int channel, int nonisdn, int inband)
-{
-	if (!ctrl || !pri_is_call_valid(ctrl, call)) {
-		return -1;
-	}
-	return q931_setup_ack(ctrl, call, channel, nonisdn, inband);
+	return q931_setup_ack(pri, call, channel, nonisdn);
 }

 int pri_answer(struct pri *pri, q931_call *call, int channel, int nonisdn)
@@ -986,7 +951,7 @@ void pri_copy_party_name_to_q931(struct q931_party_name *q931_name, const struct
 	q931_party_name_init(q931_name);
 	if (pri_name->valid) {
 		q931_name->valid = 1;
-		q931_name->presentation = pri_name->presentation & PRI_PRES_RESTRICTION;
+		q931_name->presentation = pri_name->presentation;
 		q931_name->char_set = pri_name->char_set;
 		libpri_copy_string(q931_name->str, pri_name->str, sizeof(q931_name->str));
 	}
@@ -1005,8 +970,7 @@ void pri_copy_party_number_to_q931(struct q931_party_number *q931_number, const
 	q931_party_number_init(q931_number);
 	if (pri_number->valid) {
 		q931_number->valid = 1;
-		q931_number->presentation = pri_number->presentation
-			& (PRI_PRES_RESTRICTION | PRI_PRES_NUMBER_TYPE);
+		q931_number->presentation = pri_number->presentation;
 		q931_number->plan = pri_number->plan;
 		libpri_copy_string(q931_number->str, pri_number->str, sizeof(q931_number->str));
 	}
@@ -1808,8 +1772,7 @@ char *pri_dump_info_str(struct pri *ctrl)
 			enum PRI_TIMERS_AND_COUNTERS tmr;

 			tmr = pri_timer[idx].number;
-			if (0 <= ctrl->timers[tmr]
-				|| tmr == PRI_TIMER_T316) {
+			if (0 <= ctrl->timers[tmr]) {
 				used = pri_snprintf(buf, used, buf_size, "  %s: %d\n",
 					pri_timer[idx].name, ctrl->timers[tmr]);
 			}
@@ -1977,14 +1940,13 @@ int pri_sr_set_caller(struct pri_sr *sr, char *caller, char *callername, int cal
 	q931_party_id_init(&sr->caller);
 	if (caller) {
 		sr->caller.number.valid = 1;
-		sr->caller.number.presentation = callerpres
-			& (PRI_PRES_RESTRICTION | PRI_PRES_NUMBER_TYPE);
+		sr->caller.number.presentation = callerpres;
 		sr->caller.number.plan = callerplan;
 		libpri_copy_string(sr->caller.number.str, caller, sizeof(sr->caller.number.str));

 		if (callername) {
 			sr->caller.name.valid = 1;
-			sr->caller.name.presentation = callerpres & PRI_PRES_RESTRICTION;
+			sr->caller.name.presentation = callerpres;
 			sr->caller.name.char_set = PRI_CHAR_SET_ISO8859_1;
 			libpri_copy_string(sr->caller.name.str, callername,
 				sizeof(sr->caller.name.str));
@@ -2008,8 +1970,7 @@ int pri_sr_set_redirecting(struct pri_sr *sr, char *num, int plan, int pres, int
 	q931_party_redirecting_init(&sr->redirecting);
 	if (num && num[0]) {
 		sr->redirecting.from.number.valid = 1;
-		sr->redirecting.from.number.presentation = pres
-			& (PRI_PRES_RESTRICTION | PRI_PRES_NUMBER_TYPE);
+		sr->redirecting.from.number.presentation = pres;
 		sr->redirecting.from.number.plan = plan;
 		libpri_copy_string(sr->redirecting.from.number.str, num,
 			sizeof(sr->redirecting.from.number.str));
@@ -2204,27 +2165,6 @@ void pri_cc_retain_signaling_rsp(struct pri *ctrl, int signaling_retention)
 	}
 }

-void pri_persistent_layer2_option(struct pri *ctrl, enum pri_layer2_persistence option)
-{
-	if (!ctrl) {
-		return;
-	}
-	if (PTMP_MODE(ctrl)) {
-		switch (option) {
-		case PRI_L2_PERSISTENCE_DEFAULT:
-			ctrl->l2_persistence = pri_l2_persistence_option_default(ctrl);
-			break;
-		case PRI_L2_PERSISTENCE_KEEP_UP:
-		case PRI_L2_PERSISTENCE_LEAVE_DOWN:
-			ctrl->l2_persistence = option;
-			break;
-		}
-		if (ctrl->l2_persistence == PRI_L2_PERSISTENCE_KEEP_UP) {
-			q921_bring_layer2_up(ctrl);
-		}
-	}
-}
-
 void pri_display_options_send(struct pri *ctrl, unsigned long flags)
 {
 	if (!ctrl) {
--- a/pri_cc.c
+++ b/pri_cc.c
@@ -2219,7 +2219,8 @@ void pri_cc_ptp_request(struct pri *ctrl, q931_call *call, int msgtype, const st
 				party_a.number.presentation =
 					PRI_PRES_ALLOWED | PRI_PRES_USER_NUMBER_UNSCREENED;
 			} else {
-				party_a.number.presentation = PRES_NUMBER_NOT_AVAILABLE;
+				party_a.number.presentation =
+					PRI_PRES_UNAVAILABLE | PRI_PRES_USER_NUMBER_UNSCREENED;
 			}
 		} else {
 			party_a.number.presentation =
--- a/pri_facility.c
+++ b/pri_facility.c
@@ -632,9 +632,6 @@ void rose_copy_presented_number_screened_to_q931(struct pri *ctrl,
 		rose_copy_number_to_q931(ctrl, q931_number,
 			&rose_presented->screened.number);
 		break;
-	case 2:	/* numberNotAvailableDueToInterworking */
-		q931_number->presentation = PRES_NUMBER_NOT_AVAILABLE;
-		break;
 	default:
 		q931_number->presentation |= PRI_PRES_USER_NUMBER_UNSCREENED;
 		break;
@@ -663,9 +660,6 @@ void rose_copy_presented_number_unscreened_to_q931(struct pri *ctrl,
 	case 3:	/* presentationRestrictedNumber */
 		rose_copy_number_to_q931(ctrl, q931_number, &rose_presented->number);
 		break;
-	case 2:	/* numberNotAvailableDueToInterworking */
-		q931_number->presentation = PRES_NUMBER_NOT_AVAILABLE;
-		break;
 	default:
 		break;
 	}
@@ -699,9 +693,6 @@ void rose_copy_presented_address_screened_to_id_q931(struct pri *ctrl,
 		rose_copy_subaddress_to_q931(ctrl, &q931_address->subaddress,
 			&rose_presented->screened.subaddress);
 		break;
-	case 2:	/* numberNotAvailableDueToInterworking */
-		q931_address->number.presentation = PRES_NUMBER_NOT_AVAILABLE;
-		break;
 	default:
 		q931_address->number.presentation |= PRI_PRES_USER_NUMBER_UNSCREENED;
 		break;
@@ -1880,7 +1871,6 @@ int pri_mwi_indicate_v2(struct pri *ctrl, const struct pri_party_id *mailbox,
 	}

 	pri_copy_party_id_to_q931(&called, mailbox);
-	q931_party_id_fixup(ctrl, &called);
 	if (rose_mwi_indicate_encode(ctrl, call, vm_id, basic_service, num_messages,
 		caller_id, timestamp, message_reference, message_status)
 		|| q931_facility_called(ctrl, call, &called)) {
@@ -4574,7 +4564,8 @@ void rose_handle_invoke(struct pri *ctrl, q931_call *call, int msgtype, q931_ie
 				party_id.number.presentation =
 					PRI_PRES_ALLOWED | PRI_PRES_USER_NUMBER_UNSCREENED;
 			} else {
-				party_id.number.presentation = PRES_NUMBER_NOT_AVAILABLE;
+				party_id.number.presentation =
+					PRI_PRES_UNAVAILABLE | PRI_PRES_USER_NUMBER_UNSCREENED;
 			}
 		} else {
 			q931_party_number_init(&party_id.number);
--- a/pri_internal.h
+++ b/pri_internal.h
@@ -120,8 +120,6 @@ struct pri {

 	/*! Layer 2 link control for D channel. */
 	struct q921_link link;
-	/*! Layer 2 persistence option. */
-	enum pri_layer2_persistence l2_persistence;
 	/*! T201 TEI Identity Check timer. */
 	int t201_timer;
 	/*! Number of times T201 has expired. */
@@ -625,9 +623,7 @@ struct q931_call {
 	/*! Display text ie contents. */
 	struct {
 		/*! Display ie text.  NULL if not present or consumed as remote name. */
-		const unsigned char *text;
-		/*! Full IE code of received display text */
-		int full_ie;
+		const char *text;
 		/*! Length of display text. */
 		unsigned char length;
 		/*!
@@ -647,34 +643,8 @@ struct q931_call {
 		unsigned char char_set;
 	} display;

-	/*! AOC charge requesting on Setup */
+	/* AOC charge requesting on Setup */
 	int aoc_charging_request;
-
-	/*! TRUE if the slotmap is E1 (32 bits). */
-	unsigned int slotmap_size:1;
-	/*! TRUE if need to see the channel id ie in first response to SETUP. */
-	unsigned int channel_id_ie_mandatory:1;
-
-	/*! Control the RESTART reception to the upper layer. */
-	struct {
-		/*! Timer ID of RESTART notification events to upper layer. */
-		int timer;
-		/*! Current RESTART notification index. */
-		int idx;
-		/*! Number of channels in the channel ID list. */
-		int count;
-		/*! Channel ID list */
-		char chan_no[32];
-	} restart;
-	/*! Control the RESTART retransmissions. */
-	struct {
-		/*! T316 RESTART retransmit timer. */
-		int t316_timer;
-		/*! Number of times remaining that RESTART can be transmitted. */
-		int remain;
-		/*! Encoded RESTART channel id. */
-		int channel;
-	} restart_tx;
 };

 enum CC_STATES {
--- a/pri_q921.h
+++ b/pri_q921.h
@@ -252,7 +252,7 @@ struct q921_link {
 	int n202_counter;
 	/*! Max idle time */
 	int t203_timer;
-	/*! Layer 2 persistence restart delay timer */
+	/*! PTP restart delay timer */
 	int restart_timer;

 	/* MDL variables */
@@ -273,11 +273,10 @@ static inline int Q921_ADD(int a, int b)
 }

 /* Dumps a *known good* Q.921 packet */
-extern void q921_dump(struct pri *pri, q921_h *h, int len, int debugflags, int txrx);
+extern void q921_dump(struct pri *pri, q921_h *h, int len, int showraw, int txrx);

 /* Bring up the D-channel */
 void q921_start(struct q921_link *link);
-void q921_bring_layer2_up(struct pri *ctrl);

 //extern void q921_reset(struct pri *pri, int reset_iqueue);

--- a/pri_q931.h
+++ b/pri_q931.h
@@ -138,6 +138,9 @@ typedef struct q931_ie {
 #define SERVICE_CHANGE_STATUS_REQCONTINUITYCHECK  3  /* not supported */
 #define SERVICE_CHANGE_STATUS_SHUTDOWN            4  /* not supported */

+/* Special codeset 0 IE */
+#define	NATIONAL_CHANGE_STATUS		0x1
+
 /* Q.931 / National ISDN Information Elements */
 #define Q931_LOCKING_SHIFT			0x90
 #define Q931_NON_LOCKING_SHIFT		0x98
@@ -465,7 +468,7 @@ extern int q931_notify(struct pri *pri, q931_call *call, int channel, int info);

 extern int q931_call_proceeding(struct pri *pri, q931_call *call, int channel, int info);

-extern int q931_setup_ack(struct pri *ctrl, q931_call *c, int channel, int nonisdn, int inband);
+extern int q931_setup_ack(struct pri *pri, q931_call *call, int channel, int nonisdn);

 extern int q931_information(struct pri *pri, q931_call *call, char digit);

--- a/pridump.c
+++ b/pridump.c
@@ -42,7 +42,7 @@
 #include <sys/ioctl.h>
 #include <sys/select.h>
 #include <sys/types.h>
-#include <dahdi/user.h>
+#include <zaptel/zaptel.h>
 #include "libpri.h"
 #include "pri_q921.h"
 #include "pri_q931.h"
@@ -50,18 +50,18 @@
 static int pri_open(char *dev)
 {
 	int dfd;
-	struct dahdi_params p;
+	struct zt_params p;
 	
 	dfd = open(dev, O_RDWR);
 	if (dfd < 0) {
 		fprintf(stderr, "Failed to open dchannel '%s': %s\n", dev, strerror(errno));
 		return -1;
 	}
-	if (ioctl(dfd, DAHDI_GET_PARAMS, &p)) {
+	if (ioctl(dfd, ZT_GET_PARAMS, &p)) {
 		fprintf(stderr, "Unable to get parameters on '%s': %s\n", dev, strerror(errno));
 		return -1;
 	}
-	if ((p.sigtype != DAHDI_SIG_HDLCRAW) && (p.sigtype != DAHDI_SIG_HDLCFCS)) {
+	if ((p.sigtype != ZT_SIG_HDLCRAW) && (p.sigtype != ZT_SIG_HDLCFCS)) {
 		fprintf(stderr, "%s is in %d signalling, not FCS HDLC or RAW HDLC mode\n", dev, p.sigtype);
 		return -1;
 	}
@@ -71,7 +71,7 @@ static int pri_open(char *dev)
 static void dump_packet(struct pri *pri, char *buf, int len, int txrx)
 {
 	q921_h *h = (q921_h *)buf;
-	q921_dump(pri, h, len, PRI_DEBUG_ALL, txrx);
+	q921_dump(pri, h, len, 1, txrx);
 	if (!((h->h.data[0] & Q921_FRAMETYPE_MASK) & 0x3)) {
 		q931_dump(pri, h->h.tei, (q931_h *)(h->i.data), len - 4 - 2 /* FCS */, txrx);
 	}
@@ -80,7 +80,7 @@ static void dump_packet(struct pri *pri, char *buf, int len, int txrx)
 }


-static void pri_bridge(int d1, int d2)
+static int pri_bridge(int d1, int d2)
 {
 	char buf[1024];
 	fd_set fds;
@@ -94,8 +94,8 @@ static void pri_bridge(int d1, int d2)
 		max = d1;
 		if (max < d2)
 			max = d2;
-		ioctl(d1, DAHDI_GETEVENT, &e);
-		ioctl(d2, DAHDI_GETEVENT, &e);
+		ioctl(d1, ZT_GETEVENT, &e);
+		ioctl(d2, ZT_GETEVENT, &e);
 		res = select(max + 1, &fds, NULL, NULL, NULL);
 		if (res < 0) {
 			fprintf(stderr, "Select returned %d: %s\n", res, strerror(errno));
--- a/pritest.c
+++ b/pritest.c
@@ -68,10 +68,7 @@ static void do_channel(int fd)
 	int i=0;

 	while ((res = read(fd, buf, READ_SIZE)) > 0 && (i++ < 1000)) {
-		if (write(fd, buf, res) == -1) {
-			fprintf(stderr, "--!! Failed write: %d\n", errno);
-			break;
-		}
+		write(fd, buf, res);
 	}
 }

--- a/q921.c
+++ b/q921.c
@@ -45,19 +45,13 @@
 */
 //#define RANDOM_DROPS	1

-#define Q921_INIT(fr, l_sapi, l_tei) \
-	do { \
-		(fr)->h.sapi = l_sapi; \
-		(fr)->h.ea1 = 0; \
-		(fr)->h.ea2 = 1; \
-		(fr)->h.tei = l_tei; \
-	} while (0)
-
-#define Q921_CLEAR_INIT(fr, l_sapi, l_tei) \
-	do { \
-		memset((fr), 0, sizeof(*(fr))); \
-		Q921_INIT((fr), (l_sapi), (l_tei)); \
-	} while (0)
+#define Q921_INIT(link, hf) do { \
+	memset(&(hf),0,sizeof(hf)); \
+	(hf).h.sapi = (link)->sapi; \
+	(hf).h.ea1 = 0; \
+	(hf).h.ea2 = 1; \
+	(hf).h.tei = (link)->tei; \
+} while (0)

 static void q921_dump_pri(struct q921_link *link, char direction_tag);
 static void q921_establish_data_link(struct q921_link *link);
@@ -192,7 +186,7 @@ static int q921_transmit(struct pri *ctrl, q921_h *h, int len)
 	ctrl->q921_txcount++;
 	/* Just send it raw */
 	if (ctrl->debug & (PRI_DEBUG_Q921_DUMP | PRI_DEBUG_Q921_RAW))
-		q921_dump(ctrl, h, len, ctrl->debug, 1);
+		q921_dump(ctrl, h, len, ctrl->debug & PRI_DEBUG_Q921_RAW, 1);
 	/* Write an extra two bytes for the FCS */
 	res = ctrl->write_func ? ctrl->write_func(ctrl, h, len + 2) : 0;
 	if (res != (len + 2)) {
@@ -202,14 +196,17 @@ static int q921_transmit(struct pri *ctrl, q921_h *h, int len)
 	return 0;
 }

-static void q921_mdl_send(struct pri *ctrl, enum q921_tei_identity message, int ri, int ai, int iscommand)
+static void q921_send_tei(struct pri *ctrl, enum q921_tei_identity message, int ri, int ai, int iscommand)
 {
 	q921_u *f;
+	struct q921_link *link;
+
+	link = &ctrl->link;

 	if (!(f = calloc(1, sizeof(*f) + 5)))
 		return;

-	Q921_INIT(f, Q921_SAPI_LAYER2_MANAGEMENT, Q921_TEI_GROUP);
+	Q921_INIT(link, *f);
 	f->h.c_r = (ctrl->localtype == PRI_NETWORK) ? iscommand : !iscommand;
 	f->ft = Q921_FRAMETYPE_U;
 	f->data[0] = 0x0f;	/* Management entity */
@@ -219,7 +216,7 @@ static void q921_mdl_send(struct pri *ctrl, enum q921_tei_identity message, int
 	f->data[4] = (ai << 1) | 1;
 	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
 		pri_message(ctrl,
-			"Sending MDL message: %d(%s), TEI=%d\n",
+			"Sending TEI management message %d(%s), TEI=%d\n",
 			message, q921_tei_mgmt2str(message), ai);
 	}
 	q921_transmit(ctrl, (q921_h *)f, 8);
@@ -238,10 +235,7 @@ static void t202_expire(void *vlink)
 	link->t202_timer =
 		pri_schedule_event(ctrl, ctrl->timers[PRI_TIMER_T202], t202_expire, link);

-	if (ctrl->l2_persistence != PRI_L2_PERSISTENCE_KEEP_UP) {
-		/* Only try to get a TEI for N202 times if layer 2 is not persistent. */
-		++link->n202_counter;
-	}
+	++link->n202_counter;
 	if (!link->t202_timer || link->n202_counter > ctrl->timers[PRI_TIMER_N202]) {
 		if (!link->t202_timer) {
 			pri_error(ctrl, "Could not start T202 timer.");
@@ -268,7 +262,7 @@ static void t202_expire(void *vlink)

 	/* Send TEI request */
 	link->ri = random() % 65535;
-	q921_mdl_send(ctrl, Q921_TEI_IDENTITY_REQUEST, link->ri, Q921_TEI_GROUP, 1);
+	q921_send_tei(ctrl, Q921_TEI_IDENTITY_REQUEST, link->ri, Q921_TEI_GROUP, 1);
 }

 static void q921_tei_request(struct q921_link *link)
@@ -283,8 +277,8 @@ static void q921_tei_remove(struct pri *ctrl, int tei)
 	 * Q.921 Section 5.3.2 says we should send the remove message
 	 * twice, in case of message loss.
 	 */
-	q921_mdl_send(ctrl, Q921_TEI_IDENTITY_REMOVE, 0, tei, 1);
-	q921_mdl_send(ctrl, Q921_TEI_IDENTITY_REMOVE, 0, tei, 1);
+	q921_send_tei(ctrl, Q921_TEI_IDENTITY_REMOVE, 0, tei, 1);
+	q921_send_tei(ctrl, Q921_TEI_IDENTITY_REMOVE, 0, tei, 1);
 }

 static void q921_send_dm(struct q921_link *link, int fbit)
@@ -294,7 +288,7 @@ static void q921_send_dm(struct q921_link *link, int fbit)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.u.m3 = 0;	/* M3 = 0 */
 	h.u.m2 = 3;	/* M2 = 3 */
 	h.u.p_f = fbit;	/* Final set appropriately */
@@ -313,7 +307,7 @@ static void q921_send_dm(struct q921_link *link, int fbit)
 	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
 		pri_message(ctrl, "TEI=%d Sending DM\n", link->tei);
 	}
-	q921_transmit(ctrl, &h, 3);
+	q921_transmit(ctrl, &h, 4);
 }

 static void q921_send_disc(struct q921_link *link, int pbit)
@@ -323,7 +317,7 @@ static void q921_send_disc(struct q921_link *link, int pbit)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.u.m3 = 2;	/* M3 = 2 */
 	h.u.m2 = 0;	/* M2 = 0 */
 	h.u.p_f = pbit;	/* Poll set appropriately */
@@ -342,7 +336,7 @@ static void q921_send_disc(struct q921_link *link, int pbit)
 	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
 		pri_message(ctrl, "TEI=%d Sending DISC\n", link->tei);
 	}
-	q921_transmit(ctrl, &h, 3);
+	q921_transmit(ctrl, &h, 4);
 }

 static void q921_send_ua(struct q921_link *link, int fbit)
@@ -352,7 +346,7 @@ static void q921_send_ua(struct q921_link *link, int fbit)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.u.m3 = 3;		/* M3 = 3 */
 	h.u.m2 = 0;		/* M2 = 0 */
 	h.u.p_f = fbit;	/* Final set appropriately */
@@ -381,7 +375,7 @@ static void q921_send_sabme(struct q921_link *link)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.u.m3 = 3;	/* M3 = 3 */
 	h.u.m2 = 3;	/* M2 = 3 */
 	h.u.p_f = 1;	/* Poll bit set */
@@ -536,49 +530,6 @@ static void stop_t200(struct q921_link *link)
 	}
 }

-/*!
- * \internal
- * \brief Initiate bringing up layer 2 link.
- *
- * \param link Layer 2 link to bring up.
- *
- * \return Nothing
- */
-static void kick_start_link(struct q921_link *link)
-{
-	struct pri *ctrl;
-
-	ctrl = link->ctrl;
-
-	switch (link->state) {
-	case Q921_TEI_UNASSIGNED:
-		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-			pri_message(ctrl, "Kick starting link from no TEI.\n");
-		}
-		q921_setstate(link, Q921_ESTABLISH_AWAITING_TEI);
-		q921_tei_request(link);
-		break;
-	case Q921_ASSIGN_AWAITING_TEI:
-		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-			pri_message(ctrl, "Kick starting link when awaiting TEI.\n");
-		}
-		q921_setstate(link, Q921_ESTABLISH_AWAITING_TEI);
-		break;
-	case Q921_TEI_ASSIGNED:
-		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-			pri_message(ctrl, "SAPI/TEI=%d/%d Kick starting link\n", link->sapi,
-				link->tei);
-		}
-		q921_discard_iqueue(link);
-		q921_establish_data_link(link);
-		link->l3_initiated = 1;
-		q921_setstate(link, Q921_AWAITING_ESTABLISHMENT);
-		break;
-	default:
-		break;
-	}
-}
-
 static void restart_timer_expire(void *vlink)
 {
 	struct q921_link *link = vlink;
@@ -586,14 +537,19 @@ static void restart_timer_expire(void *vlink)

 	ctrl = link->ctrl;

+	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
+		pri_message(ctrl, "SAPI/TEI=%d/%d Kick starting link\n", link->sapi, link->tei);
+	}
+
 	link->restart_timer = 0;

 	switch (link->state) {
-	case Q921_TEI_UNASSIGNED:
-	case Q921_ASSIGN_AWAITING_TEI:
 	case Q921_TEI_ASSIGNED:
 		/* Try to bring layer 2 up. */
-		kick_start_link(link);
+		q921_discard_iqueue(link);
+		q921_establish_data_link(link);
+		link->l3_initiated = 1;
+		q921_setstate(link, Q921_AWAITING_ESTABLISHMENT);
 		break;
 	default:
 		/* Looks like someone forgot to stop the restart timer. */
@@ -612,7 +568,6 @@ static void restart_timer_stop(struct q921_link *link)
 	link->restart_timer = 0;
 }

-/*! \note Only call on the transition to state Q921_TEI_ASSIGNED or already there. */
 static void restart_timer_start(struct q921_link *link)
 {
 	struct pri *ctrl;
@@ -628,27 +583,21 @@ static void restart_timer_start(struct q921_link *link)
 		pri_schedule_event(ctrl, ctrl->timers[PRI_TIMER_T200], restart_timer_expire, link);
 }

-/*! \note Only call on the transition to state Q921_TEI_ASSIGNED or already there. */
-static pri_event *q921_check_delay_restart(struct q921_link *link)
+static pri_event *q921_ptp_delay_restart(struct q921_link *link)
 {
 	pri_event *ev;
 	struct pri *ctrl;

 	ctrl = link->ctrl;

-	if (ctrl->l2_persistence == PRI_L2_PERSISTENCE_KEEP_UP) {
+	if (PTP_MODE(ctrl)) {
 		/*
-		 * For PTP links:
 		 * This is where we act a bit like L3 instead of L2, since we've
 		 * got an L3 that depends on us keeping L2 automatically alive
-		 * and happy.
-		 *
-		 * For PTMP links:
-		 * We can optionally keep L2 automatically alive and happy.
+		 * and happy for PTP links.
 		 */
 		restart_timer_start(link);
-	}
-	if (PTP_MODE(ctrl)) {
+
 		switch (link->state) {
 		case Q921_MULTI_FRAME_ESTABLISHED:
 		case Q921_TIMER_RECOVERY:
@@ -667,31 +616,6 @@ static pri_event *q921_check_delay_restart(struct q921_link *link)
 	return ev;
 }

-/*!
- * \brief Bring all layer 2 links up.
- *
- * \param ctrl D channel controller.
- *
- * \return Nothing
- */
-void q921_bring_layer2_up(struct pri *ctrl)
-{
-	struct q921_link *link;
-
-	if (PTMP_MODE(ctrl)) {
-		/* Don't start with the broadcast link. */
-		link = ctrl->link.next;
-	} else {
-		link = &ctrl->link;
-	}
-	for (; link; link = link->next) {
-		if (!link->restart_timer) {
-			/* A restart on the link is not already in the works. */
-			kick_start_link(link);
-		}
-	}
-}
-
 /* This is the equivalent of the I-Frame queued up path in Figure B.7 in MULTI_FRAME_ESTABLISHED */
 static int q921_send_queued_iframes(struct q921_link *link)
 {
@@ -812,7 +736,7 @@ static void q921_reject(struct q921_link *link, int pf)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.s.x0 = 0;	/* Always 0 */
 	h.s.ss = 2;	/* Reject */
 	h.s.ft = 1;	/* Frametype (01) */
@@ -842,7 +766,7 @@ static void q921_rr(struct q921_link *link, int pbit, int cmd)

 	ctrl = link->ctrl;

-	Q921_CLEAR_INIT(&h, link->sapi, link->tei);
+	Q921_INIT(link, h);
 	h.s.x0 = 0;	/* Always 0 */
 	h.s.ss = 0; /* Receive Ready */
 	h.s.ft = 1;	/* Frametype (01) */
@@ -935,7 +859,7 @@ static void t200_expire(void *vlink)
 			q921_send_sabme(link);
 			start_t200(link);
 		} else {
-			q921_check_delay_restart(link);
+			q921_ptp_delay_restart(link);
 			q921_discard_iqueue(link);
 			q921_mdl_error(link, 'G');
 			q921_setstate(link, Q921_TEI_ASSIGNED);
@@ -949,7 +873,7 @@ static void t200_expire(void *vlink)
 			q921_send_disc(link, 1);
 			start_t200(link);
 		} else {
-			q921_check_delay_restart(link);
+			q921_ptp_delay_restart(link);
 			q921_mdl_error(link, 'H');
 			/* DL-RELEASE confirm */
 			q931_dl_event(link, Q931_DL_EVENT_DL_RELEASE_CONFIRM);
@@ -1069,7 +993,7 @@ int q921_transmit_iframe(struct q921_link *link, void *buf, int len, int cr)

 		f = calloc(1, sizeof(struct q921_frame) + len + 2);
 		if (f) {
-			Q921_INIT(&f->h, link->sapi, link->tei);
+			Q921_INIT(link, f->h);
 			switch (ctrl->localtype) {
 			case PRI_NETWORK:
 				if (cr)
@@ -1192,7 +1116,7 @@ static void q921_dump_iqueue_info(struct q921_link *link)

 static void q921_dump_pri_by_h(struct pri *ctrl, char direction_tag, q921_h *h);

-void q921_dump(struct pri *ctrl, q921_h *h, int len, int debugflags, int txrx)
+void q921_dump(struct pri *ctrl, q921_h *h, int len, int showraw, int txrx)
 {
 	int x;
 	const char *type;
@@ -1201,11 +1125,9 @@ void q921_dump(struct pri *ctrl, q921_h *h, int len, int debugflags, int txrx)
 	direction_tag = txrx ? '>' : '<';

 	pri_message(ctrl, "\n");
-	if (debugflags & PRI_DEBUG_Q921_DUMP) {
-		q921_dump_pri_by_h(ctrl, direction_tag, h);
-	}
+	q921_dump_pri_by_h(ctrl, direction_tag, h);

-	if (debugflags & PRI_DEBUG_Q921_RAW) {
+	if (showraw) {
 		char *buf = malloc(len * 3 + 1);
 		int buflen = 0;
 		if (buf) {
@@ -1216,134 +1138,132 @@ void q921_dump(struct pri *ctrl, q921_h *h, int len, int debugflags, int txrx)
 		}
 	}

-	if (debugflags & PRI_DEBUG_Q921_DUMP) {
-		switch (h->h.data[0] & Q921_FRAMETYPE_MASK) {
+	switch (h->h.data[0] & Q921_FRAMETYPE_MASK) {
+	case 0:
+	case 2:
+		pri_message(ctrl, "%c Informational frame:\n", direction_tag);
+		break;
+	case 1:
+		pri_message(ctrl, "%c Supervisory frame:\n", direction_tag);
+		break;
+	case 3:
+		pri_message(ctrl, "%c Unnumbered frame:\n", direction_tag);
+		break;
+	}
+	
+	pri_message(ctrl, "%c SAPI: %02d  C/R: %d EA: %d\n",
+		direction_tag,
+		h->h.sapi, 
+		h->h.c_r,
+		h->h.ea1);
+	pri_message(ctrl, "%c  TEI: %03d        EA: %d\n", 
+		direction_tag,
+		h->h.tei,
+		h->h.ea2);
+
+	switch (h->h.data[0] & Q921_FRAMETYPE_MASK) {
+	case 0:
+	case 2:
+		/* Informational frame */
+		pri_message(ctrl, "%c N(S): %03d   0: %d\n",
+			direction_tag,
+			h->i.n_s,
+			h->i.ft);
+		pri_message(ctrl, "%c N(R): %03d   P: %d\n",
+			direction_tag,
+			h->i.n_r,
+			h->i.p_f);
+		pri_message(ctrl, "%c %d bytes of data\n",
+			direction_tag,
+			len - 4);
+		break;
+	case 1:
+		/* Supervisory frame */
+		type = "???";
+		switch (h->s.ss) {
 		case 0:
-		case 2:
-			pri_message(ctrl, "%c Informational frame:\n", direction_tag);
+			type = "RR (receive ready)";
 			break;
 		case 1:
-			pri_message(ctrl, "%c Supervisory frame:\n", direction_tag);
+			type = "RNR (receive not ready)";
 			break;
-		case 3:
-			pri_message(ctrl, "%c Unnumbered frame:\n", direction_tag);
+		case 2:
+			type = "REJ (reject)";
 			break;
 		}
-		
-		pri_message(ctrl, "%c SAPI: %02d  C/R: %d EA: %d\n",
+		pri_message(ctrl, "%c Zero: %d     S: %d 01: %d  [ %s ]\n",
 			direction_tag,
-			h->h.sapi, 
-			h->h.c_r,
-			h->h.ea1);
-		pri_message(ctrl, "%c  TEI: %03d        EA: %d\n", 
+			h->s.x0,
+			h->s.ss,
+			h->s.ft,
+			type);
+		pri_message(ctrl, "%c N(R): %03d P/F: %d\n",
 			direction_tag,
-			h->h.tei,
-			h->h.ea2);
-	
-		switch (h->h.data[0] & Q921_FRAMETYPE_MASK) {
-		case 0:
-		case 2:
-			/* Informational frame */
-			pri_message(ctrl, "%c N(S): %03d   0: %d\n",
-				direction_tag,
-				h->i.n_s,
-				h->i.ft);
-			pri_message(ctrl, "%c N(R): %03d   P: %d\n",
-				direction_tag,
-				h->i.n_r,
-				h->i.p_f);
-			pri_message(ctrl, "%c %d bytes of data\n",
-				direction_tag,
-				len - 4);
-			break;
-		case 1:
-			/* Supervisory frame */
-			type = "???";
-			switch (h->s.ss) {
+			h->s.n_r,
+			h->s.p_f);
+		pri_message(ctrl, "%c %d bytes of data\n",
+			direction_tag,
+			len - 4);
+		break;
+	case 3:		
+		/* Unnumbered frame */
+		type = "???";
+		if (h->u.ft == 3) {
+			switch (h->u.m3) {
 			case 0:
-				type = "RR (receive ready)";
-				break;
-			case 1:
-				type = "RNR (receive not ready)";
+				if (h->u.m2 == 3)
+					type = "DM (disconnect mode)";
+				else if (h->u.m2 == 0)
+					type = "UI (unnumbered information)";
 				break;
 			case 2:
-				type = "REJ (reject)";
+				if (h->u.m2 == 0)
+					type = "DISC (disconnect)";
+				break;
+			case 3:
+				if (h->u.m2 == 3)
+					type = "SABME (set asynchronous balanced mode extended)";
+				else if (h->u.m2 == 0)
+					type = "UA (unnumbered acknowledgement)";
+				break;
+			case 4:
+				if (h->u.m2 == 1)
+					type = "FRMR (frame reject)";
+				break;
+			case 5:
+				if (h->u.m2 == 3)
+					type = "XID (exchange identification note)";
+				break;
+			default:
 				break;
 			}
-			pri_message(ctrl, "%c Zero: %d     S: %d 01: %d  [ %s ]\n",
-				direction_tag,
-				h->s.x0,
-				h->s.ss,
-				h->s.ft,
-				type);
-			pri_message(ctrl, "%c N(R): %03d P/F: %d\n",
-				direction_tag,
-				h->s.n_r,
-				h->s.p_f);
-			pri_message(ctrl, "%c %d bytes of data\n",
-				direction_tag,
-				len - 4);
-			break;
-		case 3:		
-			/* Unnumbered frame */
-			type = "???";
-			if (h->u.ft == 3) {
-				switch (h->u.m3) {
-				case 0:
-					if (h->u.m2 == 3)
-						type = "DM (disconnect mode)";
-					else if (h->u.m2 == 0)
-						type = "UI (unnumbered information)";
-					break;
-				case 2:
-					if (h->u.m2 == 0)
-						type = "DISC (disconnect)";
-					break;
-				case 3:
-					if (h->u.m2 == 3)
-						type = "SABME (set asynchronous balanced mode extended)";
-					else if (h->u.m2 == 0)
-						type = "UA (unnumbered acknowledgement)";
-					break;
-				case 4:
-					if (h->u.m2 == 1)
-						type = "FRMR (frame reject)";
-					break;
-				case 5:
-					if (h->u.m2 == 3)
-						type = "XID (exchange identification note)";
-					break;
-				default:
-					break;
-				}
-			}
-			pri_message(ctrl, "%c   M3: %d   P/F: %d M2: %d 11: %d  [ %s ]\n",
-				direction_tag,
-				h->u.m3,
-				h->u.p_f,
-				h->u.m2,
-				h->u.ft,
-				type);
-			pri_message(ctrl, "%c %d bytes of data\n",
-				direction_tag,
-				len - 3);
-			break;
 		}
-	
-		if ((h->u.ft == 3) && (h->u.m3 == 0) && (h->u.m2 == 0) && (h->u.data[0] == 0x0f)) {
-			int ri;
-			u_int8_t *action;
-	
-			/* TEI management related */
-			type = q921_tei_mgmt2str(h->u.data[3]);
-			pri_message(ctrl, "%c MDL Message: %d(%s)\n", direction_tag, h->u.data[3], type);
-			ri = (h->u.data[1] << 8) | h->u.data[2];
-			pri_message(ctrl, "%c Ri: %d\n", direction_tag, ri);
-			action = &h->u.data[4];
-			for (x = len - (action - (u_int8_t *) h); 0 < x; --x, ++action) {
-				pri_message(ctrl, "%c Ai: %d E:%d\n",
-					direction_tag, (*action >> 1) & 0x7f, *action & 0x01);
-			}
+		pri_message(ctrl, "%c   M3: %d   P/F: %d M2: %d 11: %d  [ %s ]\n",
+			direction_tag,
+			h->u.m3,
+			h->u.p_f,
+			h->u.m2,
+			h->u.ft,
+			type);
+		pri_message(ctrl, "%c %d bytes of data\n",
+			direction_tag,
+			len - 3);
+		break;
+	}
+
+	if ((h->u.ft == 3) && (h->u.m3 == 0) && (h->u.m2 == 0) && (h->u.data[0] == 0x0f)) {
+		int ri;
+		u_int8_t *action;
+
+		/* TEI management related */
+		type = q921_tei_mgmt2str(h->u.data[3]);
+		pri_message(ctrl, "%c MDL Message: %d(%s)\n", direction_tag, h->u.data[3], type);
+		ri = (h->u.data[1] << 8) | h->u.data[2];
+		pri_message(ctrl, "%c Ri: %d\n", direction_tag, ri);
+		action = &h->u.data[4];
+		for (x = len - (action - (u_int8_t *) h); 0 < x; --x, ++action) {
+			pri_message(ctrl, "%c Ai: %d E:%d\n",
+				direction_tag, (*action >> 1) & 0x7f, *action & 0x01);
 		}
 	}
 }
@@ -1399,6 +1319,12 @@ static void t201_expire(void *vctrl)

 	ctrl = vctrl;

+	if (!ctrl->link.next) {
+		/* No TEI links remain. */
+		ctrl->t201_timer = 0;
+		return;
+	}
+
 	/* Start the TEI check timer. */
 	ctrl->t201_timer =
 		pri_schedule_event(ctrl, ctrl->timers[PRI_TIMER_T201], t201_expire, ctrl);
@@ -1459,7 +1385,7 @@ static void t201_expire(void *vctrl)
 			}
 		}
 	}
-	q921_mdl_send(ctrl, Q921_TEI_IDENTITY_CHECK_REQUEST, 0, Q921_TEI_GROUP, 1);
+	q921_send_tei(ctrl, Q921_TEI_IDENTITY_CHECK_REQUEST, 0, Q921_TEI_GROUP, 1);
 }

 static void q921_tei_check(struct pri *ctrl)
@@ -1472,19 +1398,7 @@ static void q921_tei_check(struct pri *ctrl)
 	t201_expire(ctrl);
 }

-static void q921_mdl_ignore(struct pri *ctrl, q921_u *h, const char *reason)
-{
-	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-		/*
-		 * Send out this message in debug modes since it is possible the
-		 * user has misconfigured their link for the wrong mode.
-		 */
-		pri_message(ctrl, "Ignoring MDL message: %d(%s)  %s\n",
-			h->data[3], q921_tei_mgmt2str(h->data[3]), reason);
-	}
-}
-
-static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
+static pri_event *q921_receive_MDL(struct pri *ctrl, q921_u *h, int len)
 {
 	int ri;
 	struct q921_link *sub;
@@ -1494,13 +1408,20 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 	int count;
 	int tei;

+	if (!BRI_NT_PTMP(ctrl) && !BRI_TE_PTMP(ctrl)) {
+		return pri_mkerror(ctrl,
+			"Received MDL/TEI managemement message, but configured for mode other than PTMP!\n");
+	}
+
+	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
+		pri_message(ctrl, "Received MDL message\n");
+	}
 	if (len <= &h->data[0] - (u_int8_t *) h) {
-		pri_error(ctrl, "Received short MDL frame\n");
+		pri_error(ctrl, "Received short frame\n");
 		return NULL;
 	}
 	if (h->data[0] != 0x0f) {
-		pri_error(ctrl, "Received MDL with unsupported management entity %02x\n",
-			h->data[0]);
+		pri_error(ctrl, "Received MDL with unsupported management entity %02x\n", h->data[0]);
 		return NULL;
 	}
 	if (len <= &h->data[4] - (u_int8_t *) h) {
@@ -1509,20 +1430,8 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 	}
 	if (h->data[3] != Q921_TEI_IDENTITY_CHECK_RESPONSE
 		&& !(h->data[4] & 0x01)) {
-		pri_error(ctrl, "Received MDL message: %d(%s) with Ai E bit not set.\n",
-			h->data[3], q921_tei_mgmt2str(h->data[3]));
-		return NULL;
-	}
-	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-		pri_message(ctrl, "Received MDL message: %d(%s)\n",
-			h->data[3], q921_tei_mgmt2str(h->data[3]));
-	}
-	if (PTP_MODE(ctrl) && NT_MODE(ctrl)) {
-		/*
-		 * We are not managing automatic TEI's in this mode so we can
-		 * ignore MDL messages from the CPE.
-		 */
-		q921_mdl_ignore(ctrl, h, "We are in NT-PTP mode.");
+		pri_error(ctrl, "Received %d(%s) with Ai E bit not set.\n", h->data[3],
+			q921_tei_mgmt2str(h->data[3]));
 		return NULL;
 	}
 	ri = (h->data[1] << 8) | h->data[2];
@@ -1531,14 +1440,13 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 	switch (h->data[3]) {
 	case Q921_TEI_IDENTITY_REQUEST:
 		if (!BRI_NT_PTMP(ctrl)) {
-			q921_mdl_ignore(ctrl, h, "We are not in NT-PTMP mode.");
 			return NULL;
 		}

 		if (tei != Q921_TEI_GROUP) {
 			pri_error(ctrl, "Received %s with invalid TEI %d\n",
 				q921_tei_mgmt2str(Q921_TEI_IDENTITY_REQUEST), tei);
-			q921_mdl_send(ctrl, Q921_TEI_IDENTITY_DENIED, ri, tei, 1);
+			q921_send_tei(ctrl, Q921_TEI_IDENTITY_DENIED, ri, tei, 1);
 			return NULL;
 		}

@@ -1553,7 +1461,7 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 						break;
 					}
 					pri_error(ctrl, "TEI pool exhausted.  Reclaiming dead TEIs.\n");
-					q921_mdl_send(ctrl, Q921_TEI_IDENTITY_DENIED, ri, Q921_TEI_GROUP, 1);
+					q921_send_tei(ctrl, Q921_TEI_IDENTITY_DENIED, ri, Q921_TEI_GROUP, 1);
 					q921_tei_check(ctrl);
 					return NULL;
 				}
@@ -1563,14 +1471,13 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
 			pri_message(ctrl, "Allocating new TEI %d\n", tei);
 		}
-		link = pri_link_new(ctrl, Q921_SAPI_CALL_CTRL, tei);
-		if (!link) {
+		sub->next = pri_link_new(ctrl, Q921_SAPI_CALL_CTRL, tei);
+		if (!sub->next) {
 			pri_error(ctrl, "Unable to allocate layer 2 link for new TEI %d\n", tei);
 			return NULL;
 		}
-		sub->next = link;
-		q921_setstate(link, Q921_TEI_ASSIGNED);
-		q921_mdl_send(ctrl, Q921_TEI_IDENTITY_ASSIGNED, ri, tei, 1);
+		q921_setstate(sub->next, Q921_TEI_ASSIGNED);
+		q921_send_tei(ctrl, Q921_TEI_IDENTITY_ASSIGNED, ri, tei, 1);

 		count = 0;
 		for (sub = ctrl->link.next; sub; sub = sub->next) {
@@ -1586,19 +1493,9 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 			}
 			q921_tei_check(ctrl);
 		}
-
-		if (ctrl->l2_persistence == PRI_L2_PERSISTENCE_KEEP_UP) {
-			/*
-			 * Layer 2 is persistent so give the peer some time to setup
-			 * it's new TEI and bring the link up itself before we bring the
-			 * link up.
-			 */
-			restart_timer_start(link);
-		}
 		break;
 	case Q921_TEI_IDENTITY_CHECK_RESPONSE:
 		if (!BRI_NT_PTMP(ctrl)) {
-			/* Silently ignore the message since we never asked for the check. */
 			return NULL;
 		}

@@ -1646,7 +1543,6 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 		break;
 	case Q921_TEI_IDENTITY_VERIFY:
 		if (!BRI_NT_PTMP(ctrl)) {
-			q921_mdl_ignore(ctrl, h, "We are not in NT-PTMP mode.");
 			return NULL;
 		}
 		if (tei == Q921_TEI_GROUP) {
@@ -1657,30 +1553,16 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 		q921_tei_check(ctrl);
 		break;
 	case Q921_TEI_IDENTITY_ASSIGNED:
-		if (NT_MODE(ctrl)) {
-			/* We should not be receiving this message. */
-			q921_mdl_ignore(ctrl, h, "We are the network.");
-			return NULL;
-		}
 		if (!BRI_TE_PTMP(ctrl)) {
-			/*
-			 * Silently ignore the message.  It must not be for us
-			 * since we will never ask for one.
-			 */
 			return NULL;
 		}

-		/* We're operating on the specific TEI link here */
+		/* Assuming we're operating on the specific TEI link here */
 		link = ctrl->link.next;
 		
 		switch (link->state) {
 		case Q921_TEI_UNASSIGNED:
-			/*
-			 * We do not have a TEI and we are not currently asking for one.
-			 * Start asking for one.
-			 */
-			q921_setstate(link, Q921_ASSIGN_AWAITING_TEI);
-			q921_tei_request(link);
+			/* We do not have a TEI and we are not asking for one. */
 			return NULL;
 		case Q921_ASSIGN_AWAITING_TEI:
 		case Q921_ESTABLISH_AWAITING_TEI:
@@ -1722,12 +1604,9 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 		switch (link->state) {
 		case Q921_ASSIGN_AWAITING_TEI:
 			q921_setstate(link, Q921_TEI_ASSIGNED);
-			if (ctrl->l2_persistence != PRI_L2_PERSISTENCE_KEEP_UP) {
-				ctrl->ev.gen.e = PRI_EVENT_DCHAN_UP;
-				res = &ctrl->ev;
-				break;
-			}
-			/* Fall through: Layer 2 is persistent so bring it up. */
+			ctrl->ev.gen.e = PRI_EVENT_DCHAN_UP;
+			res = &ctrl->ev;
+			break;
 		case Q921_ESTABLISH_AWAITING_TEI:
 			q921_establish_data_link(link);
 			link->l3_initiated = 1;
@@ -1740,52 +1619,29 @@ static pri_event *q921_mdl_receive(struct pri *ctrl, q921_u *h, int len)
 		}
 		break;
 	case Q921_TEI_IDENTITY_CHECK_REQUEST:
-		if (NT_MODE(ctrl)) {
-			/* We should not be receiving this message. */
-			q921_mdl_ignore(ctrl, h, "We are the network.");
+		if (!BRI_TE_PTMP(ctrl)) {
 			return NULL;
 		}

-		if (PTP_MODE(ctrl)) {
-			/*
-			 * Some telco switches/devices get very unhappy if we don't
-			 * respond to the TEI check request with our permanently
-			 * assigned TEI.
-			 */
-			link = &ctrl->link;
-		} else if (BRI_TE_PTMP(ctrl)) {
-			/* We're operating on the specific TEI link here */
-			link = ctrl->link.next;
+		/* Assuming we're operating on the specific TEI link here */
+		link = ctrl->link.next;

-			if (link->state < Q921_TEI_ASSIGNED) {
-				/* We do not have a TEI. */
-				return NULL;
-			}
-		} else {
-			/* Should never get here. */
+		if (link->state < Q921_TEI_ASSIGNED) {
+			/* We do not have a TEI. */
 			return NULL;
 		}

 		/* If it's addressed to the group TEI or to our TEI specifically, we respond */
 		if (tei == Q921_TEI_GROUP || tei == link->tei) {
-			q921_mdl_send(ctrl, Q921_TEI_IDENTITY_CHECK_RESPONSE, random() % 65535, link->tei, 1);
+			q921_send_tei(ctrl, Q921_TEI_IDENTITY_CHECK_RESPONSE, random() % 65535, link->tei, 1);
 		}
 		break;
 	case Q921_TEI_IDENTITY_REMOVE:
-		if (NT_MODE(ctrl)) {
-			/* We should not be receiving this message. */
-			q921_mdl_ignore(ctrl, h, "We are the network.");
-			return NULL;
-		}
 		if (!BRI_TE_PTMP(ctrl)) {
-			/*
-			 * Silently ignore the message.  If we are TE-PTP our
-			 * TEI is permanently assigned and cannot be removed.
-			 */
 			return NULL;
 		}

-		/* We're operating on the specific TEI link here */
+		/* Assuming we're operating on the specific TEI link here */
 		link = ctrl->link.next;

 		if (link->state < Q921_TEI_ASSIGNED) {
@@ -1909,7 +1765,7 @@ static pri_event *q921_disc_rx(struct q921_link *link, q921_h *h)
 		break;
 	case Q921_MULTI_FRAME_ESTABLISHED:
 	case Q921_TIMER_RECOVERY:
-		res = q921_check_delay_restart(link);
+		res = q921_ptp_delay_restart(link);
 		q921_discard_iqueue(link);
 		q921_send_ua(link, h->u.p_f);
 		/* DL-RELEASE indication */
@@ -1950,9 +1806,7 @@ static void q921_mdl_remove(struct q921_link *link)

 	switch (link->state) {
 	case Q921_TEI_ASSIGNED:
-		if (mdl_free_me) {
-			restart_timer_stop(link);
-		}
+		restart_timer_stop(link);
 		/* XXX: deviation! Since we don't have a UI queue, we just discard our I-queue */
 		q921_discard_iqueue(link);
 		q921_setstate(link, Q921_TEI_UNASSIGNED);
@@ -2323,7 +2177,7 @@ static pri_event *q921_ua_rx(struct q921_link *link, q921_h *h)
 		if (!h->u.p_f) {
 			q921_mdl_error(link, 'D');
 		} else {
-			res = q921_check_delay_restart(link);
+			res = q921_ptp_delay_restart(link);
 			/* DL-RELEASE confirm */
 			q931_dl_event(link, Q931_DL_EVENT_DL_RELEASE_CONFIRM);
 			stop_t200(link);
@@ -2761,7 +2615,7 @@ static pri_event *q921_dm_rx(struct q921_link *link, q921_h *h)
 		if (!h->u.p_f)
 			break;

-		res = q921_check_delay_restart(link);
+		res = q921_ptp_delay_restart(link);
 		q921_discard_iqueue(link);
 		/* DL-RELEASE indication */
 		q931_dl_event(link, Q931_DL_EVENT_DL_RELEASE_IND);
@@ -2771,7 +2625,7 @@ static pri_event *q921_dm_rx(struct q921_link *link, q921_h *h)
 	case Q921_AWAITING_RELEASE:
 		if (!h->u.p_f)
 			break;
-		res = q921_check_delay_restart(link);
+		res = q921_ptp_delay_restart(link);
 		/* DL-RELEASE confirm */
 		q931_dl_event(link, Q931_DL_EVENT_DL_RELEASE_CONFIRM);
 		stop_t200(link);
@@ -3028,32 +2882,27 @@ static pri_event *__q921_receive_qualified(struct q921_link *link, q921_h *h, in

 static pri_event *q921_handle_unmatched_frame(struct pri *ctrl, q921_h *h, int len)
 {
-	if (!BRI_NT_PTMP(ctrl)) {
-		return NULL;
-	}
-
-	if (h->h.tei < Q921_TEI_AUTO_FIRST) {
-		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-			pri_message(ctrl, "Manual TEI range is not supported in NT-PTMP mode. Discarding\n");
-		}
+	if (h->h.tei < 64) {
+		pri_error(ctrl, "Do not support manual TEI range. Discarding\n");
 		return NULL;
 	}

 	if (h->h.sapi != Q921_SAPI_CALL_CTRL) {
-		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-			pri_message(ctrl, "Message with SAPI other than CALL CTRL is discarded\n");
-		}
+		pri_error(ctrl, "Message with SAPI other than CALL CTRL is discarded\n");
 		return NULL;
 	}

-	/* This means an unrecognized TEI that we'll kill */
-	if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
-		pri_message(ctrl,
-			"Could not find a layer 2 link for received frame with SAPI/TEI of %d/%d.\n",
-			h->h.sapi, h->h.tei);
-		pri_message(ctrl, "Sending TEI release, in order to re-establish TEI state\n");
+	/* If we're NT-PTMP, this means an unrecognized TEI that we'll kill */
+	if (BRI_NT_PTMP(ctrl)) {
+		if (ctrl->debug & PRI_DEBUG_Q921_STATE) {
+			pri_message(ctrl,
+				"Could not find a layer 2 link for received frame with SAPI/TEI of %d/%d.\n",
+				h->h.sapi, h->h.tei);
+			pri_message(ctrl, "Sending TEI release, in order to re-establish TEI state\n");
+		}
+	
+		q921_tei_remove(ctrl, h->h.tei);
 	}
-	q921_tei_remove(ctrl, h->h.tei);

 	return NULL;
 }
@@ -3068,7 +2917,7 @@ static pri_event *__q921_receive(struct pri *ctrl, q921_h *h, int len)
 	len -= 2;
 	
 	if (ctrl->debug & (PRI_DEBUG_Q921_DUMP | PRI_DEBUG_Q921_RAW)) {
-		q921_dump(ctrl, h, len, ctrl->debug, 0);
+		q921_dump(ctrl, h, len, ctrl->debug & PRI_DEBUG_Q921_RAW, 0);
 	}

 	/* Check some reject conditions -- Start by rejecting improper ea's */
@@ -3077,7 +2926,7 @@ static pri_event *__q921_receive(struct pri *ctrl, q921_h *h, int len)
 	}

 	if (h->h.sapi == Q921_SAPI_LAYER2_MANAGEMENT) {
-		return q921_mdl_receive(ctrl, &h->u, len);
+		return q921_receive_MDL(ctrl, &h->u, len);
 	}

 	if (h->h.tei == Q921_TEI_GROUP && h->h.sapi != Q921_SAPI_CALL_CTRL) {
--- a/q931.c
+++ b/q931.c
--- a/testprilib.c
+++ b/testprilib.c
@@ -48,10 +48,13 @@
 #include <sys/time.h>
 #include <netinet/in.h>
 #include <sys/socket.h>
+#include <zaptel/zaptel.h>
+#ifndef SOLARIS
+#include <zap.h>
+#endif
 #include <pthread.h>
 #include <sys/select.h>
 #include "libpri.h"
-#include "pri_q921.h"
 #include "pri_q931.h"

 #ifndef AF_LOCAL
@@ -63,7 +66,7 @@
 #define PRI_DEF_NODETYPE	PRI_CPE
 #define PRI_DEF_SWITCHTYPE	PRI_SWITCH_NI2

-static struct pri *first;
+static struct pri *first, *cur;

 static pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;

@@ -156,9 +159,9 @@ static void testmsg(struct pri *pri, char *s)
 			*c = '\0';
 			c++;
 		}
-		if (keeplast || !pri)
+		if (keeplast)
 			printf("%s", s);
-		else if (pri == first)
+		else if (cur == first)
 			printf("-1 %s", s);
 		else
 			printf("-2 %s", s);
@@ -182,9 +185,9 @@ static void testerr(struct pri *pri, char *s)
 			*c = '\0';
 			c++;
 		}
-		if (keeplast || !pri)
+		if (keeplast)
 			printf("%s", s);
-		else if (pri == first)
+		else if (cur == first)
 			printf("=1 %s", s);
 		else
 			printf("=2 %s", s);
@@ -204,7 +207,7 @@ static void *dchan(void *data)
 	/* Joint D-channel */
 	struct pri *pri = data;
 	struct timeval *next, tv;
-	pri_event *e = NULL;
+	pri_event *e;
 	fd_set fds;
 	int res;
 	for(;;) {
@@ -225,6 +228,7 @@ static void *dchan(void *data)
 		FD_SET(pri_fd(pri), &fds);
 		res = select(pri_fd(pri) + 1, &fds, NULL, NULL, next ? &tv : NULL);
 		pthread_mutex_lock(&lock);
+		cur = pri;
 		if (res < 0) {
 			perror("select");
 		} else if (!res) {
@@ -234,9 +238,9 @@ static void *dchan(void *data)
 		}
 		if (e) {
 			if (first == pri) {
-				event1(pri, e);
+				event1(e->gen.pri, e);
 			} else {
-				event2(pri, e);
+				event2(e->gen.pri, e);
 			}
 		}
 		pthread_mutex_unlock(&lock);