From owner-freebsd-commit  Tue Jan 30 03:09:20 1996
Return-Path: owner-commit
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id DAA04776
          for freebsd-commit-outgoing; Tue, 30 Jan 1996 03:09:20 -0800 (PST)
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id DAA04755
          for cvs-all-outgoing; Tue, 30 Jan 1996 03:09:14 -0800 (PST)
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id DAA04726
          for cvs-usrsbin-outgoing; Tue, 30 Jan 1996 03:09:07 -0800 (PST)
Received: (from dfr@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id DAA04706
          Tue, 30 Jan 1996 03:09:02 -0800 (PST)
Date: Tue, 30 Jan 1996 03:09:02 -0800 (PST)
From: Doug Rabson <dfr>
Message-Id: <199601301109.DAA04706@freefall.freebsd.org>
To: CVS-committers, cvs-usrsbin
Subject: cvs commit: src/usr.sbin/ppp chap.c fsm.c hdlc.h ip.c lcp.c lqr.c lqr.h main.c modem.c pap.c pred.c vjcomp.c
Sender: owner-commit@FreeBSD.ORG
Precedence: bulk

dfr         96/01/30 03:08:59

  Modified:    usr.sbin/ppp  chap.c fsm.c hdlc.h ip.c lcp.c lqr.c lqr.h
                        main.c modem.c  pap.c pred.c vjcomp.c
  Log:
  Some patches to ppp which improve stability.  I have been running a
  ppp based on these patches for about 3 weeks with no downtime.
  
  The original submitters comments:
  
  Two features iijppp has over kernel ppp that I like are predictor1
  compression and demand dialing.  Here are a few bug fixes.
  
  I expanded the priority queueing scheme and discovered it was broken
  due to the assignment at ip.c line 300.  All packets were being
  queued at the same priority.
  
  Fixing priority queueing broke predictor1 compression.  Packets
  were compressed before being queued and predictor1 worked as long
  as the packets were popped off the queue in the same order they
  were pushed onto the queue.
  
  There were a few byte order problems in IP header tests also.
  
  There is a recursion problem in SendLqrReport().  LcpClose() is
  called when "Too many echo packets are lost" which winds up in
  SendLqrReport() again.  I believe the original intention was to
  just stop the LQR timer with the call to StopLqr() but the side
  effects hurt.
  
  Submitted by:	John Capo <jc@irbs.com>
  
  Revision  Changes    Path
  1.5       +2 -2      src/usr.sbin/ppp/chap.c
  1.6       +2 -2      src/usr.sbin/ppp/fsm.c
  1.4       +8 -3      src/usr.sbin/ppp/hdlc.h
  1.8       +16 -35    src/usr.sbin/ppp/ip.c
  1.8       +2 -2      src/usr.sbin/ppp/lcp.c
  1.6       +12 -6     src/usr.sbin/ppp/lqr.c
  1.3       +2 -1      src/usr.sbin/ppp/lqr.h
  1.14      +7 -3      src/usr.sbin/ppp/main.c
  1.12      +17 -8     src/usr.sbin/ppp/modem.c
  1.4       +3 -3      src/usr.sbin/ppp/pap.c
  1.5       +11 -3     src/usr.sbin/ppp/pred.c
  1.4       +3 -4      src/usr.sbin/ppp/vjcomp.c