ppp: Fix throttling bugs
[linux-2.6] / drivers / net / ppp_synctty.c
1 /*
2  * PPP synchronous tty channel driver for Linux.
3  *
4  * This is a ppp channel driver that can be used with tty device drivers
5  * that are frame oriented, such as synchronous HDLC devices.
6  *
7  * Complete PPP frames without encoding/decoding are exchanged between
8  * the channel driver and the device driver.
9  *
10  * The async map IOCTL codes are implemented to keep the user mode
11  * applications happy if they call them. Synchronous PPP does not use
12  * the async maps.
13  *
14  * Copyright 1999 Paul Mackerras.
15  *
16  * Also touched by the grubby hands of Paul Fulghum paulkf@microgate.com
17  *
18  *  This program is free software; you can redistribute it and/or
19  *  modify it under the terms of the GNU General Public License
20  *  as published by the Free Software Foundation; either version
21  *  2 of the License, or (at your option) any later version.
22  *
23  * This driver provides the encapsulation and framing for sending
24  * and receiving PPP frames over sync serial lines.  It relies on
25  * the generic PPP layer to give it frames to send and to process
26  * received frames.  It implements the PPP line discipline.
27  *
28  * Part of the code in this driver was inspired by the old async-only
29  * PPP driver, written by Michael Callahan and Al Longyear, and
30  * subsequently hacked by Paul Mackerras.
31  *
32  * ==FILEVERSION 20040616==
33  */
34
35 #include <linux/module.h>
36 #include <linux/kernel.h>
37 #include <linux/skbuff.h>
38 #include <linux/tty.h>
39 #include <linux/netdevice.h>
40 #include <linux/poll.h>
41 #include <linux/ppp_defs.h>
42 #include <linux/if_ppp.h>
43 #include <linux/ppp_channel.h>
44 #include <linux/spinlock.h>
45 #include <linux/completion.h>
46 #include <linux/init.h>
47 #include <asm/uaccess.h>
48
49 #define PPP_VERSION     "2.4.2"
50
51 /* Structure for storing local state. */
52 struct syncppp {
53         struct tty_struct *tty;
54         unsigned int    flags;
55         unsigned int    rbits;
56         int             mru;
57         spinlock_t      xmit_lock;
58         spinlock_t      recv_lock;
59         unsigned long   xmit_flags;
60         u32             xaccm[8];
61         u32             raccm;
62         unsigned int    bytes_sent;
63         unsigned int    bytes_rcvd;
64
65         struct sk_buff  *tpkt;
66         unsigned long   last_xmit;
67
68         struct sk_buff_head rqueue;
69
70         struct tasklet_struct tsk;
71
72         atomic_t        refcnt;
73         struct completion dead_cmp;
74         struct ppp_channel chan;        /* interface to generic ppp layer */
75 };
76
77 /* Bit numbers in xmit_flags */
78 #define XMIT_WAKEUP     0
79 #define XMIT_FULL       1
80
81 /* Bits in rbits */
82 #define SC_RCV_BITS     (SC_RCV_B7_1|SC_RCV_B7_0|SC_RCV_ODDP|SC_RCV_EVNP)
83
84 #define PPPSYNC_MAX_RQLEN       32      /* arbitrary */
85
86 /*
87  * Prototypes.
88  */
89 static struct sk_buff* ppp_sync_txmunge(struct syncppp *ap, struct sk_buff *);
90 static int ppp_sync_send(struct ppp_channel *chan, struct sk_buff *skb);
91 static int ppp_sync_ioctl(struct ppp_channel *chan, unsigned int cmd,
92                           unsigned long arg);
93 static void ppp_sync_process(unsigned long arg);
94 static int ppp_sync_push(struct syncppp *ap);
95 static void ppp_sync_flush_output(struct syncppp *ap);
96 static void ppp_sync_input(struct syncppp *ap, const unsigned char *buf,
97                            char *flags, int count);
98
99 static struct ppp_channel_ops sync_ops = {
100         ppp_sync_send,
101         ppp_sync_ioctl
102 };
103
104 /*
105  * Utility procedures to print a buffer in hex/ascii
106  */
107 static void
108 ppp_print_hex (register __u8 * out, const __u8 * in, int count)
109 {
110         register __u8 next_ch;
111         static const char hex[] = "0123456789ABCDEF";
112
113         while (count-- > 0) {
114                 next_ch = *in++;
115                 *out++ = hex[(next_ch >> 4) & 0x0F];
116                 *out++ = hex[next_ch & 0x0F];
117                 ++out;
118         }
119 }
120
121 static void
122 ppp_print_char (register __u8 * out, const __u8 * in, int count)
123 {
124         register __u8 next_ch;
125
126         while (count-- > 0) {
127                 next_ch = *in++;
128
129                 if (next_ch < 0x20 || next_ch > 0x7e)
130                         *out++ = '.';
131                 else {
132                         *out++ = next_ch;
133                         if (next_ch == '%')   /* printk/syslogd has a bug !! */
134                                 *out++ = '%';
135                 }
136         }
137         *out = '\0';
138 }
139
140 static void
141 ppp_print_buffer (const char *name, const __u8 *buf, int count)
142 {
143         __u8 line[44];
144
145         if (name != NULL)
146                 printk(KERN_DEBUG "ppp_synctty: %s, count = %d\n", name, count);
147
148         while (count > 8) {
149                 memset (line, 32, 44);
150                 ppp_print_hex (line, buf, 8);
151                 ppp_print_char (&line[8 * 3], buf, 8);
152                 printk(KERN_DEBUG "%s\n", line);
153                 count -= 8;
154                 buf += 8;
155         }
156
157         if (count > 0) {
158                 memset (line, 32, 44);
159                 ppp_print_hex (line, buf, count);
160                 ppp_print_char (&line[8 * 3], buf, count);
161                 printk(KERN_DEBUG "%s\n", line);
162         }
163 }
164
165
166 /*
167  * Routines implementing the synchronous PPP line discipline.
168  */
169
170 /*
171  * We have a potential race on dereferencing tty->disc_data,
172  * because the tty layer provides no locking at all - thus one
173  * cpu could be running ppp_synctty_receive while another
174  * calls ppp_synctty_close, which zeroes tty->disc_data and
175  * frees the memory that ppp_synctty_receive is using.  The best
176  * way to fix this is to use a rwlock in the tty struct, but for now
177  * we use a single global rwlock for all ttys in ppp line discipline.
178  *
179  * FIXME: Fixed in tty_io nowdays.
180  */
181 static DEFINE_RWLOCK(disc_data_lock);
182
183 static struct syncppp *sp_get(struct tty_struct *tty)
184 {
185         struct syncppp *ap;
186
187         read_lock(&disc_data_lock);
188         ap = tty->disc_data;
189         if (ap != NULL)
190                 atomic_inc(&ap->refcnt);
191         read_unlock(&disc_data_lock);
192         return ap;
193 }
194
195 static void sp_put(struct syncppp *ap)
196 {
197         if (atomic_dec_and_test(&ap->refcnt))
198                 complete(&ap->dead_cmp);
199 }
200
201 /*
202  * Called when a tty is put into sync-PPP line discipline.
203  */
204 static int
205 ppp_sync_open(struct tty_struct *tty)
206 {
207         struct syncppp *ap;
208         int err;
209         int speed;
210
211         if (tty->ops->write == NULL)
212                 return -EOPNOTSUPP;
213
214         ap = kzalloc(sizeof(*ap), GFP_KERNEL);
215         err = -ENOMEM;
216         if (!ap)
217                 goto out;
218
219         /* initialize the syncppp structure */
220         ap->tty = tty;
221         ap->mru = PPP_MRU;
222         spin_lock_init(&ap->xmit_lock);
223         spin_lock_init(&ap->recv_lock);
224         ap->xaccm[0] = ~0U;
225         ap->xaccm[3] = 0x60000000U;
226         ap->raccm = ~0U;
227
228         skb_queue_head_init(&ap->rqueue);
229         tasklet_init(&ap->tsk, ppp_sync_process, (unsigned long) ap);
230
231         atomic_set(&ap->refcnt, 1);
232         init_completion(&ap->dead_cmp);
233
234         ap->chan.private = ap;
235         ap->chan.ops = &sync_ops;
236         ap->chan.mtu = PPP_MRU;
237         ap->chan.hdrlen = 2;    /* for A/C bytes */
238         speed = tty_get_baud_rate(tty);
239         ap->chan.speed = speed;
240         err = ppp_register_channel(&ap->chan);
241         if (err)
242                 goto out_free;
243
244         tty->disc_data = ap;
245         tty->receive_room = 65536;
246         return 0;
247
248  out_free:
249         kfree(ap);
250  out:
251         return err;
252 }
253
254 /*
255  * Called when the tty is put into another line discipline
256  * or it hangs up.  We have to wait for any cpu currently
257  * executing in any of the other ppp_synctty_* routines to
258  * finish before we can call ppp_unregister_channel and free
259  * the syncppp struct.  This routine must be called from
260  * process context, not interrupt or softirq context.
261  */
262 static void
263 ppp_sync_close(struct tty_struct *tty)
264 {
265         struct syncppp *ap;
266
267         write_lock_irq(&disc_data_lock);
268         ap = tty->disc_data;
269         tty->disc_data = NULL;
270         write_unlock_irq(&disc_data_lock);
271         if (!ap)
272                 return;
273
274         /*
275          * We have now ensured that nobody can start using ap from now
276          * on, but we have to wait for all existing users to finish.
277          * Note that ppp_unregister_channel ensures that no calls to
278          * our channel ops (i.e. ppp_sync_send/ioctl) are in progress
279          * by the time it returns.
280          */
281         if (!atomic_dec_and_test(&ap->refcnt))
282                 wait_for_completion(&ap->dead_cmp);
283         tasklet_kill(&ap->tsk);
284
285         ppp_unregister_channel(&ap->chan);
286         skb_queue_purge(&ap->rqueue);
287         kfree_skb(ap->tpkt);
288         kfree(ap);
289 }
290
291 /*
292  * Called on tty hangup in process context.
293  *
294  * Wait for I/O to driver to complete and unregister PPP channel.
295  * This is already done by the close routine, so just call that.
296  */
297 static int ppp_sync_hangup(struct tty_struct *tty)
298 {
299         ppp_sync_close(tty);
300         return 0;
301 }
302
303 /*
304  * Read does nothing - no data is ever available this way.
305  * Pppd reads and writes packets via /dev/ppp instead.
306  */
307 static ssize_t
308 ppp_sync_read(struct tty_struct *tty, struct file *file,
309                unsigned char __user *buf, size_t count)
310 {
311         return -EAGAIN;
312 }
313
314 /*
315  * Write on the tty does nothing, the packets all come in
316  * from the ppp generic stuff.
317  */
318 static ssize_t
319 ppp_sync_write(struct tty_struct *tty, struct file *file,
320                 const unsigned char *buf, size_t count)
321 {
322         return -EAGAIN;
323 }
324
325 static int
326 ppp_synctty_ioctl(struct tty_struct *tty, struct file *file,
327                   unsigned int cmd, unsigned long arg)
328 {
329         struct syncppp *ap = sp_get(tty);
330         int __user *p = (int __user *)arg;
331         int err, val;
332
333         if (!ap)
334                 return -ENXIO;
335         err = -EFAULT;
336         switch (cmd) {
337         case PPPIOCGCHAN:
338                 err = -EFAULT;
339                 if (put_user(ppp_channel_index(&ap->chan), p))
340                         break;
341                 err = 0;
342                 break;
343
344         case PPPIOCGUNIT:
345                 err = -EFAULT;
346                 if (put_user(ppp_unit_number(&ap->chan), p))
347                         break;
348                 err = 0;
349                 break;
350
351         case TCFLSH:
352                 /* flush our buffers and the serial port's buffer */
353                 if (arg == TCIOFLUSH || arg == TCOFLUSH)
354                         ppp_sync_flush_output(ap);
355                 err = tty_perform_flush(tty, arg);
356                 break;
357
358         case FIONREAD:
359                 val = 0;
360                 if (put_user(val, p))
361                         break;
362                 err = 0;
363                 break;
364
365         default:
366                 err = tty_mode_ioctl(tty, file, cmd, arg);
367                 break;
368         }
369
370         sp_put(ap);
371         return err;
372 }
373
374 /* No kernel lock - fine */
375 static unsigned int
376 ppp_sync_poll(struct tty_struct *tty, struct file *file, poll_table *wait)
377 {
378         return 0;
379 }
380
381 /*
382  * This can now be called from hard interrupt level as well
383  * as soft interrupt level or mainline.
384  */
385 static void
386 ppp_sync_receive(struct tty_struct *tty, const unsigned char *buf,
387                   char *cflags, int count)
388 {
389         struct syncppp *ap = sp_get(tty);
390         unsigned long flags;
391
392         if (!ap)
393                 return;
394         spin_lock_irqsave(&ap->recv_lock, flags);
395         ppp_sync_input(ap, buf, cflags, count);
396         spin_unlock_irqrestore(&ap->recv_lock, flags);
397         if (!skb_queue_empty(&ap->rqueue))
398                 tasklet_schedule(&ap->tsk);
399         sp_put(ap);
400 }
401
402 static void
403 ppp_sync_wakeup(struct tty_struct *tty)
404 {
405         struct syncppp *ap = sp_get(tty);
406
407         clear_bit(TTY_DO_WRITE_WAKEUP, &tty->flags);
408         if (!ap)
409                 return;
410         set_bit(XMIT_WAKEUP, &ap->xmit_flags);
411         tasklet_schedule(&ap->tsk);
412         sp_put(ap);
413 }
414
415
416 static struct tty_ldisc_ops ppp_sync_ldisc = {
417         .owner  = THIS_MODULE,
418         .magic  = TTY_LDISC_MAGIC,
419         .name   = "pppsync",
420         .open   = ppp_sync_open,
421         .close  = ppp_sync_close,
422         .hangup = ppp_sync_hangup,
423         .read   = ppp_sync_read,
424         .write  = ppp_sync_write,
425         .ioctl  = ppp_synctty_ioctl,
426         .poll   = ppp_sync_poll,
427         .receive_buf = ppp_sync_receive,
428         .write_wakeup = ppp_sync_wakeup,
429 };
430
431 static int __init
432 ppp_sync_init(void)
433 {
434         int err;
435
436         err = tty_register_ldisc(N_SYNC_PPP, &ppp_sync_ldisc);
437         if (err != 0)
438                 printk(KERN_ERR "PPP_sync: error %d registering line disc.\n",
439                        err);
440         return err;
441 }
442
443 /*
444  * The following routines provide the PPP channel interface.
445  */
446 static int
447 ppp_sync_ioctl(struct ppp_channel *chan, unsigned int cmd, unsigned long arg)
448 {
449         struct syncppp *ap = chan->private;
450         int err, val;
451         u32 accm[8];
452         void __user *argp = (void __user *)arg;
453         u32 __user *p = argp;
454
455         err = -EFAULT;
456         switch (cmd) {
457         case PPPIOCGFLAGS:
458                 val = ap->flags | ap->rbits;
459                 if (put_user(val, (int __user *) argp))
460                         break;
461                 err = 0;
462                 break;
463         case PPPIOCSFLAGS:
464                 if (get_user(val, (int __user *) argp))
465                         break;
466                 ap->flags = val & ~SC_RCV_BITS;
467                 spin_lock_irq(&ap->recv_lock);
468                 ap->rbits = val & SC_RCV_BITS;
469                 spin_unlock_irq(&ap->recv_lock);
470                 err = 0;
471                 break;
472
473         case PPPIOCGASYNCMAP:
474                 if (put_user(ap->xaccm[0], p))
475                         break;
476                 err = 0;
477                 break;
478         case PPPIOCSASYNCMAP:
479                 if (get_user(ap->xaccm[0], p))
480                         break;
481                 err = 0;
482                 break;
483
484         case PPPIOCGRASYNCMAP:
485                 if (put_user(ap->raccm, p))
486                         break;
487                 err = 0;
488                 break;
489         case PPPIOCSRASYNCMAP:
490                 if (get_user(ap->raccm, p))
491                         break;
492                 err = 0;
493                 break;
494
495         case PPPIOCGXASYNCMAP:
496                 if (copy_to_user(argp, ap->xaccm, sizeof(ap->xaccm)))
497                         break;
498                 err = 0;
499                 break;
500         case PPPIOCSXASYNCMAP:
501                 if (copy_from_user(accm, argp, sizeof(accm)))
502                         break;
503                 accm[2] &= ~0x40000000U;        /* can't escape 0x5e */
504                 accm[3] |= 0x60000000U;         /* must escape 0x7d, 0x7e */
505                 memcpy(ap->xaccm, accm, sizeof(ap->xaccm));
506                 err = 0;
507                 break;
508
509         case PPPIOCGMRU:
510                 if (put_user(ap->mru, (int __user *) argp))
511                         break;
512                 err = 0;
513                 break;
514         case PPPIOCSMRU:
515                 if (get_user(val, (int __user *) argp))
516                         break;
517                 if (val < PPP_MRU)
518                         val = PPP_MRU;
519                 ap->mru = val;
520                 err = 0;
521                 break;
522
523         default:
524                 err = -ENOTTY;
525         }
526         return err;
527 }
528
529 /*
530  * This is called at softirq level to deliver received packets
531  * to the ppp_generic code, and to tell the ppp_generic code
532  * if we can accept more output now.
533  */
534 static void ppp_sync_process(unsigned long arg)
535 {
536         struct syncppp *ap = (struct syncppp *) arg;
537         struct sk_buff *skb;
538
539         /* process received packets */
540         while ((skb = skb_dequeue(&ap->rqueue)) != NULL) {
541                 if (skb->len == 0) {
542                         /* zero length buffers indicate error */
543                         ppp_input_error(&ap->chan, 0);
544                         kfree_skb(skb);
545                 }
546                 else
547                         ppp_input(&ap->chan, skb);
548         }
549
550         /* try to push more stuff out */
551         if (test_bit(XMIT_WAKEUP, &ap->xmit_flags) && ppp_sync_push(ap))
552                 ppp_output_wakeup(&ap->chan);
553 }
554
555 /*
556  * Procedures for encapsulation and framing.
557  */
558
559 static struct sk_buff*
560 ppp_sync_txmunge(struct syncppp *ap, struct sk_buff *skb)
561 {
562         int proto;
563         unsigned char *data;
564         int islcp;
565
566         data  = skb->data;
567         proto = (data[0] << 8) + data[1];
568
569         /* LCP packets with codes between 1 (configure-request)
570          * and 7 (code-reject) must be sent as though no options
571          * have been negotiated.
572          */
573         islcp = proto == PPP_LCP && 1 <= data[2] && data[2] <= 7;
574
575         /* compress protocol field if option enabled */
576         if (data[0] == 0 && (ap->flags & SC_COMP_PROT) && !islcp)
577                 skb_pull(skb,1);
578
579         /* prepend address/control fields if necessary */
580         if ((ap->flags & SC_COMP_AC) == 0 || islcp) {
581                 if (skb_headroom(skb) < 2) {
582                         struct sk_buff *npkt = dev_alloc_skb(skb->len + 2);
583                         if (npkt == NULL) {
584                                 kfree_skb(skb);
585                                 return NULL;
586                         }
587                         skb_reserve(npkt,2);
588                         skb_copy_from_linear_data(skb,
589                                       skb_put(npkt, skb->len), skb->len);
590                         kfree_skb(skb);
591                         skb = npkt;
592                 }
593                 skb_push(skb,2);
594                 skb->data[0] = PPP_ALLSTATIONS;
595                 skb->data[1] = PPP_UI;
596         }
597
598         ap->last_xmit = jiffies;
599
600         if (skb && ap->flags & SC_LOG_OUTPKT)
601                 ppp_print_buffer ("send buffer", skb->data, skb->len);
602
603         return skb;
604 }
605
606 /*
607  * Transmit-side routines.
608  */
609
610 /*
611  * Send a packet to the peer over an sync tty line.
612  * Returns 1 iff the packet was accepted.
613  * If the packet was not accepted, we will call ppp_output_wakeup
614  * at some later time.
615  */
616 static int
617 ppp_sync_send(struct ppp_channel *chan, struct sk_buff *skb)
618 {
619         struct syncppp *ap = chan->private;
620
621         ppp_sync_push(ap);
622
623         if (test_and_set_bit(XMIT_FULL, &ap->xmit_flags))
624                 return 0;       /* already full */
625         skb = ppp_sync_txmunge(ap, skb);
626         if (skb != NULL)
627                 ap->tpkt = skb;
628         else
629                 clear_bit(XMIT_FULL, &ap->xmit_flags);
630
631         ppp_sync_push(ap);
632         return 1;
633 }
634
635 /*
636  * Push as much data as possible out to the tty.
637  */
638 static int
639 ppp_sync_push(struct syncppp *ap)
640 {
641         int sent, done = 0;
642         struct tty_struct *tty = ap->tty;
643         int tty_stuffed = 0;
644
645         if (!spin_trylock_bh(&ap->xmit_lock))
646                 return 0;
647         for (;;) {
648                 if (test_and_clear_bit(XMIT_WAKEUP, &ap->xmit_flags))
649                         tty_stuffed = 0;
650                 if (!tty_stuffed && ap->tpkt) {
651                         set_bit(TTY_DO_WRITE_WAKEUP, &tty->flags);
652                         sent = tty->ops->write(tty, ap->tpkt->data, ap->tpkt->len);
653                         if (sent < 0)
654                                 goto flush;     /* error, e.g. loss of CD */
655                         if (sent < ap->tpkt->len) {
656                                 tty_stuffed = 1;
657                         } else {
658                                 kfree_skb(ap->tpkt);
659                                 ap->tpkt = NULL;
660                                 clear_bit(XMIT_FULL, &ap->xmit_flags);
661                                 done = 1;
662                         }
663                         continue;
664                 }
665                 /* haven't made any progress */
666                 spin_unlock_bh(&ap->xmit_lock);
667                 if (!(test_bit(XMIT_WAKEUP, &ap->xmit_flags)
668                       || (!tty_stuffed && ap->tpkt)))
669                         break;
670                 if (!spin_trylock_bh(&ap->xmit_lock))
671                         break;
672         }
673         return done;
674
675 flush:
676         if (ap->tpkt) {
677                 kfree_skb(ap->tpkt);
678                 ap->tpkt = NULL;
679                 clear_bit(XMIT_FULL, &ap->xmit_flags);
680                 done = 1;
681         }
682         spin_unlock_bh(&ap->xmit_lock);
683         return done;
684 }
685
686 /*
687  * Flush output from our internal buffers.
688  * Called for the TCFLSH ioctl.
689  */
690 static void
691 ppp_sync_flush_output(struct syncppp *ap)
692 {
693         int done = 0;
694
695         spin_lock_bh(&ap->xmit_lock);
696         if (ap->tpkt != NULL) {
697                 kfree_skb(ap->tpkt);
698                 ap->tpkt = NULL;
699                 clear_bit(XMIT_FULL, &ap->xmit_flags);
700                 done = 1;
701         }
702         spin_unlock_bh(&ap->xmit_lock);
703         if (done)
704                 ppp_output_wakeup(&ap->chan);
705 }
706
707 /*
708  * Receive-side routines.
709  */
710
711 /* called when the tty driver has data for us.
712  *
713  * Data is frame oriented: each call to ppp_sync_input is considered
714  * a whole frame. If the 1st flag byte is non-zero then the whole
715  * frame is considered to be in error and is tossed.
716  */
717 static void
718 ppp_sync_input(struct syncppp *ap, const unsigned char *buf,
719                 char *flags, int count)
720 {
721         struct sk_buff *skb;
722         unsigned char *p;
723
724         if (count == 0)
725                 return;
726
727         if (ap->flags & SC_LOG_INPKT)
728                 ppp_print_buffer ("receive buffer", buf, count);
729
730         /* stuff the chars in the skb */
731         skb = dev_alloc_skb(ap->mru + PPP_HDRLEN + 2);
732         if (!skb) {
733                 printk(KERN_ERR "PPPsync: no memory (input pkt)\n");
734                 goto err;
735         }
736         /* Try to get the payload 4-byte aligned */
737         if (buf[0] != PPP_ALLSTATIONS)
738                 skb_reserve(skb, 2 + (buf[0] & 1));
739
740         if (flags && *flags) {
741                 /* error flag set, ignore frame */
742                 goto err;
743         } else if (count > skb_tailroom(skb)) {
744                 /* packet overflowed MRU */
745                 goto err;
746         }
747
748         p = skb_put(skb, count);
749         memcpy(p, buf, count);
750
751         /* strip address/control field if present */
752         p = skb->data;
753         if (p[0] == PPP_ALLSTATIONS && p[1] == PPP_UI) {
754                 /* chop off address/control */
755                 if (skb->len < 3)
756                         goto err;
757                 p = skb_pull(skb, 2);
758         }
759
760         /* decompress protocol field if compressed */
761         if (p[0] & 1) {
762                 /* protocol is compressed */
763                 skb_push(skb, 1)[0] = 0;
764         } else if (skb->len < 2)
765                 goto err;
766
767         /* queue the frame to be processed */
768         skb_queue_tail(&ap->rqueue, skb);
769         return;
770
771 err:
772         /* queue zero length packet as error indication */
773         if (skb || (skb = dev_alloc_skb(0))) {
774                 skb_trim(skb, 0);
775                 skb_queue_tail(&ap->rqueue, skb);
776         }
777 }
778
779 static void __exit
780 ppp_sync_cleanup(void)
781 {
782         if (tty_unregister_ldisc(N_SYNC_PPP) != 0)
783                 printk(KERN_ERR "failed to unregister Sync PPP line discipline\n");
784 }
785
786 module_init(ppp_sync_init);
787 module_exit(ppp_sync_cleanup);
788 MODULE_LICENSE("GPL");
789 MODULE_ALIAS_LDISC(N_SYNC_PPP);